JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
really made it so you can put tasks on hold. contractor can switch logins multiple...
[contractor-progress.git] / tasks.php
index 7116567..8418502 100644 (file)
--- a/tasks.php
+++ b/tasks.php
@@ -141,8 +141,12 @@ function tasks_display_main() {
                        switch($state) {
                                case TASK_DRAFT:
                                case TASK_NEEDS_CLARIFICATION:
-                               case TASK_NEEDS_QUOTE:
                                case TASK_BUG:
+                               case TASK_ON_HOLD:
+                                       tem_show('normal_edit_link');
+                               break;
+                               case TASK_NEEDS_QUOTE:
+                                       tem_show('hold_link');
                                        tem_show('normal_edit_link');
                                break;
                                case TASK_NEEDS_GO_AHEAD:
@@ -153,6 +157,7 @@ function tasks_display_main() {
                                case TASK_QUEUED:
                                        tem_show('price_row');
                                        tem_show('warning_edit_link');
+                                       tem_show('hold_link');
                                break;
                                case TASK_WORKING:
                                        tem_show('price_row');
@@ -427,13 +432,13 @@ function tasks_edit_main() {
                } else { # better be "request_price"
                        if(description_has_fixmes($description)) {
                                $state = TASK_NEEDS_CLARIFICATION;
-                               message('The description is not ready to be priced yet because it still contains at least one "FIXME".');
+                               message("Error: Not requesting price. To get this task priced, you'll need to edit the description so it no longer contains \"FIXME\".");
                        } else {
                                $state = TASK_NEEDS_QUOTE;
                        }
                }
 
-               if(!logged_in_as_contractor() || $edit_id || $_REQUEST['client_id']) { # if you change this change the one above
+               if(!logged_in_as_contractor() || $edit_id || $_REQUEST['client_id']) {
                        if($edit_id) {
                                $tables = 'title,url,description,state';
                                $values = array($title, $url, $description, $state);
@@ -447,7 +452,7 @@ function tasks_edit_main() {
                                        array_push($values, new_lowest_priority($client_id));
                                }
                                db_update('tasks', $tables, $values, 'where id=%i', $edit_id);
-                               message('Task updated.');
+                               message('Changes saved.');
                        } else {
                                # new task
                                $paid = 0;