mQueue\View\Helper\StatusLinks::statusLinks()

Complexity

Conditions	8
Paths	10

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	8.0155

Importance

Changes

0

<?php
    public function statusLinks(Status $status)
    {
        $result = '<div class="mqueue_status_links mqueue_status_links_' . $status->getUniqueId() . '">';
        $user = User::getCurrent();

        // Deactivate links if no logged user
        if ($user) {

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

            $tag = 'a';
        } else {
            $tag = 'span';
        }

        foreach (Status::$ratings as $val => $name) {
            $class = $val . ($status->rating == $val ? ' current' : '');

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

            $url = $this->view->serverUrl() . $this->view->url([
                    'controller' => 'status',
                    'movie' => $status->idMovie,

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

                    'rating' => ($val == $status->rating && $user && $user->id == $status->idUser) ? 0 : $val,

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

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

                ], 'status', true);

            $result .= '<' . $tag . ' class="mqueue_status mqueue_status_' . $class . '"' . ($tag == 'a' ? ' href="' . $url . '"' : '') . ' title="' . $name . '"><span>' . $name . '</span></' . $tag . '>';
        }
        $result .= '<span class="preloader"></span></div>';

        return $result;
    }