LegacyController::setGetVariables() - Code Metrics - Inspection of "transferred project to laminas" - maglnet/MaglLegacyApplication - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — transform-to-laminas3 ( 1cf951 )

unknown

created 2021-04-01 08:17 UTC

LegacyController::setGetVariables() B

↳ Parent: LegacyController

Complexity

Conditions	10
Paths	11

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	36
rs	7.6666
c	0
b	0
f	0
cc	10
nc	11
nop	0

How to fix Complexity

<?php

/**
 * @author Matthias Glaub <[email protected]>
 * @license http://opensource.org/licenses/BSD-3-Clause BSD-3-Clause
 */

namespace MaglLegacyApplication\Controller;

use Laminas\Http\Response;
use Laminas\Mvc\Controller\AbstractActionController;
use Laminas\View\Model\ViewModel;
use MaglLegacyApplication\Application\MaglLegacy;
use MaglLegacyApplication\Options\LegacyControllerOptions;

class LegacyController extends AbstractActionController
{

    /**
     *
     * @var LegacyControllerOptions
     */
    private $options;

    /**
     *
     * @var MaglLegacy
     */
    private $legacy;

    public function __construct(LegacyControllerOptions $options, MaglLegacy $legacy)
    {
        $this->options = $options;
        $this->legacy = $legacy;
    }

    public function indexAction()
    {
        $docRoots = $this->options->getDocRoots();
        foreach ($docRoots as $key => $docRoot) {
            $docRoots[$key] = rtrim(getcwd() . '/' . $docRoot);
        }

        $scriptName = $this->params('script');

        if (empty($scriptName)) {
            $path = $this->params(('path')) ? $this->params('path') : '';
            foreach ($this->options->getIndexFiles() as $indexFile) {
                foreach ($docRoots as $docRoot) {
                    if (is_file($docRoot . '/' . $path . $indexFile)) {
                        $this->legacy->setLegacyScriptName($path . $indexFile);
                        return $this->runScript($docRoot . '/' . $path . $indexFile);
                    }
                }
            }
        }

        $scriptUri = '/' . ltrim($scriptName, '/'); // force leading '/'
        foreach ($docRoots as $docRoot) {
            $legacyScriptFilename = $docRoot . $scriptUri;
            if (is_file($legacyScriptFilename)) {
                //inform the application about the used script
                $this->legacy->setLegacyScriptName($scriptUri);

                //inject get and request variables
                $this->setGetVariables();

                return $this->runScript($legacyScriptFilename);
            }
        }

        /** @var Response $response */
        $response = $this->getResponse();
        $response->setStatusCode(404);
    }

    private function setGetVariables()
    {
        $globals_options = $this->options->getGlobals();

        // if we should not set any global vars, we can return safely
        if (!$globals_options['get'] && !$globals_options['request']) {
            return;
        }

        // check if $_GET is used at all (ini - variables_order ?)
        // check if $_GET is written to $_REQUEST (ini - variables_order / request_order)
        // depending on request_order, check if $_REQUEST is already written and decide if we are allowed to override

        $request_order = ini_get('request_order');

        if ($request_order === false) {
            $request_order = ini_get('variables_order');
        }

        $get_prio = stripos($request_order, 'g');
        $post_prio = stripos($request_order, 'p');

        $forceOverrideRequest = $get_prio > $post_prio;

        $routeParams = $this->getEvent()->getRouteMatch()->getParams();

        foreach ($routeParams as $paramName => $paramValue) {
            if ($globals_options['get'] && !isset($_GET[$paramName])) {
                $_GET[$paramName] = $paramValue;
            }

            if ($globals_options['request'] && ($forceOverrideRequest || !isset($_REQUEST[$paramName]))) {
                $_REQUEST[$paramName] = $paramValue;
            }
        }
    }

    private function runScript($scriptFileName)
    {
        $this->legacy->setLegacyScriptFilename($scriptFileName);

        ob_start();
        $result = include $scriptFileName;
        $output = ob_get_clean();

        if ($result instanceof ViewModel || $result instanceof Response) {
            if ($this->options->getPrependOutputBufferToResponse()) {
                echo $output;
            }
            return $result;
        }

        $result = $this->getEventManager()->trigger(MaglLegacy::EVENT_SHORT_CIRCUIT_RESPONSE, $this);
        if ($result->stopped()) {
            if ($this->options->getPrependOutputBufferToResponse()) {
                $result->last()->setContent($output . $result->last()->getContent());
            }
            return $result->last();
        }

        $this->getResponse()->setContent($output);
        return $this->getResponse();
    }
}


1			<?php
2
3			/**
4			* @author Matthias Glaub <[email protected]>
5			* @license http://opensource.org/licenses/BSD-3-Clause BSD-3-Clause
6			*/
7
8			namespace MaglLegacyApplication\Controller;
9
10			use Laminas\Http\Response;
11			use Laminas\Mvc\Controller\AbstractActionController;
12			use Laminas\View\Model\ViewModel;
13			use MaglLegacyApplication\Application\MaglLegacy;
14			use MaglLegacyApplication\Options\LegacyControllerOptions;
15
16			class LegacyController extends AbstractActionController
17			{
18
19			/**
20			*
21			* @var LegacyControllerOptions
22			*/
23			private $options;
24
25			/**
26			*
27			* @var MaglLegacy
28			*/
29			private $legacy;
30
31			public function __construct(LegacyControllerOptions $options, MaglLegacy $legacy)
32			{
33			$this->options = $options;
34			$this->legacy = $legacy;
35			}
36
37			public function indexAction()
38			{
39			$docRoots = $this->options->getDocRoots();
40			foreach ($docRoots as $key => $docRoot) {
41			$docRoots[$key] = rtrim(getcwd() . '/' . $docRoot);
42			}
43
44			$scriptName = $this->params('script');
45
46			if (empty($scriptName)) {
47			$path = $this->params(('path')) ? $this->params('path') : '';
48			foreach ($this->options->getIndexFiles() as $indexFile) {
49			foreach ($docRoots as $docRoot) {
50			if (is_file($docRoot . '/' . $path . $indexFile)) {
51			$this->legacy->setLegacyScriptName($path . $indexFile);
52			return $this->runScript($docRoot . '/' . $path . $indexFile);
53			}
54			}
55			}
56			}
57
58			$scriptUri = '/' . ltrim($scriptName, '/'); // force leading '/'
59			foreach ($docRoots as $docRoot) {
60			$legacyScriptFilename = $docRoot . $scriptUri;
61			if (is_file($legacyScriptFilename)) {
62			//inform the application about the used script
63			$this->legacy->setLegacyScriptName($scriptUri);
64
65			//inject get and request variables
66			$this->setGetVariables();
67
68			return $this->runScript($legacyScriptFilename);
69			}
70			}
71
72			/** @var Response $response */
73			$response = $this->getResponse();
74			$response->setStatusCode(404);
75			}
76
77			private function setGetVariables()
78			{
79			$globals_options = $this->options->getGlobals();
80
81			// if we should not set any global vars, we can return safely
82			if (!$globals_options['get'] && !$globals_options['request']) {
83			return;
84			}
85
86			// check if $_GET is used at all (ini - variables_order ?)
87			// check if $_GET is written to $_REQUEST (ini - variables_order / request_order)
88			// depending on request_order, check if $_REQUEST is already written and decide if we are allowed to override
89
90			$request_order = ini_get('request_order');
91
92			if ($request_order === false) {
93			$request_order = ini_get('variables_order');
94			}
95
96			$get_prio = stripos($request_order, 'g');
97			$post_prio = stripos($request_order, 'p');
98
99			$forceOverrideRequest = $get_prio > $post_prio;
100
101			$routeParams = $this->getEvent()->getRouteMatch()->getParams();
102
103			foreach ($routeParams as $paramName => $paramValue) {
104			if ($globals_options['get'] && !isset($_GET[$paramName])) {
105			$_GET[$paramName] = $paramValue;
106			}
107
108			if ($globals_options['request'] && ($forceOverrideRequest \|\| !isset($_REQUEST[$paramName]))) {
109			$_REQUEST[$paramName] = $paramValue;
110			}
111			}
112			}
113
114			private function runScript($scriptFileName)
115			{
116			$this->legacy->setLegacyScriptFilename($scriptFileName);
117
118			ob_start();
119			$result = include $scriptFileName;
120			$output = ob_get_clean();
121
122			if ($result instanceof ViewModel \|\| $result instanceof Response) {
123			if ($this->options->getPrependOutputBufferToResponse()) {
124			echo $output;
125			}
126			return $result;
127			}
128
129			$result = $this->getEventManager()->trigger(MaglLegacy::EVENT_SHORT_CIRCUIT_RESPONSE, $this);
130			if ($result->stopped()) {
131			if ($this->options->getPrependOutputBufferToResponse()) {
132			$result->last()->setContent($output . $result->last()->getContent());
133			}
134			return $result->last();
135			}
136
137			$this->getResponse()->setContent($output);
138			return $this->getResponse();
139			}
140			}
141

maglnet / MaglLegacyApplication

Push — transform-to-laminas3 ( 1cf951 )

LegacyController::setGetVariables() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like