laudeco / bbc-dolibarr-flightlog

facture.php

Spacing +4 added lines, -4 removed lines

@@ -124,12 +124,12 @@
  * Put here all code to do according to value of "action" parameter
  */
 if ($action == EXPENSE_REPORT_GENERATOR_ACTION_GENERATE) {
-    try{
-        $command = new CreateFlightBillCommand($flight->getId(), $modeReglement, $conditionReglement, $documentModel, $type, $publicNote, $privateNote,$bankAccount, $nbrPax);
+    try {
+        $command = new CreateFlightBillCommand($flight->getId(), $modeReglement, $conditionReglement, $documentModel, $type, $publicNote, $privateNote, $bankAccount, $nbrPax);
         $handler->handle($command);
         Header("Location: card.php?id=" . $flight->getId());
-    }catch (\Exception $e){
-        dol_syslog($e->getMessage(),LOG_ERR);
+    } catch (\Exception $e) {
+        dol_syslog($e->getMessage(), LOG_ERR);
         dol_htmloutput_mesg("Facture non créée", '', 'error');
     }
 }

Braces +6 added lines, -3 removed lines

@@ -128,7 +128,7 @@ 
         $command = new CreateFlightBillCommand($flight->getId(), $modeReglement, $conditionReglement, $documentModel, $type, $publicNote, $privateNote,$bankAccount, $nbrPax);
         $handler->handle($command);
         Header("Location: card.php?id=" . $flight->getId());
-    }catch (\Exception $e){
+    } catch (\Exception $e){
         dol_syslog($e->getMessage(),LOG_ERR);
         dol_htmloutput_mesg("Facture non créée", '', 'error');
     }
@@ -269,9 +269,12 @@ 
 
         <?php if (!$flightProduct || !$flight->hasReceiver()) : ?>
             <a class="butActionRefused" href="#">Générer</a>
-        <?php else: ?>
+        <?php else {
+    : ?>
             <button class="butAction" type="submit">Générer</button>
-        <?php endif; ?>
+        <?php endif;
+}
+?>
 
         <a class="butAction" href="<?php echo DOL_URL_ROOT . '/flightlog/card.php?id=' . $flight->id; ?>">Retour au
             vol</a>

class/missions/FlightMission.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
         Assert::string($kilometersComment);
         Assert::integerish($numberOfKilometers);
 
-        $this->id = (int)$id;
+        $this->id = (int) $id;
         $this->startPoint = $startPoint;
         $this->endPoint = $endPoint;
         $this->kilometersComment = $kilometersComment;

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

query/FlightForQuarterAndPilotQueryHandler.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
                 while ($i < $num) {
                     $flight = $this->db->fetch_object($resql);
                     if ($flight) {
-                        $flights[] = new FlightMission($flight->rowid, $flight->lieuD,$flight->lieuA, $flight->justif_kilometers, $flight->kilometers, new \DateTime($flight->date));
+                        $flights[] = new FlightMission($flight->rowid, $flight->lieuD, $flight->lieuA, $flight->justif_kilometers, $flight->kilometers, new \DateTime($flight->date));
                     }
                     $i++;
                 }

query/GetPilotsWithMissionsQuery.php

Spacing +3 added lines, -3 removed lines

@@ -33,14 +33,14 @@
         Assert::greaterThan($year, 0);
 
         Assert::nullOrIntegerish($quarter);
-        if($quarter !== null){
+        if ($quarter !== null) {
             Assert::greaterThan($quarter, 0);
             Assert::lessThanEq($quarter, 4);
 
         }
 
-        $this->quarter = (int)$quarter;
-        $this->year = (int)$year;
+        $this->quarter = (int) $quarter;
+        $this->year = (int) $year;
     }
 
     /**

query/FlightForQuarterAndPilotQuery.php

Spacing +3 added lines, -3 removed lines

@@ -39,9 +39,9 @@
         Assert::integerish($quarter);
         Assert::integerish($year);
 
-        $this->pilotId = (int)$pilotId;
-        $this->quarter = (int)$quarter;
-        $this->year = (int)$year;
+        $this->pilotId = (int) $pilotId;
+        $this->quarter = (int) $quarter;
+        $this->year = (int) $year;
     }
 
     /**

command/CreateExpenseNoteCommandHandler.php

Spacing +3 added lines, -3 removed lines

@@ -111,7 +111,7 @@ 
                 $isFlightInError = $this->addFlight($currentFlightForQuarter, $expenseNote) || $isFlightInError;
             }
 
-            if(!$isFlightInError){
+            if (!$isFlightInError) {
                 $expenseNote->fetch($expenseNoteId);
                 $expenseNote->setValidate($this->user);
                 $expenseNote->setApproved($this->user);
@@ -120,7 +120,7 @@ 
                 $expenseNote->setDocModel($this->user, "standard");
                 $error = $expenseNote->generateDocument($expenseNote->modelpdf, $this->langs) <= 0;
 
-                if(!$error){
+                if (!$error) {
                     dol_htmloutput_mesg(sprintf("Notes de frais crée pour %s", $currentMission->getPilotName()));
                     continue;
                 }
@@ -194,7 +194,7 @@ 
 
         $isError = $object_ligne->insert() <= 0;
 
-        if(!$isError){
+        if (!$isError) {
             $expenseReport->add_object_linked(self::FLIGHT_ELEMENT, $currentFlightForQuarter->getId());
         }
 

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');
 }