RemoveController::indexAction() - Code Metrics - Inspection of "Merge pull request #274: User account removal" - cross-solution/YAWIK - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 031ed9...9561de )

unknown

created 2016-09-02 15:01 UTC

RemoveController::indexAction() B

↳ Parent: RemoveController

Complexity

Conditions	3
Paths	3

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	25
rs	8.8571
cc	3
eloc	16
nc	3
nop	0

<?php
/**
 * @filesource
 * @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
 * @license MIT
 * @author Miroslav Fedeleš <[email protected]>
 * @since 0.27
 */
namespace Auth\Controller;

use Auth\Entity\Status;
use Zend\Mvc\Controller\AbstractActionController;
use Auth\Dependency\Manager as Dependencies;

/**
 *
 * @author Miroslav Fedeleš <[email protected]>r
 * @author Mathias Gelhausen <[email protected]>
 * @since 0.27
 */
class RemoveController extends AbstractActionController
{
    /**
     * @var Dependencies
     */
    protected $dependencies;

    /**
     * @param Dependencies $dependencies
     */
    public function __construct(Dependencies $dependencies)
    {
        $this->dependencies = $dependencies;
    }

    public function indexAction()
    {
        /* @var \Auth\AuthenticationService $auth */
        $auth = $this->serviceLocator->get('AuthenticationService');
        $user = $auth->getUser();
        $error = false;
        
        if ($this->params()->fromPost('confirm'))
        {
            if ($this->dependencies->removeItems($user)) {
                $auth->clearIdentity();
                $user->setStatus(Status::INACTIVE);
                return $this->redirect()->toRoute('lang');
            } else {
                $error = true;
            }
        }
        
        return [
            'lists' => $this->dependencies->getLists(),
            'user' => $user,
            'limit' => 20,
            'error' => $error,
        ];
    }
}


1			<?php
2			/**
3			* @filesource
4			* @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
5			* @license MIT
6			* @author Miroslav Fedeleš <[email protected]>
7			* @since 0.27
8			*/
9			namespace Auth\Controller;
10
11			use Auth\Entity\Status;
12			use Zend\Mvc\Controller\AbstractActionController;
13			use Auth\Dependency\Manager as Dependencies;
14
15			/**
16			*
17			* @author Miroslav Fedeleš <[email protected]>r
18			* @author Mathias Gelhausen <[email protected]>
19			* @since 0.27
20			*/
21			class RemoveController extends AbstractActionController
22			{
23			/**
24			* @var Dependencies
25			*/
26			protected $dependencies;
27
28			/**
29			* @param Dependencies $dependencies
30			*/
31			public function __construct(Dependencies $dependencies)
32			{
33			$this->dependencies = $dependencies;
34			}
35
36			public function indexAction()
37			{
38			/* @var \Auth\AuthenticationService $auth */
39			$auth = $this->serviceLocator->get('AuthenticationService');
40			$user = $auth->getUser();
41			$error = false;
42
43			if ($this->params()->fromPost('confirm'))
44			{
45			if ($this->dependencies->removeItems($user)) {
46			$auth->clearIdentity();
47			$user->setStatus(Status::INACTIVE);
48			return $this->redirect()->toRoute('lang');
49			} else {
50			$error = true;
51			}
52			}
53
54			return [
55			'lists' => $this->dependencies->getLists(),
56			'user' => $user,
57			'limit' => 20,
58			'error' => $error,
59			];
60			}
61			}
62

cross-solution / YAWIK

Push — develop ( 031ed9...9561de )

RemoveController::indexAction() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like