+global\ActivityController::indexAction()

Complexity

Conditions	5
Paths	9

Size

Total Lines

24

Duplication

Lines	12
Ratio	50 %

Code Coverage

Tests	13
CRAP Score	5

Importance

Changes

0

<?php
    public function indexAction(): void
    {
        // By default we show overall activity
        $item = null;
        $this->view->title = $this->view->translate('Overall activity');

        // Try to show user's actitvity
        if ($this->getRequest()->getParam('user')) {
            $item = \mQueue\Model\UserMapper::find($this->getRequest()->getParam('user'));
            if ($item) {
                $this->view->title = $this->view->translate('Activity for %s', [$item->nickname]);

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

            }
        }

        // Try to show movie's actitvity
        if ($this->getRequest()->getParam('movie')) {
            $item = \mQueue\Model\MovieMapper::find($this->getRequest()->getParam('movie'));
            if ($item) {
                $this->view->title = $this->view->translate('Activity for %s', [$item->getTitle()]);
            }
        }

        $this->view->activity = $this->_helper->createPaginator(\mQueue\Model\StatusMapper::getActivityQuery($item));
    }
}