MPOS/php-mpos

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    function yy_r44(){
    $tag = trim(substr($this->yystack[$this->yyidx + -1]->minor,$this->lex->ldel_length));
    $this->_retvalue = $this->compiler->compileTag(($tag == 'else if')? 'elseif' : $tag,array(),array('if condition'=>$this->yystack[$this->yyidx + 0]->minor));
    }

• Read up
  Read up

Method _establishSocketConnection has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function _establishSocketConnection()
    {
        $host = $this->_params['host'];
        if (!empty($this->_params['protocol'])) {
            $host = $this->_params['protocol'] . '://' . $host;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    function yy_r42(){
    $tag = trim(substr($this->yystack[$this->yyidx + -1]->minor,$this->lex->ldel_length)); 
    $this->_retvalue = $this->compiler->compileTag(($tag == 'else if')? 'elseif' : $tag,array(),array('if condition'=>$this->yystack[$this->yyidx + 0]->minor));
    }

• Read up
  Read up

Method run_102 has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function run_102() {
  // Ugly but haven't found a better way
  global $setting, $config, $coin_address, $user, $mysqli;

  // Version information

Method smarty_modifier_truncate has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function smarty_modifier_truncate($string, $length = 80, $etc = '...', $break_words = false, $middle = false)
{
    if ($length == 0)
        return '';

Method isTrustedPHPDir has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function isTrustedPHPDir($filepath)
    {
        if (empty($this->trusted_dir)) {
            throw new SmartyException("directory '{$filepath}' not allowed by security setting (no trusted_dir specified)");
        }

Method get_debug_vars has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function get_debug_vars($obj)
    {
        $config_vars = $obj->config_vars;
        $tpl_vars = array();
        foreach ($obj->tpl_vars as $key => $var) {

Method populateCompiledFilepath has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function populateCompiledFilepath(Smarty_Template_Compiled $compiled, Smarty_Internal_Template $_template)
    {
        $_compile_id = isset($_template->compile_id) ? preg_replace('![^\w\|]+!', '_', $_template->compile_id) : null;
        $_filepath = $compiled->source->uid;
        // if use_sub_dirs, break file into directories

Method resetPassword has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  public function resetPassword($token, $new1, $new2) {
    $this->debug->append("STA " . __METHOD__, 4);
    if ($aToken = $this->token->getToken($token, 'password_reset')) {
      if ($new1 !== $new2) {
        $this->setErrorMessage( 'New passwords do not match' );

Method check_acl_access has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function check_acl_access($params, $smarty)
{
  $isAuthenticated = isset($_SESSION['AUTHENTICATED']) ? true : false;
  $iAclSetting = $params['acl'];
  // $params['icon'] is deprecated, only needed for mpos compatibility

Method doHeaders has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function doHeaders($text) {
    #
    # Redefined to add id and class attribute support.
    #
        # Setext-style headers:

Method doFencedCodeBlocks has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function doFencedCodeBlocks($text) {
    #
    # Adding the fenced code block syntax to regular Markdown:
    #
    # ~~~

Consider simplifying this complex logical expression.

Open

    if (isset($aTransaction['address']) &&
      $aTransaction['address'] == $aDebitTx['coin_address'] &&
      ((string)($aTransaction['amount'] + $aTransaction['fee'])    == (string)($aDebitTx['amount'] * -1) ||     // Check against transaction - Fee total
       (string)($aTransaction['amount'] + $config['txfee_manual']) == (string)($aDebitTx['amount'] * -1) ||     // Check against txfee_manual deducted
       (string)($aTransaction['amount'] + $config['txfee_auto'])   == (string)($aDebitTx['amount'] * -1) ||     // Check against txfee_auto deducted

Consider simplifying this complex logical expression.

Open

        if ($compiler->template->caching && ((!$compiler->inheritance && !$compiler->nocache && !$compiler->tag_nocache) || ($compiler->inheritance && ($compiler->nocache ||$compiler->tag_nocache)))) {
            $_caching = self::CACHING_NOCACHE_CODE;
        }

Consider simplifying this complex logical expression.

Open

        if ((!$this->_template_dir || $this->_template_dir !== $_template_dir)
                || (!$this->_config_dir || $this->_config_dir !== $_config_dir)
                || (!empty($this->secure_dir) && (!$this->_secure_dir || $this->_secure_dir !== $this->secure_dir))
        ) {
            $this->_resource_dir = array();

Method setupInlineSubTemplate has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function setupInlineSubTemplate($template, $cache_id, $compile_id, $caching, $cache_lifetime, $data, $parent_scope, $hash)

Consider simplifying this complex logical expression.

Open

              if (empty($filter['type']) || ($filter['type'] != 'Debit_AP' && $filter['type'] != 'Debit_MP' && $filter['type'] != 'TXFee' && $filter['type'] != 'Credit_PPS' && $filter['type'] != 'Fee_PPS' && $filter['type'] != 'Donation_PPS')) {
                $aFilter[] = "( b.confirmations >= " . $this->config['confirmations'] . " OR ISNULL(b.confirmations) )";
              }

Method updateAccount has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

  public function updateAccount($userID, $address, $threshold, $donate, $email, $timezone, $is_anonymous, $strToken) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  public function getAll() {
    $this->debug->append("STA " . __METHOD__, 4);
    $stmt = $this->mysqli->prepare("SELECT n.*, a.username AS author FROM $this->table AS n LEFT JOIN " . $this->user->getTableName() . " AS a ON a.id = n.account_id ORDER BY time DESC");
    if ($stmt && $stmt->execute() && $result = $stmt->get_result())
      return $result->fetch_all(MYSQLI_ASSOC);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  public function getAllActive() {
    $this->debug->append("STA " . __METHOD__, 4);
    $stmt = $this->mysqli->prepare("SELECT n.*, a.username AS author FROM $this->table AS n LEFT JOIN " . $this->user->getTableName() . " AS a ON a.id = n.account_id WHERE active = 1 ORDER BY time DESC");
    if ($stmt && $stmt->execute() && $result = $stmt->get_result())
      return $result->fetch_all(MYSQLI_ASSOC);

• Read up
  Read up