Passed
Pull Request — master (#12)
by
unknown
02:38
created
catalog/includes/modules/payment/pagantis.php 1 patch
Spacing   +54 added lines, -54 removed lines patch added patch discarded remove patch
@@ -75,8 +75,8 @@  discard block
 block discarded – undo
75 75
 
76 76
         if (strpos($_SERVER[REQUEST_URI], "checkout_payment.php") <= 0) {
77 77
             $this->title = MODULE_PAYMENT_PAGANTIS_TEXT_ADMIN_TITLE; // Payment module title in Admin
78
-        } else {
79
-            $this->title = MODULE_PAYMENT_PAGANTIS_TEXT_CHECKOUT .'<br/><br/><div class="buttonSet" style="display:none"></div><br/>'; // Payment module title in Catalog
78
+        }else {
79
+            $this->title = MODULE_PAYMENT_PAGANTIS_TEXT_CHECKOUT.'<br/><br/><div class="buttonSet" style="display:none"></div><br/>'; // Payment module title in Catalog
80 80
         }
81 81
 
82 82
         if (defined('MODULE_PAYMENT_PAGANTIS_LANG_CODE')) {
@@ -92,12 +92,12 @@  discard block
 block discarded – undo
92 92
                 "%s://%s%s%s",
93 93
                 isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off' ? 'https' : 'http',
94 94
                 $_SERVER['SERVER_NAME'],
95
-                isset($_SERVER['SERVER_PORT']) ? ":" . $_SERVER['SERVER_PORT'] : '',
95
+                isset($_SERVER['SERVER_PORT']) ? ":".$_SERVER['SERVER_PORT'] : '',
96 96
                 $_SERVER['REQUEST_URI']
97 97
             )
98 98
         );
99 99
 
100
-        $this->form_action_url = $this->base_url . '/ext/modules/payment/pagantis/bypass.php';
100
+        $this->form_action_url = $this->base_url.'/ext/modules/payment/pagantis/bypass.php';
101 101
 
102 102
         if (defined('MODULE_PAYMENT_PAGANTIS_ERROR_MESSAGE')) {
103 103
             $this->errorMessage = strtoupper(MODULE_PAYMENT_PAGANTIS_ERROR_MESSAGE);
@@ -211,12 +211,12 @@  discard block
 block discarded – undo
211 211
             $national_id = $this->getNationalId();
212 212
             $tax_id = $this->getTaxId();
213 213
 
214
-            $fullName = $order->billing['firstname'] . ' ' . $order->billing['lastname'];
214
+            $fullName = $order->billing['firstname'].' '.$order->billing['lastname'];
215 215
             if ($fullName == ' ') {
216
-                $fullName = $order->customer['firstname'] . ' ' . $order->customer['lastname'];
216
+                $fullName = $order->customer['firstname'].' '.$order->customer['lastname'];
217 217
             }
218 218
             if ($fullName == ' ') {
219
-                $fullName = $order->delivery['firstname'] . ' ' . $order->delivery['lastname'];
219
+                $fullName = $order->delivery['firstname'].' '.$order->delivery['lastname'];
220 220
             }
221 221
             $fullName = utf8_encode($fullName);
222 222
             $userAddress = new Address();
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
                     ->setQuantity(intval($item['qty']))
288 288
                     ->setDescription($item['name']);
289 289
                 if ($promotedProduct) {
290
-                    $promotedAmount+=$product->getAmount();
290
+                    $promotedAmount += $product->getAmount();
291 291
                     $promotedMessage = $product->getDescription()."-Price:".$item['final_price']."-Qty:".$product->getQuantity();
292 292
                     $metadataOrder->addMetadata('promotedProduct', $promotedMessage);
293 293
                 }
@@ -307,9 +307,9 @@  discard block
 block discarded – undo
307 307
             );
308 308
 
309 309
             $cancelUrl = trim(tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL', false));
310
-            if ($this->extraConfig['PAGANTIS_URL_KO']!='') {
310
+            if ($this->extraConfig['PAGANTIS_URL_KO'] != '') {
311 311
                 $koUrl = $this->extraConfig['PAGANTIS_URL_KO'];
312
-            } else {
312
+            }else {
313 313
                 $koUrl = $cancelUrl;
314 314
             }
315 315
 
@@ -347,25 +347,25 @@  discard block
 block discarded – undo
347 347
             if ($pagantisOrder instanceof \Pagantis\OrdersApiClient\Model\Order) {
348 348
                 $url = $pagantisOrder->getActionUrls()->getForm();
349 349
                 $this->insertRow($this->os_order_reference, $pagantisOrder->getId(), serialize($global_vars));
350
-            } else {
350
+            }else {
351 351
                 throw new OrderNotFoundException();
352 352
             }
353 353
 
354 354
             if ($url == "") {
355 355
                 throw new UnknownException(_("No ha sido posible obtener una respuesta de Pagantis"));
356
-            } else {
356
+            }else {
357 357
                 $output = "\n";
358
-                $output .= tep_draw_hidden_field("formUrl", $url) . "\n";
359
-                $output .= tep_draw_hidden_field("cancelUrl", $cancelUrl) . "\n";
358
+                $output .= tep_draw_hidden_field("formUrl", $url)."\n";
359
+                $output .= tep_draw_hidden_field("cancelUrl", $cancelUrl)."\n";
360 360
                 return $output;
361 361
 
362 362
             } //TODO IFRAME
363 363
         } catch (\Exception $exception) {
364 364
             $this->insertLog($exception);
365 365
             $output = "\n";
366
-            $output .= tep_draw_hidden_field("cancelUrl", $cancelUrl) . "\n";
367
-            $output .= tep_draw_hidden_field("errorMessage", $exception->getMessage()) . "\n";
368
-            $output .= tep_draw_hidden_field("errorCode", $exception->getCode()) . "\n";
366
+            $output .= tep_draw_hidden_field("cancelUrl", $cancelUrl)."\n";
367
+            $output .= tep_draw_hidden_field("errorMessage", $exception->getMessage())."\n";
368
+            $output .= tep_draw_hidden_field("errorCode", $exception->getCode())."\n";
369 369
             $output .= "<p>".$this->errorMessage.", <a href='$cancelUrl' style='text-decoration:underline'><b>";
370 370
             $output .= $this->errorLinkMessage." </b></a></p>";
371 371
 
@@ -429,7 +429,7 @@  discard block
 block discarded – undo
429 429
             return 'failed';
430 430
         }
431 431
 
432
-        tep_db_query("insert into " . TABLE_CONFIGURATION . "
432
+        tep_db_query("insert into ".TABLE_CONFIGURATION."
433 433
         (
434 434
             configuration_title,
435 435
             configuration_key,
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
             ',
453 453
             now()
454 454
         )");
455
-        tep_db_query("insert into " . TABLE_CONFIGURATION . "
455
+        tep_db_query("insert into ".TABLE_CONFIGURATION."
456 456
         (
457 457
             configuration_title,
458 458
             configuration_key,
@@ -472,7 +472,7 @@  discard block
 block discarded – undo
472 472
             '0',
473 473
             now()
474 474
         )");
475
-        tep_db_query("insert into " . TABLE_CONFIGURATION . "
475
+        tep_db_query("insert into ".TABLE_CONFIGURATION."
476 476
         (
477 477
             configuration_title,
478 478
             configuration_key,
@@ -492,7 +492,7 @@  discard block
 block discarded – undo
492 492
             '0',
493 493
             now()
494 494
         )");
495
-        tep_db_query("insert into " . TABLE_CONFIGURATION . "
495
+        tep_db_query("insert into ".TABLE_CONFIGURATION."
496 496
         (
497 497
             configuration_title,
498 498
             configuration_key,
@@ -524,14 +524,14 @@  discard block
 block discarded – undo
524 524
      */
525 525
     private function installPagantisTables()
526 526
     {
527
-        $sql = "CREATE TABLE IF NOT EXISTS " . TABLE_PAGANTIS_LOG . " ( 
527
+        $sql = "CREATE TABLE IF NOT EXISTS ".TABLE_PAGANTIS_LOG." ( 
528 528
                           id int NOT NULL AUTO_INCREMENT, 
529 529
                           log text NOT NULL, 
530 530
                           createdAt timestamp DEFAULT CURRENT_TIMESTAMP, 
531 531
                           UNIQUE KEY id (id))";
532 532
         tep_db_query($sql);
533 533
 
534
-        $sql = "CREATE TABLE IF NOT EXISTS " . TABLE_PAGANTIS_CONFIG . " (
534
+        $sql = "CREATE TABLE IF NOT EXISTS ".TABLE_PAGANTIS_CONFIG." (
535 535
                             id int NOT NULL AUTO_INCREMENT, 
536 536
                             config varchar(60) NOT NULL, 
537 537
                             value varchar(200) NOT NULL, 
@@ -539,10 +539,10 @@  discard block
 block discarded – undo
539 539
         tep_db_query($sql);
540 540
 
541 541
         // check if table has records
542
-        $check_query = tep_db_query("select value from " . TABLE_PAGANTIS_CONFIG);
542
+        $check_query = tep_db_query("select value from ".TABLE_PAGANTIS_CONFIG);
543 543
         if (tep_db_num_rows($check_query) === 0) {
544 544
             foreach ((array)$this->defaultConfigs as $configKey => $configValue) {
545
-                $query = "INSERT INTO " . TABLE_PAGANTIS_CONFIG . "
545
+                $query = "INSERT INTO ".TABLE_PAGANTIS_CONFIG."
546 546
                 (
547 547
                     config,
548 548
                     value
@@ -556,7 +556,7 @@  discard block
 block discarded – undo
556 556
             }
557 557
         }
558 558
 
559
-        $sql = "CREATE TABLE IF NOT EXISTS " . TABLE_PAGANTIS_ORDERS . " (
559
+        $sql = "CREATE TABLE IF NOT EXISTS ".TABLE_PAGANTIS_ORDERS." (
560 560
                             id int NOT NULL AUTO_INCREMENT, 
561 561
                             os_order_id varchar(50), 
562 562
                             os_order_reference varchar(50) NOT NULL,
@@ -565,7 +565,7 @@  discard block
 block discarded – undo
565 565
                             UNIQUE KEY id(id))";
566 566
         tep_db_query($sql);
567 567
 
568
-        $sql = "CREATE TABLE IF NOT EXISTS " . TABLE_PAGANTIS_CONCURRENCY . " (
568
+        $sql = "CREATE TABLE IF NOT EXISTS ".TABLE_PAGANTIS_CONCURRENCY." (
569 569
                             id varchar(50) NOT NULL,
570 570
                             `timestamp` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
571 571
                             UNIQUE KEY id(id))";
@@ -577,29 +577,29 @@  discard block
 block discarded – undo
577 577
      */
578 578
     public function remove()
579 579
     {
580
-        $checkTable = tep_db_query("SHOW TABLES LIKE '" . TABLE_PAGANTIS_LOG . "'");
580
+        $checkTable = tep_db_query("SHOW TABLES LIKE '".TABLE_PAGANTIS_LOG."'");
581 581
         if (tep_db_num_rows($checkTable) > 0) {
582
-            tep_db_query("drop table " . TABLE_PAGANTIS_LOG);
582
+            tep_db_query("drop table ".TABLE_PAGANTIS_LOG);
583 583
         }
584 584
 
585
-        $checkTable = tep_db_query("SHOW TABLES LIKE '" . TABLE_PAGANTIS_CONFIG . "'");
585
+        $checkTable = tep_db_query("SHOW TABLES LIKE '".TABLE_PAGANTIS_CONFIG."'");
586 586
         if (tep_db_num_rows($checkTable) > 0) {
587
-            tep_db_query("drop table " . TABLE_PAGANTIS_CONFIG);
587
+            tep_db_query("drop table ".TABLE_PAGANTIS_CONFIG);
588 588
         }
589 589
 
590
-        $checkTable = tep_db_query("SHOW TABLES LIKE '" . TABLE_PAGANTIS_ORDERS . "'");
590
+        $checkTable = tep_db_query("SHOW TABLES LIKE '".TABLE_PAGANTIS_ORDERS."'");
591 591
         if (tep_db_num_rows($checkTable) > 0) {
592
-            tep_db_query("drop table " . TABLE_PAGANTIS_ORDERS);
592
+            tep_db_query("drop table ".TABLE_PAGANTIS_ORDERS);
593 593
         }
594 594
 
595
-        $checkTable = tep_db_query("SHOW TABLES LIKE '" . TABLE_PAGANTIS_CONCURRENCY . "'");
595
+        $checkTable = tep_db_query("SHOW TABLES LIKE '".TABLE_PAGANTIS_CONCURRENCY."'");
596 596
         if (tep_db_num_rows($checkTable) > 0) {
597
-            tep_db_query("drop table " . TABLE_PAGANTIS_CONCURRENCY);
597
+            tep_db_query("drop table ".TABLE_PAGANTIS_CONCURRENCY);
598 598
         }
599 599
 
600
-        tep_db_query("DELETE FROM ". TABLE_CONFIGURATION ." where configuration_key in ('MODULE_PAYMENT_PAGANTIS_STATUS','MODULE_PAYMENT_PAGANTIS_PK','MODULE_PAYMENT_PAGANTIS_SK')");
600
+        tep_db_query("DELETE FROM ".TABLE_CONFIGURATION." where configuration_key in ('MODULE_PAYMENT_PAGANTIS_STATUS','MODULE_PAYMENT_PAGANTIS_PK','MODULE_PAYMENT_PAGANTIS_SK')");
601 601
 
602
-        $query = "delete from " . TABLE_CONFIGURATION . " where configuration_key like '%_PAGANTIS_%'";
602
+        $query = "delete from ".TABLE_CONFIGURATION." where configuration_key like '%_PAGANTIS_%'";
603 603
         tep_db_query($query);
604 604
 
605 605
         $this->uninstallSimulator();
@@ -654,13 +654,13 @@  discard block
 block discarded – undo
654 654
      */
655 655
     private function insertRow($orderId, $pagantisOrderId, $globalVars)
656 656
     {
657
-        $query = "select * from " . TABLE_PAGANTIS_ORDERS . " where os_order_reference='$orderId'";
657
+        $query = "select * from ".TABLE_PAGANTIS_ORDERS." where os_order_reference='$orderId'";
658 658
         $resultsSelect = tep_db_query($query);
659 659
         $countResults = tep_db_num_rows($resultsSelect);
660 660
         if ($countResults == 0) {
661
-            $query = "INSERT INTO " . TABLE_PAGANTIS_ORDERS . " 
661
+            $query = "INSERT INTO ".TABLE_PAGANTIS_ORDERS." 
662 662
                 (os_order_reference, pagantis_order_id, globals) values ('$orderId', '$pagantisOrderId','$globalVars')";
663
-        } else {
663
+        }else {
664 664
             $query = "UPDATE ".TABLE_PAGANTIS_ORDERS." set pagantis_order_id='$pagantisOrderId' 
665 665
                         where os_order_reference='$orderId'";
666 666
         }
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
698 698
 
699 699
         if ($this->extraConfig['PAGANTIS_PROMOTION'] == '') {
700 700
             $promotedProducts = array();
701
-        } else {
701
+        }else {
702 702
             $promotedProducts = array_values((array)unserialize($this->extraConfig['PAGANTIS_PROMOTION']));
703 703
         }
704 704
 
@@ -711,17 +711,17 @@  discard block
 block discarded – undo
711 711
     private function getDescription()
712 712
     {
713 713
         $descriptionCode = "<img src=\"images/icon_info.gif\" border=\"0\" alt=\"Info\" title=\"Info\">&nbsp;<strong>Module version:</strong> $this->version<br/><br/>";
714
-        $descriptionCode.= "<img src=\"images/icon_info.gif\" border=\"0\">&nbsp;<a href='https://developer.pagantis.com/' target=\"_blank\" style=\"text-decoration: underline; font-weight: bold;\">View Online Documentation</a><br/><br/>";
715
-        $descriptionCode.= "<img src='images/icon_popup.gif'  border='0'>        <a href='http://pagantis.com' target='_blank' style='text-decoration: underline; font-weight: bold;'>Visit Pagantis Website</a><br/><br/><br/>";
714
+        $descriptionCode .= "<img src=\"images/icon_info.gif\" border=\"0\">&nbsp;<a href='https://developer.pagantis.com/' target=\"_blank\" style=\"text-decoration: underline; font-weight: bold;\">View Online Documentation</a><br/><br/>";
715
+        $descriptionCode .= "<img src='images/icon_popup.gif'  border='0'>        <a href='http://pagantis.com' target='_blank' style='text-decoration: underline; font-weight: bold;'>Visit Pagantis Website</a><br/><br/><br/>";
716 716
 
717 717
         if (MODULE_PAYMENT_PAGANTIS_STATUS == 'True') {
718 718
             $pagantisPromotionUrl = $this->base_url.'/admin/promotion.php';
719 719
             $linkDescription = "Si deseas ofrecer financiación sin intereses para alguno de tus productos ";
720
-            $descriptionCode.= "<img src='images/icon_info.gif' border='0'/> $linkDescription<a href='$pagantisPromotionUrl' style='text-decoration: underline; font-weight: bold;'>haz click aquí</a>";
720
+            $descriptionCode .= "<img src='images/icon_info.gif' border='0'/> $linkDescription<a href='$pagantisPromotionUrl' style='text-decoration: underline; font-weight: bold;'>haz click aquí</a>";
721 721
 
722 722
             $pagantisAllowedCountriesUrl = $this->base_url.'/admin/allowedCountries.php';
723 723
             $linkDescription = "Para gestionar paises en los que operar con Pagantis ";
724
-            $descriptionCode.= "<br/><br/><img src='images/icon_info.gif' border='0'/> $linkDescription<a href='$pagantisAllowedCountriesUrl' style='text-decoration: underline; font-weight: bold;'>haz click aquí</a>";
724
+            $descriptionCode .= "<br/><br/><img src='images/icon_info.gif' border='0'/> $linkDescription<a href='$pagantisAllowedCountriesUrl' style='text-decoration: underline; font-weight: bold;'>haz click aquí</a>";
725 725
         }
726 726
 
727 727
         return $descriptionCode;
@@ -732,18 +732,18 @@  discard block
 block discarded – undo
732 732
      */
733 733
     private function installSimulator()
734 734
     {
735
-        $checkSimulator = tep_db_query("select configuration_key, configuration_value from " .TABLE_CONFIGURATION ." 
735
+        $checkSimulator = tep_db_query("select configuration_key, configuration_value from ".TABLE_CONFIGURATION." 
736 736
                                     where configuration_key like 'MODULE_HEADER_TAGS_INSTALLED'
737 737
                                     and configuration_value like '%ht_pagantis.php%';");
738 738
         if (tep_db_num_rows($checkSimulator) > 0) {
739 739
             return true;
740 740
         }
741 741
 
742
-        $query = "UPDATE " . TABLE_CONFIGURATION . " set configuration_value = concat(configuration_value, ';ht_pagantis.php')
742
+        $query = "UPDATE ".TABLE_CONFIGURATION." set configuration_value = concat(configuration_value, ';ht_pagantis.php')
743 743
                         where configuration_key like 'MODULE_HEADER_TAGS_INSTALLED'";
744 744
         tep_db_query($query);
745 745
 
746
-        tep_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) values ('Enable Pagantis Module', 'MODULE_HEADER_TAGS_PAGANTIS_STATUS', 'True', '', '6', '1', 'tep_cfg_select_option(array(\'True\', \'False\'), ', now())");
746
+        tep_db_query("insert into ".TABLE_CONFIGURATION." (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) values ('Enable Pagantis Module', 'MODULE_HEADER_TAGS_PAGANTIS_STATUS', 'True', '', '6', '1', 'tep_cfg_select_option(array(\'True\', \'False\'), ', now())");
747 747
     }
748 748
 
749 749
     /**
@@ -751,18 +751,18 @@  discard block
 block discarded – undo
751 751
      */
752 752
     private function uninstallSimulator()
753 753
     {
754
-        $checkSimulator = tep_db_query("select configuration_key, configuration_value from " .TABLE_CONFIGURATION ." 
754
+        $checkSimulator = tep_db_query("select configuration_key, configuration_value from ".TABLE_CONFIGURATION." 
755 755
                                     where configuration_key like 'MODULE_HEADER_TAGS_INSTALLED'
756 756
                                     and configuration_value like '%ht_pagantis.php%';");
757 757
         if (tep_db_num_rows($checkSimulator) == 0) {
758 758
             return true;
759 759
         }
760 760
 
761
-        $query = "UPDATE " . TABLE_CONFIGURATION . " set configuration_value = REPLACE(configuration_value, ';ht_pagantis.php', '')
761
+        $query = "UPDATE ".TABLE_CONFIGURATION." set configuration_value = REPLACE(configuration_value, ';ht_pagantis.php', '')
762 762
                         where configuration_key like 'MODULE_HEADER_TAGS_INSTALLED'";
763 763
         tep_db_query($query);
764 764
 
765
-        $query = "delete from " . TABLE_CONFIGURATION . " where configuration_key = 'MODULE_HEADER_TAGS_PAGANTIS_STATUS'";
765
+        $query = "delete from ".TABLE_CONFIGURATION." where configuration_key = 'MODULE_HEADER_TAGS_PAGANTIS_STATUS'";
766 766
         tep_db_query($query);
767 767
     }
768 768
 
@@ -772,7 +772,7 @@  discard block
 block discarded – undo
772 772
     private function insertLog($exception)
773 773
     {
774 774
         if ($exception instanceof \Exception) {
775
-            $logEntry= new LogEntry();
775
+            $logEntry = new LogEntry();
776 776
             $logEntryJson = $logEntry->error($exception)->toJson();
777 777
             $logEntryJson = addslashes($logEntryJson);
778 778
 
@@ -791,7 +791,7 @@  discard block
 block discarded – undo
791 791
             return $order->customer['national_id'];
792 792
         } elseif (isset($order->billing['piva'])) {
793 793
             return $order->billing['piva'];
794
-        } else {
794
+        }else {
795 795
             return null;
796 796
         }
797 797
     }
@@ -806,7 +806,7 @@  discard block
 block discarded – undo
806 806
             return $order->customer['tax_id'];
807 807
         } elseif (isset($order->billing['cf'])) {
808 808
             return $order->billing['cf'];
809
-        } else {
809
+        }else {
810 810
             return null;
811 811
         }
812 812
     }
Please login to merge, or discard this patch.