blocks/alumni.php ➔ alumni_show()

Complexity

Conditions	2
Paths	2

Size

Total Lines	42
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function alumni_show($options)

function alumni_show() does not seem to conform to the naming convention (^(?:[a-z]|__)[a-zA-Z0-9]*$).

{
    $block        = [];
    $myts         = MyTextSanitizer::getInstance();

$myts is not used, you could remove the assignment.

    $blockDirName = basename(dirname(__DIR__));

    $xoops          = Xoops::getInstance();
    $helper         = Xoops::getModuleHelper('alumni');
    $module_id      = $helper->getModule()->getVar('mid');

$module_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $listingHandler = $helper->getHandler('listing');
    $groups         = $xoops->getUserGroups();
    $alumni_ids     = $xoops->getHandlerGroupPermission()->getItemIds('alumni_view', $groups, $module_id);

$alumni_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $all_ids        = implode(', ', $alumni_ids);

$all_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $criteria = new CriteriaCompo();
    $criteria->add(new Criteria('valid', 1, '='));
    $criteria->add(new Criteria('cid', '(' . $all_ids . ')', 'IN'));

$all_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $criteria->setLimit($options[1]);
    $criteria->setSort('date');
    $criteria->setOrder('DESC');
    $block_listings = $listingHandler->getall($criteria);

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    foreach (array_keys($block_listings) as $i) {

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $name   = $block_listings[$i]->getVar('name');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $mname  = $block_listings[$i]->getVar('mname');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $lname  = $block_listings[$i]->getVar('lname');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $school = $block_listings[$i]->getVar('school');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $year   = $block_listings[$i]->getVar('year');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $view   = $block_listings[$i]->getVar('view');

$block_listings does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$view is not used, you could remove the assignment.

        $a_item           = [];

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $a_item['school'] = $school;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $a_item['link']   = '<a href="' . XOOPS_URL . "/modules/{$blockDirName}/listing.php?lid=" . addslashes($block_listings[$i]->getVar('lid')) . "\"><b>$year&nbsp;-&nbsp;$name $mname $lname</b><br></a>";

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $block['items'][] = $a_item;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    $block['lang_title'] = AlumniLocale::BLOCKS_ITEM;
    $block['lang_date']  = AlumniLocale::BLOCKS_DATE;
    $block['link']       = '<a href="' . XOOPS_URL . "/modules/{$blockDirName}/index.php\"><b>" . AlumniLocale::BLOCKS_ALL_LISTINGS . '</b></a></div>';

    return $block;
}