laudeco / bbc-dolibarr-flightlog

Application/Pilot/ViewModel/Pilot.php

Spacing +20 added lines, -20 removed lines

@@ -459,7 +459,7 @@ 
             || !$this->isTrainingFirstHelpValid();
     }
 
-    public function isWarning(){
+    public function isWarning() {
         return !$this->isHoursAndTakeOffValidGroupA() || !$this->isProValid();
     }
 
@@ -469,7 +469,7 @@ 
 
         $ok = '<span class="text-success text-italic">OK</span>';
 
-        if($this->isPilot()){
+        if ($this->isPilot()) {
             $reasons .= '<br> <span class="text-bold">Médical : </span> ' . ($this->isMedicalValid() ? $ok : 'L\'échéance est atteinte ou dépassée.');
             $reasons .= '<br> <span class="text-bold">Training flight : </span> ' . ($this->isTrainingFlightValid() ? $ok : 'Pas de vol avec un FI dans les 48 derniers mois.');
             $reasons .= '<br> <span class="text-bold">Exp. récente Gr. A: </span> ' . ($this->isHoursAndTakeOffValidGroupA() ? $ok : 'Pas 6H & 10TO dans les 24 derniers mois');
@@ -481,27 +481,27 @@ 
         }
 
 
-        if($this->isPilot() || $this->isTrainingFire()){
+        if ($this->isPilot() || $this->isTrainingFire()) {
             $reasons .= '<br><span class="text-bold">Feu: </span> ' . ($this->isTrainingFireValid() ? $ok : 'Date expirée');
         }
 
-        if($this->isPilot() || $this->isTrainingFirstHelp()){
+        if ($this->isPilot() || $this->isTrainingFirstHelp()) {
             $reasons .= '<br><span class="text-bold">Premiers secours: </span> ' . ($this->isTrainingFirstHelpValid() ? $ok : 'Date expirée');
         }
 
 
         $qualification = $ok;
-        if(!$this->isPilot()){
+        if (!$this->isPilot()) {
             $qualification = '<span class="text-success">CREW</span>';
         }
-        if($this->isWarning()){
+        if ($this->isWarning()) {
             $qualification = '<span class="text-warning">Controle du carnet de vol nécessaire</span>';
         }
-        if($this->isDanger()){
+        if ($this->isDanger()) {
             $qualification = '<span class="text-danger">Vol non autorisé</span>';
         }
 
-        return '<u><span class="text-bold">Qualification </span> :  ' . $qualification .'</u>'. $reasons;
+        return '<u><span class="text-bold">Qualification </span> :  ' . $qualification . '</u>' . $reasons;
     }
 
     private function isMedicalValid(): bool
@@ -510,7 +510,7 @@ 
             return true;
         }
 
-        if(null === $this->medicalEndDate){
+        if (null === $this->medicalEndDate) {
             return false;
         }
 
@@ -520,11 +520,11 @@ 
 
     private function isTrainingFlightValid(): bool
     {
-        if(!$this->isPilot()){
+        if (!$this->isPilot()) {
             return true;
         }
 
-        if(null === $this->lastTrainingFlightDate){
+        if (null === $this->lastTrainingFlightDate) {
             return false;
         }
 
@@ -533,7 +533,7 @@ 
 
     private function isHoursAndTakeOffValidGroupA(): bool
     {
-        if(!$this->isPilot()){
+        if (!$this->isPilot()) {
             return true;
         }
 
@@ -560,7 +560,7 @@ 
 
     private function isProDateValid(): bool
     {
-        if(!$this->isPilot()){
+        if (!$this->isPilot()) {
             return true;
         }
 
@@ -574,7 +574,7 @@ 
 
     private function isProValid(): bool
     {
-        if(!$this->isPilot()){
+        if (!$this->isPilot()) {
             return true;
         }
 
@@ -582,7 +582,7 @@ 
             return true;
         }
 
-        if($this->lastOpcDate && $this->diffDateInMonths($this->lastOpcDate) < 6){
+        if ($this->lastOpcDate && $this->diffDateInMonths($this->lastOpcDate) < 6) {
             return true;
         }
 
@@ -602,7 +602,7 @@ 
     }
 
     private function diffDateInMonths(\DateTimeImmutable $dateA, \DateTimeImmutable $dateB = null): int{
-        if(null === $dateB){
+        if (null === $dateB) {
             $dateB = new \DateTimeImmutable();
         }
 
@@ -612,11 +612,11 @@ 
 
     private function isTrainingFireValid()
     {
-        if(!$this->isTrainingFire() && !$this->isPilot()){
+        if (!$this->isTrainingFire() && !$this->isPilot()) {
             return true;
         }
 
-        if(null === $this->lastTrainingFireDate){
+        if (null === $this->lastTrainingFireDate) {
             return false;
         }
 
@@ -625,11 +625,11 @@ 
 
     private function isTrainingFirstHelpValid()
     {
-        if(!$this->isTrainingFirstHelp() && !$this->isPilot() ){
+        if (!$this->isTrainingFirstHelp() && !$this->isPilot()) {
             return true;
         }
 
-        if(null === $this->lastTrainingFirstHelpDate){
+        if (null === $this->lastTrainingFirstHelpDate) {
             return false;
         }
 

Infrastructure/Pilot/Query/Repository/PilotQueryRepository.php

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
      *
      * @return Pilot
      */
-    public function byId($id){
+    public function byId($id) {
         $pilots = $this->filter(['id' => $id]);
         if (empty($pilots)) {
             return Pilot::fromArray(['id' => $id]);
@@ -90,8 +90,8 @@ 
         ))
         ';
 
-        if(isset($filters['id'])){
-            $sql .= ' AND llx_user.rowid = '.$filters['id'];
+        if (isset($filters['id'])) {
+            $sql .= ' AND llx_user.rowid = ' . $filters['id'];
         }
         $sql .= ' ORDER BY llx_user.firstname, llx_user.lastname, llx_user.rowid, flight.date';
 
@@ -107,16 +107,16 @@ 
 
         /** @var Pilot[] $pilots */
         $pilots = [];
-        for($i = 0; $i < $num ; $i++) {
+        for ($i = 0; $i < $num; $i++) {
             $values = $this->db->fetch_array($resql);
             $pilotId = $values['id'];
 
-            if(!isset($pilots[$pilotId])){
+            if (!isset($pilots[$pilotId])) {
                 $pilots[$pilotId] = Pilot::fromArray($values);
             }
 
             $flight = PilotFlight::fromArray($values, 'flight');
-            if($flight->isValid()){
+            if ($flight->isValid()) {
                 $pilots[$pilotId]->addFlight($flight);
             }
 

class/bbcvols.class.php

Spacing +27 added lines, -27 removed lines

@@ -129,7 +129,7 @@ 
      */
     public function getIdBBCVols()
     {
-        return (int)$this->idBBC_vols;
+        return (int) $this->idBBC_vols;
     }
 
     /**
@@ -137,7 +137,7 @@ 
      */
     public function getId()
     {
-        return (int)$this->getIdBBCVols();
+        return (int) $this->getIdBBCVols();
     }
 
     /**
@@ -274,13 +274,13 @@ 
         $sql .= ' ' . (!isset($this->nbrPax) ? 'NULL' : "'" . $this->db->escape($this->nbrPax) . "'") . ',';
         $sql .= ' ' . (!isset($this->remarque) ? 'NULL' : "'" . $this->db->escape($this->remarque) . "'") . ',';
         $sql .= ' ' . (!isset($this->incidents) ? 'NULL' : "'" . $this->db->escape($this->incidents) . "'") . ',';
-        $sql .= ' ' . (!isset($this->fk_type) || (int)$this->fk_type === -1 ? 'NULL' : $this->fk_type) . ',';
-        $sql .= ' ' . (!isset($this->fk_pilot) || (int)$this->fk_pilot === -1 ? 'NULL' : $this->fk_pilot) . ',';
-        $sql .= ' ' . (!isset($this->fk_organisateur) || (int)$this->fk_organisateur === -1 ? 'NULL' : $this->fk_organisateur) . ',';
+        $sql .= ' ' . (!isset($this->fk_type) || (int) $this->fk_type === -1 ? 'NULL' : $this->fk_type) . ',';
+        $sql .= ' ' . (!isset($this->fk_pilot) || (int) $this->fk_pilot === -1 ? 'NULL' : $this->fk_pilot) . ',';
+        $sql .= ' ' . (!isset($this->fk_organisateur) || (int) $this->fk_organisateur === -1 ? 'NULL' : $this->fk_organisateur) . ',';
         $sql .= ' ' . (!isset($this->is_facture) ? '0' : $this->is_facture) . ',';
         $sql .= ' ' . (!isset($this->kilometers) || empty($this->kilometers) ? '0' : $this->kilometers) . ',';
         $sql .= ' ' . (!isset($this->cost) ? 'NULL' : "'" . $this->db->escape($this->cost) . "'") . ',';
-        $sql .= ' ' . (!isset($this->fk_receiver) || (int)$this->fk_receiver === -1 ? 'NULL' : $this->fk_receiver) . ',';
+        $sql .= ' ' . (!isset($this->fk_receiver) || (int) $this->fk_receiver === -1 ? 'NULL' : $this->fk_receiver) . ',';
         $sql .= ' ' . (!isset($this->justif_kilometers) ? 'NULL' : "'" . $this->db->escape($this->justif_kilometers) . "'") . ',';
         $sql .= ' ' . "'" . date('Y-m-d H:i:s') . "'" . ',';
         $sql .= ' ' . "'" . date('Y-m-d H:i:s') . "'" . ',';
@@ -455,23 +455,23 @@ 
 
                 $this->id = $obj->idBBC_vols;
 
-                $this->idBBC_vols = (int)$obj->idBBC_vols;
+                $this->idBBC_vols = (int) $obj->idBBC_vols;
                 $this->date = $this->db->jdate($obj->date);
                 $this->lieuD = $obj->lieuD;
                 $this->lieuA = $obj->lieuA;
                 $this->heureD = $obj->heureD;
                 $this->heureA = $obj->heureA;
-                $this->BBC_ballons_idBBC_ballons = (int)$obj->BBC_ballons_idBBC_ballons;
+                $this->BBC_ballons_idBBC_ballons = (int) $obj->BBC_ballons_idBBC_ballons;
                 $this->nbrPax = $obj->nbrPax;
                 $this->remarque = $obj->remarque;
                 $this->incidents = $obj->incidents;
-                $this->fk_type = (int)$obj->fk_type;
-                $this->fk_pilot = (int)$obj->fk_pilot;
-                $this->fk_organisateur = (int)$obj->fk_organisateur;
-                $this->statut = $this->is_facture = (int)$obj->is_facture;
+                $this->fk_type = (int) $obj->fk_type;
+                $this->fk_pilot = (int) $obj->fk_pilot;
+                $this->fk_organisateur = (int) $obj->fk_organisateur;
+                $this->statut = $this->is_facture = (int) $obj->is_facture;
                 $this->kilometers = $obj->kilometers;
                 $this->total_ttc = $this->cost = $obj->cost;
-                $this->fk_receiver = (int)$obj->fk_receiver;
+                $this->fk_receiver = (int) $obj->fk_receiver;
                 $this->justif_kilometers = $obj->justif_kilometers;
                 $this->date_creation = $obj->date_creation;
                 $this->date_update = $obj->date_update;
@@ -587,13 +587,13 @@ 
         $sql .= ' nbrPax = ' . (isset($this->nbrPax) ? "'" . $this->db->escape($this->nbrPax) . "'" : "null") . ',';
         $sql .= ' remarque = ' . (isset($this->remarque) ? "'" . $this->db->escape($this->remarque) . "'" : "null") . ',';
         $sql .= ' incidents = ' . (isset($this->incidents) ? "'" . $this->db->escape($this->incidents) . "'" : "null") . ',';
-        $sql .= ' fk_type = ' . (isset($this->fk_type) && (int)$this->fk_type > 0 ? $this->fk_type : "null") . ',';
-        $sql .= ' fk_pilot = ' . (isset($this->fk_pilot) && (int)$this->fk_pilot > 0 ? $this->fk_pilot : "null") . ',';
-        $sql .= ' fk_organisateur = ' . (isset($this->fk_organisateur) && (int)$this->fk_organisateur > 0 ? $this->fk_organisateur : "null") . ',';
+        $sql .= ' fk_type = ' . (isset($this->fk_type) && (int) $this->fk_type > 0 ? $this->fk_type : "null") . ',';
+        $sql .= ' fk_pilot = ' . (isset($this->fk_pilot) && (int) $this->fk_pilot > 0 ? $this->fk_pilot : "null") . ',';
+        $sql .= ' fk_organisateur = ' . (isset($this->fk_organisateur) && (int) $this->fk_organisateur > 0 ? $this->fk_organisateur : "null") . ',';
         $sql .= ' is_facture = ' . (isset($this->is_facture) ? $this->is_facture : "0") . ',';
         $sql .= ' kilometers = ' . (!empty($this->kilometers) ? $this->kilometers : "0") . ',';
         $sql .= ' cost = ' . (isset($this->cost) ? "'" . $this->db->escape($this->cost) . "'" : "''") . ',';
-        $sql .= ' fk_receiver = ' . (isset($this->fk_receiver) && (int)$this->fk_receiver > 0 ? $this->fk_receiver : "null") . ',';
+        $sql .= ' fk_receiver = ' . (isset($this->fk_receiver) && (int) $this->fk_receiver > 0 ? $this->fk_receiver : "null") . ',';
         $sql .= ' justif_kilometers = ' . (isset($this->justif_kilometers) ? "'" . $this->db->escape($this->justif_kilometers) . "'," : "'',");
         $sql .= ' date_update = ' . "'" . date('Y-m-d H:i:s') . "',";
         $sql .= ' is_night = ' . ($this->nightFlight ? 1 : 0) . ',';
@@ -701,9 +701,9 @@ 
 
         $label = '<u>' . $langs->trans("MyModule") . '</u>';
         $label .= '<div width="100%">';
-        $label .= '<b>' . $langs->trans('Ref') . ':</b> ' . $this->idBBC_vols . ' - <b>'.$langs->trans('T') . '</b>' . $this->fk_type . '<br>';
+        $label .= '<b>' . $langs->trans('Ref') . ':</b> ' . $this->idBBC_vols . ' - <b>' . $langs->trans('T') . '</b>' . $this->fk_type . '<br>';
         $label .= '<b>' . $langs->trans('Date') . ':</b> ' . dol_print_date($this->date, '%d-%m-%Y') . '<br/>';
-        $label .= '<b>' . $langs->trans('De') . ':</b> ' . $this->lieuD . ' - '. '<b>' . $langs->trans('à') . ':</b> ' . $this->lieuA . '<br/>';
+        $label .= '<b>' . $langs->trans('De') . ':</b> ' . $this->lieuD . ' - ' . '<b>' . $langs->trans('à') . ':</b> ' . $this->lieuA . '<br/>';
         $label .= '<b>' . $langs->trans('Pilote') . ':</b> ' . $this->getPilot()->getFullName($langs) . '<br/>';
         $label .= '<b>' . $langs->trans('Organisateur') . ':</b> ' . $this->getOrganisator()->getFullName($langs) . '<br/>';
         $label .= '</div>';
@@ -903,7 +903,7 @@ 
      */
     public function getPilotId()
     {
-        return (int)$this->fk_pilot;
+        return (int) $this->fk_pilot;
     }
 
     /**
@@ -911,7 +911,7 @@ 
      */
     public function getOrganisatorId()
     {
-        return (int)$this->fk_organisateur;
+        return (int) $this->fk_organisateur;
     }
 
     /**
@@ -948,7 +948,7 @@ 
      */
     public function hasPax()
     {
-        return (int)$this->nbrPax > 0;
+        return (int) $this->nbrPax > 0;
     }
 
     /**
@@ -1017,7 +1017,7 @@ 
      */
     public function getKilometers()
     {
-        return (int)$this->kilometers;
+        return (int) $this->kilometers;
     }
 
     /**
@@ -1060,7 +1060,7 @@ 
      */
     public function getNumberOfPassengers()
     {
-        return (int)$this->nbrPax;
+        return (int) $this->nbrPax;
     }
 
     /**
@@ -1263,7 +1263,7 @@ 
      */
     public function getBBCBallonsIdBBCBallons()
     {
-        return (int)$this->BBC_ballons_idBBC_ballons;
+        return (int) $this->BBC_ballons_idBBC_ballons;
     }
 
     /**
@@ -1339,7 +1339,7 @@ 
      */
     public function getFkType()
     {
-        return (int)$this->fk_type;
+        return (int) $this->fk_type;
     }
 
     /**
@@ -1434,7 +1434,7 @@ 
      */
     public function getFkReceiver()
     {
-        return (int)$this->fk_receiver;
+        return (int) $this->fk_receiver;
     }
 
     /**

command/CreateOrderCommandHandler.php

Spacing +1 added lines, -1 removed lines

@@ -220,7 +220,7 @@
 
         $orderId = $this->order->create($this->user);
         if ($orderId <= 0) {
-            throw new Exception('Exception during the order creation ('.$this->order->error.')');
+            throw new Exception('Exception during the order creation (' . $this->order->error . ')');
         }
 
         return $this;

readFlights.php

Spacing +7 added lines, -7 removed lines

@@ -153,10 +153,10 @@ 
 $tableQueryHandler = new BillableFlightQueryHandler($db, $conf->global);
 $pilotQueryRepository = new PilotQueryRepository($db);
 
-function pilotStatus($id){
+function pilotStatus($id) {
     global $pilotQueryRepository;
     $member = $pilotQueryRepository->byId($id);
-    if($member === null){
+    if ($member === null) {
         return '';
     }
 
@@ -201,7 +201,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>';
@@ -220,8 +220,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>';
 }
 
@@ -246,7 +246,7 @@ 
 print '<td>' . $totalT3 . '</td>';
 print '<td></td>';
 
-print '<td>' . $totalT4. '</td>';
+print '<td>' . $totalT4 . '</td>';
 print '<td></td>';
 
 print '<td>' . $totalT5 . '</td>';
@@ -275,7 +275,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');
 }
 

Application/Damage/ViewModel/Damage.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
         $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);
     }

Http/Web/templates/damage/form.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     <?php echo $renderer->render($damageCreationForm->getElement('flight_id')); ?>
     <?php echo $renderer->render($damageCreationForm->getElement('token')); ?>
 
-    <?php if($damageCreationForm->has('amount')): ?>
+    <?php if ($damageCreationForm->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/Controller/AddDamageController.php

Spacing +8 added lines, -8 removed lines

@@ -15,28 +15,28 @@ 
     /**
      * Display the form.
      */
-    public function view(){
+    public function view() {
         $damageCreationForm = new DamageCreationForm('damage_creation', $this->db);
         $damageCreationForm->bind(CreateDamageCommand::create());
 
-        if($this->request->isPost()){
+        if ($this->request->isPost()) {
             $damageCreationForm->setData($this->request->getPostParameters());
 
-            if(!$damageCreationForm->validate()){
+            if (!$damageCreationForm->validate()) {
                 return $this->render('damage/form.php', [
                     'form' => $damageCreationForm,
                 ]);
             }
             $this->handle($damageCreationForm->getObject());
-            try{
+            try {
 
-            }catch(\Exception $e){
+            } catch (\Exception $e) {
                 print $e->getMessage();
                 dol_syslog($e->getMessage(), LOG_ERR);
             }
 
 
-            return $this->redirect($_SERVER["PHP_SELF"].'?r=get_damages');
+            return $this->redirect($_SERVER["PHP_SELF"] . '?r=get_damages');
         }
 
         return $this->render('damage/form.php', [
@@ -57,14 +57,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

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

Infrastructure/Common/Repository/AbstractDomainRepository.php

Spacing +15 added lines, -15 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,10 +33,10 @@ 
      *
      * @throws \Exception
      */
-    protected function write(array $elements){
+    protected function write(array $elements) {
         $columns = join(',', array_keys($elements));
         $values = [];
-        foreach($elements as $col => $val){
+        foreach ($elements as $col => $val) {
             $values[] = $this->escape($val);
         }
         $values = join(',', $values);
@@ -49,7 +49,7 @@ 
             $lasterror = $this->db->lasterror();
             dol_syslog(__METHOD__ . ' ' . $lasterror, LOG_ERR);
             $this->db->rollback();
-            throw new \Exception("Repository error - ".$lasterror. ' ('.$sql.')');
+            throw new \Exception("Repository error - " . $lasterror . ' (' . $sql . ')');
         }
 
         $id = $this->db->last_insert_id($this->tableName);
@@ -62,16 +62,16 @@ 
      *
      * @return int|string|null
      */
-    private function escape($value){
-        if((is_null($value) || null == $value) && !is_bool($value)){
+    private function escape($value) {
+        if ((is_null($value) || null == $value) && !is_bool($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;
     }
 
     /**
@@ -84,18 +84,18 @@ 
     {
         $sqlModifications = [];
 
-        foreach($elements as $field => $value){
+        foreach ($elements as $field => $value) {
             $sqlModifications[] = sprintf('%s=%s', $field, $this->escape($value));
         }
 
         $this->db->begin();
-        $resql = $this->db->query(sprintf('UPDATE %s SET %s WHERE '.$idCol.' = %s ', $this->tableName, join(',',$sqlModifications), $id));
+        $resql = $this->db->query(sprintf('UPDATE %s SET %s WHERE ' . $idCol . ' = %s ', $this->tableName, join(',', $sqlModifications), $id));
 
         if (!$resql) {
             $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);
         }
 
         $this->db->commit();
@@ -109,8 +109,8 @@ 
      *
      * @return array|null
      */
-    protected function get($id, $idCol = 'rowid'){
-        $sql = sprintf('SELECT * FROM %s WHERE '.$idCol.' = %s', $this->tableName, $id );
+    protected function get($id, $idCol = 'rowid') {
+        $sql = sprintf('SELECT * FROM %s WHERE ' . $idCol . ' = %s', $this->tableName, $id);
 
         $resql = $this->db->query($sql);
         if (!$resql) {
@@ -122,7 +122,7 @@ 
             return null;
         }
 
-        for($i = 0; $i < $num ; $i++) {
+        for ($i = 0; $i < $num; $i++) {
             return $this->db->fetch_array($resql);
         }