Merge lp:~ara/ubuntu-qa-website/bug_488707 into lp:~ubuntu-qa-website-devel/ubuntu-qa-website/trunk

[Ara Pulido]

[Ara Pulido (ara)]

Added a new check on post to avoid allowing marking a test as failed, without providing a bug number. It fixes bug 488707.

=== modified file 'code/modules/qatracker/qatracker.function.php'
--- code/modules/qatracker/qatracker.function.php	2008-03-05 20:45:53 +0000
+++ code/modules/qatracker/qatracker.function.php	2009-12-21 09:35:22 +0000
@@ -40,6 +40,9 @@
 	if ((!is_numeric($data['isot_result'])) && ($data['isot_valid'] || $data['isot_addbug'])) {
 		$data['isot_error'].="<font color=\"red\">You must select a result</font><br />";
 	}
+    if (($data['isot_result'] == 0) && (count($data['isot_bug']) == 0) &&  ($_SERVER['REQUEST_METHOD'] === 'POST'))
+		$data['isot_error'].="<font color=\"red\">To mark a test as failed, you need to provide a bug number</font><br />";
+
 	return $data;
 }