AuthAction::handle() - Code Metrics - Inspection of "Add role feature" - platine-php/starter - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 86cfdb...e29dd4 )

by nguereza

created 2023-12-06 10:51 UTC

AuthAction::handle() A

↳ Parent: AuthAction

Complexity

Conditions	5
Paths	5

Size

Total Lines	57
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	35
c	1
b	0
f	0
nc	5
nop	1
dl	0
loc	57
rs	9.0488

How to fix Long Method

<?php

declare(strict_types=1);

namespace Platine\App\Http\Action\User;

use Platine\App\Param\AuthParam;
use Platine\App\Validator\AuthValidator;
use Platine\Framework\Auth\AuthenticationInterface;
use Platine\Framework\Auth\Exception\AuthenticationException;
use Platine\Framework\Helper\Flash;
use Platine\Framework\Http\RequestData;
use Platine\Framework\Http\Response\RedirectResponse;
use Platine\Framework\Http\Response\TemplateResponse;
use Platine\Framework\Http\RouteHelper;
use Platine\Http\Handler\RequestHandlerInterface;
use Platine\Http\ResponseInterface;
use Platine\Http\ServerRequestInterface;
use Platine\Lang\Lang;
use Platine\Logger\LoggerInterface;
use Platine\Template\Template;

/**
* @class AuthAction
* @package Platine\App\Http\Action\User
*/
class AuthAction implements RequestHandlerInterface
{
    /**
    * The AuthenticationInterface instance
    * @var AuthenticationInterface
    */
    protected AuthenticationInterface $authentication;

    /**
    * The Flash instance
    * @var Flash
    */
    protected Flash $flash;

    /**
    * The RouteHelper instance
    * @var RouteHelper
    */
    protected RouteHelper $routeHelper;

    /**
    * The Lang instance
    * @var Lang
    */
    protected Lang $lang;

    /**
    * The LoggerInterface instance
    * @var LoggerInterface
    */
    protected LoggerInterface $logger;

    /**
    * The Template instance
    * @var Template
    */
    protected Template $template;



    /**
    * Create new instance
    * @param AuthenticationInterface $authentication
    * @param Flash $flash
    * @param RouteHelper $routeHelper
    * @param Lang $lang
    * @param LoggerInterface $logger
    * @param Template $template
    */
    public function __construct(
        AuthenticationInterface $authentication,
        Flash $flash,
        RouteHelper $routeHelper,
        Lang $lang,
        LoggerInterface $logger,
        Template $template
    ) {
        $this->authentication = $authentication;
        $this->flash = $flash;
        $this->routeHelper = $routeHelper;
        $this->lang = $lang;
        $this->logger = $logger;
        $this->template = $template;
    }

    /**
    * {@inheritdoc}
    */
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        $context = [];
        $param = new RequestData($request);

        $formParam = new AuthParam($param->posts());
        $context['param'] = $formParam;

        if ($request->getMethod() === 'GET') {
            return new TemplateResponse(
                $this->template,
                'user/login',
                $context
            );
        }

        $validator = new AuthValidator($formParam, $this->lang);
        if ($validator->validate() === false) {
            $context['errors'] = $validator->getErrors();

            return new TemplateResponse(
                $this->template,
                'user/login',
                $context
            );
        }

        $username = $formParam->getUsername();
        $password = $formParam->getPassword();

        $credentials = [
            'username' => $username,
            'password' => $password,
        ];

        try {
            $this->authentication->login($credentials);
        } catch (AuthenticationException $ex) {
            $this->logger->error('Authentication error: {error}', [
                'error' => $ex->getMessage()
            ]);

            $this->flash->setError('Authentication error. Please check your login and/or password.');

            return new TemplateResponse(
                $this->template,
                'user/login',
                $context
            );
        }

        $returnUrl = $this->routeHelper->generateUrl('home');
        if ($param->get('next')) {
            $returnUrl = $param->get('next');
        }

        return new RedirectResponse($returnUrl);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Platine\App\Http\Action\User;
6
7			use Platine\App\Param\AuthParam;
8			use Platine\App\Validator\AuthValidator;
9			use Platine\Framework\Auth\AuthenticationInterface;
10			use Platine\Framework\Auth\Exception\AuthenticationException;
11			use Platine\Framework\Helper\Flash;
12			use Platine\Framework\Http\RequestData;
13			use Platine\Framework\Http\Response\RedirectResponse;
14			use Platine\Framework\Http\Response\TemplateResponse;
15			use Platine\Framework\Http\RouteHelper;
16			use Platine\Http\Handler\RequestHandlerInterface;
17			use Platine\Http\ResponseInterface;
18			use Platine\Http\ServerRequestInterface;
19			use Platine\Lang\Lang;
20			use Platine\Logger\LoggerInterface;
21			use Platine\Template\Template;
22
23			/**
24			* @class AuthAction
25			* @package Platine\App\Http\Action\User
26			*/
27			class AuthAction implements RequestHandlerInterface
28			{
29			/**
30			* The AuthenticationInterface instance
31			* @var AuthenticationInterface
32			*/
33			protected AuthenticationInterface $authentication;
34
35			/**
36			* The Flash instance
37			* @var Flash
38			*/
39			protected Flash $flash;
40
41			/**
42			* The RouteHelper instance
43			* @var RouteHelper
44			*/
45			protected RouteHelper $routeHelper;
46
47			/**
48			* The Lang instance
49			* @var Lang
50			*/
51			protected Lang $lang;
52
53			/**
54			* The LoggerInterface instance
55			* @var LoggerInterface
56			*/
57			protected LoggerInterface $logger;
58
59			/**
60			* The Template instance
61			* @var Template
62			*/
63			protected Template $template;
64
65
66
67			/**
68			* Create new instance
69			* @param AuthenticationInterface $authentication
70			* @param Flash $flash
71			* @param RouteHelper $routeHelper
72			* @param Lang $lang
73			* @param LoggerInterface $logger
74			* @param Template $template
75			*/
76			public function __construct(
77			AuthenticationInterface $authentication,
78			Flash $flash,
79			RouteHelper $routeHelper,
80			Lang $lang,
81			LoggerInterface $logger,
82			Template $template
83			) {
84			$this->authentication = $authentication;
85			$this->flash = $flash;
86			$this->routeHelper = $routeHelper;
87			$this->lang = $lang;
88			$this->logger = $logger;
89			$this->template = $template;
90			}
91
92			/**
93			* {@inheritdoc}
94			*/
95			public function handle(ServerRequestInterface $request): ResponseInterface
96			{
97			$context = [];
98			$param = new RequestData($request);
99
100			$formParam = new AuthParam($param->posts());
101			$context['param'] = $formParam;
102
103			if ($request->getMethod() === 'GET') {
104			return new TemplateResponse(
105			$this->template,
106			'user/login',
107			$context
108			);
109			}
110
111			$validator = new AuthValidator($formParam, $this->lang);
112			if ($validator->validate() === false) {
113			$context['errors'] = $validator->getErrors();
114
115			return new TemplateResponse(
116			$this->template,
117			'user/login',
118			$context
119			);
120			}
121
122			$username = $formParam->getUsername();
123			$password = $formParam->getPassword();
124
125			$credentials = [
126			'username' => $username,
127			'password' => $password,
128			];
129
130			try {
131			$this->authentication->login($credentials);
132			} catch (AuthenticationException $ex) {
133			$this->logger->error('Authentication error: {error}', [
134			'error' => $ex->getMessage()
135			]);
136
137			$this->flash->setError('Authentication error. Please check your login and/or password.');
138
139			return new TemplateResponse(
140			$this->template,
141			'user/login',
142			$context
143			);
144			}
145
146			$returnUrl = $this->routeHelper->generateUrl('home');
147			if ($param->get('next')) {
148			$returnUrl = $param->get('next');
149			}
150
151			return new RedirectResponse($returnUrl);
152			}
153			}
154

platine-php / starter

Push — master ( 86cfdb...e29dd4 )

AuthAction::handle() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like