UserController

Complexity

Total Complexity

16

Size/Duplication

Total Lines	90
Duplicated Lines	15.56 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

71.43%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	indexAction()	0	4	1
A	newAction()	0	20	3
B	loginAction()	0	36	7
A	logoutAction()	0	4	1
A	viewAction()	14	14	3

<?php

class UserController extends Zend_Controller_Action

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function init(): void
    {
        // Initialize action controller here
    }

    public function indexAction(): void
    {
        $this->view->users = \mQueue\Model\UserMapper::fetchAll();
    }

    public function newAction()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $request = $this->getRequest();
        $form = new \mQueue\Form\User();

        if ($this->getRequest()->isPost()) {
            if ($form->isValid($request->getPost())) {
                $values = $form->getValues();
                $user = \mQueue\Model\UserMapper::insertUser($values);

                \mQueue\Model\User::setCurrent($user);

                $this->_helper->FlashMessenger('Subscription complete.');

                return $this->_helper->redirector('index', 'movie');
            }
        }

        $this->view->form = $form;
    }

    public function loginAction()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $request = $this->getRequest();
        $form = new \mQueue\Form\Login();

        if ($this->getRequest()->isPost()) {
            if ($form->isValid($request->getPost())) {
                $values = $form->getValues();

                $user = \mQueue\Model\UserMapper::findEmailPassword($values['email'], $values['password']);
                if ($user) {
                    \mQueue\Model\User::setCurrent($user);

                    $this->_helper->FlashMessenger('Logged in.');

                    $referrer = $values['referrer'];

                    // If we have a valid referer to one page of ourselve (except login or logout), redirect to it
                    if (mb_strpos($referrer, $this->view->serverUrl() . $this->view->baseUrl()) === 0
                            && mb_strpos($referrer, $this->view->serverUrl() . $this->view->url(['controller' => 'user', 'action' => 'login'])) !== 0
                            && mb_strpos($referrer, $this->view->serverUrl() . $this->view->url(['controller' => 'user', 'action' => 'logout'])) !== 0) {
                        return $this->_redirect($values['referrer']);

The method Zend_Controller_Action::_redirect() has been deprecated with message: Deprecated as of Zend Framework 1.7. Use redirect() instead.

                    }

                    return $this->_helper->redirector('index', 'movie');
                }
                $this->_helper->FlashMessenger(['error' => 'Login failed.']);
            }
        } else {
            $form->setDefaults([
                'referrer' => $this->getRequest()->getServer('HTTP_REFERER'),
            ]);
        }

        $this->view->form = $form;
    }

    public function logoutAction(): void
    {
        \mQueue\Model\User::setCurrent(null);
    }

    public function viewAction(): void

This method seems to be duplicated in your project.

    {
        if ($this->getRequest()->getParam('id')) {
            $this->view->user = \mQueue\Model\UserMapper::find($this->getRequest()->getParam('id'));
        } else {
            $this->view->user = \mQueue\Model\User::getCurrent();
        }

        if (!$this->view->user) {
            throw new Exception($this->view->translate('User not found'));
        }

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