MyiframeBaseHandler::insert() - Code Metrics - Inspection of "Myiframe 1.67.0 Beta 1" - XoopsModules25x/myiframe - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Pull Request — master (#8)

by Michael
created 2022-03-28 12:17 UTC
MyiframeBaseHandler::insert() B

↳ Parent: MyiframeBaseHandler
Complexity

Conditions	10
Paths	28
Size

Total Lines	70
Code Lines	54
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	10
eloc	54
c	0
b	0
f	0
nc	28
nop	2
dl	0
loc	70
rs	7.1369
How to fix Long Method Complexity

<?php declare(strict_types=1);

namespace XoopsModules\Myiframe;

/**
 * ****************************************************************************
 * MYIFRAME - MODULE FOR XOOPS
 * Copyright (c) Hervé Thouzard of Instant Zero (https://www.instant-zero.com)
 * ****************************************************************************
 */
require_once XOOPS_ROOT_PATH . '/kernel/object.php';

/**
 * Class MyiframeBaseHandler
 */
class MyiframeBaseHandler extends \XoopsObjectHandler
{
    /**
     * @param bool $isNew
     * @return \XoopsModules\Myiframe\MyiframeBase
     */
    public function create($isNew = true)
    {
        $object = new MyiframeBase();
        if ($isNew) {
            $object->setNew();
        }

        return $object;
    }

    /**
     * @param int $id
     * @return \XoopsModules\Myiframe\MyiframeBase|null
     */
    public function get($id)
    {
        $ret = null;
        $sql = 'SELECT * FROM ' . $this->db->prefix('myiframe') . '  WHERE frame_frameid=' . (int)$id;
        if (!$result = $this->db->query($sql)) {
            return $ret;
        }
        $numrows = $this->db->getRowsNum($result);
        if (1 == $numrows) {
            $object = new MyiframeBase();
            $object->assignVars($this->db->fetchArray($result));

            return $object;
        }

        return $ret;
    }

    /**
     * @param bool $force
     * @return bool
     */
    public function insert(\XoopsObject $object, $force = false)
    {
        if (!$object instanceof MyiframeBase) {
            return false;
        }
        if (!$object->isDirty()) {
            return true;
        }
        if (!$object->cleanVars()) {
            foreach ($object->getErrors() as $oneerror) {
                \trigger_error($oneerror);
            }

            return false;
        }
        foreach ($object->cleanVars as $k => $v) {
            ${$k} = $v;
        }

        if ($object->isNew()) {
            $format = 'INSERT INTO %s (frame_created, frame_uid, frame_description, frame_width, frame_height, frame_align, frame_frameborder, frame_marginwidth, frame_marginheight, frame_scrolling, frame_hits, frame_url) VALUES (%u, %u, %s, %s, %s, %d, %d, %d, %d, %d, %u, %s)';
            $sql    = \sprintf(
                $format,
                $this->db->prefix('myiframe'),
                $frame_created,

                $frame_uid,

                $this->db->quoteString($frame_description),

                $this->db->quoteString($frame_width),

                $this->db->quoteString($frame_height),

                $frame_align,

                $frame_frameborder,

                $frame_marginwidth,

                $frame_marginheight,

                $frame_scrolling,

                $frame_hits,

                $this->db->quoteString($frame_url)

            );
            $force  = true;
        } else {
            $format = 'UPDATE %s SET frame_description=%s, frame_width=%s, frame_height=%s, frame_align="%d", frame_frameborder="%d", frame_marginwidth="%d", frame_marginheight="%d", frame_scrolling="%d", frame_hits="%u", frame_url=%s WHERE frame_frameid=%u';
            $sql    = \sprintf(
                $format,
                $this->db->prefix('myiframe'),
                $this->db->quoteString($frame_description),
                $this->db->quoteString($frame_width),
                $this->db->quoteString($frame_height),
                $frame_align,
                $frame_frameborder,
                $frame_marginwidth,
                $frame_marginheight,
                $frame_scrolling,
                $frame_hits,
                $this->db->quoteString($frame_url),
                $frame_frameid

            );
        }
        if (false !== $force) {
            $result = $this->db->queryF($sql);
        } else {
            $result = $this->db->query($sql);
        }
        if (!$result) {
            return false;
        }
        if (empty($frame_frameid)) {

            $frame_frameid = $this->db->getInsertId();
        }
        $object->assignVar('frame_frameid', $frame_frameid);

        return $frame_frameid;
    }

    /**
     * @param bool $force
     * @return bool
     */
    public function delete(\XoopsObject $object, $force = false)
    {
        if (!$object instanceof \MyiframeBase) {
filter:
    dependency_paths: ["lib/*"]
            return false;
        }
        $sql = \sprintf('DELETE FROM %s WHERE frame_frameid = "%u"', $this->db->prefix('myiframe'), $object->getVar('frame_frameid'));
        $sql = \sprintf('DELETE FROM %s WHERE frame_frameid = "%u"', $this->db->prefix('myiframe'), /** @scrutinizer ignore-type */ $object->getVar('frame_frameid'));
        if (false !== $force) {
            $result = $this->db->queryF($sql);
        } else {
            $result = $this->db->query($sql);
        }
        if (!$result) {
            return false;
        }

        return true;
    }

    /**
     * @param \Criteria|null $criteria
     * @param bool           $id_as_key
     * @return array
     */
    public function &getObjects(\Criteria $criteria = null, $id_as_key = false)
    {
        $ret   = [];
        $limit = $start = 0;
        $sql   = 'SELECT * FROM ' . $this->db->prefix('myiframe');
        if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
            $sql .= ' ' . $criteria->renderWhere();
            if ('' !== $criteria->getSort()) {
                $sql .= ' ORDER BY ' . $criteria->getSort() . ' ' . $criteria->getOrder();
            }
            $limit = $criteria->getLimit();
            $start = $criteria->getStart();
        }
        $result = $this->db->query($sql, $limit, $start);
        /** @var array $myrow */
        if ($result instanceof \mysqli_result) {

            while (false !== ($myrow = $this->db->fetchArray($result))) {
                if (!$id_as_key) {
                    $ret[] = new MyiframeBase($myrow);
                } else {
                    $ret[$myrow['frame_frameid']] = new MyiframeBase($myrow);
                }
            }
        }

        return $ret;
    }

    /**
     * @param \CriteriaCompo|null $criteria
     * @return int
     */
    public function getCount(\CriteriaCompo $criteria = null)
    {
        $sql = 'SELECT COUNT(*) FROM ' . $this->db->prefix('myiframe');
        if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
            $sql .= ' ' . $criteria->renderWhere();
        }
        $result = $this->db->query($sql);
        if (!$result) {
            return 0;
        }
        [$count] = $this->db->fetchRow($result);

        return $count;
    }

    /**
     * @param \CriteriaCompo|null $criteria
     * @return bool
     */
    public function deleteAll(\CriteriaCompo $criteria = null)
    {
        $sql = 'DELETE FROM ' . $this->db->prefix('myiframe');
        if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
            $sql .= ' ' . $criteria->renderWhere();
        }
        if (!$result = $this->db->query($sql)) {

            return false;
        }

        return true;
    }

    /**
     * @param $frame_id
     */
    public function updatehits($frame_id): void
    {
        $sql = \sprintf('UPDATE %s SET frame_hits = frame_hits+1 WHERE frame_frameid="%u"', $this->db->prefix('myiframe'), (int)$frame_id);
        $this->db->queryF($sql);
    }
}

XoopsModules25x / myiframe

Pull Request — master (#8)

MyiframeBaseHandler::insert() B

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like
