diff --git a/htdocs/comm/action/class/actioncomm.class.php b/htdocs/comm/action/class/actioncomm.class.php index 4096175e87c..97d3c4e397e 100644 --- a/htdocs/comm/action/class/actioncomm.class.php +++ b/htdocs/comm/action/class/actioncomm.class.php @@ -233,14 +233,14 @@ class ActionComm extends CommonObject /** * @var User Object user of owner * @deprecated - * @see userownerid + * @see $userownerid */ public $usertodo; /** * @var User Object user that did action * @deprecated - * @see userdoneid + * @see $userdoneid */ public $userdone; @@ -257,14 +257,14 @@ class ActionComm extends CommonObject /** * @var Societe|null Company linked to action (optional) * @deprecated - * @see socid + * @see $socid */ public $societe; /** * @var Contact|null Contact linked to action (optional) * @deprecated - * @see contactid + * @see $contactid */ public $contact; @@ -824,7 +824,7 @@ class ActionComm extends CommonObject /** * Initialize this->userassigned array with list of id of user assigned to event * - * @param bool $override this->userownerid when empty + * @param bool $override Override $this->userownerid when empty. TODO This should be false by default. True is here to fix corrupted data. * @return int <0 if KO, >0 if OK */ public function fetch_userassigned($override = true) @@ -856,7 +856,7 @@ class ActionComm extends CommonObject 'transparency'=>$obj->transparency); } - if($override === true) + if ($override === true) { // If not defined (should not happened, we fix this) if (empty($this->userownerid)) @@ -883,7 +883,7 @@ class ActionComm extends CommonObject */ public function delete($notrigger = 0) { - global $user,$langs,$conf; + global $user; $error=0;