laudeco / bbc-dolibarr-flightlog

Infrastructure/Damage/Query/Repository/GetPilotDamagesQueryRepository.php

Spacing +5 added lines, -5 removed lines

@@ -35,16 +35,16 @@
     public function query($year)
     {
         $sql = 'SELECT damage.amount as amount, damage.billed as billed, damage.author_id as author, author.firstname as author_name';
-        $sql.=' FROM '.MAIN_DB_PREFIX.'bbc_flight_damages as damage';
-        $sql.=' INNER JOIN '.MAIN_DB_PREFIX.'bbc_vols as flight ON flight.rowid = damage.flight_id';
-        $sql.=' INNER JOIN '.MAIN_DB_PREFIX.'user as author ON author.rowid = damage.author_id';
-        $sql.=' WHERE YEAR(flight.date) = '.$this->db->escape($year);
+        $sql .= ' FROM ' . MAIN_DB_PREFIX . 'bbc_flight_damages as damage';
+        $sql .= ' INNER JOIN ' . MAIN_DB_PREFIX . 'bbc_vols as flight ON flight.rowid = damage.flight_id';
+        $sql .= ' INNER JOIN ' . MAIN_DB_PREFIX . 'user as author ON author.rowid = damage.author_id';
+        $sql .= ' WHERE YEAR(flight.date) = ' . $this->db->escape($year);
 
         $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);
                     $damage = TotalDamage::fromArray($properties);
                     yield $damage;

Application/Damage/Command/InvoiceDamageCommandHandler.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      *
      * @throws \Exception
      */
-    public function __invoke(InvoiceDamageCommand $command){
+    public function __invoke(InvoiceDamageCommand $command) {
         $damage = $this->damageRepository->getById(DamageId::create($command->getDamageId()));
         $damage = $damage->invoice();
         $this->damageRepository->save($damage);

Http/Web/templates/flight_damage/view.php

Braces +5 added lines, -2 removed lines

@@ -22,9 +22,12 @@
             <td>
                 <?php if($currentDamage->isInvoiced()): ?>
                     <i class="fas fa-check"></i>
-                <?php else: ?>
+                <?php else {
+    : ?>
                     <i class="fas fa-times"></i>
-                <?php endif; ?>
+                <?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>

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; ?>

Http/Web/templates/damage/view.php

Spacing +3 added lines, -3 removed lines

@@ -53,9 +53,9 @@
 ?>
 
 <table class="border centpercent">
-    <tr><td class="fieldrequired">Auteur</td><td><?php echo $damage->getAuthorName();?></td></tr>
-    <tr><td class="fieldrequired">Montant</td><td><?php echo $damage->getAmount();?>€</td></tr>
-    <tr><td class="fieldrequired">Facturé</td><td><?php echo $damage->isInvoiced()? 'Oui' : 'Non';?></td></tr>
+    <tr><td class="fieldrequired">Auteur</td><td><?php echo $damage->getAuthorName(); ?></td></tr>
+    <tr><td class="fieldrequired">Montant</td><td><?php echo $damage->getAmount(); ?>€</td></tr>
+    <tr><td class="fieldrequired">Facturé</td><td><?php echo $damage->isInvoiced() ? 'Oui' : 'Non'; ?></td></tr>
     <tr>
         <td class="fieldrequired">Vol</td>
         <td>

Http/Web/Controller/DamageController.php

Braces +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
                 'form' => new Form($this->db),
                 'flight' => $flight
             ]);
-        }catch (\Exception $e){
+        } catch (\Exception $e){
             echo $e->getMessage();
         }
     }

Spacing +6 added lines, -6 removed lines

@@ -13,14 +13,14 @@ 
     /**
      * @return GetDamageQueryRepository
      */
-    private function getDamageRepository(){
+    private function getDamageRepository() {
         return new GetDamageQueryRepository($this->db);
     }
 
-    public function view(){
+    public function view() {
         $damageId = $this->request->getParam('id');
 
-        try{
+        try {
             $damage = $this->getDamageRepository()->query($damageId);
 
             $flight = new \Bbcvols($this->db);
@@ -31,16 +31,16 @@ 
                 'form' => new Form($this->db),
                 'flight' => $flight
             ]);
-        }catch (\Exception $e){
+        } catch (\Exception $e) {
             echo $e->getMessage();
         }
     }
 
-    public function invoice(){
+    public function invoice() {
         $damageId = $this->request->getParam('id');
 
 
 
-        return $this->redirect($_SERVER["PHP_SELF"].'?id='.$damageId);
+        return $this->redirect($_SERVER["PHP_SELF"] . '?id=' . $damageId);
     }
 }
\ No newline at end of file

Infrastructure/Common/Routes/Route.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
      *
      * @return bool
      */
-    public function match($route){
+    public function match($route) {
         return $this->name === $route;
     }
 

Http/Web/Controller/WebController.php

Spacing +4 added lines, -4 removed lines

@@ -32,19 +32,19 @@
      *
      * @return Response
      */
-    protected function render($template, array $variables = []){
+    protected function render($template, array $variables = []) {
 
-        foreach($variables as $variableName => $variableValue){
+        foreach ($variables as $variableName => $variableValue) {
             $GLOBALS[$variableName] = $variableValue;
         }
 
-        return new Response(__DIR__.'/../templates/'.$template);
+        return new Response(__DIR__ . '/../templates/' . $template);
     }
 
     /**
      * @param string $html
      */
-    protected function renderHtml($html){
+    protected function renderHtml($html) {
         print $html;
     }
 

Http/Web/Controller/FlightController.php

Spacing +2 added lines, -2 removed lines

@@ -6,9 +6,9 @@
 
 final class FlightController extends WebController
 {
-    public function view(){
+    public function view() {
         $id = $this->request->getParam('id');
-        return $this->redirect(DOL_URL_ROOT.'/flightlog/card.php?id='.$id);
+        return $this->redirect(DOL_URL_ROOT . '/flightlog/card.php?id=' . $id);
     }
 
 }
\ No newline at end of file

Http/Web/Response/Redirect.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      *
      * @return Redirect
      */
-    public static function create($url){
+    public static function create($url) {
         return new self($url);
     }