pagantis / prestashop

controllers/front/log.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 
         header('HTTP/1.1 200 Ok', true, 200);
         header('Content-Type: application/json', true);
-        header('Content-Length: ' . Tools::strlen($result));
+        header('Content-Length: '.Tools::strlen($result));
 
         echo $result;
         exit();

controllers/front/api.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 
         header('HTTP/1.1 200 Ok', true, 200);
         header('Content-Type: application/json', true);
-        header('Content-Length: ' . Tools::strlen($result));
+        header('Content-Length: '.Tools::strlen($result));
 
         echo $result;
         exit();
@@ -90,7 +90,7 @@ 
 
         header('HTTP/1.1 403 Forbidden', true, 403);
         header('Content-Type: application/json', true);
-        header('Content-Length: ' . Tools::strlen($result));
+        header('Content-Length: '.Tools::strlen($result));
 
         echo $result;
         exit();

controllers/front/payment.php

Spacing +18 added lines, -18 removed lines

@@ -25,9 +25,9 @@ 
     {
         if (_PS_VERSION_ < 1.6) {
             Logger::addLog(
-                'Pagantis Exception For user ' .
-                $customer->email .
-                ' : ' .
+                'Pagantis Exception For user '.
+                $customer->email.
+                ' : '.
                 $exception->getMessage(),
                 3,
                 $exception->getCode(),
@@ -37,9 +37,9 @@ 
             );
         } else {
             PrestaShopLogger::addLog(
-                'Pagantis Exception For user ' .
-                $customer->email .
-                ' : ' .
+                'Pagantis Exception For user '.
+                $customer->email.
+                ' : '.
                 $exception->getMessage(),
                 3,
                 $exception->getCode(),
@@ -98,22 +98,22 @@ 
         );
 
         try {
-            $userAddress =  new \Pagantis\OrdersApiClient\Model\Order\User\Address();
+            $userAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
             $userAddress
                 ->setZipCode($shippingAddress->postcode)
-                ->setFullName($shippingAddress->firstname . ' ' . $shippingAddress->lastname)
+                ->setFullName($shippingAddress->firstname.' '.$shippingAddress->lastname)
                 ->setCountryCode('ES')
                 ->setCity($shippingAddress->city)
-                ->setAddress($shippingAddress->address1 . ' ' . $shippingAddress->address2)
+                ->setAddress($shippingAddress->address1.' '.$shippingAddress->address2)
             ;
 
-            $orderShippingAddress =  new \Pagantis\OrdersApiClient\Model\Order\User\Address();
+            $orderShippingAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
             $orderShippingAddress
                 ->setZipCode($shippingAddress->postcode)
-                ->setFullName($shippingAddress->firstname . ' ' . $shippingAddress->lastname)
+                ->setFullName($shippingAddress->firstname.' '.$shippingAddress->lastname)
                 ->setCountryCode('ES')
                 ->setCity($shippingAddress->city)
-                ->setAddress($shippingAddress->address1 . ' ' . $shippingAddress->address2)
+                ->setAddress($shippingAddress->address1.' '.$shippingAddress->address2)
                 ->setDni($shippingAddress->dni)
                 ->setFixPhone($shippingAddress->phone)
                 ->setMobilePhone($shippingAddress->phone_mobile)
@@ -122,10 +122,10 @@ 
             $orderBillingAddress = new \Pagantis\OrdersApiClient\Model\Order\User\Address();
             $orderBillingAddress
                 ->setZipCode($billingAddress->postcode)
-                ->setFullName($billingAddress->firstname . ' ' . $billingAddress->lastname)
+                ->setFullName($billingAddress->firstname.' '.$billingAddress->lastname)
                 ->setCountryCode('ES')
                 ->setCity($billingAddress->city)
-                ->setAddress($billingAddress->address1 . ' ' . $billingAddress->address2)
+                ->setAddress($billingAddress->address1.' '.$billingAddress->address2)
                 ->setDni($billingAddress->dni)
                 ->setFixPhone($billingAddress->phone)
                 ->setMobilePhone($billingAddress->phone_mobile)
@@ -143,7 +143,7 @@ 
                 ->setDni($shippingAddress->dni)
             ;
 
-            if ($customer->birthday!='0000-00-00') {
+            if ($customer->birthday != '0000-00-00') {
                 $orderUser->setDateOfBirth($customer->birthday);
             }
 
@@ -219,7 +219,7 @@ 
             Tools::redirect($cancelUrl);
         }
 
-        $url ='';
+        $url = '';
         try {
             $orderClient = new \Pagantis\OrdersApiClient\Client(
                 $pagantisPublicKey,
@@ -229,12 +229,12 @@ 
             if ($order instanceof \Pagantis\OrdersApiClient\Model\Order) {
                 $url = $order->getActionUrls()->getForm();
                 $orderId = $order->getId();
-                $sql = "INSERT INTO `" . _DB_PREFIX_ . "pagantis_order` (`id`, `order_id`)
+                $sql = "INSERT INTO `"._DB_PREFIX_."pagantis_order` (`id`, `order_id`)
                      VALUES ('$cart->id','$orderId') 
                      ON DUPLICATE KEY UPDATE `order_id` = '$orderId'";
                 $result = Db::getInstance()->execute($sql);
                 if (!$result) {
-                    throw new UnknownException('Unable to save pagantis-order-id in database: '. $sql);
+                    throw new UnknownException('Unable to save pagantis-order-id in database: '.$sql);
                 }
             } else {
                 throw new OrderNotFoundException();

Indentation +2 added lines, -2 removed lines

@@ -83,8 +83,8 @@
         $pagantisPublicKey = Configuration::get('pagantis_public_key');
         $pagantisPrivateKey = Configuration::get('pagantis_private_key');
         $okUrl = _PS_BASE_URL_.__PS_BASE_URI__
-                 .'index.php?canonical=true&fc=module&module=pagantis&controller=notify&'
-                 .http_build_query($query)
+                    .'index.php?canonical=true&fc=module&module=pagantis&controller=notify&'
+                    .http_build_query($query)
         ;
 
         $shippingAddress = new Address($cart->id_address_delivery);

controllers/front/AbstractController.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
     {
         $parsedUrl = parse_url($url);
         $separator = ($parsedUrl['query'] == null) ? '?' : '&';
-        $redirectUrl = $url. $separator . http_build_query($parameters);
+        $redirectUrl = $url.$separator.http_build_query($parameters);
         Tools::redirect($redirectUrl);
     }
 
@@ -76,8 +76,8 @@ 
                     $response = $exception->getMessage();
                 } else {
                     $response = 'Unable to serialize log.'.
-                        'data: '. json_encode($data).
-                        'exception: '. json_encode($exception);
+                        'data: '.json_encode($data).
+                        'exception: '.json_encode($exception);
                 }
             }
 

controllers/front/config.php

Spacing +5 added lines, -5 removed lines

@@ -18,12 +18,12 @@ 
     public function initContent()
     {
         $this->authorize();
-        $method = Tools::strtolower($_SERVER['REQUEST_METHOD']) . "Method";
+        $method = Tools::strtolower($_SERVER['REQUEST_METHOD'])."Method";
         if (method_exists($this, $method)) {
             header('HTTP/1.1 200 Ok', true, 200);
             header('Content-Type: application/json', true);
             $result = json_encode($this->{$method}());
-            header('Content-Length: ' . Tools::strlen($result));
+            header('Content-Length: '.Tools::strlen($result));
             echo $result;
             exit();
         }
@@ -38,7 +38,7 @@ 
      */
     public function getExtraConfigs()
     {
-        $sql_content = 'select * from ' . _DB_PREFIX_. 'pagantis_config';
+        $sql_content = 'select * from '._DB_PREFIX_.'pagantis_config';
         $dbConfigs = Db::getInstance()->executeS($sql_content);
 
         $simpleDbConfigs = array();
@@ -67,7 +67,7 @@ 
                     Db::getInstance()->update(
                         'pagantis_config',
                         array('value' => pSQL($value)),
-                        'config = \''. pSQL($config) .'\''
+                        'config = \''.pSQL($config).'\''
                     );
                 } else {
                     $errors[$config] = $value;
@@ -91,7 +91,7 @@ 
      */
     public function getMethod()
     {
-        $sql_content = 'select * from ' . _DB_PREFIX_. 'pagantis_config';
+        $sql_content = 'select * from '._DB_PREFIX_.'pagantis_config';
         $dbConfigs = Db::getInstance()->executeS($sql_content);
 
         $simpleDbConfigs = array();

pagantis.php

Spacing +19 added lines, -19 removed lines

@@ -11,7 +11,7 @@ 
     exit;
 }
 
-define('_PS_PAGANTIS_DIR', _PS_MODULE_DIR_. '/pagantis');
+define('_PS_PAGANTIS_DIR', _PS_MODULE_DIR_.'/pagantis');
 
 require _PS_PAGANTIS_DIR.'/vendor/autoload.php';
 
@@ -195,18 +195,18 @@ 
     public function checkHooks()
     {
         try {
-            $sql_content = 'select * from ' . _DB_PREFIX_. 'hook_module where 
-            id_module = \'' . Module::getModuleIdByName($this->name) . '\' and 
-            id_shop = \'' . Shop::getContextShopID() . '\' and 
-            id_hook = \'' . Hook::getIdByName('header') . '\'';
+            $sql_content = 'select * from '._DB_PREFIX_.'hook_module where 
+            id_module = \'' . Module::getModuleIdByName($this->name).'\' and 
+            id_shop = \'' . Shop::getContextShopID().'\' and 
+            id_hook = \'' . Hook::getIdByName('header').'\'';
             $hook_exists = Db::getInstance()->ExecuteS($sql_content);
             if (empty($hook_exists)) {
-                $sql_insert = 'insert into ' . _DB_PREFIX_.  'hook_module 
+                $sql_insert = 'insert into '._DB_PREFIX_.'hook_module 
             (id_module, id_shop, id_hook, position)
             values
-            (\''. Module::getModuleIdByName($this->name) . '\',
-            \''. Shop::getContextShopID() . '\',
-            \''. Hook::getIdByName('header') . '\',
+            (\''. Module::getModuleIdByName($this->name).'\',
+            \''. Shop::getContextShopID().'\',
+            \''. Hook::getIdByName('header').'\',
             150)';
                 Db::getInstance()->execute($sql_insert);
             }
@@ -220,7 +220,7 @@ 
      */
     public function checkEnvVariables()
     {
-        $sql_content = 'select * from ' . _DB_PREFIX_. 'pagantis_config';
+        $sql_content = 'select * from '._DB_PREFIX_.'pagantis_config';
         $dbConfigs = Db::getInstance()->executeS($sql_content);
 
         // Convert a multimple dimension array for SQL insert statements into a simple key/value
@@ -317,7 +317,7 @@ 
         } else {
             $this->context->controller->addJS('https://cdn.pagantis.com/js/pg-v2/sdk.js');
         }
-        $this->context->controller->addJS($this->getPathUri(). 'views/js/simulator.js');
+        $this->context->controller->addJS($this->getPathUri().'views/js/simulator.js');
     }
 
     /**
@@ -366,7 +366,7 @@ 
         $paymentOption
             ->setCallToActionText($pagantisTitle)
             ->setAction($link->getModuleLink('pagantis', 'payment'))
-            ->setLogo($this->getPathUri(). 'logo.gif')
+            ->setLogo($this->getPathUri().'logo.gif')
             ->setModuleName(__CLASS__)
         ;
 
@@ -396,7 +396,7 @@ 
                 'input' => array(
                     array(
                         'name' => 'pagantis_is_enabled',
-                        'type' =>  (version_compare(_PS_VERSION_, '1.6')<0) ?'radio' :'switch',
+                        'type' =>  (version_compare(_PS_VERSION_, '1.6') < 0) ? 'radio' : 'switch',
                         'label' => $this->l('Module is enabled'),
                         'prefix' => '<i class="icon icon-key"></i>',
                         'class' => 't',
@@ -436,7 +436,7 @@ 
                     ),
                     array(
                         'name' => 'pagantis_simulator_is_enabled',
-                        'type' => (version_compare(_PS_VERSION_, '1.6')<0) ?'radio' :'switch',
+                        'type' => (version_compare(_PS_VERSION_, '1.6') < 0) ? 'radio' : 'switch',
                         'label' => $this->l('Simulator is enabled'),
                         'prefix' => '<i class="icon icon-key"></i>',
                         'class' => 't',
@@ -552,7 +552,7 @@ 
             $message = $this->displayError($error);
         }
 
-        $logo = $this->getPathUri(). 'views/img/logo_pagantis.png';
+        $logo = $this->getPathUri().'views/img/logo_pagantis.png';
         $tpl = $this->local_path.'views/templates/admin/config-info.tpl';
         $this->context->smarty->assign(array(
             'logo' => $logo,
@@ -747,10 +747,10 @@ 
      */
     public function checkLogoExists()
     {
-        $logo = _PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments/'. Tools::strtolower(__CLASS__). '.png';
-        if (!file_exists($logo) && is_dir(_PS_MODULE_DIR_ . '/onepagecheckoutps/views/img/payments')) {
+        $logo = _PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments/'.Tools::strtolower(__CLASS__).'.png';
+        if (!file_exists($logo) && is_dir(_PS_MODULE_DIR_.'/onepagecheckoutps/views/img/payments')) {
             copy(
-                _PS_PAGANTIS_DIR . '/views/img/logo-64x64.png',
+                _PS_PAGANTIS_DIR.'/views/img/logo-64x64.png',
                 $logo
             );
         }
@@ -762,7 +762,7 @@ 
             return '';
         }
 
-        $sql = 'SELECT value FROM '._DB_PREFIX_.'pagantis_config where config = \'' . pSQL($config) . '\' limit 1';
+        $sql = 'SELECT value FROM '._DB_PREFIX_.'pagantis_config where config = \''.pSQL($config).'\' limit 1';
         if ($results = Db::getInstance()->ExecuteS($sql)) {
             if (is_array($results) && count($results) === 1 && isset($results[0]['value'])) {
                 return $results[0]['value'];

controllers/front/notify.php

Spacing +13 added lines, -13 removed lines

@@ -196,7 +196,7 @@ 
     private function getPagantisOrderId()
     {
         try {
-            $this->pagantisOrderId= Db::getInstance()->getValue(
+            $this->pagantisOrderId = Db::getInstance()->getValue(
                 'select order_id from '._DB_PREFIX_.'pagantis_order where id = '.$this->merchantOrderId
             );
 
@@ -285,8 +285,8 @@ 
                 Configuration::get('PS_OS_PAYMENT'),
                 $this->merchantOrder->getOrderTotal(true),
                 $this->module->displayName,
-                'pagantisOrderId: ' . $this->pagantisOrder->getId().
-                'pagantisOrderStatus: '. $this->pagantisOrder->getStatus(),
+                'pagantisOrderId: '.$this->pagantisOrder->getId().
+                'pagantisOrderStatus: '.$this->pagantisOrder->getStatus(),
                 array('transaction_id' => $this->pagantisOrderId),
                 null,
                 false,
@@ -308,9 +308,9 @@ 
             $this->orderClient->confirmOrder($this->pagantisOrderId);
             try {
                 $mode = ($_SERVER['REQUEST_METHOD'] == 'POST') ? 'NOTIFICATION' : 'REDIRECTION';
-                $message = 'Order CONFIRMED. The order was confirmed by a ' . $mode .
-                    '. Pagantis OrderId=' . $this->pagantisOrderId .
-                    '. Prestashop OrderId=' . $this->merchantOrderId;
+                $message = 'Order CONFIRMED. The order was confirmed by a '.$mode.
+                    '. Pagantis OrderId='.$this->pagantisOrderId.
+                    '. Prestashop OrderId='.$this->merchantOrderId;
                 $this->saveLog(array(
                     'message' => $message
                 ));
@@ -355,7 +355,7 @@ 
     protected function unblockConcurrency()
     {
         try {
-            Db::getInstance()->delete('pagantis_cart_process', 'timestamp < ' . (time() - 6));
+            Db::getInstance()->delete('pagantis_cart_process', 'timestamp < '.(time() - 6));
         } catch (\Exception $exception) {
             throw new ConcurrencyException();
         }
@@ -373,15 +373,15 @@ 
     {
         if ($this->merchantOrder && $this->processError === true) {
             sleep(5);
-            $id = (!is_null($this->pagantisOrder))?$this->pagantisOrder->getId():null;
-            $status = (!is_null($this->pagantisOrder))?$this->pagantisOrder->getStatus():null;
+            $id = (!is_null($this->pagantisOrder)) ? $this->pagantisOrder->getId() : null;
+            $status = (!is_null($this->pagantisOrder)) ? $this->pagantisOrder->getStatus() : null;
             $this->module->validateOrder(
                 $this->merchantOrderId,
                 Configuration::get('PS_OS_ERROR'),
                 $this->merchantOrder->getOrderTotal(true),
                 $this->module->displayName,
-                ' pagantisOrderId: ' . $id.
-                ' pagantisOrderStatus: '. $status,
+                ' pagantisOrderId: '.$id.
+                ' pagantisOrderStatus: '.$status,
                 null,
                 null,
                 false,
@@ -417,9 +417,9 @@ 
             'id_cart' => $this->merchantOrderId,
             'key' => $this->config['secureKey'],
             'id_module' => $this->module->id,
-            'id_order' => ($this->pagantisOrder)?$this->pagantisOrder->getId(): null,
+            'id_order' => ($this->pagantisOrder) ? $this->pagantisOrder->getId() : null,
         );
-        $url = ($error)? $this->config['urlKO'] : $this->config['urlOK'];
+        $url = ($error) ? $this->config['urlKO'] : $this->config['urlOK'];
         return $this->redirect($url, $parameters);
     }
 }