laudeco / bbc-dolibarr-flightlog

Http/Web/Controller/AddFlightDamageController.php

Spacing +10 added lines, -10 removed lines

@@ -19,16 +19,16 @@ 
     /**
      * Display the form.
      */
-    public function view(){
+    public function view() {
         $id = $this->request->getParam('id');
 
-        if($id === null){
+        if ($id === null) {
             print '<p>Paramètre non fournis.</p>';
             return;
         }
 
         $flight = new \Bbcvols($this->db);
-        if($flight->fetch($id) <= 0){
+        if ($flight->fetch($id) <= 0) {
             print '<p>Vol non trouvé</p>';
             return;
         }
@@ -37,24 +37,24 @@ 
         $form = new DamageCreationForm('damage_creation', $this->db);
         $form->bind($command);
 
-        if($this->request->isPost()){
+        if ($this->request->isPost()) {
             $form->setData($this->request->getPostParameters());
 
-            if(!$form->validate()){
+            if (!$form->validate()) {
                 return $this->render('flight_damage/form.php', [
                     'form' => $form,
                 ]);
             }
 
-            try{
+            try {
                 $this->handle($form->getObject());
-            }catch(\Exception $e){
+            } catch (\Exception $e) {
                 print $e->getMessage();
                 dol_syslog($e->getMessage(), LOG_ERR);
             }
 
 
-            $this->redirect($_SERVER["PHP_SELF"].'?id='.$id);
+            $this->redirect($_SERVER["PHP_SELF"] . '?id=' . $id);
             exit;
         }
 
@@ -76,14 +76,14 @@ 
     /**
      * @return CreateDamageCommandHandler
      */
-    private function getHandler(){
+    private function getHandler() {
         return new CreateDamageCommandHandler($this->db, $this->getDamageRepository());
     }
 
     /**
      * @return FlightDamageRepository
      */
-    private function getDamageRepository(){
+    private function getDamageRepository() {
         return new FlightDamageRepository($this->db);
     }
 

Braces +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
             try{
                 $this->handle($form->getObject());
-            }catch(\Exception $e){
+            } catch(\Exception $e){
                 print $e->getMessage();
                 dol_syslog($e->getMessage(), LOG_ERR);
             }

Http/Web/Controller/FlightDamageController.php

Spacing +2 added lines, -2 removed lines

@@ -13,11 +13,11 @@
     /**
      * @return GetDamagesForFlightQueryRepositoryInterface
      */
-    private function getDamagesRepository(){
+    private function getDamagesRepository() {
         return new GetDamagesForFlightQueryRepository($this->db);
     }
 
-    public function view(){
+    public function view() {
         $flightId = $this->request->getParam('id');
 
         return $this->render('flight_damage/view.php', [

Infrastructure/Common/Repository/AbstractDomainRepository.php

Spacing +8 added lines, -8 removed lines

@@ -23,7 +23,7 @@ 
     public function __construct(\DoliDB $db, $tableName)
     {
         $this->db = $db;
-        $this->tableName = strpos($tableName, MAIN_DB_PREFIX) === 0 ? $tableName : MAIN_DB_PREFIX.$tableName;
+        $this->tableName = strpos($tableName, MAIN_DB_PREFIX) === 0 ? $tableName : MAIN_DB_PREFIX . $tableName;
     }
 
     /**
@@ -33,18 +33,18 @@ 
      *
      * @throws \Exception
      */
-    protected function write(array $elements){
+    protected function write(array $elements) {
         $columns = join(',', array_keys($elements));
-        $values = join(',', array_map(function($value){
-                if(is_null($value)){
+        $values = join(',', array_map(function($value) {
+                if (is_null($value)) {
                     return null;
                 }
 
-                if(is_bool($value)){
-                    return (int)$value;
+                if (is_bool($value)) {
+                    return (int) $value;
                 }
 
-                return is_string($value) ? $this->db->escape($value) : $value ;
+                return is_string($value) ? $this->db->escape($value) : $value;
             }, array_values($elements)));
 
         $this->db->begin();
@@ -54,7 +54,7 @@ 
             $lasterror = $this->db->lasterror();
             dol_syslog(__METHOD__ . ' ' . $lasterror, LOG_ERR);
             $this->db->rollback();
-            throw new \Exception("Repository error - ".$lasterror);
+            throw new \Exception("Repository error - " . $lasterror);
         }
 
         $id = $this->db->last_insert_id($this->tableName);

Infrastructure/Damage/Query/Repository/GetDamagesForFlightQueryRepository.php

Spacing +4 added lines, -4 removed lines

@@ -31,9 +31,9 @@ 
     public function __invoke($flightId)
     {
         $sql = 'SELECT damage.amount, author.login as author_name';
-        $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);
 
         $damages = [];
 
@@ -41,7 +41,7 @@ 
         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);
                     $damages[] = Damage::fromArray($properties);
                 }

Infrastructure/Damage/Query/Repository/GetPilotDamagesQueryRepository.php

Spacing +5 added lines, -5 removed lines

@@ -30,10 +30,10 @@ 
     public function query($year)
     {
         $sql = 'SELECT SUM(damage.amount) as total_amount, damage.author_id as author';
-        $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.=' WHERE YEAR(flight.date) = '.$this->db->escape($year);
-        $sql.=' GROUP BY damage.author_id';
+        $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 .= ' WHERE YEAR(flight.date) = ' . $this->db->escape($year);
+        $sql .= ' GROUP BY damage.author_id';
 
         $damages = [];
 
@@ -41,7 +41,7 @@ 
         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);
                     $damages[] = TotalDamage::fromArray($properties);
                 }

Form/Form.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
      */
     public function add(FormElementInterface $element)
     {
-        if(array_key_exists($element->getName(), $this->elements)){
+        if (array_key_exists($element->getName(), $this->elements)) {
             throw new \InvalidArgumentException('Element already exists');
         }
         $this->elements[$element->getName()] = $element;
@@ -141,7 +141,7 @@ 
      */
     public function getErrorMessages()
     {
-        if(!$this->validator){
+        if (!$this->validator) {
             return [];
         }
 

class/Common/ValueObject/Id.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      *
      * @return Id
      */
-    public static function create($id){
+    public static function create($id) {
         return new self($id);
     }
 

class/Damage/FlightDamage.php

Indentation +4 added lines, -4 removed lines

@@ -33,10 +33,10 @@
      */
     private function __construct(FlightId $flightId, DamageAmount $amount, $billed, AuthorId $authorId)
     {
-       $this->flight = $flightId;
-       $this->amount = $amount;
-       $this->billed = $billed;
-       $this->author = $authorId;
+        $this->flight = $flightId;
+        $this->amount = $amount;
+        $this->billed = $billed;
+        $this->author = $authorId;
     }
 
     /**

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
      *
      * @return FlightDamage
      */
-    public static function damage(FlightId $flightId, DamageAmount $amount, AuthorId $authorId){
+    public static function damage(FlightId $flightId, DamageAmount $amount, AuthorId $authorId) {
         return new self($flightId, $amount, false, $authorId);
     }
 
@@ -61,14 +61,14 @@ 
     /**
      * @return FlightId
      */
-    public function getFlightId(){
+    public function getFlightId() {
         return $this->flight;
     }
 
     /**
      * @return DamageAmount
      */
-    public function amount(){
+    public function amount() {
         return $this->amount;
     }
 

class/card/TabCollection.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      * @return array
      */
     public function toArray() {
-        return array_map(function(Tab $tab){
+        return array_map(function(Tab $tab) {
             return $tab->toArray();
         }, $this->tabs);
     }