Avoid creating tasks without a title (#1152)
This commit is contained in:
parent
d690844582
commit
7c1293be22
1 changed files with 11 additions and 5 deletions
|
@ -148,12 +148,14 @@ function rcube_tasklist_ui(settings)
|
||||||
// quick-add a task
|
// quick-add a task
|
||||||
$(rcmail.gui_objects.quickaddform).submit(function(e){
|
$(rcmail.gui_objects.quickaddform).submit(function(e){
|
||||||
var tasktext = this.elements.text.value,
|
var tasktext = this.elements.text.value,
|
||||||
rec = { id:-(++idcount), title:tasktext, readonly:true, mask:0, complete:0 };
|
rec = { id:-(++idcount), title:tasktext, readonly:true, mask:0, complete:0 };
|
||||||
|
|
||||||
save_task({ tempid:rec.id, raw:tasktext, list:me.selected_list }, 'new');
|
if (tasktext && tasktext.length) {
|
||||||
render_task(rec);
|
save_task({ tempid:rec.id, raw:tasktext, list:me.selected_list }, 'new');
|
||||||
|
render_task(rec);
|
||||||
|
|
||||||
$('#listmessagebox').hide();
|
$('#listmessagebox').hide();
|
||||||
|
}
|
||||||
|
|
||||||
// clear form
|
// clear form
|
||||||
this.reset();
|
this.reset();
|
||||||
|
@ -1059,7 +1061,11 @@ function rcube_tasklist_ui(settings)
|
||||||
me.selected_task.attachments = [];
|
me.selected_task.attachments = [];
|
||||||
|
|
||||||
// do some basic input validation
|
// do some basic input validation
|
||||||
if (me.selected_task.startdate && me.selected_task.date) {
|
if (!me.selected_task.title || me.selected_task.title.length) {
|
||||||
|
title.focus();
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
else if (me.selected_task.startdate && me.selected_task.date) {
|
||||||
var startdate = $.datepicker.parseDate(datepicker_settings.dateFormat, me.selected_task.startdate, datepicker_settings);
|
var startdate = $.datepicker.parseDate(datepicker_settings.dateFormat, me.selected_task.startdate, datepicker_settings);
|
||||||
var duedate = $.datepicker.parseDate(datepicker_settings.dateFormat, me.selected_task.date, datepicker_settings);
|
var duedate = $.datepicker.parseDate(datepicker_settings.dateFormat, me.selected_task.date, datepicker_settings);
|
||||||
if (startdate > duedate) {
|
if (startdate > duedate) {
|
||||||
|
|
Loading…
Add table
Reference in a new issue