[WIP] PHP Code Standart fix [WIP]

controllers/admin/AdminDPDAjaxOnBoardController.php

@@ -56,7 +56,7 @@ private function nextOnBoardStep($nextStep)
             return;
         }
 
-        if (!Configuration::updateValue(Config::ON_BOARD_STEP, $nextStep) ) {
+        if (!Configuration::updateValue(Config::ON_BOARD_STEP, $nextStep)) {
             $this->returnResponse(
                 array(
                     'status' => false,
@@ -104,4 +104,4 @@ private function returnResponse(array $response)
         $response = json_encode($response);
         $this->ajaxDie($response);
     }
-}
+}

controllers/admin/AdminDPDBalticsAddressTemplateController.php

@@ -58,7 +58,6 @@ public function __construct()
         $this->initList();
         $this->initForm();
         $this->addressTemplateService = $this->module->getModuleContainer()->get('invertus.dpdbaltics.service.address.address_template_service');
-
     }
 
     public function renderForm()

controllers/admin/AdminDPDBalticsAjaxController.php

@@ -30,7 +30,7 @@ public function ajaxProcessImportZones()
 
         $selectedCountry = Tools::getValue('country');
         switch ($selectedCountry) {
-            case 'latvia' :
+            case 'latvia':
                 $this->ajaxDie(json_encode($importOnLoginService->importLatviaZones()));
                 break;
             case 'lithuania':

controllers/admin/AdminDPDBalticsAjaxShipmentsController.php

@@ -198,7 +198,6 @@ public function printMultipleLabelsFromList($orderIds)
         $shipmentService = $this->module->getModuleContainer('invertus.dpdbaltics.service.shipment_service');
 
         return $shipmentService->formatMultipleLabelShipmentPrintResponse($orderIds);
-
     }
 
 
@@ -294,7 +293,7 @@ private function printLabel($shipmentId, $labelFormat, $labelPosition)
         return $labelPrintingService->setLabelOptions($shipmentId, $labelFormat, $labelPosition);
     }
 
-    private function  printReturnLabel($shipmentId)
+    private function printReturnLabel($shipmentId)
     {
         /** @var LabelApiService $labelApiService */
         $labelApiService = $this->module->getModuleContainer('invertus.dpdbaltics.service.api.label_api_service');

controllers/admin/AdminDPDBalticsCourierRequestController.php

@@ -219,7 +219,6 @@ private function initForm()
         if (Tools::getIsset('submitAdddpd_courier_request')) {
             $phoneData['sender_phone_code'] = Tools::getValue('sender_phone_code');
             $phoneData['sender_phone'] = Tools::getValue('sender_phone');
-
         } elseif (Tools::getIsset('id_dpd_courier_request')) {
 
             /** @var CourierRequestRepository $courierRequestRepository */

controllers/admin/AdminDPDBalticsImportExportController.php

@@ -433,10 +433,10 @@ private function postProcessImport($importOption, $deleteOnImport, $importFile =
         );
 
         $this->confirmations[] = sprintf(
-                $this->l('Successfully imported %s with %d row(s)'),
-                $name,
-                $importable->getImportedRowsCount()
-            ) . $break;
+            $this->l('Successfully imported %s with %d row(s)'),
+            $name,
+            $importable->getImportedRowsCount()
+        ) . $break;
 
         $this->confirmations = array_merge($this->confirmations, $importable->getConfirmations());
     }

controllers/admin/AdminDPDBalticsOrderReturnController.php

@@ -140,7 +140,8 @@ public function displayViewOrderLink($token, $idDpdShipment)
             [
                 'vieworder' => '1',
                 'id_order' => (int)$shipment->id_order
-            ]);
+            ]
+        );
 
         $params = [
             'href' => $orderUrl,

controllers/admin/AdminDPDBalticsProductsAvailabilityController.php

@@ -116,7 +116,8 @@ public function renderForm()
             $this->module->l('Friday'),
         ];
 
-        $this->context->smarty->assign([
+        $this->context->smarty->assign(
+            [
             'daysList' => $deliveryDays,
             'productId' => Tools::getValue('id_dpd_product')
             ]

controllers/admin/AdminDPDBalticsSettingsController.php

@@ -238,7 +238,6 @@ public function postProcess()
             try {
                 $regenerationHandler->handle();
                 $this->confirmations[] = $this->l('Prestashop carriers regenerated successfully');
-
             } catch (DpdCarrierException $e) {
                 $logger->error($e->getMessage());
                 $this->errors[] = $this->l('Could not regenerate carriers, please refer to module logs for more information');

controllers/admin/AdminDPDBalticsShipmentSettingsController.php

@@ -144,7 +144,6 @@ protected function initOptions()
                         $this->module->display($this->module->getPathUri(), 'views/templates/admin/redirect/redirect-closing.tpl')
                     );
             }
-
         }
 
         /** @var LabelPositionService $labelPositionService */

controllers/admin/AdminDPDBalticsZonesController.php

@@ -378,7 +378,7 @@ private function updateOnBoardAndAddTemplateToResponseOnZoneSave($response)
             $onBoardStepActionService = $this->module->getModuleContainer('invertus.dpdbaltics.on_board.service.on_board_step_action_service');
             $onBoardStepActionService->nextStep(Config::STEP_MANUAL_ZONES_6);
 
-            if(Tools::getValue('buttonName') === Config::ZONES_SAVE_AND_STAY_BUTTON) {
+            if (Tools::getValue('buttonName') === Config::ZONES_SAVE_AND_STAY_BUTTON) {
                 /** @var OnBoardService $onBoardService */
                 $onBoardService = $this->module->getModuleContainer('invertus.dpdbaltics.on_board.service.on_board_service');
                 $response['onBoardStepTemplate'] = $onBoardService->makeStepActionWithTemplateReturn(true);
@@ -387,5 +387,4 @@ private function updateOnBoardAndAddTemplateToResponseOnZoneSave($response)
 
         return $response;
     }
-
 }

controllers/front/Ajax.php

@@ -77,12 +77,12 @@ public function init()
 
     public function postProcess()
     {
-
         $action = Tools::getValue('action');
         $currentCountryProvider = $this->module->getModuleContainer('invertus.dpdbaltics.provider.current_country_provider');
         $countryCode = $currentCountryProvider->getCurrentCountryIsoCode();
         $city = Tools::getValue('city_name');
-        $carrierId = (int)Tools::getValue('id_carrier');
+        $carrierId = (int) Tools::getValue('id_carrier');
+
         switch ($action) {
             case 'searchPudoServices':
                 $cartId = Context::getContext()->cart->id;

controllers/front/ShipmentReturn.php

@@ -45,7 +45,6 @@ public function postProcess()
 
     private function printReturnLabel($orderId, $returnTemplateId)
     {
-
         $response['status'] = false;
 
         /** @var ShipmentRepository $shipmentRepo */

src/Config/Config.php

@@ -491,6 +491,7 @@ public static function getDefaultServiceWeights($countryIso, $productReference)
                     default:
                         return false;
                 }
+                // no break
             case self::LITHUANIA_ISO_CODE:
             case self::LATVIA_ISO_CODE:
                 switch ($productReference) {
@@ -504,6 +505,7 @@ public static function getDefaultServiceWeights($countryIso, $productReference)
                     default:
                         return false;
                 }
+                // no break
             case self::PORTUGAL_ISO_CODE:
                 switch ($productReference) {
                     case self::PRODUCT_TYPE_PUDO:
@@ -516,6 +518,7 @@ public static function getDefaultServiceWeights($countryIso, $productReference)
                     default:
                         return false;
                 }
+                // no break
             default:
                 switch ($productReference) {
                     case self::PRODUCT_TYPE_PUDO:
@@ -629,7 +632,6 @@ public static function getMinimalTimeIntervalForCountry($countryIso)
     public static function isPrestashopVersionAbove177()
     {
         if (_PS_VERSION_ >= self::PS_VERSION_1_7_7) {
-
             return true;
         }
 

src/Controller/OrderLabelController.php

@@ -156,7 +156,7 @@ private function printLabel($shipmentId)
                 $exceptionService->getAPIErrorMessages()
             ));
         } catch (\Exception $e) {
-            return $this->redirectWithError('admin_orders_index',$this->module->l('Failed to print label: ') . $e->getMessage());
+            return $this->redirectWithError('admin_orders_index', $this->module->l('Failed to print label: ') . $e->getMessage());
         }
 
         if (!empty($parcelPrintResponse->getErrLog())) {
@@ -179,11 +179,11 @@ private function printMultipleLabels($shipmentIds)
                 $exceptionService->getAPIErrorMessages()
             ));
         } catch (\Exception $e) {
-            return $this->redirectWithError('admin_orders_index',$this->module->l('Failed to print label: ') . $e->getMessage());
+            return $this->redirectWithError('admin_orders_index', $this->module->l('Failed to print label: ') . $e->getMessage());
         }
 
         if (!empty($parcelPrintResponse->getErrLog())) {
-            return $this->redirectWithError('admin_orders_index',$parcelPrintResponse->getErrLog());
+            return $this->redirectWithError('admin_orders_index', $parcelPrintResponse->getErrLog());
         }
 
         return null;

src/Core/Shared/Repository/LanguageRepository.php

@@ -20,7 +20,6 @@
 
 namespace Invertus\dpdBaltics\Core\Shared\Repository;
 
-
 use Invertus\dpdBaltics\Infrastructure\Repository\CollectionRepository;
 
 class LanguageRepository extends CollectionRepository implements LanguageRepositoryInterface

src/DTO/CourierRequestData.php

@@ -401,5 +401,4 @@ public function setNonStandard($nonStandard)
     {
         $this->nonStandard = $nonStandard;
     }
-
 }

src/Entity/DPDCourierRequest.php

@@ -12,7 +12,6 @@
 
 class DPDCourierRequest extends ObjectModel
 {
-
     public $sender_name;
 
     public $sender_phone_code;