From b43c6500bf79225a41aa91d740a569965d1c9e0b Mon Sep 17 00:00:00 2001
From: Adam Zammit <adam.zammit@acspri.org.au>
Date: Thu, 8 Aug 2013 14:45:22 +1000
Subject: [PATCH] Fixes:1209486 Supervisor can assign a blank outcome to a case
 which makes it unavailable

---
 admin/supervisor.php | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/admin/supervisor.php b/admin/supervisor.php
index 7ced9ad5..43ce1692 100644
--- a/admin/supervisor.php
+++ b/admin/supervisor.php
@@ -116,10 +116,16 @@ if (isset($_GET['call_id']))
 	if (isset($_GET['set_outcome_id']))
 	{
 		$outcome_id = bigintval($_GET['set_outcome_id']);
-		$sql = "UPDATE `call`
-			SET outcome_id = '$outcome_id'
-			WHERE call_id = '$call_id'";
-		$db->Execute($sql);
+
+		if ($outcome_id > 0)
+		{
+
+			$sql = "UPDATE `call`
+				SET outcome_id = '$outcome_id'
+				WHERE call_id = '$call_id'";
+
+			$db->Execute($sql);
+		}
 	}
 	else
 	{
@@ -156,11 +162,14 @@ if ($case_id != false)
 	{
 		$outcome_id = bigintval($_GET['outcome_id']);
 
-		$sql = "UPDATE `case`
-			SET current_outcome_id = $outcome_id
-			WHERE case_id = '$case_id'";
-
-		$db->Execute($sql);
+		if ($outcome_id > 0)
+		{
+			$sql = "UPDATE `case`
+				SET current_outcome_id = $outcome_id
+				WHERE case_id = '$case_id'";
+	
+			$db->Execute($sql);
+		}
 	}
 
 	if (isset($_GET['operator_id']))