ActivityController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	40
Duplicated Lines	30 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	23	5
A	init()	0	11	1

<?php

class ActivityController extends Zend_Controller_Action
{
    public function init(): void
    {
        // Init the Context Switch Action helper
        $contextSwitch = $this->_helper->contextSwitch();

        // Add the new context
        $contextSwitch->setContexts([
            'rss' => ['suffix' => 'rss'],
        ]);

        $contextSwitch->addActionContext('index', 'rss')->initContext();
    }

    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));
    }
}