XoopsModules\Smartobject\Customtag

Complexity

Total Complexity

15

Size/Duplication

Total Lines	224
Duplicated Lines	43.75 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	23	1
A	getVar()	0	9	3
A	render()	0	9	2
A	renderWithPhp()	0	20	4
A	getXoopsCode()	0	6	1
A	getCloneLink()	0	6	1
A	emptyString()	0	4	1
B	generateTag()	98	98	1
A	getCustomtagName()	0	6	1

<?php namespace XoopsModules\Smartobject;

/*
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 */

/**
 * @copyright    XOOPS Project https://xoops.org/
 * @license      GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @package
 * @since
 * @author     XOOPS Development Team
 */

use XoopsModules\Smartobject;

// defined('XOOPS_ROOT_PATH') || die('Restricted access');

70% of this comment could be valid code. Did you maybe forget this after debugging?

//require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobject.php';

38% of this comment could be valid code. Did you maybe forget this after debugging?

/**
 * Class SmartobjectCustomtag
 */
class Customtag extends Smartobject\BaseSmartObject
{
    public $content = false;

    /**
     * SmartobjectCustomtag constructor.
     */
    public function __construct()
    {
        $this->quickInitVar('customtagid', XOBJ_DTYPE_INT, true);
        $this->quickInitVar('name', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_CUSTOMTAG_NAME, _CO_SOBJECT_CUSTOMTAG_NAME_DSC);
        $this->quickInitVar('description', XOBJ_DTYPE_TXTAREA, false, _CO_SOBJECT_CUSTOMTAG_DESCRIPTION, _CO_SOBJECT_CUSTOMTAG_DESCRIPTION_DSC);
        $this->quickInitVar('content', XOBJ_DTYPE_TXTAREA, true, _CO_SOBJECT_CUSTOMTAG_CONTENT, _CO_SOBJECT_CUSTOMTAG_CONTENT_DSC);
        $this->quickInitVar('language', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_CUSTOMTAG_LANGUAGE, _CO_SOBJECT_CUSTOMTAG_LANGUAGE_DSC);

        $this->initNonPersistableVar('dohtml', XOBJ_DTYPE_INT, 'class', 'dohtml', '', true);
        $this->initNonPersistableVar('doimage', XOBJ_DTYPE_INT, 'class', 'doimage', '', true);
        $this->initNonPersistableVar('doxcode', XOBJ_DTYPE_INT, 'class', 'doxcode', '', true);
        $this->initNonPersistableVar('dosmiley', XOBJ_DTYPE_INT, 'class', 'dosmiley', '', true);

        $this->setControl('content', [
            'name'        => 'textarea',
            'form_editor' => 'textarea',
            'form_rows'   => 25
        ]);
        $this->setControl('language', [
            'name' => 'language',
            'all'  => true
        ]);
    }

    /**
     * @param  string $key
     * @param  string $format
     * @return mixed
     */
    public function getVar($key, $format = 's')
    {
        if ('s' === $format && in_array($key, [])) {
            //            return call_user_func(array($this, $key));

72% of this comment could be valid code. Did you maybe forget this after debugging?

            return $this->{$key}();
        }

        return parent::getVar($key, $format);
    }

    /**
     * @return bool|mixed
     */
    public function render()
    {
        if (!$this->content) {
            $ret           = $this->getVar('content');
            $this->content = $ret;
        }

        return $this->content;
    }

    /**
     * @return bool|mixed|string
     */
    public function renderWithPhp()
    {
        if (!$this->content) {
            $ret           = $this->getVar('content');
            $this->content = $ret;
        } else {
            $ret = $this->content;
        }

        // check for PHP if we are not on admin side
        if (!defined('XOOPS_CPFUNC_LOADED') && !(false === strpos($ret, '[php]'))) {
            $ret = str_replace('[php]', '', $ret);
            // we have PHP code, let's evaluate
            eval($ret);

It is generally not recommended to use eval unless absolutely required.

            return '';
        }

        return $this->content;
    }

    /**
     * @return string
     */
    public function getXoopsCode()
    {
        $ret = '[customtag]' . $this->getVar('tag', 'n') . '[/customtag]';

        return $ret;
    }

    /**
     * @return string
     */
    public function getCloneLink()
    {
        $ret = '<a href="' . SMARTOBJECT_URL . 'admin/customtag.php?op=clone&customtagid=' . $this->id() . '"><img src="' . SMARTOBJECT_IMAGES_ACTIONS_URL . 'editcopy.png" style="vertical-align: middle;" alt="' . _CO_SOBJECT_CUSTOMTAG_CLONE . '" title="' . _CO_SOBJECT_CUSTOMTAG_CLONE . '"></a>';

        return $ret;
    }

    /**
     * @param $var
     * @return bool
     */
    public function emptyString($var)
    {
        return (strlen($var) > 0);
    }

    /**
     * @return mixed|string
     */
    public function generateTag()

This method seems to be duplicated in your project.

    {
        $title = rawurlencode(strtolower($this->getVar('description', 'e')));
        $title = xoops_substr($title, 0, 10, '');
        // Transformation des ponctuations
        $pattern = [
            '/%09/', // Tab
            '/%20/', // Space
            '/%21/', // !
            '/%22/', // "
            '/%23/', // #
            '/%25/', // %
            '/%26/', // &
            '/%27/', // '
            '/%28/', // (
            '/%29/', // )
            '/%2C/', // ,
            '/%2F/', // /
            '/%3A/', // :
            '/%3B/', // ;
            '/%3C/', // <
            '/%3D/', // =
            '/%3E/', // >
            '/%3F/', // ?
            '/%40/', // @
            '/%5B/', // [
            '/%5C/', // \
            '/%5D/', // ]
            '/%5E/', // ^
            '/%7B/', // {
            '/%7C/', // |
            '/%7D/', // }
            '/%7E/', // ~
            "/\./" // .
        ];
        $rep_pat = [
            '-',
            '-',
            '-',
            '-',
            '-',
            '-100',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-at-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-',
            '-'
        ];
        $title   = preg_replace($pattern, $rep_pat, $title);

        // Transformation des caractères accentués
        $pattern = [
            '/%B0/', // °
            '/%E8/', // è
            '/%E9/', // é
            '/%EA/', // ê
            '/%EB/', // ë
            '/%E7/', // ç
            '/%E0/', // à
            '/%E2/', // â
            '/%E4/', // ä
            '/%EE/', // î
            '/%EF/', // ï
            '/%F9/', // ù
            '/%FC/', // ü
            '/%FB/', // û
            '/%F4/', // ô
            '/%F6/', // ö
        ];
        $rep_pat = ['-', 'e', 'e', 'e', 'e', 'c', 'a', 'a', 'a', 'i', 'i', 'u', 'u', 'u', 'o', 'o'];
        $title   = preg_replace($pattern, $rep_pat, $title);

        $tableau = explode('-', $title); // Transforme la chaine de caract�res en tableau
        $tableau = array_filter($tableau, [$this, 'emptyString']); // Supprime les chaines vides du tableau
        $title   = implode('-', $tableau); // Transforme un tableau en chaine de caract�res s�par� par un tiret

        $title .= time();
        $title = md5($title);

        return $title;
    }

    /**
     * @return mixed
     */
    public function getCustomtagName()
    {
        $ret = $this->getVar('name');

        return $ret;
    }
}