b_xoopsfaq_random_show()

Complexity

Conditions	10
Paths	108

Size

Total Lines	62
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php declare(strict_types=1);

function b_xoopsfaq_random_show(array $options)

{

    $moduleDirName = \basename(\dirname(__DIR__));

    $myts          = \MyTextSanitizer::getInstance();


    /** @var CategoryHandler $categoryHandler */

    /** @var ContentsHandler $contentsHandler */

    /** @var Helper $helper */

    $helper          = Helper::getInstance();

    $permHelper      = new Permission($moduleDirName);

    $contentsHandler = $helper->getHandler('Contents');

    $block           = [];


    $criteria = new \CriteriaCompo();

    $criteria->add(new \Criteria('contents_active', Constants::ACTIVE, '='));


    // Filter out cats based on group permissions

    $options[1] = $options[1] ?? [0];

    $cTu        = $catsToUse = (false === mb_strpos($options[1], ',')) ? (array)$options[1] : explode(',', $options[1]);

It seems like $options[1] can also be of type array<integer,integer>; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

It seems like $options[1] can also be of type array<integer,integer>; however, parameter $haystack of mb_strpos() does only seem to accept string, maybe add an additional type check?

    if (in_array(0, $catsToUse, true) || empty($catsToUse)) {

        // Get a list of all cats

        $categoryHandler = $helper->getHandler('Category');

        $catListArray    = $categoryHandler->getList();

        $cTu             = $catsToUse = array_keys($catListArray);

    }

    // Remove any cats this user doesn't have rights to view

    foreach ($cTu as $key => $thisCat) {

        if (false === $permHelper->checkPermission('viewcat', $thisCat)) {

            unset($catsToUse[$key]);

        }

    }

    if (!empty($catsToUse)) {

        $criteria->add(new \Criteria('contents_cid', '(' . implode(',', $catsToUse) . ')', 'IN'));

    } else {

        return $block;

    }

    $xpFaqObjArray = $contentsHandler->getAll($criteria);

    $faqCount      = (is_array($xpFaqObjArray) && !empty($xpFaqObjArray)) ? count($xpFaqObjArray) : 0;


    if ($faqCount > 0) {

        $faqNum   = array_rand($xpFaqObjArray, 1);

        $xpFaqObj = $xpFaqObjArray[$faqNum];

        $faq      = $myts->displayTarea($xpFaqObj->getVar('contents_title'));

        $txtAns   = strip_tags($xpFaqObj->getVar('contents_contents')); // get rid of html for block

        $faqAns   = $myts->displayTarea(xoops_substr($txtAns, 0, $options[0]), 0, 0, 0, 0, 0);


        $categoryHandler = $helper->getHandler('Category');

        $catObj          = $categoryHandler->get($xpFaqObj->getVar('contents_cid'));

        $cid             = $catObj->getVar('category_id');

        $block           = [

            'title'    => _MB_XOOPSFAQ_RANDOM_TITLE,

            'faq'      => $faq,

            'faqans'   => $faqAns,

            'morelink' => $helper->url('index.php?cat_id=' . $cid . '#q' . $xpFaqObj->getVar('contents_id')),

            'linktxt'  => _MB_XOOPSFAQ_SEE_MORE,

            'catlink'  => $helper->url('index.php?cat_id=' . $cid),

            'cattxt'   => $catObj->getVar('category_title'),

        ];

        unset($xpFaqObj, $catObj);

    }


    return $block;

}