mQueue\View\Helper\LoginState::loginState()

Complexity

Conditions	2
Paths	2

Size

Total Lines	20
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	2.1165

Importance

Changes

0

<?php
    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>';
    }