XoopsModules\Smartobject\CustomtagHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	getCustomtagsByName()	0	29	5

<?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 SmartobjectCustomtagHandler
 */
class CustomtagHandler extends Smartobject\PersistableObjectHandler
{
    public $objects = false;

    /**
     * SmartobjectCustomtagHandler constructor.
     * @param \XoopsDatabase $db
     */
    public function __construct(\XoopsDatabase $db)
    {
        parent::__construct($db, Customtag::class, 'customtagid', 'name', 'description', 'smartobject');
        $this->addPermission('view', _CO_SOBJECT_CUSTOMTAG_PERMISSION_VIEW, _CO_SOBJECT_CUSTOMTAG_PERMISSION_VIEW_DSC);
    }

    /**
     * @return array|bool
     */
    public function getCustomtagsByName()
    {
        if (!$this->objects) {
            global $xoopsConfig;

            $ret = [];

            $criteria = new \CriteriaCompo();

            $criteria_language = new \CriteriaCompo();
            $criteria_language->add(new \Criteria('language', $xoopsConfig['language']));
            $criteria_language->add(new \Criteria('language', 'all'), 'OR');
            $criteria->add($criteria_language);

            $smartobjectPermissionsHandler = new PermissionHandler($this);
            $granted_ids                   = $smartobjectPermissionsHandler->getGrantedItems('view');

            if ($granted_ids && count($granted_ids) > 0) {

The expression $granted_ids of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                $criteria->add(new \Criteria('customtagid', '(' . implode(', ', $granted_ids) . ')', 'IN'));
                $customtagsObj =& $this->getObjects($criteria, true);
                foreach ($customtagsObj as $customtagObj) {
                    $ret[$customtagObj->getVar('name')] = $customtagObj;
                }
            }
            $this->objects = $ret;

It seems like $ret of type array is incompatible with the declared type boolean of property $objects.

        }

        return $this->objects;
    }
}