pagantis / pagamastarde-prestashop

backward_compatibility/Context.php

Doc Comments +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
 class ControllerBackwardModule
 {
     /**
-     * @param $js_uri
+     * @param string $js_uri
      * @return void
      */
     public function addJS($js_uri)

Indentation +1 added lines, -1 removed lines

@@ -299,7 +299,7 @@
     {
         //1.3 compatibility
         if (_PS_VERSION_ >= '1.4') {
-          Tools::addJS($js_uri);
+            Tools::addJS($js_uri);
         }
     }
 

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 */
 
 if ((bool)Configuration::get('PS_MOBILE_DEVICE'))
-    require_once(_PS_MODULE_DIR_ . '/mobile_theme/Mobile_Detect.php');
+    require_once(_PS_MODULE_DIR_.'/mobile_theme/Mobile_Detect.php');
 
 // Retro 1.3, 'class_exists' cause problem with autoload...
 if (version_compare(_PS_VERSION_, '1.4', '<'))

Braces +32 added lines, -23 removed lines

@@ -24,8 +24,9 @@ 
 *  International Registered Trademark & Property of PrestaShop SA
 */
 
-if ((bool)Configuration::get('PS_MOBILE_DEVICE'))
+if ((bool)Configuration::get('PS_MOBILE_DEVICE')) {
     require_once(_PS_MODULE_DIR_ . '/mobile_theme/Mobile_Detect.php');
+}
 
 // Retro 1.3, 'class_exists' cause problem with autoload...
 if (version_compare(_PS_VERSION_, '1.4', '<'))
@@ -163,8 +164,7 @@ 
             $this->country = new Country((int)$cookie->id_country);
             $this->customer = new CustomerBackwardModule((int)$cookie->id_customer);
             $this->employee = new Employee((int)$cookie->id_employee);
-        }
-        else
+        } else
         {
             $this->currency = null;
             $this->language = null;
@@ -175,8 +175,9 @@ 
 
         $this->shop = new ShopBackwardModule();
 
-        if ((bool)Configuration::get('PS_MOBILE_DEVICE'))
-            $this->mobile_detect = new Mobile_Detect();
+        if ((bool)Configuration::get('PS_MOBILE_DEVICE')) {
+                    $this->mobile_detect = new Mobile_Detect();
+        }
     }
 
     public function getMobileDevice()
@@ -189,16 +190,19 @@ 
                 switch ((int)Configuration::get('PS_MOBILE_DEVICE'))
                 {
                     case 0: // Only for mobile device
-                        if ($this->mobile_detect->isMobile() && !$this->mobile_detect->isTablet())
-                            $this->mobile_device = true;
+                        if ($this->mobile_detect->isMobile() && !$this->mobile_detect->isTablet()) {
+                                                    $this->mobile_device = true;
+                        }
                         break;
                     case 1: // Only for touchpads
-                        if ($this->mobile_detect->isTablet() && !$this->mobile_detect->isMobile())
-                            $this->mobile_device = true;
+                        if ($this->mobile_detect->isTablet() && !$this->mobile_detect->isMobile()) {
+                                                    $this->mobile_device = true;
+                        }
                         break;
                     case 2: // For touchpad or mobile devices
-                        if ($this->mobile_detect->isMobile() || $this->mobile_detect->isTablet())
-                            $this->mobile_device = true;
+                        if ($this->mobile_detect->isMobile() || $this->mobile_detect->isTablet()) {
+                                                    $this->mobile_device = true;
+                        }
                         break;
                 }
             }
@@ -221,8 +225,9 @@ 
      */
     public static function getContext()
     {
-        if (!isset(self::$instance))
-            self::$instance = new Context();
+        if (!isset(self::$instance)) {
+                    self::$instance = new Context();
+        }
         return self::$instance;
     }
 
@@ -241,8 +246,9 @@ 
      */
     public static function shop()
     {
-        if (!self::$instance->shop->getContextType())
-            return ShopBackwardModule::CONTEXT_ALL;
+        if (!self::$instance->shop->getContextType()) {
+                    return ShopBackwardModule::CONTEXT_ALL;
+        }
         return self::$instance->shop->getContextType();
     }
 }
@@ -315,10 +321,11 @@ 
 
     public function addJquery()
     {
-        if (_PS_VERSION_ < '1.5')
-            $this->addJS(_PS_JS_DIR_.'jquery/jquery-1.4.4.min.js');
-        elseif (_PS_VERSION_ >= '1.5')
-            $this->addJS(_PS_JS_DIR_.'jquery/jquery-1.7.2.min.js');
+        if (_PS_VERSION_ < '1.5') {
+                    $this->addJS(_PS_JS_DIR_.'jquery/jquery-1.4.4.min.js');
+        } elseif (_PS_VERSION_ >= '1.5') {
+                    $this->addJS(_PS_JS_DIR_.'jquery/jquery-1.7.2.min.js');
+        }
     }
 
 }
@@ -339,12 +346,14 @@ 
      */
     public function isLogged($with_guest = false)
     {
-        if (!$with_guest && $this->is_guest == 1)
-            return false;
+        if (!$with_guest && $this->is_guest == 1) {
+                    return false;
+        }
 
         /* Customer is valid only if it can be load and if object password is the same as database one */
-        if ($this->logged == 1 && $this->id && Validate::isUnsignedId($this->id) && Customer::checkPassword($this->id, $this->passwd))
-            return true;
+        if ($this->logged == 1 && $this->id && Validate::isUnsignedId($this->id) && Customer::checkPassword($this->id, $this->passwd)) {
+                    return true;
+        }
         return false;
     }
 }

paylater.php

Indentation +7 added lines, -7 removed lines

@@ -115,7 +115,7 @@ 
             return false;
         }
 
-         return true;
+            return true;
     }
 
     public function postProcess()
@@ -295,7 +295,7 @@ 
                             'id' => 'id_type',
                             'name' => 'name'
                     )
-                 ),
+                    ),
                 array(
                     'type' => 'text',
                     'label' => $this->l('Minimum amount'),
@@ -392,7 +392,7 @@ 
                             'id' => 'id_type',
                             'name' => 'name'
                     )
-                 ),
+                    ),
                 array(
                     'type' => 'select',
                     'name' => 'PAYLATER_CART_WIDGET',
@@ -413,7 +413,7 @@ 
                             'id' => 'id_type',
                             'name' => 'name'
                     )
-                 ),
+                    ),
             ),
             'submit' => array(
                 'name' => 'submitPaylaterSettings',
@@ -501,12 +501,12 @@ 
         if (Configuration::get('PAYLATER_ENVIRONMENT') == 0) {
             if (trim(Configuration::get('PAYLATER_ACCOUNT_ID_TEST')) == false ||
                trim(Configuration::get('PAYLATER_ACCOUNT_KEY_TEST')) == false ) {
-                   return false;
+                    return false;
             }
         } else {
             if (trim(Configuration::get('PAYLATER_ACCOUNT_ID_LIVE')) == false ||
             trim(Configuration::get('PAYLATER_ACCOUNT_KEY_LIVE')) == false) {
-                  return false;
+                    return false;
             }
         }
 
@@ -523,7 +523,7 @@ 
                         'quantity' => $p['cart_quantity'],
                         'amount' => number_format($p['total_wt'], 2, '.', '')
             );
-             $desciption[]=  $p['name']. " (".$p['cart_quantity'].")";
+                $desciption[]=  $p['name']. " (".$p['cart_quantity'].")";
         }
 
         //Shipping address

Spacing +31 added lines, -31 removed lines

@@ -46,7 +46,7 @@ 
         $this->module_key = '2b9bc901b4d834bb7069e7ea6510438f';
         $this->ps_versions_compliancy['min'] = '1.5.0';
         $this->ps_versions_compliancy['max'] = '1.6.9';
-        $this->url='https://pmt.pagantis.com/v1/installments';
+        $this->url = 'https://pmt.pagantis.com/v1/installments';
         $this->product_tab = 0;
         $this->limited_currencies = array('EUR');
         self::initModuleAccess();
@@ -58,7 +58,7 @@ 
 
         /* Backward compatibility */
         if (version_compare(_PS_VERSION_, "1.5", "<")) {
-            require(_PS_MODULE_DIR_ . $this->name . '/backward_compatibility/backward.php');
+            require(_PS_MODULE_DIR_.$this->name.'/backward_compatibility/backward.php');
         }
     }
 
@@ -491,7 +491,7 @@ 
             return;
         }
         $currency_id = $params['cart']->id_currency;
-        $currency = new Currency((int) $currency_id);
+        $currency = new Currency((int)$currency_id);
         if (in_array($currency->iso_code, $this->limited_currencies) == false) {
             return false;
         }
@@ -500,7 +500,7 @@ 
         //prevent opening payment method if no credentials introduced
         if (Configuration::get('PAYLATER_ENVIRONMENT') == 0) {
             if (trim(Configuration::get('PAYLATER_ACCOUNT_ID_TEST')) == false ||
-               trim(Configuration::get('PAYLATER_ACCOUNT_KEY_TEST')) == false ) {
+               trim(Configuration::get('PAYLATER_ACCOUNT_KEY_TEST')) == false) {
                    return false;
             }
         } else {
@@ -513,26 +513,26 @@ 
         $cart_products = $this->context->cart->getProducts();
 
         $items = array();
-        $desciption=array();
+        $desciption = array();
 
         $discount = Configuration::get('PAYLATER_DISCOUNT');
-        $discount_boolean = $discount == 'true' ? 1:0;
+        $discount_boolean = $discount == 'true' ? 1 : 0;
         foreach ($cart_products as $p) {
             $items[] = array(
                         'description' => $p['name'],
                         'quantity' => $p['cart_quantity'],
                         'amount' => number_format($p['total_wt'], 2, '.', '')
             );
-             $desciption[]=  $p['name']. " (".$p['cart_quantity'].")";
+             $desciption[] = $p['name']." (".$p['cart_quantity'].")";
         }
 
         //Shipping address
         $saddress = new Address($this->context->cart->id_address_delivery);
-        $sstreet=$saddress->address1.' '.$saddress->address2;
-        $scity=$saddress->city;
+        $sstreet = $saddress->address1.' '.$saddress->address2;
+        $scity = $saddress->city;
         $suser_state = new State($saddress->id_state);
-        $sprovince=$suser_state->name;
-        $szipcode=$saddress->postcode;
+        $sprovince = $suser_state->name;
+        $szipcode = $saddress->postcode;
         $sphone = $saddress->phone;
         $smobile_phone = $saddress->phone_mobile;
         $shipping_last_updated = Tools::substr($saddress->date_upd, 0, 10);
@@ -541,11 +541,11 @@ 
 
         //billing address
         $billing_address = new Address($this->context->cart->id_address_invoice);
-        $street=$billing_address->address1.' '.$billing_address->address2;
-        $city=$billing_address->city;
+        $street = $billing_address->address1.' '.$billing_address->address2;
+        $city = $billing_address->city;
         $user_state2 = new State($billing_address->id_state);
-        $province=$user_state2->name;
-        $zipcode=$billing_address->postcode;
+        $province = $user_state2->name;
+        $zipcode = $billing_address->postcode;
         $phone = $billing_address->phone;
         $mobile_phone = $billing_address->phone_mobile;
         $billing_last_updated = Tools::substr($billing_address->date_upd, 0, 10);
@@ -557,14 +557,14 @@ 
         $dob = '';
         $dni = '';
         if (property_exists($billing_address, 'dni')) {
-            $dni=$billing_address->dni;
+            $dni = $billing_address->dni;
             $billing_dni = $billing_address->dni;
         }
 
         if (property_exists($saddress, 'dni')) {
             if (trim($dni) == '' && $saddress->firstname == $this->context->cookie->customer_firstname
               && $saddress->lastname == $this->context->cookie->customer_lastname) {
-                $dni=$saddress->dni;
+                $dni = $saddress->dni;
             }
             $shipping_dni = $saddress->dni;
         }
@@ -582,13 +582,13 @@ 
         }
 
         $link = Context::getContext()->link;
-        $callback_url =  $link->getModuleLink('paylater', 'validation', array());
+        $callback_url = $link->getModuleLink('paylater', 'validation', array());
         $confirmationQuery = array(
             'id_cart' => $this->context->cart->id,
             'key' => $this->context->cart->secure_key,
         );
 
-        $url_OK =  $link->getModuleLink('paylater', 'confirmation', $confirmationQuery);
+        $url_OK = $link->getModuleLink('paylater', 'confirmation', $confirmationQuery);
 
         if (Configuration::get('PS_ORDER_PROCESS_TYPE') == 1) {
             $url_NOK = $link->getPageLink('order-opc');
@@ -604,7 +604,7 @@ 
                         'quantity' => 1,
                         'amount' => number_format($shippingCost, 2, '.', '')
             );
-            $desciption[]= $this->l('Shipping cost');
+            $desciption[] = $this->l('Shipping cost');
         }
         if (Configuration::get('PAYLATER_ENVIRONMENT') == 1) {
             //mode live
@@ -634,7 +634,7 @@ 
                 $gender = '';
                 break;
         }
-        $discount_boolean = $discount == 'true' ? 1:0;
+        $discount_boolean = $discount == 'true' ? 1 : 0;
 
         $iframe = Configuration::get('PAYLATER_IFRAME');
 
@@ -668,7 +668,7 @@ 
             $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_TEST');
         }
 
-        $widget_type = Configuration::get('PAYLATER_WIDGET') == 'false' ?  '0' : '1';
+        $widget_type = Configuration::get('PAYLATER_WIDGET') == 'false' ? '0' : '1';
 
         $total_paid = 0;
         $num_prev_orders = 0;
@@ -704,7 +704,7 @@ 
             $db_paid_statuses = Db::getInstance()->executeS($sql);
             $paid_statuses = array();
             foreach ($db_paid_statuses as $p) {
-                $paid_statuses[]=$p['id_order_state'];
+                $paid_statuses[] = $p['id_order_state'];
             }
             if ($this->context->cookie->logged) {
                 $orders = Order::getCustomerOrders($this->context->customer->id);
@@ -738,7 +738,7 @@ 
         $opc_enabled = Configuration::get('PS_ORDER_PROCESS_TYPE') == 1 ? 1 : 0;
 
         //d($key_to_use.$account_id.$order_id.$amount.$this->context->currency->iso_code.$url_OK.$url_NOK);
-        $text=$key_to_use.$account_id.$order_id.$amount.$this->context->currency->iso_code.
+        $text = $key_to_use.$account_id.$order_id.$amount.$this->context->currency->iso_code.
         $url_OK.$url_NOK.$callback_url.$discount.$cancelled_url;
         $signature = hash('sha512', $text);
         $this->smarty->assign(array(
@@ -754,11 +754,11 @@ 
             'items' => $items,
             'signature' => $signature,
             'customer_name' => ($this->context->cookie->logged ? $this->context->cookie->customer_firstname.
-            ' '.$this->context->cookie->customer_lastname :   $customer->firstname." ".$customer->lastname),
+            ' '.$this->context->cookie->customer_lastname : $customer->firstname." ".$customer->lastname),
             'customer_email' => ($this->context->cookie->logged ? $this->context->cookie->email : $customer->email),
             'locale' => $this->context->language->iso_code,
             'cart_products' => $cart_products,
-            'billing_full_name' => $billing_address->firstname. " ".$billing_address->lastname ,
+            'billing_full_name' => $billing_address->firstname." ".$billing_address->lastname,
             'billing_dni' => $billing_dni,
             'billing_alias' => $billing_address->alias,
             'billing_company' => $billing_address->company,
@@ -777,7 +777,7 @@ 
             'shipping_alias' => $saddress->alias,
             'shipping_company' => $saddress->company,
             'shipping_vat' => $saddress->vat_number,
-            'shipping_full_name' => $saddress->firstname. " ".$saddress->lastname ,
+            'shipping_full_name' => $saddress->firstname." ".$saddress->lastname,
             'sphone' => $sphone,
             'smobile_phone' => $smobile_phone,
             'dni' => $dni,
@@ -802,7 +802,7 @@ 
             'customer_gender' => $gender,
             'opc_enabled' => $opc_enabled,
             'is_guest' => $this->context->customer->is_guest,
-            'platform' => 'prestashop ' . _PS_VERSION_,
+            'platform' => 'prestashop '._PS_VERSION_,
             'version4' => version_compare(_PS_VERSION_, "1.5", "<"),
             'version3' => version_compare(_PS_VERSION_, "1.4", "<"),
             'content' => "javascript:$('#paylater_form').submit();",
@@ -962,7 +962,7 @@ 
         }
 
         $discount = Configuration::get('PAYLATER_DISCOUNT');
-        $discount_boolean = $discount == 'true' ? 1:0;
+        $discount_boolean = $discount == 'true' ? 1 : 0;
         $this->smarty->assign(array(
             'price' => ($params['total_price']),
             'discount' => $discount_boolean,
@@ -979,7 +979,7 @@ 
     {
         if (isset($params['product'])) {
             $product = $params['product'];
-        } elseif (Tools::getValue('controller')=='product' && Tools::getValue('id_product')) {
+        } elseif (Tools::getValue('controller') == 'product' && Tools::getValue('id_product')) {
             $product = new Product(Tools::getValue('id_product'));
         } else {
             if (method_exists($this->context->controller, 'getProduct')) {
@@ -1046,7 +1046,7 @@ 
         }
 
         $discount = Configuration::get('PAYLATER_DISCOUNT');
-        $discount_boolean = $discount == 'true' ? 1:0;
+        $discount_boolean = $discount == 'true' ? 1 : 0;
         $this->smarty->assign(array(
             'price' => ($price),
             'discount' => $discount_boolean,

controllers/front/validation.php

Indentation +3 added lines, -3 removed lines

@@ -41,9 +41,9 @@ 
 
             //validate the callback
             if (Configuration::get('PAYLATER_ENVIRONMENT') == 1) {
-                  $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_LIVE');
+                    $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_LIVE');
             } else {
-                  $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_TEST');
+                    $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_TEST');
             }
             $signature_check = sha1(
                 $key_to_use .
@@ -87,7 +87,7 @@ 
                 //nothing happen
             }
         } else {
-          //nothing happen
+            //nothing happen
         }
     }
 }

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
             $currency_id = (int)Context::getContext()->currency->id;
 
             //$json = file_get_contents('php://input');
-            $json =  Tools::file_get_contents('php://input');
+            $json = Tools::file_get_contents('php://input');
             //$data = json_decode($json, true);
             $data = Tools::jsonDecode($json);
 
@@ -46,10 +46,10 @@ 
                   $key_to_use = Configuration::get('PAYLATER_ACCOUNT_KEY_TEST');
             }
             $signature_check = sha1(
-                $key_to_use .
-                $data->account_id .
-                $data->api_version .
-                $data->event .
+                $key_to_use.
+                $data->account_id.
+                $data->api_version.
+                $data->event.
                 $data->data->id
             );
             if ($signature_check != $data->signature) {

controllers/front/redirect.php

Indentation +3 added lines, -3 removed lines

@@ -26,9 +26,9 @@
 
 class PaylaterRedirectModuleFrontController extends ModuleFrontController
 {
-  /**
-    * Do whatever you have to before redirecting the customer on the website of your payment processor.
-    */
+    /**
+     * Do whatever you have to before redirecting the customer on the website of your payment processor.
+     */
     public function postProcess()
     {
         /**

controllers/front/confirmation.php

Indentation +2 added lines, -2 removed lines

@@ -46,8 +46,8 @@
         $link = Context::getContext()->link;
         $history_link = $link->getPageLink('history');
         $this->context->smarty->assign(array(
-          'total' => $order->total_paid,
-          'history_link' => $history_link
+            'total' => $order->total_paid,
+            'history_link' => $history_link
         ));
         return $this->setTemplate('confirmation.tpl');
     }

backward_compatibility/Display.php

Braces +5 added lines, -4 removed lines

@@ -32,10 +32,11 @@
     // Assign template, on 1.4 create it else assign for 1.5
     public function setTemplate($template)
     {
-        if (_PS_VERSION_ >= '1.5')
-            parent::setTemplate($template);
-        else
-            $this->template = $template;
+        if (_PS_VERSION_ >= '1.5') {
+                    parent::setTemplate($template);
+        } else {
+                    $this->template = $template;
+        }
     }
 
     // Overload displayContent for 1.4

backward_compatibility/backward.php

Indentation +3 added lines, -3 removed lines

@@ -35,9 +35,9 @@
 
 //1.3 compatibility
 if (_PS_VERSION_ >= '1.4') {
-  // Get out if the Display (BWDisplay to avoid any conflict)) is already defined
-  if (!in_array('BWDisplay', get_declared_classes()))
-      require_once(dirname(__FILE__).'/Display.php');
+    // Get out if the Display (BWDisplay to avoid any conflict)) is already defined
+    if (!in_array('BWDisplay', get_declared_classes()))
+        require_once(dirname(__FILE__).'/Display.php');
 }
 // If not under an object we don't have to set the context
 if (!isset($this))

Braces +8 added lines, -6 removed lines

@@ -30,19 +30,21 @@
  */
 
 // Get out if the context is already defined
-if (!in_array('Context', get_declared_classes()))
+if (!in_array('Context', get_declared_classes())) {
     require_once(dirname(__FILE__).'/Context.php');
+}
 
 //1.3 compatibility
 if (_PS_VERSION_ >= '1.4') {
   // Get out if the Display (BWDisplay to avoid any conflict)) is already defined
-  if (!in_array('BWDisplay', get_declared_classes()))
-      require_once(dirname(__FILE__).'/Display.php');
-}
+  if (!in_array('BWDisplay', get_declared_classes())) {
+        require_once(dirname(__FILE__).'/Display.php');
+  }
+  }
 // If not under an object we don't have to set the context
-if (!isset($this))
+if (!isset($this)) {
     return;
-else if (isset($this->context))
+} else if (isset($this->context))
 {
     // If we are under an 1.5 version and backoffice, we have to set some backward variable
     if (_PS_VERSION_ >= '1.5' && isset($this->context->employee->id) && $this->context->employee->id && isset(AdminController::$currentIndex) && !empty(AdminController::$currentIndex))