XoopsModules\Yogurt\RegstepHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	delete()	0	9	2
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace XoopsModules\Yogurt;

/**
 * Extended User Profile
 *
 * 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       (c) 2000-2016 XOOPS Project (www.xoops.org)
 * @license             GNU GPL 2 (http://www.gnu.org/licenses/gpl-2.0.html)
 * @package             profile
 * @since               2.3.0
 * @author              Jan Pedersen
 * @author              Taiwen Jiang <phppp@users.sourceforge.net>
 */


/**
 * Class RegstepHandler
 */
class RegstepHandler extends \XoopsPersistableObjectHandler
{
    /**
     * @param null|object $db
     */

    public function __construct($db)
    {
        parent::__construct($db, 'yogurt_profile_regstep', Regstep::class, 'step_id', 'step_name');
    }

    /**
     * Delete an object from the database
     * @param \XoopsObject $obj
     * @param bool         $force
     *
     * @return bool
     * @see XoopsPersistableObjectHandler
     */

    public function delete(\XoopsObject $obj, $force = false)
    {
        if (parent::delete($obj, $force)) {
            $field_handler = \xoops_getModuleHandler('field');

            return $field_handler->updateAll('step_id', 0, new \Criteria('step_id', $obj->getVar('step_id')), $force);

It seems like $obj->getVar('step_id') can also be of type array and array; however, parameter $value of Criteria::__construct() does only seem to accept string, maybe add an additional type check?

The method updateAll() does not exist on XoopsObjectHandler. It seems like you code against a sub-type of XoopsObjectHandler such as XoopsPersistableObjectHandler.

        }

        return false;
    }
}