Code comment

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 @@ public function fetchResources()
     /**
      *    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 @@ public function fetch_userassigned($override = true)
                                                                   '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 @@ public function fetch_userassigned($override = true)
      */
     public function delete($notrigger = 0)
     {
-        global $user,$langs,$conf;
+        global $user;
 
         $error=0;