csphere\plugins\users\classes\Menu

Complexity

Total Complexity

2

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	show()	0	27	2

<?php

/**
 * User menu details
 *
 * PHP Version 5
 *
 * @category  Plugins
 * @package   Users
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

namespace csphere\plugins\users\classes;

/**
 * User menu details
 *
 * @category  Plugins
 * @package   Users
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

abstract class Menu
{
    /**
     * User menu if login was passed
     *
     * @param boolean $box    If view target area is a box
     * @param string  $layout Template layout to use
     *
     * @return void
     **/

There must be no blank lines after the function comment

    public static function show($box = false, $layout = '')
    {
        // Get view object
        $loader = \csphere\core\service\Locator::get();
        $view   = $loader->load('view');

        // Get user details from session
        $session = new \csphere\core\session\Session();

        // Set template content
        $data = ['user_name' => $session->get('user_name'),
                 'user_id'   => $session->get('user_id')];

        // Set template layout
        $tpl    = 'login' . $layout . '_menu';

        // Box mode needs a different tpl file
        if ($box == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $view->template('users', $tpl, $data);

        } else {

            $view->template('users', "side".$tpl, $data, true);

        }
    }
}