alxarafe / alixar

Dolibarr/Code/Salaries/Classes/PaymentSalary.php

Indentation +4 added lines, -4 removed lines

@@ -688,7 +688,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Mise a jour du lien entre le paiement de  salaire et la ligne dans llx_bank generee
      *
@@ -697,7 +697,7 @@ 
      */
     public function update_fk_bank($id_bank)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "UPDATE " . MAIN_DB_PREFIX . "payment_salary SET fk_bank = " . ((int) $id_bank) . " WHERE rowid = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::update_fk_bank", LOG_DEBUG);
@@ -776,7 +776,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -786,7 +786,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs; // TODO Renvoyer le libelle anglais et faire traduction a affichage
 
         $langs->load('compta');

Spacing +1 added lines, -1 removed lines

@@ -294,7 +294,7 @@
             $this->amount = $totalamount;
             $this->total = $totalamount; // deprecated
             $this->db->commit();
-            return $this->id;   // id of the last payment inserted
+            return $this->id; // id of the last payment inserted
         } else {
             $this->error = $this->db->error();
             $this->db->rollback();

Dolibarr/Code/Salaries/Classes/Salary.php

Indentation +12 added lines, -12 removed lines

@@ -448,7 +448,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update link between payment salary and line generate into llx_bank
      *
@@ -457,7 +457,7 @@ 
      */
     public function update_fk_bank($id_bank)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = 'UPDATE ' . MAIN_DB_PREFIX . 'salary SET fk_bank = ' . ((int) $id_bank);
         $sql .= " WHERE rowid = " . ((int) $this->id);
         $result = $this->db->query($sql);
@@ -665,7 +665,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Tag social contribution as paid completely
      *
@@ -676,7 +676,7 @@ 
      */
     public function set_paid($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_paid is deprecated, use setPaid instead", LOG_NOTICE);
         return $this->setPaid($user);
     }
@@ -703,7 +703,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Remove tag paid on social contribution
      *
@@ -712,7 +712,7 @@ 
      */
     public function set_unpaid($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "UPDATE " . MAIN_DB_PREFIX . "salary SET";
         $sql .= " paye = 0";
         $sql .= " WHERE rowid = " . ((int) $this->id);
@@ -740,7 +740,7 @@ 
         return $this->LibStatut($this->paye, $mode, $alreadypaid);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a given status
      *
@@ -751,7 +751,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $alreadypaid = -1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         // Load translation files required by the page
@@ -834,7 +834,7 @@ 
         return $return;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a withdrawal request for a direct debit order or a credit transfer order.
      *  Use the remain to pay excluding all existing open direct debit requests.
@@ -848,7 +848,7 @@ 
      */
     public function demande_prelevement($fuser, $amount = 0, $type = 'direct-debit', $sourcetype = 'salaire', $checkduplicateamongall = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $mysoc;
 
         $error = 0;
@@ -956,7 +956,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a direct debit request or a credit transfer request
      *
@@ -966,7 +966,7 @@ 
      */
     public function demande_prelevement_delete($fuser, $did)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = 'DELETE FROM ' . $this->db->prefix() . 'prelevement_demande';
         $sql .= ' WHERE rowid = ' . ((int) $did);
         $sql .= ' AND traite = 0';

Spacing +1 added lines, -1 removed lines

@@ -619,7 +619,7 @@
                 } elseif ($multicurrency) {
                     //$this->sumpayed_multicurrency = $obj->multicurrency_amount;
                     //return (float) $obj->multicurrency_amount;
-                    return -1;      // Not yet supported
+                    return -1; // Not yet supported
                 } else {
                     //$this->sumpayed = $obj->amount;
                     return (float) $obj->amount;

Dolibarr/Code/Salaries/Api/Salaries.php

Indentation +2 added lines, -2 removed lines

@@ -455,7 +455,7 @@ 
         return $paymentsalary;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -464,7 +464,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->rowid);

Braces +3 added lines, -1 removed lines

@@ -445,7 +445,9 @@
     {
         $paymentsalary = array();
         $fields = Salaries::$FIELDSPAYMENT;
-        if (isModEnabled("bank")) array_push($fields, "accountid");
+        if (isModEnabled("bank")) {
+            array_push($fields, "accountid");
+        }
         foreach ($fields as $field) {
             if (!isset($data[$field])) {
                 throw new RestException(400, "$field field missing");

Dolibarr/Code/Zapier/Classes/Hook.php

Spacing +2 added lines, -2 removed lines

@@ -403,7 +403,7 @@ 
         $sql .= ' t.rowid';
         // TODO Get all fields
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.entity = ' . ((int)$conf->entity);
+        $sql .= ' WHERE t.entity = ' . ((int) $conf->entity);
 
         // Manage filter
         $errormessage = '';
@@ -611,7 +611,7 @@ 
         $sql = 'SELECT rowid, date_creation as datec, tms as datem,';
         $sql .= ' fk_user_creat, fk_user_modif';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.rowid = ' . ((int)$id);
+        $sql .= ' WHERE t.rowid = ' . ((int) $id);
         $result = $this->db->query($sql);
         if ($result) {
             if ($this->db->num_rows($result)) {

Dolibarr/Code/Zapier/Api/Zapier.php

Indentation +27 added lines, -27 removed lines

@@ -295,7 +295,7 @@ 
         );
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -304,30 +304,30 @@ 
      */
     public function _cleanObjectDatas($object)
     {
-		// phpcs:disable
-		$object = parent::_cleanObjectDatas($object);
-
-		return $object;
-	}
-
-	/**
-	 * Validate fields before create or update object
-	 *
-	 * @param   array       $data       Array of data to validate
-	 * @param   array       $fields     Array of fields needed
-	 * @return  array
-	 *
-	 * @throws  RestException
-	 */
-	private function validate($data, $fields)
-	{
-		$hook = array();
-		foreach ($fields as $field) {
-			if (!isset($data[$field])) {
-				throw new RestException(400, $field." field missing");
-			}
-			$hook[$field] = $data[$field];
-		}
-		return $hook;
-	}
+        // phpcs:disable
+        $object = parent::_cleanObjectDatas($object);
+
+        return $object;
+    }
+
+    /**
+     * Validate fields before create or update object
+     *
+     * @param   array       $data       Array of data to validate
+     * @param   array       $fields     Array of fields needed
+     * @return  array
+     *
+     * @throws  RestException
+     */
+    private function validate($data, $fields)
+    {
+        $hook = array();
+        foreach ($fields as $field) {
+            if (!isset($data[$field])) {
+                throw new RestException(400, $field." field missing");
+            }
+            $hook[$field] = $data[$field];
+        }
+        return $hook;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -324,7 +324,7 @@
 		$hook = array();
 		foreach ($fields as $field) {
 			if (!isset($data[$field])) {
-				throw new RestException(400, $field." field missing");
+				throw new RestException(400, $field . " field missing");
 			}
 			$hook[$field] = $data[$field];
 		}

Dolibarr/Code/Recruitement/Classes/RecruitmentJobPosition.php

Indentation +2 added lines, -2 removed lines

@@ -893,7 +893,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -903,7 +903,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("recruitment");

Dolibarr/Code/Recruitement/Classes/RecruitmentCandidature.php

Indentation +2 added lines, -2 removed lines

@@ -816,7 +816,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -826,7 +826,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("recruitment@recruitment");

Dolibarr/Code/Recruitement/Api/Recruitments.php

Indentation +11 added lines, -11 removed lines

@@ -347,15 +347,15 @@ 
     }
 
     /**
-    * Create candidature object
-    *
-    * @param array $request_data   Request datas
-    * @return int  ID of candidature
-    *
-    * @throws RestException
-    *
-    * @url  POST candidature/
-    */
+     * Create candidature object
+     *
+     * @param array $request_data   Request datas
+     * @return int  ID of candidature
+     *
+     * @throws RestException
+     *
+     * @url  POST candidature/
+     */
     public function postCandidature($request_data = null)
     {
         if (!DolibarrApiAccess::$user->hasRight('recruitment', 'recruitmentjobposition', 'write')) {
@@ -556,7 +556,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -565,7 +565,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->rowid);

Dolibarr/Code/Resource/Classes/FormResource.php

Indentation +8 added lines, -8 removed lines

@@ -65,7 +65,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output html form to select a resource
      *
@@ -85,7 +85,7 @@ 
      */
     public function select_resource_list($selected = 0, $htmlname = 'fk_resource', array $filter = [], $showempty = 0, $showtype = 0, $forcecombo = 0, $event = [], $filterkey = [], $outputmode = 0, $limit = 20, $morecss = 'minwidth100', $multiple = false)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $out = '';
@@ -166,7 +166,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return html list of tickets type
      *
@@ -181,7 +181,7 @@ 
      */
     public function select_types_resource($selected = '', $htmlname = 'type_resource', $filtertype = '', $format = 0, $empty = 0, $noadmininfo = 0, $maxlength = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $user;
 
         $resourcestat = new Dolresource($this->db);
@@ -244,7 +244,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return a select list with zip codes and their town
      *
@@ -259,7 +259,7 @@ 
      */
     public function select_ziptown($selected = '', $htmlname = 'zipcode', $fields = array(), $fieldsize = 0, $disableautocomplete = 0, $moreattrib = '', $morecss = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $out = '';
@@ -278,7 +278,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Returns the drop-down list of departments/provinces/cantons for all countries or for a given country.
      *   In the case of an all-country list, the display breaks on the country.
@@ -294,7 +294,7 @@ 
      */
     public function select_state($selected = 0, $country_codeid = 0, $htmlname = 'state_id', $morecss = 'maxwidth200onsmartphone  minwidth300')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         dol_syslog(get_class($this) . "::select_departement selected=" . $selected . ", country_codeid=" . $country_codeid, LOG_DEBUG);