* Refactor code.
This commit is contained in:
@@ -202,8 +202,8 @@ class messageModel extends model
|
||||
}
|
||||
if(empty($toList) and $objectType == 'task' and $object->mode == 'multi')
|
||||
{
|
||||
$assignedTo = $this->loadModel('task')->getTeamTaskAssignedTo($object->id);
|
||||
$toList = array_filter($assignedTo, function($account){
|
||||
$teamMembers = $this->loadModel('task')->getTeamMembers($object->id);
|
||||
$toList = array_filter($teamMembers, function($account){
|
||||
return $account != $this->app->user->account;
|
||||
});
|
||||
$toList = implode(',', $toList);
|
||||
|
||||
@@ -3926,7 +3926,7 @@ class taskModel extends model
|
||||
$toTeamTaskList = '';
|
||||
if($task->mode == 'multi')
|
||||
{
|
||||
$toTeamTaskList = $this->getTeamTaskAssignedTo($task->id);
|
||||
$toTeamTaskList = $this->getTeamMembers($task->id);
|
||||
$toTeamTaskList = implode(',', $toTeamTaskList);
|
||||
$toList = $toTeamTaskList;
|
||||
}
|
||||
@@ -4348,17 +4348,17 @@ class taskModel extends model
|
||||
}
|
||||
|
||||
/**
|
||||
* Get teamTask assignedTo.
|
||||
* Get teamTask members.
|
||||
*
|
||||
* @param int $taskID
|
||||
* @access public
|
||||
* @return array
|
||||
*/
|
||||
public function getTeamTaskAssignedTo($taskID)
|
||||
public function getTeamMembers($taskID)
|
||||
{
|
||||
$taskType = $this->dao->select('mode')->from(TABLE_TASK)->where('id')->eq($taskID)->fetch('mode');
|
||||
if($taskType != 'multi') return array();
|
||||
$assignedToList = $this->dao->select('account')->from(TABLE_TASKTEAM)->where('task')->eq($taskID)->fetchPairs();
|
||||
return empty($assignedToList) ? array() : array_keys($assignedToList);
|
||||
$teamMembers = $this->dao->select('account')->from(TABLE_TASKTEAM)->where('task')->eq($taskID)->fetchPairs();
|
||||
return empty($teamMembers) ? $teamMembers : array_keys($teamMembers);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user