Function _loadAllChildItemIds has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function _loadAllChildItemIds($itemId, &$childIds)
    {
        if (!empty($this->_itemTree[$itemId]['children'])) {
            $first_child = $this->_itemTree[$itemId]['children'];
            foreach ($first_child as $fcid) {

• Read up
  Read up

Function wflinks_notify_iteminfo has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

function wflinks_notify_iteminfo($category, $item_id)
{
    $moduleDirName = basename(dirname(__DIR__));
    global $xoopsModule, $xoopsModuleConfig, $xoopsConfig;

• Read up
  Read up

Method loadSampleData has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function loadSampleData()
{
    global $xoopsConfig;
    $moduleDirName      = basename(dirname(__DIR__));
    $moduleDirNameUpper = mb_strtoupper($moduleDirName);

Method b_waiting_wflinks has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_waiting_wflinks()
{
    $xoopsDB = \XoopsDatabaseFactory::getDatabaseConnection();
    $ret     = [];

Method setAddress has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $postoffice = '',
        $extended = '',
        $street = '',
        $city = '',
        $region = '',

Method setLabel has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $postoffice = '',
        $extended = '',
        $street = '',
        $city = '',
        $region = '',

Method __construct has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $path = 'uploads',
        $value = null,
        $selected = '',
        $size = 1,
        $emptyselect = 0,

Function rmove has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

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

• Read up
  Read up

Function rcopy has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

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

• Read up
  Read up

Function recurseCopy has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public static function recurseCopy($src, $dst)
    {
        $dir = \opendir($src);
        //        @mkdir($dst);
        if (!@\mkdir($dst) && !\is_dir($dst)) {

• Read up
  Read up

Method isBlockCloned has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function isBlockCloned($bid, $bside, $bweight, $bvisible, $bcachetime, $bmodule, $options)

Method uploadFiles has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $FILES,
        $uploaddir = 'uploads',
        $allowed_mimetypes = '',
        $redirecturl = 'index.php',
        $num = 0,

Avoid deeply nested control flow statements.

Open

                if (false !== $all_ok || in_array($modversion['category'], $ok_syscats)) {
                    echo "<td class='$class' class='txtcenter;' valign='bottom' width='19%'>";
                    echo "<a href='" . XOOPS_URL . '/modules/system/admin.php?fct=' . $file . "'><b>" . trim($modversion['name']) . "</b></a>\n";
                    echo '</td>';
                    ++$counter;

Function getForum has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public static function getForum($type, $selected)
    {
        global $xoopsDB;
        switch (\xoops_trim($type)) {
            case 2:

• Read up
  Read up

Method makeMySelBox has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function makeMySelBox($title, $order = '', $preset_id = 0, $none = 0, $sel_name = '', $onchange = '')

Avoid deeply nested control flow statements.

Open

                if ($counter > 4) {
                    $counter = 0;
                    echo '</tr>';
                    echo '<tr>';
                }

Method setOrder has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function setOrder($bid, $title, $weight, $visible, $side, $bcachetime)

Function __construct has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function __construct($img_name = null, $img_path = null, $img_savepath = null)
    {
        if (!\preg_match("/\.(jpg|gif|png|jpeg)$/i", $img_name)) {
            //            return false;
        }

• Read up
  Read up

Function wflinks_tag_iteminfo has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function wflinks_tag_iteminfo(&$items)
{
    $moduleDirName = basename(dirname(__DIR__));

    if (empty($items) || !is_array($items)) {

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if ($space > 0) {
                        $subcategories .= "<a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewcat.php?cid=' . $ele['cid'] . "'>" . $chtitle . '</a><br>';
                    }