AuthorizationMiddleware::execute() - Code Metrics - Inspection of "workflow" - linemob/core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#18)

by ツ

created 2017-09-07 09:23 UTC

AuthorizationMiddleware::execute() B

↳ Parent: AuthorizationMiddleware

Complexity

Conditions	4
Paths	3

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	22
rs	8.9197
c	0
b	0
f	0
cc	4
eloc	12
nc	3
nop	2

<?php

namespace LineMob\Core\Mocky\Auth;

use League\Tactician\Middleware;
use LineMob\Core\Mocky\Doctrine\Model\User;

class AuthorizationMiddleware implements Middleware
{
    /**
     * @var string
     */
    private $expirationPeriod;

    public function __construct($expirationPeriod = '1 day')
    {
        $this->expirationPeriod = $expirationPeriod;
    }

    /**
     * {@inheritdoc}
     */
    public function execute($command, callable $next)
    {
        if (!$command->storage) {
            throw new \LogicException("Require storage before using this middleware!");
        }

        if (!$command->secured) {
            return $next($command);
        }

        if (!$this->isSecureExpired($command->storage)) {
            return $next($command);
        }

        $cmd = new LoginCommand();
        $cmd->input = $command->input;
        $cmd->storage = $command->storage;
        $cmd->logs = $command->logs;

        $command['switchTo'] = $cmd;

        return $next($command);

    }

    /**
     * @param User $user
     *
     * @return bool
     */
    private function isSecureExpired(User $user)
    {
        // never expired
        if (!$this->expirationPeriod) {
            return false;
        }

        if (!$lastLogin = $user->getLineLastLogin()) {
            return true;
        }

        return (new \DateTime('-'.$this->expirationPeriod)) > $lastLogin;
    }
}


1			<?php
2
3			namespace LineMob\Core\Mocky\Auth;
4
5			use League\Tactician\Middleware;
6			use LineMob\Core\Mocky\Doctrine\Model\User;
7
8			class AuthorizationMiddleware implements Middleware
9			{
10			/**
11			* @var string
12			*/
13			private $expirationPeriod;
14
15			public function __construct($expirationPeriod = '1 day')
16			{
17			$this->expirationPeriod = $expirationPeriod;
18			}
19
20			/**
21			* {@inheritdoc}
22			*/
23			public function execute($command, callable $next)
24			{
25			if (!$command->storage) {
26			throw new \LogicException("Require storage before using this middleware!");
27			}
28
29			if (!$command->secured) {
30			return $next($command);
31			}
32
33			if (!$this->isSecureExpired($command->storage)) {
34			return $next($command);
35			}
36
37			$cmd = new LoginCommand();
38			$cmd->input = $command->input;
39			$cmd->storage = $command->storage;
40			$cmd->logs = $command->logs;
41
42			$command['switchTo'] = $cmd;
43
44			return $next($command);
45
46			}
47
48			/**
49			* @param User $user
50			*
51			* @return bool
52			*/
53			private function isSecureExpired(User $user)
54			{
55			// never expired
56			if (!$this->expirationPeriod) {
57			return false;
58			}
59
60			if (!$lastLogin = $user->getLineLastLogin()) {
61			return true;
62			}
63
64			return (new \DateTime('-'.$this->expirationPeriod)) > $lastLogin;
65			}
66			}
67

linemob / core

Pull Request — master (#18)

AuthorizationMiddleware::execute() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like