laudeco / bbc-dolibarr-flightlog

facture.php

Spacing +4 added lines, -4 removed lines

@@ -127,12 +127,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);
+    try {
+        $command = new CreateFlightBillCommand($flight->getId(), $modeReglement, $conditionReglement, $documentModel, $type, $publicNote, $privateNote, $bankAccount);
         $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

@@ -131,7 +131,7 @@ 
         $command = new CreateFlightBillCommand($flight->getId(), $modeReglement, $conditionReglement, $documentModel, $type, $publicNote, $privateNote,$bankAccount);
         $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');
     }
@@ -267,9 +267,12 @@ 
 
         <?php if (!$flightProduct) : ?>
             <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>

command/CreateFlightBillCommandHandlerFactory.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
-require_once __DIR__.'/CreateFlightBillCommandHandler.php';
-require_once __DIR__.'/CreateFlightBillTransactionalCommandHandler.php';
+require_once __DIR__ . '/CreateFlightBillCommandHandler.php';
+require_once __DIR__ . '/CreateFlightBillTransactionalCommandHandler.php';
 
 /**
  * @author Laurent De Coninck <lau.deconinck@gmail.com>
@@ -11,7 +11,7 @@ 
     /**
      * @return CreateFlightBillCommandHandler
      */
-    public static function factory($db, $conf, $user, $langs){
+    public static function factory($db, $conf, $user, $langs) {
         return new CreateFlightBillTransactionalCommandHandler($db, $conf, $user, $langs);
     }
 

class/bbcvols.class.php

Doc Comments +3 added lines, -3 removed lines

@@ -86,7 +86,7 @@ 
     private $pilot;
 
     /**
-     * @return int
+     * @return string
      */
     public function getIdBBCVols()
     {
@@ -94,7 +94,7 @@ 
     }
 
     /**
-     * @return int
+     * @return string
      */
     public function getId()
     {
@@ -951,7 +951,7 @@ 
     }
 
     /**
-     * @return int
+     * @return string
      */
     public function getKilometers(){
         return $this->kilometers;

Spacing +8 added lines, -8 removed lines

@@ -149,10 +149,10 @@ 
             $this->lieuA = trim($this->lieuA);
         }
         if (isset($this->heureD)) {
-            $this->heureD = trim($this->heureD).'00';
+            $this->heureD = trim($this->heureD) . '00';
         }
         if (isset($this->heureA)) {
-            $this->heureA = trim($this->heureA).'00';
+            $this->heureA = trim($this->heureA) . '00';
         }
         if (isset($this->BBC_ballons_idBBC_ballons)) {
             $this->BBC_ballons_idBBC_ballons = trim($this->BBC_ballons_idBBC_ballons);
@@ -891,7 +891,7 @@ 
      */
     public function hasPax()
     {
-        return (int)$this->nbrPax > 0;
+        return (int) $this->nbrPax > 0;
     }
 
     /**
@@ -917,14 +917,14 @@ 
     /**
      * @return int
      */
-    public function getAmountReceived(){
+    public function getAmountReceived() {
         return $this->cost;
     }
 
     /**
      * @return int
      */
-    public function getAmountPerPassenger(){
+    public function getAmountPerPassenger() {
         return $this->cost / $this->nbrPax;
     }
 
@@ -939,21 +939,21 @@ 
     /**
      * @return boolean
      */
-    public function hasKilometers(){
+    public function hasKilometers() {
         return !empty($this->kilometers);
     }
 
     /**
      * @return boolean
      */
-    public function hasKilometersDescription(){
+    public function hasKilometersDescription() {
         return !empty(trim($this->justif_kilometers));
     }
 
     /**
      * @return int
      */
-    public function getKilometers(){
+    public function getKilometers() {
         return $this->kilometers;
     }
 }

validators/FlightValidator.php

Spacing +7 added lines, -7 removed lines

@@ -47,16 +47,16 @@ 
             $this->addError('heures', 'L\'heure de depart est plus grande  que l\'heure d\'arrivee');
         }
 
-        if(empty($vol->lieuD)){
+        if (empty($vol->lieuD)) {
             $this->addError('lieuD', 'Le lieu de départ est vide');
         }
 
-        if(empty($vol->lieuA)){
+        if (empty($vol->lieuA)) {
             $this->addError('lieuA', 'Le lieu d\'arrivée est vide');
         }
 
         // PAX
-        if (!is_numeric($vol->nbrPax) || (int)$vol->nbrPax < 0) {
+        if (!is_numeric($vol->nbrPax) || (int) $vol->nbrPax < 0) {
             $this->addError('nbrPax', 'Erreur le nombre de passager est un nombre négatif.');
         }
 
@@ -74,18 +74,18 @@ 
                 'Erreur ce type de vol doit être payant, mais personne n\'a été signalé comme recepteur d\'argent.');
         }
 
-        if($vol->getFlightType()->isBillingRequired() && ($vol->getAmountPerPassenger()) < $this->getMinPrice()){
+        if ($vol->getFlightType()->isBillingRequired() && ($vol->getAmountPerPassenger()) < $this->getMinPrice()) {
             $this->addError('cost',
                 sprintf('Le montant demandé pour ce vol n\'est pas suffisant un minimum de %s euros est demandé', $this->getMinPrice()));
         }
 
         //Kilometers
-        if($vol->hasKilometers() && !$vol->getKilometers() > 0){
+        if ($vol->hasKilometers() && !$vol->getKilometers() > 0) {
             $this->addError('kilometers',
                 'Les kilometres doivent être un nombre positif');
         }
 
-        if($vol->hasKilometers() && !$vol->hasKilometersDescription()){
+        if ($vol->hasKilometers() && !$vol->hasKilometersDescription()) {
             $this->addError('justif_kilometers',
                 'Vous essayez d\'encoder des kilometres sans justificatif.');
         }
@@ -128,7 +128,7 @@ 
      *
      * @return int
      */
-    public function getMinPrice(){
+    public function getMinPrice() {
         if ($this->defaultService->price_base_type == 'TTC') {
             return $this->defaultService->price_min;
         }