alxarafe / alixar

htdocs/core/class/CSMSFile.class.php

Spacing +4 added lines, -4 removed lines

@@ -150,10 +150,10 @@ 
         if (!getDolGlobalString('MAIN_DISABLE_ALL_SMS')) {
             // Action according to the chose sending method
             if (getDolGlobalString('MAIN_SMS_SENDMODE')) {
-                $sendmode = getDolGlobalString('MAIN_SMS_SENDMODE');    // $conf->global->MAIN_SMS_SENDMODE looks like a value 'module'
-                $classmoduleofsender = getDolGlobalString('MAIN_MODULE_' . strtoupper($sendmode) . '_SMS', $sendmode);  // $conf->global->MAIN_MODULE_XXX_SMS looks like a value 'class@module'
+                $sendmode = getDolGlobalString('MAIN_SMS_SENDMODE'); // $conf->global->MAIN_SMS_SENDMODE looks like a value 'module'
+                $classmoduleofsender = getDolGlobalString('MAIN_MODULE_' . strtoupper($sendmode) . '_SMS', $sendmode); // $conf->global->MAIN_MODULE_XXX_SMS looks like a value 'class@module'
                 if ($classmoduleofsender == 'ovh') {
-                    $classmoduleofsender = 'ovhsms@ovh';    // For backward compatibility
+                    $classmoduleofsender = 'ovhsms@ovh'; // For backward compatibility
                 }
 
                 $tmp = explode('@', $classmoduleofsender);
@@ -209,7 +209,7 @@ 
                 // Send sms method not correctly defined
                 // --------------------------------------
                 $this->error = 'Bad value for MAIN_SMS_SENDMODE constant';
-                $res = false;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                $res = false; // @phan-suppress-current-line PhanPluginRedundantAssignment
             }
         } else {
             $this->error = 'No sms sent. Feature is disabled by option MAIN_DISABLE_ALL_SMS';

htdocs/core/class/fileupload.class.php

Spacing +1 added lines, -1 removed lines

@@ -428,7 +428,7 @@
         // Sanitize to avoid stream execution when calling file_size(). Not that this is a second security because
         // most streams are already disabled by stream_wrapper_unregister() in filefunc.inc.php
         $uploaded_file = preg_replace('/\s*(http|ftp)s?:/i', '', $uploaded_file);
-        $uploaded_file = realpath($uploaded_file);  // A hack to be sure the file point to an existing file on disk (and is not a SSRF attack)
+        $uploaded_file = realpath($uploaded_file); // A hack to be sure the file point to an existing file on disk (and is not a SSRF attack)
 
         $validate = $this->validate($uploaded_file, $file, $error, $index);
 

htdocs/core/class/utils.class.php

Spacing +9 added lines, -9 removed lines

@@ -132,14 +132,14 @@ 
             if ($choice == 'allfiles') {
                 // Delete all files (except install.lock, do not follow symbolic links)
                 if ($dolibarr_main_data_root) {
-                    $filesarray = dol_dir_list($dolibarr_main_data_root, "all", 0, '', 'install\.lock$', 'name', SORT_ASC, 0, 0, '', 1);    // No need to use recursive, we will delete directory
+                    $filesarray = dol_dir_list($dolibarr_main_data_root, "all", 0, '', 'install\.lock$', 'name', SORT_ASC, 0, 0, '', 1); // No need to use recursive, we will delete directory
                 }
             }
 
             if ($choice == 'allfilesold') {
                 // Delete all files (except install.lock, do not follow symbolic links)
                 if ($dolibarr_main_data_root) {
-                    $filesarray = dol_dir_list($dolibarr_main_data_root, "files", 1, '', 'install\.lock$', 'name', SORT_ASC, 0, 0, '', 1, $nbsecondsold);   // No need to use recursive, we will delete directory
+                    $filesarray = dol_dir_list($dolibarr_main_data_root, "files", 1, '', 'install\.lock$', 'name', SORT_ASC, 0, 0, '', 1, $nbsecondsold); // No need to use recursive, we will delete directory
                 }
             }
 
@@ -467,7 +467,7 @@ 
                         $langs->load("errors");
                         dol_syslog("Datadump retval after exec=" . $retval, LOG_ERR);
                         $errormsg = 'Error ' . $retval;
-                        $ok = 0;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                        $ok = 0; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     } else {
                         $i = 0;
                         if (!empty($output_arr)) {
@@ -1344,14 +1344,14 @@ 
         }
         if ($tmpfiles && is_array($tmpfiles)) {
             foreach ($tmpfiles as $key => $val) {
-                if ($key  == 'fullname') {
+                if ($key == 'fullname') {
                     $filepath = array($val);
                     $filesize = dol_filesize($val);
                 }
-                if ($key  == 'type') {
+                if ($key == 'type') {
                     $mimetype = array($val);
                 }
-                if ($key  == 'relativename') {
+                if ($key == 'relativename') {
                     $filename = array($val);
                 }
             }
@@ -1360,7 +1360,7 @@ 
         if ($filepath) {
             if ($filesize > $sizelimit) {
                 $message .= '<br>' . $langs->trans("BackupIsTooLargeSend");
-                $documenturl =  $dolibarr_main_url_root . '/document.php?modulepart=systemtools&atachement=1&file=backup/' . urlencode($filename[0]);
+                $documenturl = $dolibarr_main_url_root . '/document.php?modulepart=systemtools&atachement=1&file=backup/' . urlencode($filename[0]);
                 $message .= '<br><a href=' . $documenturl . '>Lien de téléchargement</a>';
                 $filepath = '';
                 $mimetype = '';
@@ -1432,7 +1432,7 @@ 
         }
 
         $cron_job = new Cronjob($db);
-        $cron_job->fetchAll('DESC', 't.rowid', 100, 0, 1, [], 1);   // Fetch jobs that are currently running
+        $cron_job->fetchAll('DESC', 't.rowid', 100, 0, 1, [], 1); // Fetch jobs that are currently running
 
         // Iterate over all jobs in processing (this can't be this job since his state is set to 0 before)
         foreach ($cron_job->lines as $job_line) {
@@ -1450,7 +1450,7 @@ 
             }
 
             // Calling posix_kill with the 0 kill signal will return true if the process is running, false otherwise.
-            if (! posix_kill($job->pid, 0)) {
+            if (!posix_kill($job->pid, 0)) {
                 // Clean processing and pid values
                 $job->processing = 0;
                 $job->pid = null;

htdocs/core/class/doleditor.class.php

Spacing +2 added lines, -2 removed lines

@@ -158,7 +158,7 @@ 
                     define('REQUIRE_CKEDITOR', '1');
                 }
 
-                $skin = getDolGlobalString('FCKEDITOR_SKIN', 'moono-lisa');     // default with ckeditor 4.6 : moono-lisa
+                $skin = getDolGlobalString('FCKEDITOR_SKIN', 'moono-lisa'); // default with ckeditor 4.6 : moono-lisa
 
                 $pluginstodisable = 'elementspath,save,flash,div,anchor';
                 if (!getDolGlobalString('FCKEDITOR_ENABLE_SPECIALCHAR')) {
@@ -259,7 +259,7 @@ 
                                filebrowserImageWindowWidth : \'900\',
                                filebrowserImageWindowHeight : \'500\'';
                 }
-                $out .= '	})' . $morejs;    // end CKEditor.replace
+                $out .= '	})' . $morejs; // end CKEditor.replace
                 // Show the CKEditor javascript object once loaded is ready 'For debug)
                 //$out .= '; CKEDITOR.on(\'instanceReady\', function(ck) { ck.editor.removeMenuItem(\'maximize\'); ck.editor.removeMenuItem(\'Undo\'); ck.editor.removeMenuItem(\'undo\'); console.log(ck.editor); console.log(ck.editor.toolbar[0]); }); ';
                 $out .= '});' . "\n"; // end document.ready

htdocs/core/class/dolreceiptprinter.class.php

Spacing +1 added lines, -1 removed lines

@@ -944,7 +944,7 @@
                         }
                         break;
                     default:
-                        $parameters = array('vals' => $vals[$tplline],'object' => $object,'nbcharactbyline' => $nbcharactbyline);
+                        $parameters = array('vals' => $vals[$tplline], 'object' => $object, 'nbcharactbyline' => $nbcharactbyline);
                         $action = '';
                         $reshook = $hookmanager->executeHooks('sendToPrinterAfter', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
 

htdocs/core/class/conf.class.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
 
     public $liste_limit;
 
-    public $tzuserinputkey = 'tzserver';        // Use 'tzuserrel' to always store date in GMT and show date in time zone of user.
+    public $tzuserinputkey = 'tzserver'; // Use 'tzuserrel' to always store date in GMT and show date in time zone of user.
 
 
     // TODO Remove this part.
@@ -352,7 +352,7 @@ 
                             $value = $_ENV['DOLIBARR_' . $key];
                         }
 
-                        $this->global->$key = dolDecrypt($value);   // decrypt data excrypted with dolibarr_set_const($db, $name, $value)
+                        $this->global->$key = dolDecrypt($value); // decrypt data excrypted with dolibarr_set_const($db, $name, $value)
 
                         if ($value && strpos($key, 'MAIN_MODULE_') === 0) {
                             $reg = array();
@@ -408,7 +408,7 @@ 
                                 // deprecated in php 8.2
                                 //if (version_compare(phpversion(), '8.2') < 0) {
                                 if (!isset($this->$modulename) || !is_object($this->$modulename)) {
-                                    $this->$modulename = new stdClass();    // We need this to use the ->enabled and the ->multidir, ->dir...
+                                    $this->$modulename = new stdClass(); // We need this to use the ->enabled and the ->multidir, ->dir...
                                 }
                                 $this->$modulename->enabled = true; // TODO Remove this
                                 //}
@@ -740,7 +740,7 @@ 
             }
 
             if (!isset($this->global->MAIN_ENABLE_AJAX_TOOLTIP)) {
-                $this->global->MAIN_ENABLE_AJAX_TOOLTIP = 0;    // Not enabled by default (still trouble of persistent tooltip)
+                $this->global->MAIN_ENABLE_AJAX_TOOLTIP = 0; // Not enabled by default (still trouble of persistent tooltip)
             }
 
             // By default, suppliers objects can be linked to all projects

htdocs/core/class/CMailFile.class.php

Spacing +8 added lines, -8 removed lines

@@ -350,7 +350,7 @@ 
                     }
                 }
                 if ($emailtoadd && preg_match('/' . preg_quote($emailtoadd, '/') . '/i', $to)) {
-                    $emailtoadd = '';   // Email already in the "To"
+                    $emailtoadd = ''; // Email already in the "To"
                 }
                 if ($emailtoadd) {
                     $listofemailstoadd[$key] = $emailtoadd;
@@ -1107,8 +1107,8 @@ 
                     }
 
                     $smtperrorcode = 0;
-                    if (! $result) {
-                        $smtperrorcode = $this->smtps->lastretval;  // SMTP error code
+                    if (!$result) {
+                        $smtperrorcode = $this->smtps->lastretval; // SMTP error code
                         dol_syslog("CMailFile::sendfile: mail SMTP error code " . $smtperrorcode, LOG_WARNING);
 
                         if ($smtperrorcode == '421') {  // Try later
@@ -1126,10 +1126,10 @@ 
                         }
                     }
 
-                    $result = $this->smtps->getErrors();    // applicative error code (not SMTP error code)
+                    $result = $this->smtps->getErrors(); // applicative error code (not SMTP error code)
                     if (empty($this->error) && empty($result)) {
                         dol_syslog("CMailFile::sendfile: mail end success", LOG_DEBUG);
-                        $res = true;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                        $res = true; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     } else {
                         if (empty($this->error)) {
                             $this->error = $result;
@@ -1376,7 +1376,7 @@ 
 
         if (@is_writable($dolibarr_main_data_root)) {   // Avoid fatal error on fopen with open_basedir
             $outputfile = $dolibarr_main_data_root . "/dolibarr_mail.log";
-            $fp = fopen($outputfile, "w");  // overwrite
+            $fp = fopen($outputfile, "w"); // overwrite
 
             if ($this->sendmode == 'mail') {
                 fwrite($fp, $this->headers);
@@ -1634,7 +1634,7 @@ 
             // Similar code to forge a text from html is also in smtps.class.php
             $strContentAltText = preg_replace("/<br\s*[^>]*>/", " ", $strContent);
             // TODO We could replace <img ...> with [Filename.ext] like Gmail do.
-            $strContentAltText = html_entity_decode(strip_tags($strContentAltText));    // Remove any HTML tags
+            $strContentAltText = html_entity_decode(strip_tags($strContentAltText)); // Remove any HTML tags
             $strContentAltText = trim(wordwrap($strContentAltText, 75, !getDolGlobalString('MAIN_FIX_FOR_BUGGED_MTA') ? "\r\n" : "\n"));
 
             // Check if html header already in message, if not complete the message
@@ -2019,7 +2019,7 @@ 
                 // We save the image to send in disk
                 $filecontent = $matches[2][$key];
 
-                $cid = 'cid000' . dol_hash($filecontent, 'md5');      // The id must not change if image is same
+                $cid = 'cid000' . dol_hash($filecontent, 'md5'); // The id must not change if image is same
 
                 $destfiletmp = $images_dir . '/' . $cid . '.' . $ext;
 

htdocs/core/class/vcard.class.php

Spacing +2 added lines, -2 removed lines

@@ -135,7 +135,7 @@ 
     {
         // $type = "GIF" | "JPEG"
         //$this->properties["PHOTO;MEDIATYPE=$type;ENCODING=BASE64"] = base64_encode($photo);
-        $this->properties["PHOTO;MEDIATYPE=$type"] = $photo;        // must be url of photo
+        $this->properties["PHOTO;MEDIATYPE=$type"] = $photo; // must be url of photo
         //$this->properties["PHOTO;TYPE=$type;ENCODING=BASE64"] = base64_encode($photo);   // must be content of image
     }
 
@@ -359,7 +359,7 @@ 
     public function getVCard()
     {
         $text = "BEGIN:VCARD\r\n";
-        $text .= "VERSION:4.0\r\n";     // With V4, all encoding are UTF-8
+        $text .= "VERSION:4.0\r\n"; // With V4, all encoding are UTF-8
         //$text.= "VERSION:2.1\r\n";
         foreach ($this->properties as $key => $value) {
             $newkey = preg_replace('/-.*$/', '', $key); // remove suffix -twitter, -facebook, ...

htdocs/core/modules/delivery/doc/pdf_storm.modules.php

Spacing +1 added lines, -1 removed lines

@@ -822,7 +822,7 @@
             if ($this->page_largeur < 210) {
                 $widthrecbox = 84; // To work with US executive format
             }
-            $posy = 42;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $posy = 42; // @phan-suppress-current-line PhanPluginRedundantAssignment
             $posx = $this->page_largeur - $this->marge_droite - $widthrecbox;
             if (getDolGlobalString('MAIN_INVERT_SENDER_RECIPIENT')) {
                 $posx = $this->marge_gauche;