X-Git-Url: https://jasonwoof.com/gitweb/?a=blobdiff_plain;f=tasks.php;h=58737a3b86b46457aae9ee16a408d1712eab2de6;hb=98110d79b6536290baffd1468f22cb8b046902cd;hp=6d382bc6413fec1df281661ef173f3520ee16a34;hpb=fa236b27bfc3bc87902f0a92d0d017b9f2d444be;p=contractor-progress.git

diff --git a/tasks.php b/tasks.php
index 6d382bc..58737a3 100644
--- 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');
@@ -294,7 +299,7 @@ function prioritize_task($id, $change) {
 
 function tasks_edit_main() {
 	$state = TASK_DRAFT; # will be overwritten
-	$client_id = logged_in(); # fixed shortly if we're contractor
+	$client_id = logged_in(); # fixed shortly if we're contractor, unless it's a new task by the contractor
 	$edit_id = format_int($_REQUEST['tasks_edit_id']);
 	unset($_REQUEST['tasks_edit_id']);
 	if($edit_id) {
@@ -328,6 +333,13 @@ function tasks_edit_main() {
 		$state = TASK_BUG;
 	}
 
+	if(isset($_REQUEST['tasks_hold_id'])) {
+		$id = $_REQUEST['tasks_hold_id'];
+		db_update('tasks', 'state', TASK_ON_HOLD, 'where id=%i', $id);
+		message("Task removed from Jason's to-do list.");
+		return './';
+	}
+
 	if(isset($_REQUEST['tasks_mark_paid_id'])) {
 		if(!logged_in_as_contractor()) {
 			message("Error: only Jason can mark tasks as paid.");
@@ -399,6 +411,9 @@ function tasks_edit_main() {
 	if(isset($_REQUEST['title'])) {
 		list($title, $url, $description, $price) = tasks_get_fields();
 		$queuing = false;
+		if(logged_in_as_contractor() && $_REQUEST['client_id']) {
+			$client_id = format_int($_REQUEST['client_id']);
+		}
 
 		# FIXME
 		if(isset($_REQUEST['save_draft'])) {
@@ -420,13 +435,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);
@@ -440,7 +455,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;
@@ -510,7 +525,7 @@ function tasks_edit_main() {
 	} else {
 		tem_show('new_msg');
 		if(logged_in_as_contractor()) {
-			pulldown('client_id', db_get_rows('people', 'id,name', 'where id > 1'), PULLDOWN_2D);
+			pulldown('client_id', db_get_rows('people', 'id,name', 'where id > 1 order by name'), PULLDOWN_2D);
 			tem_set('client_id', format_int($_REQUEST['client_id']));
 			tem_show('client_row');
 		}