XoopsModules\Smartobject\AdsenseHandler

Complexity

Total Complexity

7

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	66	1
A	getFormats()	0	4	1
A	beforeSave()	0	8	2
A	getAdsensesByTag()	0	13	3

<?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 SmartobjectAdsenseHandler
 */
class AdsenseHandler extends Smartobject\PersistableObjectHandler
{
    public $adFormats;
    public $adFormatsList;
    public $objects = false;
//    public $classname = Adsense::class;

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

    /**
     * SmartobjectAdsenseHandler constructor.
     * @param \XoopsDatabase $db
     */
    public function __construct(\XoopsDatabase $db)
    {
        parent::__construct($db, Adsense::class, 'adsenseid', 'description', '', 'smartobject');
        $this->adFormats     = [];
        $this->adFormatsList = [];

        $this->adFormats['728x90_as']['caption'] = '728 X 90 Leaderboard';
        $this->adFormats['728x90_as']['width']   = 728;
        $this->adFormats['728x90_as']['height']  = 90;
        $this->adFormatsList['728x90_as']        = $this->adFormats['728x90_as']['caption'];

        $this->adFormats['468x60_as']['caption'] = '468 X 60 Banner';
        $this->adFormats['468x60_as']['width']   = 468;
        $this->adFormats['468x60_as']['height']  = 60;
        $this->adFormatsList['468x60_as']        = $this->adFormats['468x60_as']['caption'];

        $this->adFormats['234x60_as']['caption'] = '234 X 60 Half Banner';
        $this->adFormats['234x60_as']['width']   = 234;
        $this->adFormats['234x60_as']['height']  = 60;
        $this->adFormatsList['234x60_as']        = $this->adFormats['234x60_as']['caption'];

        $this->adFormats['120x600_as']['caption'] = '120 X 600 Skyscraper';
        $this->adFormats['120x600_as']['width']   = 120;
        $this->adFormats['120x600_as']['height']  = 600;
        $this->adFormatsList['120x600_as']        = $this->adFormats['120x600_as']['caption'];

        $this->adFormats['160x600_as']['caption'] = '160 X 600 Wide Skyscraper';
        $this->adFormats['160x600_as']['width']   = 160;
        $this->adFormats['160x600_as']['height']  = 600;
        $this->adFormatsList['160x600_as']        = $this->adFormats['160x600_as']['caption'];

        $this->adFormats['120x240_as']['caption'] = '120 X 240 Vertical Banner';
        $this->adFormats['120x240_as']['width']   = 120;
        $this->adFormats['120x240_as']['height']  = 240;
        $this->adFormatsList['120x240_as']        = $this->adFormats['120x240_as']['caption'];

        $this->adFormats['336x280_as']['caption'] = '336 X 280 Large Rectangle';
        $this->adFormats['336x280_as']['width']   = 136;
        $this->adFormats['336x280_as']['height']  = 280;
        $this->adFormatsList['336x280_as']        = $this->adFormats['336x280_as']['caption'];

        $this->adFormats['300x250_as']['caption'] = '300 X 250 Medium Rectangle';
        $this->adFormats['300x250_as']['width']   = 300;
        $this->adFormats['300x250_as']['height']  = 250;
        $this->adFormatsList['300x250_as']        = $this->adFormats['300x250_as']['caption'];

        $this->adFormats['250x250_as']['caption'] = '250 X 250 Square';
        $this->adFormats['250x250_as']['width']   = 250;
        $this->adFormats['250x250_as']['height']  = 250;
        $this->adFormatsList['250x250_as']        = $this->adFormats['250x250_as']['caption'];

        $this->adFormats['200x200_as']['caption'] = '200 X 200 Small Square';
        $this->adFormats['200x200_as']['width']   = 200;
        $this->adFormats['200x200_as']['height']  = 200;
        $this->adFormatsList['200x200_as']        = $this->adFormats['200x200_as']['caption'];

        $this->adFormats['180x150_as']['caption'] = '180 X 150 Small Rectangle';
        $this->adFormats['180x150_as']['width']   = 180;
        $this->adFormats['180x150_as']['height']  = 150;
        $this->adFormatsList['180x150_as']        = $this->adFormats['180x150_as']['caption'];

        $this->adFormats['125x125_as']['caption'] = '125 X 125 Button';
        $this->adFormats['125x125_as']['width']   = 125;
        $this->adFormats['125x125_as']['height']  = 125;
        $this->adFormatsList['125x125_as']        = $this->adFormats['125x125_as']['caption'];
    }

    /**
     * @return array
     */
    public function getFormats()
    {
        return $this->adFormatsList;
    }

    /**
     * @param $obj
     * @return bool
     */
    public function beforeSave($obj)
    {
        if ('' === $obj->getVar('tag')) {
            $obj->setVar('tag', $title = $obj->generateTag());
        }

        return true;
    }

    /**
     * @return array|bool
     */
    public function getAdsensesByTag()
    {
        if (!$this->objects) {
            $adsensesObj =& $this->getObjects(null, true);
            $ret         = [];
            foreach ($adsensesObj as $adsenseObj) {
                $ret[$adsenseObj->getVar('tag')] = $adsenseObj;
            }
            $this->objects = $ret;

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

        }

        return $this->objects;
    }
}