From 30634f3615f941631e463654a801801660de1d36 Mon Sep 17 00:00:00 2001 From: Julio Cesar Laura Date: Wed, 25 Sep 2013 12:15:29 -0400 Subject: [PATCH] IMPROVEMENT - In some cicunstances the $_POST['task'] variable is sent empty (not a bug), so we added a validation to avoid another bug when this variable is empty --- .../methods/cases/proxyReassignUsersList.php | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/workflow/engine/methods/cases/proxyReassignUsersList.php b/workflow/engine/methods/cases/proxyReassignUsersList.php index 797b1247b..5b8a7cbe9 100755 --- a/workflow/engine/methods/cases/proxyReassignUsersList.php +++ b/workflow/engine/methods/cases/proxyReassignUsersList.php @@ -70,13 +70,15 @@ $oConf = new Configurations(); $aUsersInvolved = Array(); $ConfEnv = $oConf->getFormats(); -G::LoadClass( 'tasks' ); -$task = new Task(); -$tasks = $task->load($TaskUid); -$rows = $oCases->getUsersToReassign($TaskUid, $_SESSION['USER_LOGGED'], $tasks['PRO_UID']); -foreach ($rows as $row) { - $sCaseUser = G::getFormatUserList( $ConfEnv['format'], $row ); - $aUsersInvolved[] = array ('userUid' => $row['USR_UID'], 'userFullname' => $sCaseUser); +if ($TaskUid != '') { + G::LoadClass( 'tasks' ); + $task = new Task(); + $tasks = $task->load($TaskUid); + $rows = $oCases->getUsersToReassign($TaskUid, $_SESSION['USER_LOGGED'], $tasks['PRO_UID']); + foreach ($rows as $row) { + $sCaseUser = G::getFormatUserList( $ConfEnv['format'], $row ); + $aUsersInvolved[] = array ('userUid' => $row['USR_UID'], 'userFullname' => $sCaseUser); + } } // $oTmp = new stdClass();