alxarafe / alixar

public/htdocs/comm/mailing/card.php

Spacing +3 added lines, -3 removed lines

@@ -471,7 +471,7 @@ 
     }
 
     // Action send test emailing
-    if ($action == 'send' && ! $cancel && $permissiontovalidatesend) {
+    if ($action == 'send' && !$cancel && $permissiontovalidatesend) {
         $error = 0;
 
         $upload_dir = $conf->mailing->dir_output . "/" . get_exdir($object->id, 2, 0, 1, $object, 'mailing');
@@ -549,7 +549,7 @@ 
     if ($action == 'add' && $permissiontocreate) {
         $mesgs = array();
 
-        $object->messtype       = (string) GETPOST("messtype");
+        $object->messtype = (string) GETPOST("messtype");
         if ($object->messtype == 'sms') {
             $object->email_from     = (string) GETPOST("from_phone", 'alphawithlgt'); // Must allow 'name <email>'
         } else {
@@ -1515,7 +1515,7 @@ 
                 $maxfilesizearray = getMaxFileSizeArray();
                 $maxmin = $maxfilesizearray['maxmin'];
                 if ($maxmin > 0) {
-                    $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';  // MAX_FILE_SIZE must precede the field type=file
+                    $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
                 }
                 $out .= '<input type="file" class="flat" id="addedfile" name="addedfile" value="' . $langs->trans("Upload") . '" />';
                 $out .= ' ';

public/htdocs/comm/mailing/cibles.php

Spacing +4 added lines, -4 removed lines

@@ -394,7 +394,7 @@ 
             }
         }
         if (empty($nbemail)) {
-            $nbemail .= ' ' . img_warning($langs->trans('ToAddRecipientsChooseHere'));//.' <span class="warning">'.$langs->trans("NoTargetYet").'</span>';
+            $nbemail .= ' ' . img_warning($langs->trans('ToAddRecipientsChooseHere')); //.' <span class="warning">'.$langs->trans("NoTargetYet").'</span>';
         }
         if ($text) {
             // @phan-suppress-next-line PhanPluginSuspiciousParamPosition
@@ -549,12 +549,12 @@ 
                     }
                     print img_object($langs->trans("EmailingTargetSelector") . ': ' . get_class($obj), $obj->picto, 'class="valignmiddle width25 size15x"');
                     print '</div>';
-                    print '<div class="tagtd valignmiddle">';   //  style="height: 4em"
+                    print '<div class="tagtd valignmiddle">'; //  style="height: 4em"
                     print $obj->getDesc();
                     print '</div>';
 
                     try {
-                        $obj->evenunsubscribe = $object->evenunsubscribe;   // Set flag to include/exclude email that has opt-out.
+                        $obj->evenunsubscribe = $object->evenunsubscribe; // Set flag to include/exclude email that has opt-out.
 
                         $nbofrecipient = $obj->getNbOfRecipients('');
                     } catch (Exception $e) {
@@ -899,7 +899,7 @@ 
                 print '<td class="center nowraponall">';
                 if ($obj->status != $object::STATUS_DRAFT) {        // If status of target line is not draft
                     // Date sent
-                    print $obj->date_envoi;     // @TODO Must store date in date format
+                    print $obj->date_envoi; // @TODO Must store date in date format
                 }
                 print '</td>';
 

public/htdocs/comm/mailing/list.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 // Initialize technical objects
 $object = new Mailing($db);
 $extrafields = new ExtraFields($db);
-$hookmanager->initHooks(array($contextpage));   // Note that conf->hooks_modules contains array of activated contexes
+$hookmanager->initHooks(array($contextpage)); // Note that conf->hooks_modules contains array of activated contexes
 
 // Fetch optionals attributes and labels
 $extrafields->fetch_name_optionals_label($object->table_element);

public/htdocs/comm/mailing/class/advtargetemailing.class.php

Indentation +8 added lines, -8 removed lines

@@ -265,7 +265,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load object in memory from the database
      *
@@ -274,7 +274,7 @@ 
      */
     public function fetch_by_mailing($id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $sql = "SELECT";
         $sql .= " t.rowid,";
@@ -327,7 +327,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load object in memory from the database
      *
@@ -337,7 +337,7 @@ 
      */
     public function fetch_by_element($id = 0, $type_element = 'mailing')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $sql = "SELECT";
         $sql .= " t.rowid,";
@@ -519,7 +519,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load object in memory from database
      *
@@ -528,7 +528,7 @@ 
      */
     public function query_thirdparty($arrayquery)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf, $extrafields;
 
         $sql = "SELECT";
@@ -676,7 +676,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load object in memory from database
      *
@@ -686,7 +686,7 @@ 
      */
     public function query_contact($arrayquery, $withThirdpartyFilter = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $sql = "SELECT";

public/htdocs/comm/mailing/class/mailing.class.php

Indentation +6 added lines, -6 removed lines

@@ -621,7 +621,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete targets emailing
      *
@@ -629,7 +629,7 @@ 
      */
     public function delete_targets()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "DELETE FROM " . MAIN_DB_PREFIX . "mailing_cibles";
         $sql .= " WHERE fk_mailing = " . ((int) $this->id);
 
@@ -646,7 +646,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Change status of each recipient
      *
@@ -655,7 +655,7 @@ 
      */
     public function reset_targets_status($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "UPDATE " . MAIN_DB_PREFIX . "mailing_cibles";
         $sql .= " SET statut = 0";
         $sql .= " WHERE fk_mailing = " . ((int) $this->id);
@@ -870,7 +870,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -880,7 +880,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load("mailing");
 

Spacing +2 added lines, -2 removed lines

@@ -384,7 +384,7 @@ 
                 $this->title = $obj->title;
                 $this->messtype = $obj->messtype;
 
-                $this->statut = $obj->status;   // deprecated
+                $this->statut = $obj->status; // deprecated
                 $this->status = $obj->status;
 
                 $this->nbemail = $obj->nbemail;
@@ -727,7 +727,7 @@ 
                 $sql .= ' WHERE rowid = ' . ((int) $this->id);
 
                 $resqlupdate = $this->db->query($sql);
-                if (! $resqlupdate) {
+                if (!$resqlupdate) {
                     $this->error = $this->db->lasterror();
                     return -1;
                 } else {

public/htdocs/comm/action/card.php

Spacing +10 added lines, -10 removed lines

@@ -75,7 +75,7 @@ 
 $offsetunit = GETPOST('offsetunittype_duration', 'aZ09');
 $remindertype = GETPOST('selectremindertype', 'aZ09');
 $modelmail = GETPOSTINT('actioncommsendmodel_mail');
-$complete = GETPOST('complete', 'alpha');   // 'na' must be allowed
+$complete = GETPOST('complete', 'alpha'); // 'na' must be allowed
 $private = GETPOST('private', 'alphanohtml');
 if ($complete == 'na' || $complete == -2) {
     $complete = -1;
@@ -610,12 +610,12 @@ 
             if ($selectedrecurrulefreq == 'WEEKLY' && !empty($selectedrecurrulebyday)) {
                 $firstdatearray = dol_get_first_day_week(GETPOSTINT("apday"), GETPOSTINT("apmonth"), GETPOSTINT("apyear"));
                 $datep = dol_mktime($fulldayevent ? '00' : GETPOSTINT("aphour"), $fulldayevent ? '00' : GETPOSTINT("apmin"), $fulldayevent ? '00' : GETPOSTINT("apsec"), $firstdatearray['month'], $firstdatearray['first_day'], $firstdatearray['year'], $tzforfullday ? $tzforfullday : 'tzuserrel');
-                $datep = dol_time_plus_duree($datep, $selectedrecurrulebyday + 6, 'd');//We begin the week after
+                $datep = dol_time_plus_duree($datep, $selectedrecurrulebyday + 6, 'd'); //We begin the week after
                 $dayoffset = 7;
                 $monthoffset = 0;
             } elseif ($selectedrecurrulefreq == 'MONTHLY' && !empty($selectedrecurrulebymonthday)) {
                 $firstday = $selectedrecurrulebymonthday;
-                $firstmonth = GETPOST("apday") > $selectedrecurrulebymonthday ? GETPOSTINT("apmonth") + 1 : GETPOSTINT("apmonth");//We begin the week after
+                $firstmonth = GETPOST("apday") > $selectedrecurrulebymonthday ? GETPOSTINT("apmonth") + 1 : GETPOSTINT("apmonth"); //We begin the week after
                 $datep = dol_mktime($fulldayevent ? '00' : GETPOSTINT("aphour"), $fulldayevent ? '00' : GETPOSTINT("apmin"), $fulldayevent ? '00' : GETPOSTINT("apsec"), $firstmonth, $firstday, GETPOSTINT("apyear"), $tzforfullday ? $tzforfullday : 'tzuserrel');
                 $dayoffset = 0;
                 $monthoffset = 1;
@@ -819,7 +819,7 @@ 
             reset($object->socpeopleassigned);
             $object->contact_id = key($object->socpeopleassigned);
         }
-        $object->fk_project  = GETPOSTINT("projectid");
+        $object->fk_project = GETPOSTINT("projectid");
         $object->note_private = trim(GETPOST("note", "restricthtml"));
 
         if (GETPOST("elementtype", 'alpha')) {
@@ -1063,7 +1063,7 @@ 
 if (empty($reshook) && GETPOST('actionmove', 'alpha') == 'mupdate') {
     $error = 0;
 
-    $shour = dol_print_date($object->datep, "%H", 'tzuserrel');     // We take the date visible by user $newdate is also date visible by user.
+    $shour = dol_print_date($object->datep, "%H", 'tzuserrel'); // We take the date visible by user $newdate is also date visible by user.
     $smin = dol_print_date($object->datep, "%M", 'tzuserrel');
 
     $newdate = GETPOST('newdate', 'alpha');
@@ -1273,7 +1273,7 @@ 
         $default = getDolGlobalString('AGENDA_USE_EVENT_TYPE_DEFAULT', 'AC_RDV');
         print img_picto($langs->trans("ActionType"), 'square', 'class="fawidth30 inline-block" style="color: #ddd;"');
         $selectedvalue = GETPOSTISSET("actioncode") ? GETPOST("actioncode", 'aZ09') : ($object->type_code ? $object->type_code : $default);
-        print $formactions->select_type_actions($selectedvalue, "actioncode", "systemauto", 0, -1, 0, 1);   // TODO Replace 0 with -2 in onlyautoornot
+        print $formactions->select_type_actions($selectedvalue, "actioncode", "systemauto", 0, -1, 0, 1); // TODO Replace 0 with -2 in onlyautoornot
         print '</td></tr>';
     }
 
@@ -1587,7 +1587,7 @@ 
         }
         //var_dump('origin='.$origin.' originid='.$originid.' hasPermissionOnLinkedObject='.$hasPermissionOnLinkedObject);
 
-        if (! in_array($origin, array('societe', 'project', 'task', 'user'))) {
+        if (!in_array($origin, array('societe', 'project', 'task', 'user'))) {
             // We do not use link for object that already contains a hard coded field to make links with agenda events
             print '<tr><td class="titlefieldcreate">' . $langs->trans("LinkedObject") . '</td>';
             print '<td colspan="3">';
@@ -1738,7 +1738,7 @@ 
             $object->socpeopleassigned[$id] = array('id' => $tmpid);
         }
         $object->contact_id   = GETPOSTINT("contactid");
-        $object->fk_project  = GETPOSTINT("projectid");
+        $object->fk_project = GETPOSTINT("projectid");
 
         $object->note_private = GETPOST("note", 'restricthtml');
     }
@@ -1814,7 +1814,7 @@ 
         if ($backtopage) {
             print '<input type="hidden" name="backtopage" value="' . ($backtopage != '1' ? $backtopage : '') . '">';
         }
-        if (!getDolGlobalString('AGENDA_USE_EVENT_TYPE') && ! preg_match('/^TICKET_MSG_PRIVATE/', $object->code)) {
+        if (!getDolGlobalString('AGENDA_USE_EVENT_TYPE') && !preg_match('/^TICKET_MSG_PRIVATE/', $object->code)) {
             print '<input type="hidden" name="actioncode" value="' . $object->type_code . '">';
         }
 
@@ -2187,7 +2187,7 @@ 
                    })';
             print '</script>' . "\n";
 
-            print '</div>';     // End of div for reminderparameters
+            print '</div>'; // End of div for reminderparameters
         }
 
         print dol_get_fiche_end();

public/htdocs/comm/action/pertype.php

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
 //if (! ($usergroup > 0) && ! ($filtert > 0)) $filtert = $user->id;
 
 // $showbirthday = empty($conf->use_javascript_ajax)?GETPOST("showbirthday","int"):1;
-$showbirthday = 0;    // will be hidden here
+$showbirthday = 0; // will be hidden here
 
 // If not choice done on calendar owner, we filter on user.
 if (empty($filtert) && !getDolGlobalString('AGENDA_ALL_CALENDARS')) {
@@ -997,8 +997,8 @@ 
     // We are in a particular day for $username, now we scan all events
     foreach ($eventarray as $daykey => $notused) {
         $annee = dol_print_date($daykey, '%Y', 'tzuserrel');
-        $mois =  dol_print_date($daykey, '%m', 'tzuserrel');
-        $jour =  dol_print_date($daykey, '%d', 'tzuserrel');
+        $mois = dol_print_date($daykey, '%m', 'tzuserrel');
+        $jour = dol_print_date($daykey, '%d', 'tzuserrel');
 
         if ($day == $jour && (int) $month == (int) $mois && $year == $annee) {  // Is it the day we are looking for when calling function ?
             // Scan all event for this date

public/htdocs/comm/action/list.php

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
 
 $mode = GETPOST('mode', 'aZ09');
 if (empty($mode) && preg_match('/show_/', $action)) {
-    $mode = $action;    // For backward compatibility
+    $mode = $action; // For backward compatibility
 }
 $resourceid = GETPOSTINT("search_resourceid") ? GETPOSTINT("search_resourceid") : GETPOSTINT("resourceid");
 $pid = GETPOSTINT("search_projectid", 3) ? GETPOSTINT("search_projectid", 3) : GETPOSTINT("projectid", 3);
@@ -179,7 +179,7 @@ 
 
 $object->fields = dol_sort_array($object->fields, 'position');
 $arrayfields = dol_sort_array($arrayfields, 'position');
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
 
 $result = restrictedArea($user, 'agenda', 0, '', 'myactions');
 if ($user->socid && $socid) {
@@ -414,7 +414,7 @@ 
 if (isModEnabled('category') && $user->hasRight('agenda', 'myactions', 'create')) {
     $arrayofmassactions['preaffecttag'] = img_picto('', 'category', 'class="pictofixedwidth"') . $langs->trans("AffectTag");
 }
-if (GETPOSTINT('nomassaction') || in_array($massaction, array('presend', 'predelete','preaffecttag'))) {
+if (GETPOSTINT('nomassaction') || in_array($massaction, array('presend', 'predelete', 'preaffecttag'))) {
     $arrayofmassactions = array();
 }
 $massactionbutton = $form->selectMassAction('', $arrayofmassactions);

public/htdocs/comm/action/peruser.php

Spacing +2 added lines, -2 removed lines

@@ -1156,8 +1156,8 @@
     // We are in a particular day for $username, now we scan all events
     foreach ($eventarray as $daykey => $notused) {
         $annee = dol_print_date($daykey, '%Y', 'tzuserrel');
-        $mois =  dol_print_date($daykey, '%m', 'tzuserrel');
-        $jour =  dol_print_date($daykey, '%d', 'tzuserrel');
+        $mois = dol_print_date($daykey, '%m', 'tzuserrel');
+        $jour = dol_print_date($daykey, '%d', 'tzuserrel');
         //var_dump("daykey=$daykey day=$day jour=$jour, month=$month mois=$mois, year=$year annee=$annee");