blocks/menu_block.php ➔ menu_block()

Complexity

Conditions	39
Paths	> 20000

Size

Total Lines	192
Code Lines	137

Duplication

Lines	24
Ratio	12.5 %

Importance

Changes

0

<?php
function menu_block()
{
    global $xoopsTpl, $xoopsUser, $pedigree;
    $moduleDirName = basename(dirname(__DIR__));

The assignment to $moduleDirName is dead and can be removed.

    if (!class_exists('PedigreePedigree')) {
        $pedigree = PedigreePedigree::getInstance(false);
    }
/*
    //get module configuration
    $moduleHandler = xoops_getHandler('module');
    $module        = $moduleHandler->getByDirname($moduleDirName);
    $configHandler = xoops_getHandler('config');
    $moduleConfig  = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
*/
    //colour variables
    $colors  = explode(',', $pedigree->getConfig('colourscheme'));
    $actlink = $colors[0];

The assignment to $actlink is dead and can be removed.

    $even    = $colors[1];

The assignment to $even is dead and can be removed.

    $odd     = $colors[2];

The assignment to $odd is dead and can be removed.

    $text    = $colors[3];

The assignment to $text is dead and can be removed.

    $hovlink = $colors[4];

The assignment to $hovlink is dead and can be removed.

    $head    = $colors[5];

The assignment to $head is dead and can be removed.

    $body    = $colors[6];

The assignment to $body is dead and can be removed.

    $title   = $colors[7];

The assignment to $title is dead and can be removed.

    /*
        //inline-css
        echo '<style>';
        //text-colour
        echo 'body {margin: 0;padding: 0;background: ' . $body . ';color: ' . $text . ";font-size: 62.5%; font-family: 'Lucida Grande', Verdana, Arial, Sans-Serif; text-align: left;}";
        //link-colour
        echo 'a, h2 a:hover, h3 a:hover { color: ' . $actlink . '; text-decoration: none; }';
        //link hover colour
        echo 'a:hover { color: ' . $hovlink . '; text-decoration: underline; }';
        //th
        echo 'th {padding: 2px;color: #ffffff;background: ' . $title . ';font-family: Verdana, Arial, Helvetica, sans-serif;vertical-align: middle;}';
        echo 'td#centercolumn th { color: #fff; background: ' . $title . '; vertical-align: middle; }';
        //head
        echo '.head {background-color: ' . $head . '; padding: 3px; font-weight: normal;}';
        //even
        echo '.even {background-color: ' . $even . '; padding: 3px;}';
        echo 'tr.even td {background-color: ' . $even . '; padding: 3px;}';
        //odd
        echo '.odd {background-color: ' . $odd . '; padding: 3px;}';
        echo 'tr.odd td {background-color: ' . $odd . '; padding: 3px;}';
        echo '</style>';
    */

    //iscurrent user a module admin ?
    $xoopsModule = XoopsModule::getByDirname('pedigree');
    if ((!empty($xoopsUser)) && ($GLOBALS['xoopsUser'] instanceof XoopsUser) && $xoopsUser->isAdmin($xoopsModule->mid())) {
        $modadmin = true;
    } else {
        $modadmin = false;
    }
    $counter   = 1;
    $menuwidth = 4;

    $x       = $_SERVER['PHP_SELF'];
    $lastpos = my_strrpos($x, '/');
    $len     = strlen($x);
    $curpage = substr($x, $lastpos, $len);

It seems like $lastpos can also be of type false; however, parameter $start of substr() does only seem to accept integer, maybe add an additional type check?

    if (1 == $pedigree->getConfig('showwelcome')) {
        if ('/welcome.php' === $curpage) {
            $title = '<b>Welcome</b>';
        } else {
            $title = 'Welcome';
        }
        $menuarray[] = array('title' => $title, 'link' => 'welcome.php', 'counter' => $counter);

$menuarray was never initialized. Although not strictly required by PHP, it is generally a good practice to add $menuarray = array(); before regardless.

        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
    }
    if ($curpage === '/index.php' || $curpage === '/result.php') {
        $title = '<b>View/Search ' . $pedigree->getConfig('animalTypes') . '</b>';
    } else {
        $title = 'View/Search ' . $pedigree->getConfig('animalTypes');
    }
    $menuarray[] = array('title' => $title, 'link' => 'index.php', 'counter' => $counter);
    ++$counter;
    if ($counter == $menuwidth) {
        $counter = 1;
    }
    if ($curpage === '/add_dog.php') {
        $title = '<b>Add a ' . $pedigree->getConfig('animalType') . '</b>';
    } else {
        $title = 'Add a ' . $pedigree->getConfig('animalType');
    }
    $menuarray[] = array('title' => $title, 'link' => 'add_dog.php', 'counter' => $counter);
    ++$counter;
    if ($counter == $menuwidth) {
        $counter = 1;
    }
    if ('1' == $pedigree->getConfig('uselitter')) {
        if ($curpage === '/add_litter.php') {
            $title = '<b>Add a ' . $pedigree->getConfig('litter') . '</b>';
        } else {
            $title = 'Add a ' . $pedigree->getConfig('litter');
        }
        $menuarray[] = array('title' => $title, 'link' => 'add_litter.php', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
    }
    if ('1' == $pedigree->getConfig('ownerbreeder')) {
        if ($curpage === '/breeder.php' || $curpage === '/owner.php') {
            $title = '<b>View owners/breeders</b>';
        } else {
            $title = 'View owners/breeders';
        }
        $menuarray[] = array('title' => $title, 'link' => 'breeder.php', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
        if ($curpage === '/add_breeder.php') {
            $title = '<b>Add an owner/breeder</b>';
        } else {
            $title = 'Add an owner/breeder';
        }
        $menuarray[] = array('title' => $title, 'link' => 'add_breeder.php', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
    }
    if ($curpage === '/advanced.php') {
        $title = '<b>Advanced info</b>';
    } else {
        $title = 'Advanced info';
    }
    $menuarray[] = array('title' => $title, 'link' => 'advanced.php', 'counter' => $counter);
    ++$counter;
    if ($counter == $menuwidth) {
        $counter = 1;
    }
    if ('1' == $pedigree->getConfig('proversion')) {
        if ($curpage === '/virtual.php') {
            $title = '<b>Virtual mating</b>';
        } else {
            $title = 'Virtual Mating';
        }
        $menuarray[] = array('title' => $title, 'link' => 'virtual.php', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
    }
    if ($curpage === '/latest.php') {
        $title = '<b>latest additions</b>';
    } else {
        $title = 'latest additions';
    }
    $menuarray[] = array('title' => $title, 'link' => 'latest.php', 'counter' => $counter);
    ++$counter;
    if ($counter == $menuwidth) {
        $counter = 1;
    }
    if ($modadmin === true) {
        if ($curpage === '/tools.php') {
            $title = '<b>Webmaster tools</b>';
        } else {
            $title = 'Webmaster tools';
        }
        $menuarray[] = array('title' => $title, 'link' => 'tools.php?op=index', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
        $title       = 'Logout';
        $menuarray[] = array('title' => $title, 'link' => '../../user.php?op=logout', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;

The assignment to $counter is dead and can be removed.

        }
    } else {
        if ($curpage === '/user.php') {
            $title = '<b>User login</b>';
        } else {
            $title = 'User login';
        }
        $menuarray[] = array('title' => $title, 'link' => '../../user.php', 'counter' => $counter);
        ++$counter;
        if ($counter == $menuwidth) {
            $counter = 1;
        }
    }

    //create path taken
    //showpath();
    $xoopsTpl->assign('menuarray', $menuarray);