ByTIC\Hello\Modules\AbstractModule\Controllers\Traits\LoginControllerTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	26	4
A	generateModelName()	0	3	1
A	oauth()	0	4	1

<?php

namespace ByTIC\Hello\Modules\AbstractModule\Controllers\Traits;

use Nip\Controllers\Traits\AbstractControllerTrait;
use Users;

The type Users was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Trait LoginControllerTrait
 * @package ByTIC\Hello\Modules\AbstractModule\Controllers\Traits
 */
trait LoginControllerTrait
{
    use AbstractControllerTrait;

    public function index()
    {
        $usersManager = $this->getModelManager();

It seems like getModelManager() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->getView()->set('headerTitle', $usersManager->getLabel('login-title'));

        $loginForm = $this->_getUser()->getForm('login');

It seems like _getUser() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($loginForm->submited()) {
            if ($loginForm->processRequest()) {
                $this->doSuccessRedirect();

It seems like doSuccessRedirect() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            }
        } else {
            if ($this->getRequest()->query->has('message')) {
                $loginForm->addMessage($this->getRequest()->query->get('message'), 'info');
            }
        }

        $this->forms['login'] = $loginForm;

The property forms does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->getView()->set('redirect', $this->getRedirectURL());

It seems like getRedirectURL() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->_setMeta('login');

It seems like _setMeta() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->getView()->Breadcrumbs()->addItem(
            $this->getModelManager()->getLabel('login-title'),
            $this->_getUser()->getManager()->compileURL('login')
        );
        $this->getView()->Meta()->prependTitle($this->getModelManager()->getLabel('login-title'));
    }

    public function oauth()
    {
        $jwt = $this->getRequest()->get('token');
        $this->_getUser()->getManager()->authenticateWithToken($jwt);
    }

    /**
     * @inheritdoc
     */
    protected function generateModelName()
    {
        return get_class($this->_getUser()->getManager());
    }
}