Intraface_Controller_Login

Complexity

Total Complexity

7

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	execute()	0	5	1
A	renderHtml()	0	6	1
A	postForm()	0	11	2
A	selectUser()	0	13	2

<?php
/**
 * Logout
 *
 * @package Intraface
 * @author  Lars Olesen <lars@legestue.net>
 * @since   0.1.0
 * @version @package-version@
 */
class Intraface_Controller_Login extends k_Component
{
    protected $template;
    protected $kernel;
    protected $auth;
    protected $mdb2;

    function __construct(k_TemplateFactory $template, Intraface_Auth $auth, MDB2_Driver_Common $mdb2, Intraface_Log $log)
    {
        $this->template = $template;
        $this->auth = $auth;
        $this->mdb2 = $mdb2;
        $this->log = $log;

The property log does not exist. Did you maybe forget to declare it?

    }

    function execute()
    {
        $this->url_state->init("continue", $this->url('/restricted'));
        return parent::execute();
    }

    function renderHtml()
    {
        $this->document->setTitle('Login');
        $smarty = $this->template->create(dirname(__FILE__) . '/templates/login');
        return $smarty->render($this);
    }

    function postForm()
    {
        $user = $this->selectUser($this->body('email'), $this->body('password'));
        if ($user) {
            $this->session()->set('intraface_identity', $user);

The method set cannot be called on $this->session() (of type string).

            return new k_SeeOther($this->query('continue'));
        } else {
            return new k_SeeOther($this->url(null, array('flare' => 'Wrong credentials')));
        }
        return $this->render();

return $this->render(); does not seem to be reachable.

    }

    protected function selectUser($username, $password)
    {
        $adapter = new Intraface_Auth_User($this->mdb2, $this->session()->sessionId(), $username, $password);

The method sessionId cannot be called on $this->session() (of type string).

        // $this->auth->attachObserver($this->log);

67% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->auth->authenticate($adapter);

        if (!$this->auth->hasIdentity()) {
            return false;
        }

        return new Intraface_AuthenticatedUser($username, $this->language());
    }
}