Passed
Pull Request — master (#45)
by Raúl
05:08
created
pagantis.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
     exit;
12 12
 }
13 13
 
14
-define('_PS_PAGANTIS_DIR', _PS_MODULE_DIR_. '/pagantis');
14
+define('_PS_PAGANTIS_DIR', _PS_MODULE_DIR_.'/pagantis');
15 15
 define('PROMOTIONS_CATEGORY', 'pagantis-promotion-product');
16 16
 define('PROMOTIONS_CATEGORY_NAME', 'Pagantis Promoted Product');
17 17
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
         if (count($langArray) != 2 && isset($lang['locale'])) {
98 98
             $langArray = explode("-", $lang['locale']);
99 99
         }
100
-        $this->language = Tools::strtoupper($langArray[count($langArray)-1]);
100
+        $this->language = Tools::strtoupper($langArray[count($langArray) - 1]);
101 101
 
102 102
         // Prevent null language detection
103 103
         $this->language = ($this->language) ? $this->language : 'ES';
@@ -202,18 +202,18 @@  discard block
 block discarded – undo
202 202
     public function checkHooks()
203 203
     {
204 204
         try {
205
-            $sql_content = 'select * from ' . _DB_PREFIX_. 'hook_module where 
206
-            id_module = \'' . Module::getModuleIdByName($this->name) . '\' and 
207
-            id_shop = \'' . Shop::getContextShopID() . '\' and 
208
-            id_hook = \'' . Hook::getIdByName('header') . '\'';
205
+            $sql_content = 'select * from '._DB_PREFIX_.'hook_module where 
206
+            id_module = \'' . Module::getModuleIdByName($this->name).'\' and 
207
+            id_shop = \'' . Shop::getContextShopID().'\' and 
208
+            id_hook = \'' . Hook::getIdByName('header').'\'';
209 209
             $hook_exists = Db::getInstance()->ExecuteS($sql_content);
210 210
             if (empty($hook_exists)) {
211
-                $sql_insert = 'insert into ' . _DB_PREFIX_.  'hook_module 
211
+                $sql_insert = 'insert into '._DB_PREFIX_.'hook_module 
212 212
             (id_module, id_shop, id_hook, position)
213 213
             values
214
-            (\''. Module::getModuleIdByName($this->name) . '\',
215
-            \''. Shop::getContextShopID() . '\',
216
-            \''. Hook::getIdByName('header') . '\',
214
+            (\''. Module::getModuleIdByName($this->name).'\',
215
+            \''. Shop::getContextShopID().'\',
216
+            \''. Hook::getIdByName('header').'\',
217 217
             150)';
218 218
                 Db::getInstance()->execute($sql_insert);
219 219
             }
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
      */
228 228
     public function checkEnvVariables()
229 229
     {
230
-        $sql_content = 'select * from ' . _DB_PREFIX_. 'pagantis_config';
230
+        $sql_content = 'select * from '._DB_PREFIX_.'pagantis_config';
231 231
         $dbConfigs = Db::getInstance()->executeS($sql_content);
232 232
 
233 233
         // Convert a multimple dimension array for SQL insert statements into a simple key/value
@@ -330,7 +330,7 @@  discard block
 block discarded – undo
330 330
         } else {
331 331
             $this->context->controller->addJS($url);
332 332
         }
333
-        $this->context->controller->addJS($this->getPathUri(). 'views/js/simulator.js');
333
+        $this->context->controller->addJS($this->getPathUri().'views/js/simulator.js');
334 334
     }
335 335
 
336 336
     /**
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
         $paymentOption
395 395
             ->setCallToActionText($pagantisTitle)
396 396
             ->setAction($link->getModuleLink('pagantis', 'payment'))
397
-            ->setLogo($this->getPathUri(). 'views/img/' . $logo)
397
+            ->setLogo($this->getPathUri().'views/img/'.$logo)
398 398
             ->setModuleName(__CLASS__)
399 399
         ;
400 400
 
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
421 421
                 'input' => array(
422 422
                     array(
423 423
                         'name' => 'pagantis_is_enabled',
424
-                        'type' =>  (version_compare(_PS_VERSION_, '1.6')<0) ?'radio' :'switch',
424
+                        'type' =>  (version_compare(_PS_VERSION_, '1.6') < 0) ? 'radio' : 'switch',
425 425
                         'label' => $this->l('Module is enabled'),
426 426
                         'prefix' => '<i class="icon icon-key"></i>',
427 427
                         'class' => 't',
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
                     ),
462 462
                     array(
463 463
                         'name' => 'pagantis_simulator_is_enabled',
464
-                        'type' => (version_compare(_PS_VERSION_, '1.6')<0) ?'radio' :'switch',
464
+                        'type' => (version_compare(_PS_VERSION_, '1.6') < 0) ? 'radio' : 'switch',
465 465
                         'label' => $this->l('Simulator is enabled'),
466 466
                         'prefix' => '<i class="icon icon-key"></i>',
467 467
                         'class' => 't',
@@ -577,9 +577,9 @@  discard block
 block discarded – undo
577 577
             $message = $this->displayError($error);
578 578
         }
579 579
 
580
-        $logo = $this->getPathUri(). 'views/img/logo_pagantis.png';
580
+        $logo = $this->getPathUri().'views/img/logo_pagantis.png';
581 581
         if ($this->language == 'ES' || $this->language == null) {
582
-            $logo = $this->getPathUri(). 'views/img/logo_pagamastarde.png';
582
+            $logo = $this->getPathUri().'views/img/logo_pagamastarde.png';
583 583
         }
584 584
         $tpl = $this->local_path.'views/templates/admin/config-info.tpl';
585 585
         $this->context->smarty->assign(array(
@@ -627,7 +627,7 @@  discard block
 block discarded – undo
627 627
         $items = $cart->getProducts(true);
628 628
         foreach ($items as $key => $item) {
629 629
             $itemCategories = ProductCore::getProductCategoriesFull($item['id_product']);
630
-            if(in_array(PROMOTIONS_CATEGORY_NAME, array_column($itemCategories, 'name')) !== false) {
630
+            if (in_array(PROMOTIONS_CATEGORY_NAME, array_column($itemCategories, 'name')) !== false) {
631 631
                 $promotedAmount += Product::getPriceStatic($item['id_product']);
632 632
             }
633 633
         }
@@ -803,17 +803,17 @@  discard block
 block discarded – undo
803 803
      */
804 804
     public function checkLogoExists()
805 805
     {
806
-        $logoPmt = _PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments/pagamastarde.png';
807
-        $logoPg = _PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments/pagantis.png';
808
-        if (!file_exists($logoPmt) && is_dir(_PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments')) {
806
+        $logoPmt = _PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments/pagamastarde.png';
807
+        $logoPg = _PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments/pagantis.png';
808
+        if (!file_exists($logoPmt) && is_dir(_PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments')) {
809 809
             copy(
810
-                _PS_PAGANTIS_DIR . '/views/img/logo_pagamastarde.png',
810
+                _PS_PAGANTIS_DIR.'/views/img/logo_pagamastarde.png',
811 811
                 $logoPmt
812 812
             );
813 813
         }
814
-        if (!file_exists($logoPg) && is_dir(_PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments')) {
814
+        if (!file_exists($logoPg) && is_dir(_PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments')) {
815 815
             copy(
816
-                _PS_PAGANTIS_DIR . '/views/img/logo_pagantis.png',
816
+                _PS_PAGANTIS_DIR.'/views/img/logo_pagantis.png',
817 817
                 $logoPg
818 818
             );
819 819
         }
@@ -829,13 +829,13 @@  discard block
 block discarded – undo
829 829
             /** @var CategoryCore $category */
830 830
             $category = new Category();
831 831
             $category->is_root_category = false;
832
-            $category->link_rewrite = array( 1=> PROMOTIONS_CATEGORY );
833
-            $category->meta_description = array( 1=> PROMOTIONS_CATEGORY );
834
-            $category->meta_keywords = array( 1=> PROMOTIONS_CATEGORY );
835
-            $category->meta_title = array( 1=> PROMOTIONS_CATEGORY );
836
-            $category->name = array( 1=> PROMOTIONS_CATEGORY_NAME );
832
+            $category->link_rewrite = array(1=> PROMOTIONS_CATEGORY);
833
+            $category->meta_description = array(1=> PROMOTIONS_CATEGORY);
834
+            $category->meta_keywords = array(1=> PROMOTIONS_CATEGORY);
835
+            $category->meta_title = array(1=> PROMOTIONS_CATEGORY);
836
+            $category->name = array(1=> PROMOTIONS_CATEGORY_NAME);
837 837
             $category->id_parent = Configuration::get('PS_HOME_CATEGORY');
838
-            $category->active=0;
838
+            $category->active = 0;
839 839
             $description = 'Pagantis: Products with this category have free financing assumed by the merchant. Use it to promote your products or brands.';
840 840
             $category->description = $this->l($description);
841 841
             $category->save();
@@ -849,7 +849,7 @@  discard block
 block discarded – undo
849 849
             return '';
850 850
         }
851 851
 
852
-        $sql = 'SELECT value FROM '._DB_PREFIX_.'pagantis_config where config = \'' . pSQL($config) . '\' limit 1';
852
+        $sql = 'SELECT value FROM '._DB_PREFIX_.'pagantis_config where config = \''.pSQL($config).'\' limit 1';
853 853
         if ($results = Db::getInstance()->ExecuteS($sql)) {
854 854
             if (is_array($results) && count($results) === 1 && isset($results[0]['value'])) {
855 855
                 return $results[0]['value'];
Please login to merge, or discard this patch.
controllers/front/payment.php 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -89,8 +89,8 @@  discard block
 block discarded – undo
89 89
         $pagantisPublicKey = Configuration::get('pagantis_public_key');
90 90
         $pagantisPrivateKey = Configuration::get('pagantis_private_key');
91 91
         $okUrl = _PS_BASE_URL_.__PS_BASE_URI__
92
-                 .'index.php?canonical=true&fc=module&module=pagantis&controller=notify&'
93
-                 .http_build_query($query)
92
+                    .'index.php?canonical=true&fc=module&module=pagantis&controller=notify&'
93
+                    .http_build_query($query)
94 94
         ;
95 95
 
96 96
         $shippingAddress = new Address($cart->id_address_delivery);
@@ -192,9 +192,9 @@  discard block
 block discarded – undo
192 192
                     $promotedAmount+=$product->getAmount();
193 193
                     $finalPrice = Product::getPriceStatic($item['id_product']);
194 194
                     $promotedMessage = 'Promoted Item: ' . $product->getDescription() .
195
-                         ' Price: ' . $finalPrice .
196
-                         ' Qty: ' . $product->getQuantity() .
197
-                         ' Item ID: ' . $item['id_product'];
195
+                            ' Price: ' . $finalPrice .
196
+                            ' Qty: ' . $product->getQuantity() .
197
+                            ' Item ID: ' . $item['id_product'];
198 198
                     $metadataOrder->addMetadata('promotedProduct', $promotedMessage);
199 199
                 }
200 200
                 $details->addProduct($product);
Please login to merge, or discard this patch.
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -25,9 +25,9 @@  discard block
 block discarded – undo
25 25
     {
26 26
         if (_PS_VERSION_ < 1.6) {
27 27
             Logger::addLog(
28
-                'Pagantis Exception For user ' .
29
-                $customer->email .
30
-                ' : ' .
28
+                'Pagantis Exception For user '.
29
+                $customer->email.
30
+                ' : '.
31 31
                 $exception->getMessage(),
32 32
                 3,
33 33
                 $exception->getCode(),
@@ -37,9 +37,9 @@  discard block
 block discarded – undo
37 37
             );
38 38
         } else {
39 39
             PrestaShopLogger::addLog(
40
-                'Pagantis Exception For user ' .
41
-                $customer->email .
42
-                ' : ' .
40
+                'Pagantis Exception For user '.
41
+                $customer->email.
42
+                ' : '.
43 43
                 $exception->getMessage(),
44 44
                 3,
45 45
                 $exception->getCode(),
@@ -105,24 +105,24 @@  discard block
 block discarded – undo
105 105
         );
106 106
 
107 107
         try {
108
-            $userAddress =  new \Pagantis\OrdersApiClient\Model\Order\User\Address();
108
+            $userAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
109 109
             $userAddress
110 110
                 ->setZipCode($shippingAddress->postcode)
111
-                ->setFullName($shippingAddress->firstname . ' ' . $shippingAddress->lastname)
111
+                ->setFullName($shippingAddress->firstname.' '.$shippingAddress->lastname)
112 112
                 ->setCountryCode($this->language)
113 113
                 ->setCity($shippingAddress->city)
114
-                ->setAddress($shippingAddress->address1 . ' ' . $shippingAddress->address2)
114
+                ->setAddress($shippingAddress->address1.' '.$shippingAddress->address2)
115 115
                 ->setTaxId($this->getTaxId($customer, $shippingAddress, $billingAddress))
116 116
                 ->setNationalId($this->getNationalId($customer, $shippingAddress, $billingAddress))
117 117
             ;
118 118
 
119
-            $orderShippingAddress =  new \Pagantis\OrdersApiClient\Model\Order\User\Address();
119
+            $orderShippingAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
120 120
             $orderShippingAddress
121 121
                 ->setZipCode($shippingAddress->postcode)
122
-                ->setFullName($shippingAddress->firstname . ' ' . $shippingAddress->lastname)
122
+                ->setFullName($shippingAddress->firstname.' '.$shippingAddress->lastname)
123 123
                 ->setCountryCode($this->language)
124 124
                 ->setCity($shippingAddress->city)
125
-                ->setAddress($shippingAddress->address1 . ' ' . $shippingAddress->address2)
125
+                ->setAddress($shippingAddress->address1.' '.$shippingAddress->address2)
126 126
                 ->setTaxId($this->getTaxId($customer, $shippingAddress, $billingAddress))
127 127
                 ->setNationalId($this->getNationalId($customer, $shippingAddress, $billingAddress))
128 128
                 ->setFixPhone($shippingAddress->phone)
@@ -132,10 +132,10 @@  discard block
 block discarded – undo
132 132
             $orderBillingAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
133 133
             $orderBillingAddress
134 134
                 ->setZipCode($billingAddress->postcode)
135
-                ->setFullName($billingAddress->firstname . ' ' . $billingAddress->lastname)
135
+                ->setFullName($billingAddress->firstname.' '.$billingAddress->lastname)
136 136
                 ->setCountryCode($this->language)
137 137
                 ->setCity($billingAddress->city)
138
-                ->setAddress($billingAddress->address1 . ' ' . $billingAddress->address2)
138
+                ->setAddress($billingAddress->address1.' '.$billingAddress->address2)
139 139
                 ->setTaxId($this->getTaxId($customer, $billingAddress, $shippingAddress))
140 140
                 ->setNationalId($this->getNationalId($customer, $billingAddress, $shippingAddress))
141 141
                 ->setFixPhone($billingAddress->phone)
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
                 ->setNationalId($this->getNationalId($customer, $shippingAddress, $billingAddress))
156 156
             ;
157 157
 
158
-            if ($customer->birthday!='0000-00-00') {
158
+            if ($customer->birthday != '0000-00-00') {
159 159
                 $orderUser->setDateOfBirth($customer->birthday);
160 160
             }
161 161
 
@@ -189,12 +189,12 @@  discard block
 block discarded – undo
189 189
                     ->setQuantity($item['quantity'])
190 190
                     ->setDescription($item['name']);
191 191
                 if ($promotedProduct) {
192
-                    $promotedAmount+=$product->getAmount();
192
+                    $promotedAmount += $product->getAmount();
193 193
                     $finalPrice = Product::getPriceStatic($item['id_product']);
194
-                    $promotedMessage = 'Promoted Item: ' . $product->getDescription() .
195
-                         ' Price: ' . $finalPrice .
196
-                         ' Qty: ' . $product->getQuantity() .
197
-                         ' Item ID: ' . $item['id_product'];
194
+                    $promotedMessage = 'Promoted Item: '.$product->getDescription().
195
+                         ' Price: '.$finalPrice.
196
+                         ' Qty: '.$product->getQuantity().
197
+                         ' Item ID: '.$item['id_product'];
198 198
                     $metadataOrder->addMetadata('promotedProduct', $promotedMessage);
199 199
                 }
200 200
                 $details->addProduct($product);
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
244 244
             Tools::redirect($cancelUrl);
245 245
         }
246 246
 
247
-        $url ='';
247
+        $url = '';
248 248
         try {
249 249
             $orderClient = new \Pagantis\OrdersApiClient\Client(
250 250
                 $pagantisPublicKey,
@@ -255,12 +255,12 @@  discard block
 block discarded – undo
255 255
             if ($order instanceof \Pagantis\OrdersApiClient\Model\Order) {
256 256
                 $url = $order->getActionUrls()->getForm();
257 257
                 $orderId = $order->getId();
258
-                $sql = "INSERT INTO `" . _DB_PREFIX_ . "pagantis_order` (`id`, `order_id`)
258
+                $sql = "INSERT INTO `"._DB_PREFIX_."pagantis_order` (`id`, `order_id`)
259 259
                      VALUES ('$cart->id','$orderId') 
260 260
                      ON DUPLICATE KEY UPDATE `order_id` = '$orderId'";
261 261
                 $result = Db::getInstance()->execute($sql);
262 262
                 if (!$result) {
263
-                    throw new UnknownException('Unable to save pagantis-order-id in database: '. $sql);
263
+                    throw new UnknownException('Unable to save pagantis-order-id in database: '.$sql);
264 264
                 }
265 265
             } else {
266 266
                 throw new OrderNotFoundException();
Please login to merge, or discard this patch.
controllers/front/notify.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
     private function getPagantisOrderId()
196 196
     {
197 197
         try {
198
-            $this->pagantisOrderId= Db::getInstance()->getValue(
198
+            $this->pagantisOrderId = Db::getInstance()->getValue(
199 199
                 'select order_id from '._DB_PREFIX_.'pagantis_order where id = '.$this->merchantOrderId
200 200
             );
201 201
 
@@ -277,14 +277,14 @@  discard block
 block discarded – undo
277 277
         $merchantAmount = explode('.', explode(',', $merchantAmount)[0])[0];
278 278
         if ($totalAmount != $merchantAmount) {
279 279
             try {
280
-                $psTotalAmount = substr_replace($merchantAmount, '.', (Tools::strlen($merchantAmount) -2), 0);
280
+                $psTotalAmount = substr_replace($merchantAmount, '.', (Tools::strlen($merchantAmount) - 2), 0);
281 281
 
282 282
                 $pgTotalAmountInCents = (string) $this->pagantisOrder->getShoppingCart()->getTotalAmount();
283
-                $pgTotalAmount = substr_replace($pgTotalAmountInCents, '.', (Tools::strlen($pgTotalAmountInCents) -2), 0);
283
+                $pgTotalAmount = substr_replace($pgTotalAmountInCents, '.', (Tools::strlen($pgTotalAmountInCents) - 2), 0);
284 284
 
285
-                $this->amountMismatchError = '. Amount mismatch in PrestaShop Order #'. $this->merchantOrderId .
286
-                    ' compared with Pagantis Order: ' . $this->pagantisOrderId . '. The order in PrestaShop has an amount'.
287
-                    ' of ' . $psTotalAmount . ' and in Pagantis ' . $pgTotalAmount . ' PLEASE REVIEW THE ORDER';
285
+                $this->amountMismatchError = '. Amount mismatch in PrestaShop Order #'.$this->merchantOrderId.
286
+                    ' compared with Pagantis Order: '.$this->pagantisOrderId.'. The order in PrestaShop has an amount'.
287
+                    ' of '.$psTotalAmount.' and in Pagantis '.$pgTotalAmount.' PLEASE REVIEW THE ORDER';
288 288
                 $this->saveLog(array(
289 289
                     'message' => $this->amountMismatchError
290 290
                 ));
@@ -315,9 +315,9 @@  discard block
 block discarded – undo
315 315
                 Configuration::get('PS_OS_PAYMENT'),
316 316
                 $this->merchantOrder->getOrderTotal(true),
317 317
                 $this->module->displayName,
318
-                'pagantisOrderId: ' . $this->pagantisOrder->getId() . ' ' .
319
-                'pagantisOrderStatus: '. $this->pagantisOrder->getStatus() .
320
-                $this->amountMismatchError .
318
+                'pagantisOrderId: '.$this->pagantisOrder->getId().' '.
319
+                'pagantisOrderStatus: '.$this->pagantisOrder->getStatus().
320
+                $this->amountMismatchError.
321 321
                 $metadataInfo,
322 322
                 array('transaction_id' => $this->pagantisOrderId),
323 323
                 null,
@@ -340,9 +340,9 @@  discard block
 block discarded – undo
340 340
             $this->orderClient->confirmOrder($this->pagantisOrderId);
341 341
             try {
342 342
                 $mode = ($_SERVER['REQUEST_METHOD'] == 'POST') ? 'NOTIFICATION' : 'REDIRECTION';
343
-                $message = 'Order CONFIRMED. The order was confirmed by a ' . $mode .
344
-                    '. Pagantis OrderId=' . $this->pagantisOrderId .
345
-                    '. Prestashop OrderId=' . $this->merchantOrderId;
343
+                $message = 'Order CONFIRMED. The order was confirmed by a '.$mode.
344
+                    '. Pagantis OrderId='.$this->pagantisOrderId.
345
+                    '. Prestashop OrderId='.$this->merchantOrderId;
346 346
                 $this->saveLog(array(
347 347
                     'message' => $message
348 348
                 ));
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
     protected function unblockConcurrency()
399 399
     {
400 400
         try {
401
-            Db::getInstance()->delete('pagantis_cart_process', 'timestamp < ' . (time() - 6));
401
+            Db::getInstance()->delete('pagantis_cart_process', 'timestamp < '.(time() - 6));
402 402
         } catch (\Exception $exception) {
403 403
             throw new ConcurrencyException();
404 404
         }
@@ -442,9 +442,9 @@  discard block
 block discarded – undo
442 442
             'id_cart' => $this->merchantOrderId,
443 443
             'key' => $this->config['secureKey'],
444 444
             'id_module' => $this->module->id,
445
-            'id_order' => ($this->pagantisOrder)?$this->pagantisOrder->getId(): null,
445
+            'id_order' => ($this->pagantisOrder) ? $this->pagantisOrder->getId() : null,
446 446
         );
447
-        $url = ($error)? $this->config['urlKO'] : $this->config['urlOK'];
447
+        $url = ($error) ? $this->config['urlKO'] : $this->config['urlOK'];
448 448
         return $this->redirect($url, $parameters);
449 449
     }
450 450
 }
Please login to merge, or discard this patch.