laudeco / bbc-dolibarr-flightlog

Http/Web/templates/flight_damage/form.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     <?php echo $renderer->render($form->getElement('flight_id')); ?>
     <?php echo $renderer->render($form->getElement('token')); ?>
 
-    <?php if($form->has('amount')): ?>
+    <?php if ($form->has('amount')): ?>
         <section class="form-section">
             <h1 class="form-section-title"><?php echo $langs->trans('Ajout de dégâts'); ?></h1>
             <table class="border" width="50%">

Http/Web/templates/flight_damage/view.php

Spacing +6 added lines, -6 removed lines

@@ -16,21 +16,21 @@
         <td colspan="2">&nbsp;</td>
     </tr>
     <?php $total = 0; ?>
-    <?php foreach($damages as $currentDamage):?>
+    <?php foreach ($damages as $currentDamage):?>
         <?php $total += $currentDamage->getAmount(); ?>
         <tr>
-            <td><?php echo $currentDamage->getAuthorName();?></td>
-            <td><?php echo $currentDamage->getLabel();?></td>
-            <td><?php echo $currentDamage->getAmount();?>€</td>
+            <td><?php echo $currentDamage->getAuthorName(); ?></td>
+            <td><?php echo $currentDamage->getLabel(); ?></td>
+            <td><?php echo $currentDamage->getAmount(); ?>€</td>
             <td>
-                <?php if($currentDamage->isInvoiced()): ?>
+                <?php if ($currentDamage->isInvoiced()): ?>
                     <i class="fas fa-check"></i>
                 <?php else: ?>
                     <i class="fas fa-times"></i>
                 <?php endif; ?>
             </td>
             <td>
-                <a href="<?php print sprintf('%s/flightlog/index.php?r=get_one_damage&id=%s', DOL_URL_ROOT, $currentDamage->getId());?>"><i class="fas fa-link"></i></a>
+                <a href="<?php print sprintf('%s/flightlog/index.php?r=get_one_damage&id=%s', DOL_URL_ROOT, $currentDamage->getId()); ?>"><i class="fas fa-link"></i></a>
             </td>
         </tr>
     <?php endforeach; ?>

readFlights.php

Spacing +5 added lines, -5 removed lines

@@ -189,7 +189,7 @@ 
     print '<td>' . $pilot->getCountForType('orga_T6')->getCount() . '</td>';
     print '<td>' . $pilot->getCountForType('orga_T6')->getCost()->getValue() . '</td>';
 
-    print sprintf('<td class="%s">', $pilot->getFlightBonus()->getValue() === 0?'text-muted':'text-bold'). $pilot->getFlightBonus()->getValue() . ' pts</td>';
+    print sprintf('<td class="%s">', $pilot->getFlightBonus()->getValue() === 0 ? 'text-muted' : 'text-bold') . $pilot->getFlightBonus()->getValue() . ' pts</td>';
 
     print '<td>' . $pilot->getCountForType('3')->getCount() . '</td>';
     print '<td>' . price($pilot->getCountForType('3')->getCost()->getValue()) . '€</td>';
@@ -208,8 +208,8 @@ 
     print '<td>' . price($pilot->damageCost()->getValue()) . '€</td>';
     print '<td>' . price($pilot->invoicedDamageCost()->getValue()) . '€</td>';
 
-    print sprintf('<td class="%s">', $pilot->getFlightsCost()->getValue() === 0?'text-muted':'text-bold'). price($pilot->getFlightsCost()->getValue()) . '€ </td>';
-    print sprintf('<td class="%s">', $pilot->isBillable(FlightBonus::zero())?'text-bold':'text-muted'). price($pilot->getTotalBill()->getValue()) . '€</td>';
+    print sprintf('<td class="%s">', $pilot->getFlightsCost()->getValue() === 0 ? 'text-muted' : 'text-bold') . price($pilot->getFlightsCost()->getValue()) . '€ </td>';
+    print sprintf('<td class="%s">', $pilot->isBillable(FlightBonus::zero()) ? 'text-bold' : 'text-muted') . price($pilot->getTotalBill()->getValue()) . '€</td>';
     print '</tr>';
 }
 
@@ -234,7 +234,7 @@ 
 print '<td>' . $totalT3 . '</td>';
 print '<td></td>';
 
-print '<td>' . $totalT4. '</td>';
+print '<td>' . $totalT4 . '</td>';
 print '<td></td>';
 
 print '<td>' . $totalT5 . '</td>';
@@ -263,7 +263,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');
 }
 

Infrastructure/Damage/Query/Repository/GetDamagesForFlightQueryRepository.php

Spacing +4 added lines, -4 removed lines

@@ -31,15 +31,15 @@
     public function __invoke($flightId)
     {
         $sql = 'SELECT damage.author_id as author_id, damage.rowid as id, damage.amount, damage.label, CONCAT(author.firstname, " " , author.lastname) as author_name, damage.billed as invoiced';
-        $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.=' WHERE damage.flight_id = '.$this->db->escape($flightId);
+        $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 .= ' WHERE damage.flight_id = ' . $this->db->escape($flightId);
 
         $resql = $this->db->query($sql);
         if ($resql) {
             $num = $this->db->num_rows($resql);
             if ($num) {
-                for($i = 0; $i < $num ; $i++) {
+                for ($i = 0; $i < $num; $i++) {
                     $properties = $this->db->fetch_array($resql);
                     yield Damage::fromArray($properties);
                 }

Infrastructure/Damage/Repository/FlightDamageRepository.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
             'label' => $flightDamage->getLabel()->value(),
         ];
 
-        if($flightDamage->getId()){
+        if ($flightDamage->getId()) {
             $this->update($flightDamage->getId()->getId(), $fields);
             return $flightDamage->getId()->getId();
         }
@@ -52,10 +52,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');
         }
 

Application/Damage/ViewModel/Damage.php

Spacing +2 added lines, -2 removed lines

@@ -72,14 +72,14 @@
      *
      * @return Damage
      */
-    public static function fromArray(array $properties){
+    public static function fromArray(array $properties) {
         $authorId = $properties['author_id'];
         $author = $properties['author_name'];
         $label = $properties['label'];
         $amount = $properties['amount'];
         $id = $properties['id'];
         $flightId = isset($properties['flight_id']) ? $properties['flight_id'] : null;
-        $invoiced = (bool)$properties['invoiced'];
+        $invoiced = (bool) $properties['invoiced'];
 
         return new self($authorId, $author, $amount, $id, $invoiced, $flightId, $label);
     }

Application/Damage/Command/CreateDamageCommandHandler.php

Spacing +6 added lines, -6 removed lines

@@ -42,7 +42,7 @@ 
     public function __invoke(CreateDamageCommand $command)
     {
         $damage = FlightDamage::waiting(new DamageAmount($command->getAmount()), AuthorId::create($command->getAuthorId()), new DamageLabel($command->getLabel()));
-        if(null !== $command->getFlightId()){
+        if (null !== $command->getFlightId()) {
             $damage = FlightDamage::damage(FlightId::create($command->getFlightId()), new DamageLabel($command->getLabel()), new DamageAmount($command->getAmount()), AuthorId::create($command->getAuthorId()));
         }
 
@@ -60,7 +60,7 @@ 
      */
     private function linkSupplierInvoice($damageId, $invoiceId)
     {
-        if($invoiceId <= 0){
+        if ($invoiceId <= 0) {
             return;
         }
 
@@ -81,8 +81,8 @@ 
      * @param int $targetId
      * @param string $targetType
      */
-    private function insertLinks($damageId, $targetId, $targetType){
-        $sql = "INSERT INTO ".MAIN_DB_PREFIX."element_element (";
+    private function insertLinks($damageId, $targetId, $targetType) {
+        $sql = "INSERT INTO " . MAIN_DB_PREFIX . "element_element (";
         $sql .= "fk_source";
         $sql .= ", sourcetype";
         $sql .= ", fk_target";
@@ -90,8 +90,8 @@ 
         $sql .= ") VALUES (";
         $sql .= $damageId;
         $sql .= ", 'flightlog_damage'";
-        $sql .= ", ".$targetId;
-        $sql .= ", '".$targetType."'";
+        $sql .= ", " . $targetId;
+        $sql .= ", '" . $targetType . "'";
         $sql .= ")";
 
         if ($this->db->query($sql))

class/Damage/ValueObject/DamageLabel.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         $this->label = $label;
     }
 
-    public function value(){
+    public function value() {
         return $this->label;
     }
 

class/flight/Pilot.php

Spacing +11 added lines, -11 removed lines

@@ -106,11 +106,11 @@ 
         return new Pilot($this->name, $this->id, $types);
     }
 
-    public function addDamage(FlightDamageCount $damage){
+    public function addDamage(FlightDamageCount $damage) {
         $this->damages[] = $damage;
     }
 
-    public function addInvoicedDamage(FlightInvoicedDamageCount $damage){
+    public function addInvoicedDamage(FlightInvoicedDamageCount $damage) {
         $this->damages[] = $damage;
     }
 
@@ -161,21 +161,21 @@ 
         return $flightsCost;
     }
 
-    public function totalDamageCost(){
+    public function totalDamageCost() {
         $flightCost = FlightCost::zero();
 
-        foreach($this->damages as $damage){
+        foreach ($this->damages as $damage) {
             $flightCost = $flightCost->addCost($damage->getCost());
         }
 
         return $flightCost;
     }
 
-    public function damageCost(){
+    public function damageCost() {
         $flightCost = FlightCost::zero();
 
-        foreach($this->damages as $damage){
-            if(!$damage instanceof FlightDamageCount){
+        foreach ($this->damages as $damage) {
+            if (!$damage instanceof FlightDamageCount) {
                 continue;
             }
 
@@ -185,11 +185,11 @@ 
         return $flightCost;
     }
 
-    public function invoicedDamageCost(){
+    public function invoicedDamageCost() {
         $flightCost = FlightCost::zero();
 
-        foreach($this->damages as $damage){
-            if(!$damage instanceof FlightInvoicedDamageCount){
+        foreach ($this->damages as $damage) {
+            if (!$damage instanceof FlightInvoicedDamageCount) {
                 continue;
             }
 
@@ -199,7 +199,7 @@ 
         return $flightCost;
     }
 
-    public function damages(){
+    public function damages() {
         return $this->damages;
     }