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/payment.php

Indentation +2 added lines, -2 removed lines

@@ -72,8 +72,8 @@
         $paylaterPublicKey = Configuration::get('pmt_public_key');
         $paylaterPrivateKey = Configuration::get('pmt_private_key');
         $okUrl = _PS_BASE_URL_.__PS_BASE_URI__
-                 .'index.php?canonical=true&fc=module&module=paylater&controller=notify&'
-                 .http_build_query($query)
+                    .'index.php?canonical=true&fc=module&module=paylater&controller=notify&'
+                    .http_build_query($query)
         ;
 
         $shippingAddress = new Address($cart->id_address_delivery);

Spacing +20 added lines, -20 removed lines

@@ -15,9 +15,9 @@ 
     {
         if (_PS_VERSION_ < 1.6) {
             Logger::addLog(
-                'PagaMasTarde Exception For user ' .
-                $customer->email .
-                ' : ' .
+                'PagaMasTarde Exception For user '.
+                $customer->email.
+                ' : '.
                 $exception->getMessage(),
                 3,
                 $exception->getCode(),
@@ -27,9 +27,9 @@ 
             );
         } else {
             PrestaShopLogger::addLog(
-                'PagaMasTarde Exception For user ' .
-                $customer->email .
-                ' : ' .
+                'PagaMasTarde Exception For user '.
+                $customer->email.
+                ' : '.
                 $exception->getMessage(),
                 3,
                 $exception->getCode(),
@@ -88,22 +88,22 @@ 
         );
 
         try {
-            $userAddress =  new \PagaMasTarde\OrdersApiClient\Model\Order\User\Address();
+            $userAddress = new \PagaMasTarde\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 \PagaMasTarde\OrdersApiClient\Model\Order\User\Address();
+            $orderShippingAddress = new \PagaMasTarde\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)
@@ -112,10 +112,10 @@ 
             $orderBillingAddress = new \PagaMasTarde\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)
@@ -133,7 +133,7 @@ 
                 ->setDni($shippingAddress->dni)
             ;
 
-            if ($customer->birthday!='0000-00-00') {
+            if ($customer->birthday != '0000-00-00') {
                 $orderUser->setDateOfBirth($customer->birthday);
             }
 
@@ -207,13 +207,13 @@ 
         } catch (\Exception $exception) {
             $this->saveLog(
                 array(
-                    'exception' => 'Exception for user ' . $customer->email . ' : ' . $exception->getMessage()
+                    'exception' => 'Exception for user '.$customer->email.' : '.$exception->getMessage()
                 )
             );
             Tools::redirect($cancelUrl);
         }
 
-        $url ='';
+        $url = '';
         try {
             $orderClient = new \PagaMasTarde\OrdersApiClient\Client(
                 $paylaterPublicKey,
@@ -224,7 +224,7 @@ 
                 $url = $order->getActionUrls()->getForm();
                 $orderId = $order->getId();
                 $result = Db::getInstance()->execute(
-                    "INSERT INTO `" . _DB_PREFIX_ . "pmt_order` (`id`, `order_id`)
+                    "INSERT INTO `"._DB_PREFIX_."pmt_order` (`id`, `order_id`)
                      VALUES ('$cart->id','$orderId') 
                      ON DUPLICATE KEY UPDATE `order_id` = '$orderId'"
                 );
@@ -237,7 +237,7 @@ 
         } catch (\Exception $exception) {
             $this->saveLog(
                 array(
-                    'exception' => 'Exception for user ' . $customer->email . ' : ' . $exception->getMessage()
+                    'exception' => 'Exception for user '.$customer->email.' : '.$exception->getMessage()
                 )
             );
             Tools::redirect($cancelUrl);
@@ -260,7 +260,7 @@ 
             } catch (\Exception $exception) {
                 $this->saveLog(
                     array(
-                        'exception' => 'Exception for user ' . $customer->email . ' : ' . $exception->getMessage()
+                        'exception' => 'Exception for user '.$customer->email.' : '.$exception->getMessage()
                     )
                 );
                 Tools::redirect($url);

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/AbstractController.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,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);
     }
 
@@ -62,7 +62,7 @@ 
             429 => "Too Many Requests",
             500 => "Internal Server Error",
         );
-        return isset($httpStatusCodes)? $httpStatusCodes[$statusCode] : $httpStatusCodes[200];
+        return isset($httpStatusCodes) ? $httpStatusCodes[$statusCode] : $httpStatusCodes[200];
     }
 
     /**

controllers/front/notify.php

Spacing +8 added lines, -8 removed lines

@@ -194,7 +194,7 @@ 
     private function getPmtOrderId()
     {
         try {
-            $this->pmtOrderId= Db::getInstance()->getValue(
+            $this->pmtOrderId = Db::getInstance()->getValue(
                 'select order_id from '._DB_PREFIX_.'pmt_order where id = '.$this->merchantOrderId
             );
 
@@ -256,7 +256,7 @@ 
     public function validateAmount()
     {
         $totalAmount = $this->pmtOrder->getShoppingCart()->getTotalAmount();
-        $merchantAmount = (int)((string) (100 * $this->merchantOrder->getOrderTotal(true)));
+        $merchantAmount = (int) ((string) (100 * $this->merchantOrder->getOrderTotal(true)));
         if ($totalAmount != $merchantAmount) {
             throw new AmountMismatchException($totalAmount, $merchantAmount);
         }
@@ -275,7 +275,7 @@ 
                 Configuration::get('PS_OS_PAYMENT'),
                 $this->merchantOrder->getOrderTotal(true),
                 $this->module->displayName,
-                'pmtOrderId: ' . $this->pmtOrder->getId(),
+                'pmtOrderId: '.$this->pmtOrder->getId(),
                 array('transaction_id' => $this->pmtOrderId),
                 null,
                 false,
@@ -334,7 +334,7 @@ 
     protected function unblockConcurrency()
     {
         try {
-            Db::getInstance()->delete('pmt_cart_process', 'timestamp < ' . (time() - 6));
+            Db::getInstance()->delete('pmt_cart_process', 'timestamp < '.(time() - 6));
         } catch (\Exception $exception) {
             throw new ConcurrencyException();
         }
@@ -349,13 +349,13 @@ 
     public function cancelProcess()
     {
         if ($this->merchantOrder) {
-            $id = (!is_null($this->pmtOrder))?$this->pmtOrder->getId():null;
+            $id = (!is_null($this->pmtOrder)) ? $this->pmtOrder->getId() : null;
             $this->module->validateOrder(
                 $this->merchantOrderId,
                 Configuration::get('PS_OS_ERROR'),
                 $this->merchantOrder->getOrderTotal(true),
                 $this->module->displayName,
-                'pmtOrderId: ' . $id,
+                'pmtOrderId: '.$id,
                 null,
                 null,
                 false,
@@ -390,9 +390,9 @@ 
             'id_cart' => $this->merchantOrderId,
             'key' => $this->config['secureKey'],
             'id_module' => $this->module->id,
-            'id_order' => ($this->pmtOrder)?$this->pmtOrder->getId(): null,
+            'id_order' => ($this->pmtOrder) ? $this->pmtOrder->getId() : null,
         );
-        $url = ($error)? $this->config['urlKO'] : $this->config['urlOK'];
+        $url = ($error) ? $this->config['urlKO'] : $this->config['urlOK'];
         return $this->redirect($url, $parameters);
     }
 }

paylater.php

Spacing +25 added lines, -25 removed lines

@@ -11,7 +11,7 @@ 
     exit;
 }
 
-define('_PS_PAYLATER_DIR', _PS_MODULE_DIR_. '/paylater');
+define('_PS_PAYLATER_DIR', _PS_MODULE_DIR_.'/paylater');
 
 require _PS_PAYLATER_DIR.'/vendor/autoload.php';
 
@@ -63,24 +63,24 @@ 
             $this->upgrade();
         } else {
             copy(
-                $_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env.dist',
-                $_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env'
+                $_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env.dist',
+                $_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env'
             );
         }
 
 
-        $sql_file = dirname($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name).'/sql/install.sql';
+        $sql_file = dirname($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name).'/sql/install.sql';
         $this->loadSQLFile($sql_file);
 
         try {
-            $envFile = new Dotenv\Dotenv($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name);
+            $envFile = new Dotenv\Dotenv($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name);
             $envFile->load();
         } catch (\Exception $exception) {
-            $this->context->controller->errors[] = $this->l('Unable to read file') .
-                ' ' . $_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env ' .
+            $this->context->controller->errors[] = $this->l('Unable to read file').
+                ' '.$_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env '.
                 $this->l('Ensure that the file exists and have the correct permissions');
-            $this->dotEnvError = $this->l('Unable to read file') .
-                ' ' . $_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env ' .
+            $this->dotEnvError = $this->l('Unable to read file').
+                ' '.$_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env '.
                 $this->l('Ensure that the file exists and have the correct permissions');
         }
 
@@ -149,16 +149,16 @@ 
     public function upgrade()
     {
 
-        if (!is_writable($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env')) {
+        if (!is_writable($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env')) {
             return false;
         }
-        $envFileVariables = $this->readEnvFileAsArray($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env');
-        $distFileVariables = $this->readEnvFileAsArray($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env.dist');
-        $distFile = Tools::file_get_contents($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env.dist');
+        $envFileVariables = $this->readEnvFileAsArray($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env');
+        $distFileVariables = $this->readEnvFileAsArray($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env.dist');
+        $distFile = Tools::file_get_contents($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env.dist');
 
         $newEnvFileArr = array_merge($distFileVariables, $envFileVariables);
         $newEnvFile = $this->replaceEnvFileValues($distFile, $newEnvFileArr);
-        file_put_contents($_SERVER['DOCUMENT_ROOT'] . '/modules/' . $this->name . '/.env', $newEnvFile);
+        file_put_contents($_SERVER['DOCUMENT_ROOT'].'/modules/'.$this->name.'/.env', $newEnvFile);
         return true;
     }
 
@@ -201,8 +201,8 @@ 
             $from = strpos($envFile, $key);
             if ($from !== false) {
                 $to = strpos($envFile, '#', $from);
-                $fromReplace = Tools::substr($envFile, $from, (($to - $from)-1));
-                $toReplace = $key . '=' . $value;
+                $fromReplace = Tools::substr($envFile, $from, (($to - $from) - 1));
+                $toReplace = $key.'='.$value;
                 $envFile = str_replace($fromReplace, $toReplace, $envFile);
             }
         }
@@ -244,9 +244,9 @@ 
         $cart                       = $this->context->cart;
         $currency                   = new Currency($cart->id_currency);
         $availableCurrencies        = array('EUR');
-        $pmtDisplayMinAmount          = getenv('PMT_DISPLAY_MIN_AMOUNT');
+        $pmtDisplayMinAmount = getenv('PMT_DISPLAY_MIN_AMOUNT');
         $pmtPublicKey               = Configuration::get('pmt_public_key');
-        $pmtPrivateKey             = Configuration::get('pmt_private_key');
+        $pmtPrivateKey = Configuration::get('pmt_private_key');
 
         return (
             $cart->getOrderTotal() >= $pmtDisplayMinAmount &&
@@ -309,7 +309,7 @@ 
         $paymentOption
             ->setCallToActionText($pmtTitle)
             ->setAction($link->getModuleLink('paylater', 'payment'))
-            ->setLogo($this->getPathUri(). 'logo.gif')
+            ->setLogo($this->getPathUri().'logo.gif')
             ->setModuleName(__CLASS__)
         ;
 
@@ -343,7 +343,7 @@ 
                 'input' => array(
                     array(
                         'name' => 'pmt_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',
@@ -383,7 +383,7 @@ 
                     ),
                     array(
                         'name' => 'pmt_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',
@@ -502,7 +502,7 @@ 
             $message = $this->displayError($this->dotEnvError);
         }
 
-        $logo = $this->getPathUri(). 'views/img/logo_pagamastarde.png';
+        $logo = $this->getPathUri().'views/img/logo_pagamastarde.png';
         $tpl = $this->local_path.'views/templates/admin/config-info.tpl';
         $this->context->smarty->assign(array(
             'logo' => $logo,
@@ -679,10 +679,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_PAYLATER_DIR . '/views/img/logo-64x64.png',
+                _PS_PAYLATER_DIR.'/views/img/logo-64x64.png',
                 $logo
             );
         }