laudeco / bbc-dolibarr-flightlog

class/missions/PilotMissions.php

Spacing +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
         Assert::nullOrString($pilotLastname);
         Assert::nullOrString($pilotFirstname);
 
-        $this->pilotId = (int)$pilotId;
+        $this->pilotId = (int) $pilotId;
         $this->pilotFirstname = $pilotFirstname;
         $this->pilotLastname = $pilotLastname;
 
@@ -59,9 +59,9 @@ 
      * @param int $numberOfFlights
      * @param int $numberOfKilometers
      */
-    public function addQuarter($quarter, $numberOfFlights, $numberOfKilometers){
+    public function addQuarter($quarter, $numberOfFlights, $numberOfKilometers) {
         Assert::integerish($quarter);
-        $quarter = (int)$quarter;
+        $quarter = (int) $quarter;
 
         Assert::keyNotExists($this->quarterMissions, $quarter);
 
@@ -119,13 +119,13 @@ 
      *
      * @return QuarterMission
      */
-    private function getQuarterMission($quarter){
+    private function getQuarterMission($quarter) {
         Assert::integer($quarter);
         Assert::greaterThan($quarter, 0);
         Assert::lessThanEq($quarter, 4);
 
-        if(!isset($this->quarterMissions[$quarter])){
-            return new QuarterMission($quarter,0,0);
+        if (!isset($this->quarterMissions[$quarter])) {
+            return new QuarterMission($quarter, 0, 0);
         }
 
         return $this->quarterMissions[$quarter];
@@ -136,6 +136,6 @@ 
      */
     public function getPilotName()
     {
-        return $this->pilotFirstname.' '.$this->pilotLastname;
+        return $this->pilotFirstname . ' ' . $this->pilotLastname;
     }
 }
\ No newline at end of file

generateExpenseNote.php

Spacing +8 added lines, -8 removed lines

@@ -87,9 +87,9 @@ 
     try {
         $command = new CreateExpenseNoteCommand($year, $quarter, $userValidatorId, $privateNote, $publicNote);
         $commandHandler->__invoke($command);
-    } catch(PeriodNotFinishedException $e){
+    } catch (PeriodNotFinishedException $e) {
         dol_htmloutput_errors("Le quadri n'est pas encore fini !");
-    } catch(\Exception $e){
+    } catch (\Exception $e) {
         dol_syslog($e->getMessage(), LOG_ERR);
         dol_htmloutput_errors('Error : ' . $e->getMessage());
     }
@@ -105,15 +105,15 @@ 
 $form = new Form($db);
 
 $tabLinks = [];
-foreach($flightYears as $currentFlightYear){
+foreach ($flightYears as $currentFlightYear) {
     $tabLinks[] = [
-        DOL_URL_ROOT."/flightlog/generateExpenseNote.php?year=".$currentFlightYear,
+        DOL_URL_ROOT . "/flightlog/generateExpenseNote.php?year=" . $currentFlightYear,
         $currentFlightYear,
-        "tab_".$currentFlightYear
+        "tab_" . $currentFlightYear
     ];
 }
 
-dol_fiche_head($tabLinks, "tab_".$year);
+dol_fiche_head($tabLinks, "tab_" . $year);
 
 ?>
     <form method="POST">
@@ -134,7 +134,7 @@ 
         <select name="quarter" id="field_quarter">
             <option value="1" <?= ($year == $currentYear && $currentQuarter <= 1) ? 'disabled="disabled"' : "" ?>>1</option>
             <option value="2" <?= ($year == $currentYear && $currentQuarter <= 2) ? 'disabled="disabled"' : "" ?>>2</option>
-            <option value="3" <?= ($year == $currentYear && $currentQuarter <= 3) ? 'disabled="disabled"': "" ?>>3</option>
+            <option value="3" <?= ($year == $currentYear && $currentQuarter <= 3) ? 'disabled="disabled"' : "" ?>>3</option>
             <option value="4" <?= ($year == $currentYear && $currentQuarter <= 4) ? 'disabled="disabled"' : "" ?>>4</option>
         </select>
         <br/>
@@ -143,7 +143,7 @@ 
         <label><?= $langs->trans("Validateur de la note de frais")?></label><br/>
         <?php
             $include_users = $object->fetch_users_approver_expensereport();
-            print $form->select_dolusers($user->id,"fk_user_validator",1,"",0,$include_users);
+            print $form->select_dolusers($user->id, "fk_user_validator", 1, "", 0, $include_users);
         ?>
         <br/>
 

readFlights.php

Spacing +2 added lines, -2 removed lines

@@ -266,7 +266,7 @@ 
 print '<td>' . $totalT3 . '</td>';
 print '<td></td>';
 
-print '<td>' . $totalT4. '</td>';
+print '<td>' . $totalT4 . '</td>';
 print '<td></td>';
 
 print '<td>' . $totalT5 . '</td>';
@@ -292,7 +292,7 @@ 
 //table km
 $tauxRemb = isset($conf->global->BBC_FLIGHT_LOG_TAUX_REMB_KM) ? $conf->global->BBC_FLIGHT_LOG_TAUX_REMB_KM : 0;
 $year = GETPOST("year", 'int');
-if(empty($year)){
+if (empty($year)) {
     $year = date('Y');
 }
 

lib/flightLog.lib.php

Doc Comments +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 }
 
 /**
- * @param null $year
+ * @param integer $year
  * @param null $pilotId
  * @param null $quarter
  * @param bool $groupBy

card.php

Spacing +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
 $extralabels = $extrafields->fetch_name_optionals_label($object->table_element);
 
 // Load object
-include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php';  // Must be include, not include_once. Include fetch and fetch_thirdparty but not fetch_optionals
+include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php'; // Must be include, not include_once. Include fetch and fetch_thirdparty but not fetch_optionals
 include DOL_DOCUMENT_ROOT . '/core/actions_dellink.inc.php';
 
 // Initialize technical object to manage hooks of modules. Note that conf->hooks_modules contains array array
@@ -142,7 +142,7 @@ 
 
 $parameters = array();
 $reshook = $hookmanager->executeHooks('doActions', $parameters, $object,
-    $action);    // Note that $action and $object may have been modified by some hooks
+    $action); // Note that $action and $object may have been modified by some hooks
 if ($reshook < 0) {
     setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
 }
@@ -526,7 +526,7 @@ 
     print '<div class="tabsAction">' . "\n";
     $parameters = array();
     $reshook = $hookmanager->executeHooks('addMoreActionsButtons', $parameters, $object,
-        $action);    // Note that $action and $object may have been modified by hook
+        $action); // Note that $action and $object may have been modified by hook
     if ($reshook < 0) {
         setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
     }

Form/SimpleFormRenderer.php

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
      */
     private function renderElement(FormElementInterface $element)
     {
-        switch($element->getType()){
+        switch ($element->getType()) {
             case FormElementInterface::TYPE_TEXTAREA:
                 return sprintf('<textarea name="%s" %s>%s</textarea>', $element->getName(), $this->formatOptions($element->getOptions()), $element->getValue());
 
@@ -97,14 +97,14 @@ 
     {
         $selectElement = sprintf('<select name="%s" >', $element->getName());
 
-        if($element->getValueOptions()){
-            foreach($element->getValueOptions() as $optionValue => $optionLabel){
+        if ($element->getValueOptions()) {
+            foreach ($element->getValueOptions() as $optionValue => $optionLabel) {
                 $selectedAttribute = $optionValue === $element->getValue() ? 'selected' : '';
                 $selectElement .= sprintf('<option value="%s" %s >%s</option>', $optionValue, $selectedAttribute, $optionLabel);
             }
         }
 
-        $selectElement.='</select>';
+        $selectElement .= '</select>';
 
         return $selectElement;
     }

Form/BaseInput.php

Spacing +2 added lines, -2 removed lines

@@ -102,7 +102,7 @@ 
     /**
      * @return $this
      */
-    public function required(){
+    public function required() {
         $this->options['attr']['required'] = 'required';
 
         return $this;
@@ -111,7 +111,7 @@ 
     /**
      * @return $this
      */
-    public function disable(){
+    public function disable() {
         $this->options['attr']['disabled'] = 'disabled';
         return $this;
     }

Form/Form.php

Spacing +11 added lines, -11 removed lines

@@ -78,7 +78,7 @@ 
     /**
      * @inheritDoc
      */
-    public function getOptions(){
+    public function getOptions() {
         return $this->options;
     }
 
@@ -98,11 +98,11 @@ 
      */
     public function validate()
     {
-        if(!$this->validator){
+        if (!$this->validator) {
             return true;
         }
 
-        if(null === $this->object){
+        if (null === $this->object) {
             throw new \InvalidArgumentException('Object not bind');
         }
 
@@ -126,10 +126,10 @@ 
      */
     public function bind($object)
     {
-        foreach($this->elements as $element){
+        foreach ($this->elements as $element) {
             $name = $this->camelCase($element->getName());
-            $methodName = 'get'.$name;
-            if(!method_exists($object, $methodName)){
+            $methodName = 'get' . $name;
+            if (!method_exists($object, $methodName)) {
                 continue;
             }
 
@@ -146,15 +146,15 @@ 
      */
     public function setData(array $data)
     {
-        foreach($data as $fieldName => $currentData){
-            if(!key_exists($fieldName, $this->elements)){
+        foreach ($data as $fieldName => $currentData) {
+            if (!key_exists($fieldName, $this->elements)) {
                 continue;
             }
 
             $this->elements[$fieldName]->setValue($currentData);
 
-            $methodName = 'set'.$this->camelCase($fieldName);
-            if(null === $this->object || !method_exists($this->object, $methodName)){
+            $methodName = 'set' . $this->camelCase($fieldName);
+            if (null === $this->object || !method_exists($this->object, $methodName)) {
                 continue;
             }
 
@@ -177,7 +177,7 @@ 
      */
     public function getElement($elementName)
     {
-        if(!key_exists($elementName, $this->elements)){
+        if (!key_exists($elementName, $this->elements)) {
             throw new \InvalidArgumentException(sprintf('Element %s not found ', $elementName));
         }