ht_pagantis

Complexity

Total Complexity

12

Size/Duplication

Total Lines	110
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getExtraConfig()	0	13	3
A	ht_pagantis()	0	10	2
A	getConfig()	0	6	1
A	isEnabled()	0	2	1
A	execute()	0	39	1
A	remove()	0	2	1
A	check()	0	2	1
A	install()	0	2	1
A	keys()	0	2	1

<?php
/*
  $Id$

  osCommerce, Open Source E-Commerce Solutions
  http://www.oscommerce.com

  Copyright (c) 2010 osCommerce

  Released under the GNU General Public License
*/

define('TABLE_PAGANTIS_CONFIG', 'pagantis_config');

class ht_pagantis {
    var $code = 'ht_pagantis';
    var $group = 'header_tags';
    var $title;
    var $description;
    var $enabled = false;


    function ht_pagantis() {

It is generally recommended to explicitly declare the visibility for methods.

        $this->title = MODULE_HEADER_TAGS_PAGANTIS_TITLE;
        $this->description = MODULE_HEADER_TAGS_PAGANTIS_DESCRIPTION;
        $this->sort_order = 0;

The property sort_order does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if ( defined('MODULE_HEADER_TAGS_PAGANTIS_STATUS') ) {
            $this->enabled = (MODULE_HEADER_TAGS_PAGANTIS_STATUS == 'True');
        }
        $this->extraConfig = $this->getExtraConfig();

The property extraConfig does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->pk = $this->getConfig('MODULE_PAYMENT_PAGANTIS_PK');

The property pk does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * @return array
     */
    private function getExtraConfig()
    {
        $checkTable = tep_db_query("SHOW TABLES LIKE '".TABLE_PAGANTIS_CONFIG."'");

The function tep_db_query was not found. Maybe you did not declare it correctly or list all dependencies?

        $response = array();
        if (tep_db_num_rows($checkTable) > 0) {

The function tep_db_num_rows was not found. Maybe you did not declare it correctly or list all dependencies?

            $query       = "select * from ".TABLE_PAGANTIS_CONFIG;
            $result      = tep_db_query($query);
            $response    = array();
            while ($resultArray = tep_db_fetch_array($result)) {

The function tep_db_fetch_array was not found. Maybe you did not declare it correctly or list all dependencies?

                $response[$resultArray['config']] = $resultArray['value'];
            }
        }
        return $response;
    }

    /**
     * @param string $config
     * @return array
     */
    private function getConfig($config = '')
    {
            $query       = "select configuration_value from ".TABLE_CONFIGURATION . " where configuration_key = '" . $config . "'";

The constant TABLE_CONFIGURATION was not found. Maybe you did not declare it correctly or list all dependencies?

            $result      = tep_db_query($query);

The function tep_db_query was not found. Maybe you did not declare it correctly or list all dependencies?

            $resultArray = tep_db_fetch_array($result);

The function tep_db_fetch_array was not found. Maybe you did not declare it correctly or list all dependencies?

            return $resultArray['configuration_value'];
    }

    function execute() {

It is generally recommended to explicitly declare the visibility for methods.

        echo "<script src='https://cdn.pagantis.com/js/pg-v2/sdk.js'></script>". PHP_EOL;
        echo '<script>'. PHP_EOL;

        echo '        function loadSimulator()'. PHP_EOL;
        echo '        {'. PHP_EOL;
        echo '           if (typeof pgSDK != \'undefined\') {'. PHP_EOL;
        echo '               var positionSelector = \'' . $this->extraConfig['PAGANTIS_SIMULATOR_CSS_POSITION_SELECTOR']. '\';'. PHP_EOL;
        echo '               var priceSelector = \'' . $this->extraConfig['PAGANTIS_SIMULATOR_CSS_PRICE_SELECTOR']. '\';'. PHP_EOL;
        echo '               var quantitySelector = \'' . $this->extraConfig['PAGANTIS_SIMULATOR_CSS_QUANTITY_SELECTOR']. '\';'. PHP_EOL;

        echo '               if (positionSelector === \'default\') {'. PHP_EOL;
        echo '                   positionSelector = \'.buttonSet\''. PHP_EOL;
        echo '               }'. PHP_EOL;

        echo '               if (priceSelector === \'default\') {'. PHP_EOL;
        echo '                   priceSelector = \'#bodyContent>form>div>h1\''. PHP_EOL;
        echo '               }'. PHP_EOL;

        echo '               pgSDK.product_simulator = {};'. PHP_EOL;
        echo '               pgSDK.product_simulator.id = \'product-simulator\';'. PHP_EOL;
        echo '               pgSDK.product_simulator.publicKey = \'' . $this->pk . '\';'. PHP_EOL;

Are you sure $this->pk of type array can be used in concatenation?

        echo '               pgSDK.product_simulator.selector = positionSelector;'. PHP_EOL;
        echo '               pgSDK.product_simulator.numInstalments = \'' . $this->extraConfig['PAGANTIS_SIMULATOR_START_INSTALLMENTS'] . '\';'. PHP_EOL;
        echo '               pgSDK.product_simulator.type = ' . $this->extraConfig['PAGANTIS_SIMULATOR_DISPLAY_TYPE'] . ';'. PHP_EOL;
        echo '               pgSDK.product_simulator.skin = ' . $this->extraConfig['PAGANTIS_SIMULATOR_DISPLAY_SKIN'] . ';'. PHP_EOL;
        echo '               pgSDK.product_simulator.position = ' . $this->extraConfig['PAGANTIS_SIMULATOR_DISPLAY_CSS_POSITION'] . ';'. PHP_EOL;
        echo '               pgSDK.product_simulator.itemAmountSelector = priceSelector;'. PHP_EOL;

        echo '               pgSDK.simulator.init(pgSDK.product_simulator);'. PHP_EOL;
        echo '               clearInterval(window.PSSimulatorId);'. PHP_EOL;
        echo '               return true;'. PHP_EOL;
        echo '           }'. PHP_EOL;
        echo '           return false;'. PHP_EOL;
        echo '       }'. PHP_EOL;
        echo '       window.PSSimulatorId = setInterval(function () {'. PHP_EOL;
        echo '          loadSimulator();'. PHP_EOL;
        echo '       }, 2000);'. PHP_EOL;
        echo '</script>'. PHP_EOL;
    }

    function isEnabled() {

It is generally recommended to explicitly declare the visibility for methods.

        return $this->enabled;
    }

    function check() {

It is generally recommended to explicitly declare the visibility for methods.

        return defined('MODULE_HEADER_TAGS_PAGANTIS_STATUS');
    }

    function install() {

It is generally recommended to explicitly declare the visibility for methods.

        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())");

The function tep_db_query was not found. Maybe you did not declare it correctly or list all dependencies?

The constant TABLE_CONFIGURATION was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    function remove() {

It is generally recommended to explicitly declare the visibility for methods.

        tep_db_query("delete from " . TABLE_CONFIGURATION . " where configuration_key in ('" . implode("', '", $this->keys()) . "')");

The constant TABLE_CONFIGURATION was not found. Maybe you did not declare it correctly or list all dependencies?

The function tep_db_query was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    function keys() {

It is generally recommended to explicitly declare the visibility for methods.

        return array('MODULE_HEADER_TAGS_PAGANTIS_STATUS');
    }
}
?>

It is not recommended to use PHP's closing tag ?> in files other than templates.