FrontendModule\LoginPresenter

Complexity

Total Complexity

9

Size/Duplication

Total Lines	84
Duplicated Lines	25 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeRender()	0	8	2
A	startup()	0	4	1
B	createComponentSignInForm()	0	28	1
A	renderDefault()	0	4	1
A	signInFormSubmitted()	21	21	3
A	actionOut()	0	6	1

<?php

namespace FrontendModule;

use Nette\Application\UI;
use Kdyby\BootstrapFormRenderer\BootstrapRenderer;
use Nette\Security as NS;

class LoginPresenter extends \FrontendModule\BasePresenter
{
    public function beforeRender()
    {
        parent::beforeRender();

        if ($this->getUser()->isLoggedIn()) {
            $this->forward('Homepage:');
        }
    }

    protected function startup()
    {
        parent::startup();
    }

    /**
     * Sign in form component factory.
     * @return UI\Form
     */
    protected function createComponentSignInForm($name)

The parameter $name is not used and could be removed.

    {
        $form = new UI\Form();

        $form->getElementPrototype()->action = $this->link('default', array(
            'path' => 'loginfe',
            'abbr' => $this->abbr,
            'do' => 'signInForm-submit',
        ));

        $form->setRenderer(new BootstrapRenderer());

        $form->addText('username', 'Přihlašovací jméno')
            ->setRequired('Please provide a username.')
            ->setAttribute('placeholder', 'Přihlašovací jméno');

'Přihlašovací jméno' is of type string, but the function expects a boolean.

        $form->addPassword('password', 'Heslo')
            ->setRequired('Please provide a password.')
            ->setAttribute('placeholder', 'Heslo');

'Heslo' is of type string, but the function expects a boolean.

        $form->addCheckbox('remember', 'Permanent login?');

        $form->addSubmit('send', 'Přihlásit');

        $form->onSuccess[] = callback($this, 'signInFormSubmitted');

        return $form;
    }

    public function renderDefault()
    {
        $this->setLayout('login');
    }

    public function signInFormSubmitted($form)

This method seems to be duplicated in your project.

    {

        try {
            $values = $form->getValues();
            if ($values->remember) {
                $this->getUser()->setExpiration('+ 14 days', false);
            } else {
                $this->getUser()->setExpiration('+ 20 minutes', true);
            }
            $this->getUser()->login($values->username, $values->password);
            $this->flashMessage('Přihlášení bylo úspěšné', 'success');

            //TODO bude upraveno - ted nestiham :)
            //$this->forward('Homepage:');
            $this->redirectUrl('https://www.zajistenainvestice.cz/uvod');
        } catch (NS\AuthenticationException $e) {
            $this->flashMessage($this->translation[$e->getMessage()], 'danger');
            $form->addError($e->getMessage());
        }
    }

    public function actionOut()
    {
        $this->getUser()->logout();
        $this->flashMessage('Odhlášení bylo úspěšné', 'success');
        $this->forward('Homepage:');
    }

}