The class Post has 23 fields. Consider redesigning Post to keep the number of fields under 15.

Open

class Post extends \XoopsObject
{
    private $post_id;
    private $topic_id;
    private $forum_id;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Person {
   protected $one;
   private $two;
   private $three;
   [... many more fields ...]
}

The class Poll has 16 fields. Consider redesigning Poll to keep the number of fields under 15.

Open

class Poll extends \XoopsObject
{
    private int    $poll_id;
    private string $question;
    private string $description;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Person {
   protected $one;
   private $two;
   private $three;
   [... many more fields ...]
}

Method editBlock has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function editBlock(int $bid): void
    {
        //        require_once \dirname(__DIR__,2) . '/admin/admin_header.php';
        //        \xoops_cp_header();
        \xoops_loadLanguage('admin', 'system');

Method assignForm has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function assignForm(\XoopsTpl $tpl): void
    {
        $myts       = \MyTextSanitizer::getInstance();
        $optionObjs = $this->optionHandler->getAllByPollId($this->pollObj->getVar('poll_id'));

Method getPostBody has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function &getPostBody()
    {
        global $xoopsConfig, $xoopsUser, $myts;
        /** @var Xoopspoll\Helper $helper */
        $helper = Xoopspoll\Helper::getInstance();

Method vote has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function vote(int $optionId, string $ip, int $time): bool
    {
        if (!empty($optionId) && $this->isAllowedToVote()) {
            $voteTime      = empty($time) ? \time() : (int)$time;
            $uid           = ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser']->uid() : 0;

Function render has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function render(?array $block = null): void
    {
        \xoops_load('XoopsFormLoader');
        \xoops_loadLanguage('common', $this->moduleDirNameUpper);

• Read up
  Read up

Function displayAttachment has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function displayAttachment($asSource = false)
    {
        global $xoopsModule;
        /** @var Xoopspoll\Helper $helper */
        $helper = Xoopspoll\Helper::getInstance();

• Read up
  Read up

Method getPostBody has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function &getPostBody()
    {
        global $myts;
        $GLOBALS['xoopsModuleConfig'] = newbb_load_config(); // irmtfan  load all newbb configs - newbb config in blocks activated in some modules like profile
        //        mod_loadFunctions('user', 'newbb');

Method orderBlock has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function orderBlock(
        array $bid,
        array $oldtitle,
        array $oldside,
        array $oldweight,

Method __construct has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct($id = null)
    {
        parent::__construct();
        //        $timestamp = xoops_getUserTimestamp(time());
        $currentTimestamp = \time();

Method hasVoted has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function hasVoted(?int $pid, string $ip, int $uid = 0): bool
    {
        $uid        = (int)$uid;
        $pid        = (int)$pid;
        $voted      = true;

Function _getDateDatepicker has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.datepicker._getDateDatepicker = function (target, noDefault) {
        var inst = this._getInst(target);
        if (!inst) {
            return;
        }

Method isResultVisible has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function isResultVisible()
    {
        $visibleMsg = '';
        \xoops_loadLanguage('main', 'xoopspoll');
        switch ($this->getVar('visibility')) {

Method delete has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function delete($post, $isDeleteOne = true, $force = false)
    {
        $retVal = false;
        if (($post instanceof Post) && ($post->getVar('post_id') > 0)) {
            if ($isDeleteOne) {

Method processOrderBlockAction has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function processOrderBlockAction(BlockData $blockData)
    {
        $blockData->bid           = Request::getArray('bid', []);
        $blockData->oldtitle      = Request::getArray('oldtitle', [], 'POST');
        $blockData->oldside       = Request::getArray('oldside', [], 'POST');

Function mailResults has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function mailResults(mixed $pollObj = null): array
    {
        $criteria = new \CriteriaCompo();
        $criteria->add(new \Criteria('end_time', \time(), '<'));  // expired polls
        $criteria->add(new \Criteria('mail_status', Constants::POLL_NOT_MAILED, '=')); // email not previously sent

• Read up
  Read up

Function rrmdir has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public static function rrmdir(string $src): bool
    {
        // Only continue if user is a 'global' Admin
        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
            return false;

• Read up
  Read up

Function hasVoted has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function hasVoted(?int $pid, string $ip, int $uid = 0): bool
    {
        $uid        = (int)$uid;
        $pid        = (int)$pid;
        $voted      = true;

• Read up
  Read up

Function assignResults has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function assignResults(\XoopsTpl $tpl): void
    {
        $myts             = \MyTextSanitizer::getInstance();
        $xuEndTimestamp   = \xoops_getUserTimestamp($this->pollObj->getVar('end_time'));
        $xuEndFormatted   = \ucfirst(\date(_MEDIUMDATESTRING, (int)$xuEndTimestamp));

• Read up
  Read up