laudeco / bbc-dolibarr-flightlog

Form/InputDate.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function setValue($value)
     {
-        if($value instanceof \DateTime){
+        if ($value instanceof \DateTime) {
             return parent::setValue($value->format('Y-m-d'));
         }
 

Form/FlightForm.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
      * @param array $options
      * @param \User $user
      */
-    public function __construct(\ValidatorInterface $validator, $baseObject, \DoliDB $db, $options,\User $user)
+    public function __construct(\ValidatorInterface $validator, $baseObject, \DoliDB $db, $options, \User $user)
     {
 
         parent::__construct('flight_form', FormInterface::METHOD_POST, $this->buildOptionsfromConfiguration($options));
@@ -83,7 +83,7 @@ 
         $flight = $object;
 
         // Quick fix - Fixme by a factory on this form.
-        if($this->user->rights->flightlog->vol->advanced){
+        if ($this->user->rights->flightlog->vol->advanced) {
             return parent::bind($object);
         }
 

command/CreateExpenseNoteCommandHandler.php

Spacing +1 added lines, -1 removed lines

@@ -272,7 +272,7 @@
         $expenseNoteUser = new User($this->db);
         $expenseNoteUser->id = $currentMissionUserId;
         $id = $expenseNote->create($expenseNoteUser);
-        if($id < 0){
+        if ($id < 0) {
             return null;
         }
 

query/GetPilotsWithMissionsQueryHandler.php

Spacing +3 added lines, -3 removed lines

@@ -52,9 +52,9 @@ 
                         $pilotLastname = $obj->lastname;
                         $pilotFirstname = $obj->firstname;
 
-                        if($query->isPilotsOnly()){
+                        if ($query->isPilotsOnly()) {
                             $result->addPilot($pilotId, $pilotFirstname, $pilotLastname);
-                        }else{
+                        } else {
                             $result->addMission($obj->quartil, $pilotId, $pilotFirstname, $pilotLastname,
                                 $obj->number_flights, $obj->total_kilometers);
                         }
@@ -79,7 +79,7 @@ 
         $sql .= " , SUM(VOL.kilometers) as total_kilometers ";
         $sql .= " , COUNT(VOL.idBBC_vols) as number_flights";
 
-        if(!$query->isPilotsOnly()){
+        if (!$query->isPilotsOnly()) {
             $sql .= " , QUARTER(VOL.date) as quartil ";
         }
 

command/CreateFlightCommand.php

Spacing +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
     /**
      * @return bool
      */
-    public function hasOrderId(){
+    public function hasOrderId() {
         return count($this->orderIds) > 0;
     }
 }
\ No newline at end of file

command/ClassifyFlight.php

Spacing +2 added lines, -2 removed lines

@@ -26,8 +26,8 @@
      */
     public function __construct($flightId, $projectId)
     {
-        $this->flightId = (int)$flightId;
-        $this->projectId = (int)$projectId;
+        $this->flightId = (int) $flightId;
+        $this->projectId = (int) $projectId;
     }
 
     /**

command/CreateOrderCommand.php

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
         $this->language = $form->language;
         $this->userId = $userId;
         $this->publicComment = $form->isCommentPublic == 1;
-        $this->socid = (int)$form->socid;
+        $this->socid = (int) $form->socid;
     }
 
     /**

validators/SimpleOrderValidator.php

Spacing +3 added lines, -3 removed lines

@@ -52,12 +52,12 @@
         if (!isset($value->socid) && empty($value->email) && empty($value->phone)) {
             $this->addWarning('Soit l\'e-mail soit le téléphone n\'a pas été complété');
         }
-        if (empty($value->nbrPax) || (int)$value->nbrPax <= 0) {
+        if (empty($value->nbrPax) || (int) $value->nbrPax <= 0) {
             $this->addError('nbrPax', 'Le nombre de passagers doit être plus grand que 0.');
         }
 
-        if ((int)$value->nbrPax > 0) {
-            $costPerPax = $value->cost / (int)$value->nbrPax;
+        if ((int) $value->nbrPax > 0) {
+            $costPerPax = $value->cost / (int) $value->nbrPax;
             if ($costPerPax < $this->getMinPrice()) {
                 $this->addError('cost',
                     sprintf('Le prix demandé par passagé est trop peu élevé. Un minimum de %s est demandé',

addOrder.php

Spacing +34 added lines, -34 removed lines

@@ -12,32 +12,32 @@ 
 dol_include_once("/flightlog/lib/flightLog.lib.php");
 dol_include_once("/flightlog/validators/SimpleOrderValidator.php");
 dol_include_once("/flightlog/command/CreateOrderCommandHandler.php");
-require_once DOL_DOCUMENT_ROOT.'/core/class/html.formcompany.class.php';
-require_once DOL_DOCUMENT_ROOT.'/core/class/html.formadmin.class.php';
+require_once DOL_DOCUMENT_ROOT . '/core/class/html.formcompany.class.php';
+require_once DOL_DOCUMENT_ROOT . '/core/class/html.formadmin.class.php';
 
 // Load object modCodeTiers
-$module=(! empty($conf->global->SOCIETE_CODECLIENT_ADDON)?$conf->global->SOCIETE_CODECLIENT_ADDON:'mod_codeclient_leopard');
+$module = (!empty($conf->global->SOCIETE_CODECLIENT_ADDON) ? $conf->global->SOCIETE_CODECLIENT_ADDON : 'mod_codeclient_leopard');
 if (substr($module, 0, 15) == 'mod_codeclient_' && substr($module, -3) == 'php')
 {
-    $module = substr($module, 0, dol_strlen($module)-4);
+    $module = substr($module, 0, dol_strlen($module) - 4);
 }
-$dirsociete=array_merge(array('/core/modules/societe/'),$conf->modules_parts['societe']);
+$dirsociete = array_merge(array('/core/modules/societe/'), $conf->modules_parts['societe']);
 foreach ($dirsociete as $dirroot)
 {
-    $res=dol_include_once($dirroot.$module.'.php');
+    $res = dol_include_once($dirroot . $module . '.php');
     if ($res) break;
 }
 $modCodeClient = new $module;
 // Load object modCodeFournisseur
-$module=(! empty($conf->global->SOCIETE_CODECLIENT_ADDON)?$conf->global->SOCIETE_CODECLIENT_ADDON:'mod_codeclient_leopard');
+$module = (!empty($conf->global->SOCIETE_CODECLIENT_ADDON) ? $conf->global->SOCIETE_CODECLIENT_ADDON : 'mod_codeclient_leopard');
 if (substr($module, 0, 15) == 'mod_codeclient_' && substr($module, -3) == 'php')
 {
-    $module = substr($module, 0, dol_strlen($module)-4);
+    $module = substr($module, 0, dol_strlen($module) - 4);
 }
-$dirsociete=array_merge(array('/core/modules/societe/'),$conf->modules_parts['societe']);
+$dirsociete = array_merge(array('/core/modules/societe/'), $conf->modules_parts['societe']);
 foreach ($dirsociete as $dirroot)
 {
-    $res=dol_include_once($dirroot.$module.'.php');
+    $res = dol_include_once($dirroot . $module . '.php');
     if ($res) break;
 }
 $modCodeFournisseur = new $module;
@@ -59,7 +59,7 @@ 
 $msg = '';
 $socid = null;
 
-if (GETPOST('socid', 'int') > 0){
+if (GETPOST('socid', 'int') > 0) {
     $socid = GETPOST('socid', 'int');
     $customer->fetch($socid);
 }
@@ -70,9 +70,9 @@ 
 
         $formObject = new stdClass();
         $formObject->socid = $socid;
-        $formObject->name = GETPOST('name','alpha');
-        $formObject->firstname = GETPOST('firstname','alpha');
-        $formObject->zip = GETPOST('zipcode','alpha');
+        $formObject->name = GETPOST('name', 'alpha');
+        $formObject->firstname = GETPOST('firstname', 'alpha');
+        $formObject->zip = GETPOST('zipcode', 'alpha');
         $formObject->town = GETPOST('town', 'alpha');
         $formObject->state = GETPOST('state_id', 'int');
         $formObject->phone = GETPOST('phone', 'alpha');
@@ -89,8 +89,8 @@ 
 
         if ($validator->isValid($formObject, $_REQUEST)) {
             $createOrderCommand = new CreateOrderCommand($formObject, $user->id);
-            try{
-                $handler = new CreateOrderCommandHandler($db, $conf,$user,$langs,$modCodeClient, $modCodeFournisseur);
+            try {
+                $handler = new CreateOrderCommandHandler($db, $conf, $user, $langs, $modCodeClient, $modCodeFournisseur);
                 $handler->handle($createOrderCommand);
 
                 $msg = '<div class="success ok">Commande et tiers créés. </div>';
@@ -125,18 +125,18 @@ 
 ?>
 
     <!-- Success message with reference -->
-    <?php if($successMessage): ?>
+    <?php if ($successMessage): ?>
         <div class="bbc-style">
             <p class="cadre_msg1">
                 Vous avez généré la facture et créé un tiers.<br/>
                 Merci,
             </p>
 
-            <?php if(count($validator->getWarningMessages()) > 0):?>
+            <?php if (count($validator->getWarningMessages()) > 0):?>
                 <div class="box box-warning">
                     <p>Ceci sont des messages d'attention mais qui ne bloquent pas la création de la commande</p>
                     <ul>
-                        <?php foreach($validator->getWarningMessages() as $warningMessage): ?>
+                        <?php foreach ($validator->getWarningMessages() as $warningMessage): ?>
                             <li class="warning"><?php echo $warningMessage; ?></li>
                         <?php endforeach; ?>
                     </ul>
@@ -195,7 +195,7 @@ 
         <p>
             Cette page vous permettra de créer une commande. La commande est <b>obligatoire</b> si vous désirez faire payer les passagers directement sur le compte du club.<br>
             Si vous avez un doute sur la manière d'encoder la commande, veuillez me contacter AVANT de soumettre le dit formulaire.<br/>
-            Si vous avez <b>déjà</b> encodé une commande, et que vous voulez la retrouver veuillez vous rendre sur : <a href="<?php echo sprintf(DOL_URL_ROOT.'/commande/list.php?search_sale=%s', $user->id); ?>">mes commandes.</a>
+            Si vous avez <b>déjà</b> encodé une commande, et que vous voulez la retrouver veuillez vous rendre sur : <a href="<?php echo sprintf(DOL_URL_ROOT . '/commande/list.php?search_sale=%s', $user->id); ?>">mes commandes.</a>
         </p>
     </div>
     <form class="flight-form" name='add' method="post">
@@ -217,7 +217,7 @@ 
                         $(document).ready(function() {
                             $("#socid").change(function() {
                                 var socid = $(this).val();
-                                window.location.href = "<?php echo $_SERVER["PHP_SELF"].'?socid="+socid' ?>
+                                window.location.href = "<?php echo $_SERVER["PHP_SELF"] . '?socid="+socid' ?>
                             });
                         });
                     </script>
@@ -226,7 +226,7 @@ 
             </tr>
 
 
-            <?php if($socid === null): ?>
+            <?php if ($socid === null): ?>
                 <!-- Nom -->
                 <tr>
                     <td class="fieldrequired">
@@ -295,7 +295,7 @@ 
                         <?php echo $langs->trans('DefaultLang'); ?>
                     </td>
                     <td>
-                        <?php echo $formAdmin->select_language($conf->global->MAIN_LANG_DEFAULT,'default_lang',0,0,1,0,0,'maxwidth200onsmartphone'); ?>
+                        <?php echo $formAdmin->select_language($conf->global->MAIN_LANG_DEFAULT, 'default_lang', 0, 0, 1, 0, 0, 'maxwidth200onsmartphone'); ?>
                     </td>
                 </tr>
 
@@ -305,16 +305,16 @@ 
                         <?php echo $langs->trans('Region'); ?>
                     </td>
                     <td>
-                        <?php print $formcompany->select_state($formObject->state,'BE'); ?>
+                        <?php print $formcompany->select_state($formObject->state, 'BE'); ?>
                     </td>
                 </tr>
 
                 <?php
                 // Zip / Town
-                print '<tr><td>'.fieldLabel('Zip','zipcode').'</td><td>';
-                        print $formcompany->select_ziptown($formObject->town,'zipcode',array('town','selectcountry_id','state_id'), 0, 0, '', 'maxwidth100 quatrevingtpercent');
-                        print '</td><td>'.fieldLabel('Town','town').'</td><td>';
-                        print $formcompany->select_ziptown($formObject->zip,'town',array('zipcode','selectcountry_id','state_id'), 0, 0, '', 'maxwidth100 quatrevingtpercent');
+                print '<tr><td>' . fieldLabel('Zip', 'zipcode') . '</td><td>';
+                        print $formcompany->select_ziptown($formObject->town, 'zipcode', array('town', 'selectcountry_id', 'state_id'), 0, 0, '', 'maxwidth100 quatrevingtpercent');
+                        print '</td><td>' . fieldLabel('Town', 'town') . '</td><td>';
+                        print $formcompany->select_ziptown($formObject->zip, 'town', array('zipcode', 'selectcountry_id', 'state_id'), 0, 0, '', 'maxwidth100 quatrevingtpercent');
                         print '</td></tr>';
                 ?>
 
@@ -338,16 +338,16 @@ 
                         <?php
                         if (empty($conf->global->MAIN_DISABLEVATCHECK)): ?>
 
-                            <?php if (! empty($conf->use_javascript_ajax)): ?>
+                            <?php if (!empty($conf->use_javascript_ajax)): ?>
                                 <script language="JavaScript" type="text/javascript">
                                 function CheckVAT(a) {
-                                    <?php print "newpopup('".DOL_URL_ROOT."/societe/checkvat/checkVatPopup.php?vatNumber='+a,'".dol_escape_js($langs->trans("VATIntraCheckableOnEUSite"))."',500,300);"; ?>
+                                    <?php print "newpopup('" . DOL_URL_ROOT . "/societe/checkvat/checkVatPopup.php?vatNumber='+a,'" . dol_escape_js($langs->trans("VATIntraCheckableOnEUSite")) . "',500,300);"; ?>
                                 }
                                 </script>
                                 <a href="#" class="hideonsmartphone" onclick="javascript: CheckVAT(document.add.tva_intra.value);"><?php echo $langs->trans("VATIntraCheck"); ?></a>
-                                <?php echo $html->textwithpicto($s,$langs->trans("VATIntraCheckDesc",$langs->trans("VATIntraCheck")),1); ?>
+                                <?php echo $html->textwithpicto($s, $langs->trans("VATIntraCheckDesc", $langs->trans("VATIntraCheck")), 1); ?>
                             <?php else: ?>
-                                <a href="<?php echo $langs->transcountry("VATIntraCheckURL",$object->country_id); ?>" target="_blank"><?php echo img_picto($langs->trans("VATIntraCheckableOnEUSite"),'help'); ?></a>
+                                <a href="<?php echo $langs->transcountry("VATIntraCheckURL", $object->country_id); ?>" target="_blank"><?php echo img_picto($langs->trans("VATIntraCheckableOnEUSite"), 'help'); ?></a>
                             <?php endif; ?>
                        <?php endif; ?>
                     </td>
@@ -402,10 +402,10 @@ 
             <tr>
                 <td><?php echo $langs->trans('Le commentaire doit-il figurer sur la commande') ?></td>
                 <td>
-                    <input type="radio" id="public_comment" name="public_comment" value="1" <?php echo ($formObject->isCommentPublic == 1)?'checked="checked"' : ''; ?>/>
+                    <input type="radio" id="public_comment" name="public_comment" value="1" <?php echo ($formObject->isCommentPublic == 1) ? 'checked="checked"' : ''; ?>/>
                     <label for="public_comment">Oui</label>
                     -
-                    <input type="radio" id="private_comment" name="public_comment" value="0" <?php echo ($formObject == null || $formObject->isCommentPublic === null || $formObject->isCommentPublic === 0)?'checked="checked"' : ''; ?>/>
+                    <input type="radio" id="private_comment" name="public_comment" value="0" <?php echo ($formObject == null || $formObject->isCommentPublic === null || $formObject->isCommentPublic === 0) ? 'checked="checked"' : ''; ?>/>
                     <label for="private_comment">Non</label>
                 </td>
             </tr>