actionRecorderAdmin

Complexity

Total Complexity

10

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
D	__construct()	0	37	10

<?php
/*
  $Id$

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

  Copyright (c) 2010 osCommerce

  Released under the GNU General Public License
*/

  use OSC\OM\HTML;
  use OSC\OM\OSCOM;
  use OSC\OM\Registry;

  require(OSCOM::getConfig('dir_root', 'Shop') . 'includes/classes/action_recorder.php');

  class actionRecorderAdmin extends actionRecorder {
    function __construct($module, $user_id = null, $user_name = null) {
      global $PHP_SELF;

      $this->lang = Registry::get('Language');

      $module = HTML::sanitize(str_replace(' ', '', $module));

      if (defined('MODULE_ACTION_RECORDER_INSTALLED') && tep_not_null(MODULE_ACTION_RECORDER_INSTALLED)) {
        if (tep_not_null($module) && in_array($module . '.' . substr($PHP_SELF, (strrpos($PHP_SELF, '.')+1)), explode(';', MODULE_ACTION_RECORDER_INSTALLED))) {
          if (!class_exists($module)) {
            if (is_file(OSCOM::getConfig('dir_root', 'Shop') . 'includes/modules/action_recorder/' . $module . '.' . substr($PHP_SELF, (strrpos($PHP_SELF, '.')+1)))) {
              $this->lang->loadDefinitions('Shop/modules/action_recorder/' . $module);
              include(OSCOM::getConfig('dir_root', 'Shop') . 'includes/modules/action_recorder/' . $module . '.' . substr($PHP_SELF, (strrpos($PHP_SELF, '.')+1)));
            } else {
              return false;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

            }
          }
        } else {
          return false;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

        }
      } else {
        return false;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

      }

      $this->_module = $module;

      if (!empty($user_id) && is_numeric($user_id)) {
        $this->_user_id = $user_id;
      }

      if (!empty($user_name)) {
        $this->_user_name = $user_name;
      }

      $GLOBALS[$this->_module] = new $module();
      $GLOBALS[$this->_module]->setIdentifier();
    }
  }
?>

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