diff --git a/module/task/model.php b/module/task/model.php index cdbb008a2e..1b7d064e60 100755 --- a/module/task/model.php +++ b/module/task/model.php @@ -1805,10 +1805,23 @@ class taskModel extends model /* Fix bug#3036. */ foreach($record->consumed as $id => $item) $record->consumed[$id] = trim($item); - foreach($record->left as $id => $item) $record->left[$id] = trim($item); - foreach($record->consumed as $id => $item) if(!is_numeric($item) and !empty($item)) dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->totalNumber; - foreach($record->left as $id => $item) if(!is_numeric($item) and !empty($item)) dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->leftNumber; - foreach($record->dates as $id => $item) if($item > $today) dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->date; + foreach($record->consumed as $id => $item) + { + if(!is_numeric($item) and !empty($item)) + { + dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->totalNumber; + } + elseif(is_numeric($item) and $item <= 0) + { + dao::$errors[] = sprintf($this->lang->error->gt, 'ID #' . $id . ' ' . $this->lang->task->record, '0'); + } + } + foreach($record->left as $id => $item) + { + $record->left[$id] = trim($item); + if(!is_numeric($item) and !empty($item)) dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->leftNumber; + } + foreach($record->dates as $id => $item) if($item > $today) dao::$errors[] = 'ID #' . $id . ' ' . $this->lang->task->error->date; if(dao::isError()) return false; $estimates = array();