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

diff --git a/tasks.php b/tasks.php
index ed35222..6d382bc 100644
--- a/tasks.php
+++ b/tasks.php
@@ -31,14 +31,6 @@ function description_has_fixmes($description) {
 	return (strpos($description, 'FIXME') !== false);
 }
 
-# encode as html, make it display newlines and leading spaces
-function enc_htmlbrtab($str) {
-	$str = enc_htmlbr($str);
-	$space_to_nbsp = create_function('$matches', 'return str_repeat(\' \', strlen($matches[0]) * 2);');
-	$str = preg_replace_callback("|^ *|m", $space_to_nbsp, $str);
-	return $str;
-}
-
 function tasks_get_fields() {
 	$title = format_oneline($_REQUEST['title']);
 	$url = format_oneline($_REQUEST['url']);
@@ -84,18 +76,26 @@ function tasks_display_main() {
 	$task_id = format_int($_REQUEST['tasks_id']);;
 	$client_id = logged_in();
 	if(logged_in_as_contractor()) {
-		$row = db_get_row('tasks', 'title,url,description,state,price,client_id', 'where id=%i', $task_id);
+		$row = db_get_row('tasks', 'title,url,description,state,price,client_id,paid,finished_at,tested_at', 'where id=%i', $task_id);
 	} else {
-		$row = db_get_row('tasks', 'title,url,description,state,price,client_id', 'where id=%i && client_id=%i', $task_id, $client_id);
+		$row = db_get_row('tasks', 'title,url,description,state,price,client_id,paid,finished_at,tested_at', 'where id=%i && client_id=%i', $task_id, $client_id);
 	}
 	if($row) {
-		list($title, $url, $description, $state, $price, $owner_id) = $row;
+		list($title, $url, $description, $state, $price, $owner_id, $paid, $finished_at, $tested_at) = $row;
 		tem_set('task_id', $task_id);
 		tem_set('title', $title);
 		tem_set('url', $url);
 		tem_set('description', $description);
 		tem_set('state', task_state_pretty($state));
 		tem_set('price', $price);
+		if($finished_at) {
+			tem_set('finished_at', $finished_at);
+			tem_show('finished_at_section');
+		}
+		if($tested_at) {
+			tem_set('tested_at', $tested_at);
+			tem_show('tested_at_section');
+		}
 		if($state == TASK_BUG) {
 			tem_show('bug_title');
 		} else {
@@ -129,8 +129,12 @@ function tasks_display_main() {
 					}
 					# FALL THROUGH
 				case TASK_FINISHED:
+					if($paid) {
+						tem_show('marked_paid');
+					} else {
+						tem_show('mark_paid_link');
+					}
 					tem_show('price_row');
-					tem_show('mark_paid_link'); # FIXME
 				break;
 			}
 		} else {
@@ -377,7 +381,7 @@ function tasks_edit_main() {
 			message("Error: can't test a task entered by/for another client.");
 			return './';
 		}
-		db_update('tasks', 'state', TASK_FINISHED, 'where id=%i', $id);
+		db_update('tasks', 'state,tested_at', TASK_FINISHED, date('Y-m-d'), 'where id=%i', $id);
 		message('Task marked as finished.');
 		# FIXME also mark it as paid if client's balance can cover it
 		return './';
@@ -422,7 +426,7 @@ function tasks_edit_main() {
 			}
 		}
 
-		if("you're happy with the POSTed values") { # if you change this change the one above
+		if(!logged_in_as_contractor() || $edit_id || $_REQUEST['client_id']) { # if you change this change the one above
 			if($edit_id) {
 				$tables = 'title,url,description,state';
 				$values = array($title, $url, $description, $state);
@@ -443,6 +447,7 @@ function tasks_edit_main() {
 				$client_id = logged_in();
 				if(logged_in_as_contractor() && $_REQUEST['client_id']) {
 					$client_id = format_int($_REQUEST['client_id']);
+					$client_name = db_get_value('people', 'name', 'where id=%i', $client_id);
 				} else {
 					# if client entered the task, no price is set
 					$price = 0;
@@ -453,7 +458,11 @@ function tasks_edit_main() {
 					$ord = 0;
 				}
 				db_insert('tasks', 'client_id,title,url,description,state,paid,price,ord', $client_id, $title, $url, $description, $state, $paid, $price, $ord);
-				message('Task saved.');
+				if(logged_in_as_contractor()) {
+					message("Task saved for $client_name.");
+				} else {
+					message('Task saved.');
+				}
 			}
 			if($GLOBALS['tasks_form_recipient'] != "fixme@example.com") {
 				$to = $GLOBALS['tasks_form_recipient'];
@@ -477,6 +486,8 @@ function tasks_edit_main() {
 			if($error !== true) {
 				return './';
 			}
+		} else {
+			message('Error: you must select a client for the task');
 		}
 		# otherwise, we display the form again. tasks_get_fields() has
 		# already put the posted values back into the template engine, so they will
@@ -484,7 +495,7 @@ function tasks_edit_main() {
 		# fix their entry in whatever way you require.
 	} elseif($edit_id) {
 		# we've recieved an edit id, but no data. So we grab the values to be edited from the database
-		list($title, $url, $description, $state, $paid) = db_get_row('tasks', 'title,url,description,state,price', 'where id=%i', $edit_id);
+		list($title, $url, $description, $state, $price) = db_get_row('tasks', 'title,url,description,state,price', 'where id=%i', $edit_id);
 		tasks_tem_sets($title, $url, $description, $price);
 	} else {
 		# form not submitted, you can set default values like so: