laudeco / bbc-dolibarr-flightlog

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, 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/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.author_id as  author_id, damage.rowid as id, damage.amount, CONCAT(author.firstname, " " , author.lastname) 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;
             }
 

Application/Damage/ViewModel/Damage.php

Spacing +2 added lines, -2 removed lines

@@ -66,13 +66,13 @@
      *
      * @return Damage
      */
-    public static function fromArray(array $properties){
+    public static function fromArray(array $properties) {
         $authorId = $properties['author_id'];
         $author = $properties['author_name'];
         $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);
     }

Http/Web/Controller/StatisticalGraphController.php

Spacing +15 added lines, -15 removed lines

@@ -31,19 +31,19 @@ 
 
         $i = 10;
         foreach ($series[0] as $key => $year) {
-            if($key == 0){
+            if ($key == 0) {
                 continue;
             }
 
             $graphType[] = $key == 5 ? "lines" : 'linesnopoint';
             $color[] = $graph->datacolor[$key];
 
-            $i+=30;
+            $i += 30;
         }
-        $color[count($color)-1] = [255,0,0]; // Average is red
+        $color[count($color) - 1] = [255, 0, 0]; // Average is red
 
-        $color[count($color)-2] = [0,100,255]; //Current year
-        $graphType[count($color)-2] = 'bars';
+        $color[count($color) - 2] = [0, 100, 255]; //Current year
+        $graphType[count($color) - 2] = 'bars';
 
         $graph->SetDataColor($color);
 
@@ -82,27 +82,27 @@ 
             /**
              * @var int $month
              * @var Statistic $stat */
-            foreach($months as $month => $stat){
-               if(!isset($series[$month-1])){
-                   $series[$month-1] = [substr(\DateTime::createFromFormat('!m', $month)->format('F'),0,1)];
+            foreach ($months as $month => $stat) {
+               if (!isset($series[$month - 1])) {
+                   $series[$month - 1] = [substr(\DateTime::createFromFormat('!m', $month)->format('F'), 0, 1)];
                };
-               $series[$month-1][$i] = $stat->number();
+               $series[$month - 1][$i] = $stat->number();
 
-               if($year === 2020){
+               if ($year === 2020) {
                    continue;
                }
 
-               if(!isset($average[$month-1])){
-                   $average[$month-1] = 0;
+               if (!isset($average[$month - 1])) {
+                   $average[$month - 1] = 0;
                }
-               $average[$month-1] += $stat->number();
+               $average[$month - 1] += $stat->number();
             }
 
             $i++;
         }
 
-        foreach($series as $monthIndex => $seriesValue){
-            $series[$monthIndex][] = $average[$monthIndex] / ($numberOfYears-1);
+        foreach ($series as $monthIndex => $seriesValue) {
+            $series[$monthIndex][] = $average[$monthIndex] / ($numberOfYears - 1);
         }
 
         return $series;

Infrastructure/Flight/Query/Repository/GetBillableFlightPerMonthQueryRepository.php

Spacing +2 added lines, -2 removed lines

@@ -64,13 +64,13 @@
 
         $stats = new BillableFlightByYearMonth();
 
-        for($i = 0; $i < $num ; $i++) {
+        for ($i = 0; $i < $num; $i++) {
             $properties = $this->db->fetch_array($resql);
             $stat = Statistic::fromArray($properties);
             $stats->add($stat);
         }
 
-        $stats->add(new Statistic(0,1,(new \DateTime())->format('Y')));
+        $stats->add(new Statistic(0, 1, (new \DateTime())->format('Y')));
 
         return $stats;
     }

Application/Flight/ViewModel/BillableFlightByYearMonth.php

Spacing +4 added lines, -4 removed lines

@@ -16,9 +16,9 @@ 
 
     public function add(Statistic $stat)
     {
-        if(!isset($this->data[$stat->year()])){
+        if (!isset($this->data[$stat->year()])) {
             $this->data[$stat->year()] = [];
-            for($i = 1 ; $i <= 12 ; $i++){
+            for ($i = 1; $i <= 12; $i++) {
                 $this->data[$stat->year()][$i] = new Statistic(0, $stat->month(), $stat->year());
             }
         }
@@ -26,11 +26,11 @@ 
         $this->data[$stat->year()][$stat->month()] = $stat;
     }
 
-    public function data(){
+    public function data() {
         return $this->data;
     }
 
-    public function years(){
+    public function years() {
         return array_keys($this->data);
     }
 

Application/Flight/ViewModel/Statistic.php

Spacing +2 added lines, -2 removed lines

@@ -44,11 +44,11 @@
         return $this->number;
     }
 
-    public function year(){
+    public function year() {
         return $this->year;
     }
 
-    public function month(){
+    public function month() {
         return $this->month;
     }