laudeco / bbc-dolibarr-flightlog

card_tab_damage.php

Spacing +8 added lines, -8 removed lines

@@ -49,7 +49,7 @@ 
 
 $id = GETPOST('id', 'int') ?: GETPOST('idBBC_vols', 'int');
 $action = GETPOST('action', 'alpha');
-$permissiondellink=$user->rights->flightlog->vol->financial;
+$permissiondellink = $user->rights->flightlog->vol->financial;
 
 $object = new Bbcvols($db);
 
@@ -57,8 +57,8 @@ 
 $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  // Must be include, not include_once. Include fetch and fetch_thirdparty but not fetch_optionals
-include DOL_DOCUMENT_ROOT.'/core/actions_dellink.inc.php';
+include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php'; // Must be include, not include_once  // 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
 $hookmanager->initHooks(array('bbcvols'));
@@ -85,12 +85,12 @@ 
         FLIGHTLOG_ACTION_HANDLE_ADD_DAMAGE => [AddFlightDamageController::class, 'view'],
     ];
 
-    if(isset($routes[$action])){
+    if (isset($routes[$action])) {
         $controllerName = $routes[$action][0];
         $actionName = $routes[$action][1];
 
         call_user_func([new $controllerName($db), $actionName]);
-    }else{
+    } else {
         echo 'Route non trouvée.';
     }
 ?>
@@ -102,13 +102,13 @@ 
 // Buttons
 print '<div class="tabsAction">' . "\n";
 
-if($user->rights->flightlog->vol->financial){
-    print '<div class="inline-block divButAction"><a class="butAction" href="' . DOL_URL_ROOT . '/flightlog/card_tab_damage.php?id=' . $object->id.'&action='.FLIGHTLOG_ACTION_ADD_DAMAGE.'">' . $langs->trans("Ajouter") . '</a></div>' . "\n";
+if ($user->rights->flightlog->vol->financial) {
+    print '<div class="inline-block divButAction"><a class="butAction" href="' . DOL_URL_ROOT . '/flightlog/card_tab_damage.php?id=' . $object->id . '&action=' . FLIGHTLOG_ACTION_ADD_DAMAGE . '">' . $langs->trans("Ajouter") . '</a></div>' . "\n";
 }
 
 print '</div>' . "\n";
 
-if($user->rights->flightlog->vol->financial){
+if ($user->rights->flightlog->vol->financial) {
     print '<div class="fichecenter"><div class="fichehalfleft">';
     $form->showLinkedObjectBlock($object);
     print '</div></div>';

Infrastructure/Damage/Query/Repository/GetDamageQueryRepository.php

Spacing +12 added lines, -12 removed lines

@@ -22,7 +22,7 @@ 
         $this->db = $db;
     }
 
-    private function element($objectType, $rowid){
+    private function element($objectType, $rowid) {
         $element = null;
         $subelement = null;
 
@@ -33,7 +33,7 @@ 
             $subelement = $regs[2];
         }
 
-        $classpath = $element.'/class';
+        $classpath = $element . '/class';
 
         if ($objectType == 'facture') {
             $classpath = 'compta/facture/class';
@@ -85,7 +85,7 @@ 
             $classfile = 'subscription'; $classname = 'Subscription';
         }
 
-        dol_include_once('/'.$classpath.'/'.$classfile.'.class.php');
+        dol_include_once('/' . $classpath . '/' . $classfile . '.class.php');
         if (!class_exists($classname))
         {
             return null;
@@ -104,14 +104,14 @@ 
      *
      * @throws \Exception
      */
-    public function query($damageId){
+    public function query($damageId) {
 
         $sql = 'SELECT damage.rowid as id, damage.amount, author.login as author_name, damage.billed as invoiced, damage.flight_id as flight_id';
-        $sql.= ', element.rowid as element_id ,element.fk_target as element_fk_target, element.targettype as element_target_type';
-        $sql.=' FROM '.MAIN_DB_PREFIX.'bbc_flight_damages as damage';
-        $sql.=' INNER JOIN '.MAIN_DB_PREFIX.'user as author ON author.rowid = damage.author_id';
-        $sql.=' LEFT JOIN '.MAIN_DB_PREFIX.'element_element as element ON (element.fk_source = damage.rowid AND element.sourcetype = "flightlog_damage")';
-        $sql.=' WHERE damage.rowid = '.$damageId;
+        $sql .= ', element.rowid as element_id ,element.fk_target as element_fk_target, element.targettype as element_target_type';
+        $sql .= ' FROM ' . MAIN_DB_PREFIX . 'bbc_flight_damages as damage';
+        $sql .= ' INNER JOIN ' . MAIN_DB_PREFIX . 'user as author ON author.rowid = damage.author_id';
+        $sql .= ' LEFT JOIN ' . MAIN_DB_PREFIX . 'element_element as element ON (element.fk_source = damage.rowid AND element.sourcetype = "flightlog_damage")';
+        $sql .= ' WHERE damage.rowid = ' . $damageId;
 
         $resql = $this->db->query($sql);
         if (!$resql) {
@@ -125,13 +125,13 @@ 
 
         /** @var Damage $damage */
         $damage = null;
-        for($i = 0; $i < $num ; $i++) {
+        for ($i = 0; $i < $num; $i++) {
             $properties = $this->db->fetch_array($resql);
-            if(!$damage){
+            if (!$damage) {
                 $damage = Damage::fromArray($properties);
             }
 
-            if(!isset($properties['element_id'])){
+            if (!isset($properties['element_id'])) {
                 continue;
             }
 

Braces +13 added lines, -26 removed lines

@@ -37,29 +37,21 @@ 
 
         if ($objectType == 'facture') {
             $classpath = 'compta/facture/class';
-        }
-        elseif ($objectType == 'facturerec') {
+        } elseif ($objectType == 'facturerec') {
             $classpath = 'compta/facture/class';
-        }
-        elseif ($objectType == 'propal') {
+        } elseif ($objectType == 'propal') {
             $classpath = 'comm/propal/class';
-        }
-        elseif ($objectType == 'supplier_proposal') {
+        } elseif ($objectType == 'supplier_proposal') {
             $classpath = 'supplier_proposal/class';
-        }
-        elseif ($objectType == 'shipping') {
+        } elseif ($objectType == 'shipping') {
             $classpath = 'expedition/class'; $subelement = 'expedition';
-        }
-        elseif ($objectType == 'delivery') {
+        } elseif ($objectType == 'delivery') {
             $classpath = 'livraison/class'; $subelement = 'livraison';
-        }
-        elseif ($objectType == 'invoice_supplier' || $objectType == 'order_supplier') {
+        } elseif ($objectType == 'invoice_supplier' || $objectType == 'order_supplier') {
             $classpath = 'fourn/class';
-        }
-        elseif ($objectType == 'fichinter') {
+        } elseif ($objectType == 'fichinter') {
             $classpath = 'fichinter/class'; $subelement = 'fichinter';
-        }
-        elseif ($objectType == 'subscription') {
+        } elseif ($objectType == 'subscription') {
             $classpath = 'adherents/class';
         }
 
@@ -68,20 +60,15 @@ 
 
         if ($objectType == 'order') {
             $classfile = 'commande'; $classname = 'Commande';
-        }
-        elseif ($objectType == 'invoice_supplier') {
+        } elseif ($objectType == 'invoice_supplier') {
             $classfile = 'fournisseur.facture'; $classname = 'FactureFournisseur';
-        }
-        elseif ($objectType == 'order_supplier') {
+        } elseif ($objectType == 'order_supplier') {
             $classfile = 'fournisseur.commande'; $classname = 'CommandeFournisseur';
-        }
-        elseif ($objectType == 'supplier_proposal') {
+        } elseif ($objectType == 'supplier_proposal') {
             $classfile = 'supplier_proposal'; $classname = 'SupplierProposal';
-        }
-        elseif ($objectType == 'facturerec') {
+        } elseif ($objectType == 'facturerec') {
             $classfile = 'facture-rec'; $classname = 'FactureRec';
-        }
-        elseif ($objectType == 'subscription') {
+        } elseif ($objectType == 'subscription') {
             $classfile = 'subscription'; $classname = 'Subscription';
         }
 

Infrastructure/Damage/Repository/FlightDamageRepository.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
             'author_id' => $flightDamage->getAuthor()->getId()
         ];
 
-        if($flightDamage->getId()){
+        if ($flightDamage->getId()) {
             $this->update($flightDamage->getId()->getId(), $fields);
             return $flightDamage->getId()->getId();
         }
@@ -50,10 +50,10 @@ 
      *
      * @throws \Exception
      */
-    public function getById(DamageId $id){
+    public function getById(DamageId $id) {
         $damage = $this->get($id->getId());
 
-        if(null === $damage){
+        if (null === $damage) {
             throw new \Exception('Damage not found');
         }
 

Infrastructure/Common/Routes/RouteManager.php

Spacing +5 added lines, -5 removed lines

@@ -29,8 +29,8 @@ 
      *
      * @throws \Exception
      */
-    public function add(Route $route){
-        if(isset($this->routes[$route->getName()])){
+    public function add(Route $route) {
+        if (isset($this->routes[$route->getName()])) {
             throw new \Exception('Route exists');
         }
 
@@ -42,8 +42,8 @@ 
      *
      * @throws \Exception
      */
-    public function load(array $routes){
-        foreach($routes as $currentRoute){
+    public function load(array $routes) {
+        foreach ($routes as $currentRoute) {
             $this->add($currentRoute);
         }
     }
@@ -55,7 +55,7 @@ 
      */
     public function __invoke($name)
     {
-        if(!isset($this->routes[$name])){
+        if (!isset($this->routes[$name])) {
             throw new \Exception('Route not found');
         }