mQueue\View\Helper\LoginState

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

69.23%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	loginState()	0	20	2

<?php

namespace mQueue\View\Helper;

use mQueue\Model\User;
use Zend_View_Helper_Abstract;

class LoginState extends Zend_View_Helper_Abstract
{
    /**
     * Returns a string displaying the login state of the user and buttons to login/off
     *
     * @return string
     */
    public function loginState()
    {
        $result = '<div class="loginState">';

        $user = User::getCurrent();
        if ($user) {

$user is of type mQueue\Model\User, thus it always evaluated to true.

            $result .= '<a href="' . $this->view->serverUrl() . $this->view->url(
                    ['controller' => 'user', 'action' => 'view', 'id' => $user->id], 'singleid', true) . '">' . $this->view->gravatar($user) . ' ' . $this->view->escape($user->nickname) . '</a> ';

The property id does not exist on mQueue\Model\User. Since you implemented __get, consider adding a @property annotation.

The property nickname does not exist on mQueue\Model\User. Since you implemented __get, consider adding a @property annotation.

            $result .= '<a href="' . $this->view->serverUrl() . $this->view->url(
                    ['controller' => 'user', 'action' => 'logout'], 'default', true) . '">' . $this->view->translate('Logout') . '</a> ';
        } else {
            $result .= ' <a href="' . $this->view->serverUrl() . $this->view->url(
                    ['controller' => 'user', 'action' => 'new'], 'default', true) . '">' . $this->view->translate('Subscribe') . '</a> ';

            $result .= '<a href="' . $this->view->serverUrl() . $this->view->url(
                    ['controller' => 'user', 'action' => 'login'], 'default', true) . '">' . $this->view->translate('Login') . '</a>';
        }

        return $result . '</div>';
    }
}