Pictures::getAllPictures() - Code Metrics - mambax7/adslight - Measure and Improve Code Quality continuously with Scrutinizer

Pictures::getAllPictures() B
last analyzed 2023-04-20 11:18 UTC

↳ Parent: Pictures
Complexity

Conditions	11
Paths	12
Size

Total Lines	43
Code Lines	26
Duplication

Lines	0
Ratio	0 %
Importance

Changes	2
Bugs	0	Features	1
Metric	Value
eloc	26
c	2
b	0
f	1
dl	0
loc	43
rs	7.3166
cc	11
nc	12
nop	6
How to fix Complexity

<?php declare(strict_types=1);

namespace XoopsModules\Adslight;

/*
 * 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.0 or later (https://www.gnu.org/licenses/gpl-2.0.html)
 * @author       XOOPS Development Team
 * @author       Pascal Le Boustouller: original author ([email protected])
 * @author       Luc Bizet (www.frxoops.org)
 * @author       jlm69 (www.jlmzone.com)
 * @author       mamba (www.xoops.org)
 */

/**
 * Protection against inclusion outside the site
 */

/**
 * Includes of form objects and uploader
 */
require_once XOOPS_ROOT_PATH . '/class/uploader.php';
require_once XOOPS_ROOT_PATH . '/kernel/object.php';
require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
require_once XOOPS_ROOT_PATH . '/kernel/object.php';

/**
 * light_pictures class.
 * $this class is responsible for providing data access mechanisms to the data source
 * of XOOPS user class objects.
 */
class Pictures extends \XoopsObject
{
    private $cod_img;

    private $title;

    private $date_created;

    private $date_updated;

    private $lid;

    private $uid_owner;

    private $url;

    /**
     * @var \XoopsMySQLDatabase
     */
    public $db;
    public $helper;
    // constructor

    /**
     * @param null|int   $id
     * @param array|null $lid
     */
    public function __construct(
        $id = null,
        /** @scrutinizer ignore-unused */ $id = null,
        $lid = null
    ) {
        $this->helper = Helper::getInstance();
        $this->db     = \XoopsDatabaseFactory::getDatabaseConnection();
        $this->initVar('cod_img', \XOBJ_DTYPE_INT, null, false, 10);
        $this->initVar('title', \XOBJ_DTYPE_TXTBOX, null, false);
        $this->initVar('date_created', \XOBJ_DTYPE_INT, 0, false);
        $this->initVar('date_updated', \XOBJ_DTYPE_INT, 0, false);
        $this->initVar('lid', \XOBJ_DTYPE_INT, null, false, 10);
        $this->initVar('uid_owner', \XOBJ_DTYPE_TXTBOX, null, false);
        $this->initVar('url', \XOBJ_DTYPE_TXTBOX, null, false);
        if (!empty($lid)) {
            if (\is_array($lid)) {

                $this->assignVars($lid);
            } else {
                $this->load((int)$lid);
            }
        } else {
            $this->setNew();
        }
    }

    /**
     * @param int $id
     */
    public function load($id): void
    {
        $sql   = 'SELECT * FROM ' . $this->db->prefix('adslight_pictures') . ' WHERE cod_img=' . $id . ' ';
        $result = $this->db->query($sql);
        if (!$this->db->isResultSet($result)) {
            \trigger_error("Query Failed! SQL: $sql- Error: " . $this->db->error(), E_USER_ERROR);
        }
        $myrow = $this->db->fetchArray($result);
        $myrow = $this->db->fetchArray(/** @scrutinizer ignore-type */ $result);
        $this->assignVars($myrow);
        $this->assignVars(/** @scrutinizer ignore-type */ $myrow);
        if (!$myrow) {
            $this->setNew();
        }
    }

    /**
     * @param array  $criteria
     * @param bool   $asobject
     * @param string $sort
     * @param string $cat_order
     * @param int    $limit
     * @param int    $start
     * @internal   param string $order
     * @deprecated this should be handled through {@see PicturesHandler}
     */
    public function getAllPictures(
        $criteria = [],
        $asobject = false,
        $sort = 'cod_img',
        $cat_order = 'ASC',
        $limit = 0,
        $start = 0
    ): array {
        /** @var \XoopsMySQLDatabase $xoopsDB */
        $xoopsDB     = \XoopsDatabaseFactory::getDatabaseConnection();
        $ret         = [];
        $where_query = '';
        if (\is_array($criteria) && \count($criteria) > 0) {
            $where_query = ' WHERE';
            foreach ($criteria as $c) {
                $where_query .= " {$c} AND";
            }
            $where_query = \mb_substr($where_query, 0, -4);
        } elseif (!\is_array($criteria) && $criteria) {
            $where_query = " WHERE {$criteria}";
        }
        if ($asobject) {
            $sql    = 'SELECT * FROM ' . $xoopsDB->prefix('adslight_pictures') . "{$where_query} ORDER BY {$sort} {$cat_order}";
            $result = $xoopsDB->query($sql, $limit, $start);
            if ($xoopsDB->isResultSet($result)) {
                while (false !== ($myrow = $xoopsDB->fetchArray($result))) {
                while (false !== ($myrow = $xoopsDB->fetchArray(/** @scrutinizer ignore-type */ $result))) {
                    $ret[] = new self($myrow);
                    $ret[] = new self(/** @scrutinizer ignore-type */ $myrow);
                }
            } else {
                \trigger_error("Query Failed! SQL: $sql- Error: " . $xoopsDB->error(), E_USER_ERROR);
            }
        } else {
            $sql    = 'SELECT cod_img FROM ' . $xoopsDB->prefix('adslight_pictures') . "{$where_query} ORDER BY {$sort} {$cat_order}";
            $result = $xoopsDB->query($sql, $limit, $start);
            if (!$xoopsDB->isResultSet($result)) {
                \trigger_error("Query Failed! SQL: $sql- Error: " . $xoopsDB->error(), E_USER_ERROR);
            }
            while (false !== ($myrow = $xoopsDB->fetchArray($result))) {
                $ret[] = $myrow['cog_img'];
            }
        }

        return $ret;
    }

    /**
     * Get form
     *
     * @param null
     * @return Form\PicturesForm
     */
    public function getForm(): Form\PicturesForm
    {
        $form = new Form\PicturesForm($this);

        return $form;
    }
}

// -------------------------------------------------------------------------
// ------------------light_pictures user handler class -------------------
// -------------------------------------------------------------------------

mambax7 / adslight

Pictures::getAllPictures() B last analyzed 2023-04-20 11:18 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Pictures::getAllPictures() B
last analyzed 2023-04-20 11:18 UTC
