Index::__construct() - Code Metrics - Inspection of "MAG2-362 - Fixed race condition in TransactionStat..." - PAYONE-GmbH/magento-2 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#600)

unknown

created 2025-07-28 13:11 UTC

Index::__construct() A

↳ Parent: Index

Complexity

Conditions	1
Paths	1

Size

Total Lines	21
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	9
c	1
b	0
f	0
nc	1
nop	10
dl	0
loc	21
rs	9.9666

How to fix Many Parameters

<?php

/**
 * PAYONE Magento 2 Connector is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Lesser General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * PAYONE Magento 2 Connector is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public License
 * along with PAYONE Magento 2 Connector. If not, see <http://www.gnu.org/licenses/>.
 *
 * PHP version 5
 *
 * @category  Payone
 * @package   Payone_Magento2_Plugin
 * @author    FATCHIP GmbH <[email protected]>
 * @copyright 2003 - 2016 Payone GmbH
 * @license   <http://www.gnu.org/licenses/> GNU Lesser General Public License
 * @link      http://www.payone.de
 */

namespace Payone\Core\Controller\Transactionstatus;

use Magento\Sales\Model\Order;
filter:
    dependency_paths: ["lib/*"]

/**
 * TransactionStatus receiver
 */
class Index extends \Payone\Core\Controller\ExternalAction
{
    /**
     * TransactionStatus model
     *
     * @var \Payone\Core\Model\ResourceModel\TransactionStatus
     */
    protected $transactionStatus;

    /**
     * PAYONE toolkit helper
     *
     * @var \Payone\Core\Helper\Toolkit
     */
    protected $toolkitHelper;

    /**
     * PAYONE environment helper
     *
     * @var \Payone\Core\Helper\Environment
     */
    protected $environmentHelper;

    /**
     * PAYONE order helper
     *
     * @var \Payone\Core\Helper\Order
     */
    protected $orderHelper;

    /**
     * TransactionStatus handler
     *
     * @var \Payone\Core\Model\Handler\TransactionStatus
     */
    protected $transactionStatusHandler;

    /**
     * Result factory for file-download
     *
     * @var \Magento\Framework\Controller\Result\RawFactory
filter:
    dependency_paths: ["lib/*"]
     */
    protected $resultRawFactory;

    /**
     * PAYONE substitute order handler
     *
     * @var \Payone\Core\Model\Handler\SubstituteOrder
     */
    protected $substituteOrder;

    /**
     * CacheInterface object
     *
     * @var \Magento\Framework\App\CacheInterface
filter:
    dependency_paths: ["lib/*"]
     */
    protected $cache;

    /**
     * Constructor
     *
     * @param \Magento\Framework\App\Action\Context              $context
     * @param \Magento\Framework\Data\Form\FormKey               $formKey
     * @param \Payone\Core\Model\ResourceModel\TransactionStatus $transactionStatus
     * @param \Payone\Core\Helper\Toolkit                        $toolkitHelper
     * @param \Payone\Core\Helper\Environment                    $environmentHelper
     * @param \Payone\Core\Helper\Order                          $orderHelper
     * @param \Payone\Core\Model\Handler\TransactionStatus       $transactionStatusHandler,
     * @param \Magento\Framework\Controller\Result\RawFactory    $resultRawFactory
     * @param \Payone\Core\Model\Handler\SubstituteOrder         $substituteOrder
     * @param \Magento\Framework\App\CacheInterface              $cache
     */
    public function __construct(
        \Magento\Framework\App\Action\Context $context,
filter:
    dependency_paths: ["lib/*"]
        \Magento\Framework\Data\Form\FormKey $formKey,
filter:
    dependency_paths: ["lib/*"]
        \Payone\Core\Model\ResourceModel\TransactionStatus $transactionStatus,
        \Payone\Core\Helper\Toolkit $toolkitHelper,
        \Payone\Core\Helper\Environment $environmentHelper,
        \Payone\Core\Helper\Order $orderHelper,
        \Payone\Core\Model\Handler\TransactionStatus $transactionStatusHandler,
        \Magento\Framework\Controller\Result\RawFactory $resultRawFactory,
        \Payone\Core\Model\Handler\SubstituteOrder $substituteOrder,
        \Magento\Framework\App\CacheInterface $cache
    ) {
        parent::__construct($context, $formKey);
        $this->transactionStatus = $transactionStatus;
        $this->toolkitHelper = $toolkitHelper;
        $this->environmentHelper = $environmentHelper;
        $this->orderHelper = $orderHelper;
        $this->transactionStatusHandler = $transactionStatusHandler;
        $this->resultRawFactory = $resultRawFactory;
        $this->substituteOrder = $substituteOrder;
        $this->cache = $cache;
    }

    /**
     * Return request parameter value
     *
     * @param  string $sKey
     * @return string
     */
    protected function getParam($sKey)
    {
        return $this->getRequest()->getParam($sKey, '');
    }

    /**
     * Return Post array
     *
     * @return array
     */
    protected function getPostArray()
    {
        return $this->getRequest()->getPostValue();
    }

    /**
     * Write the TransactionStatus to the database
     *
     * @param  Order $oOrder
     * @param  array $aRequest
     * @param  bool  $blWillBeHandled
     * @return void
     */
    protected function logTransactionStatus(Order $oOrder, $aRequest, $blWillBeHandled)
    {
        $this->transactionStatus->addTransactionLogEntry($aRequest, $oOrder, $blWillBeHandled);
    }

    /**
     * Main method for executing all needed actions for the incoming TransactionStatus
     *
     * @return string
     */
    protected function handleTransactionStatus()
    {
        if (!$this->environmentHelper->isRemoteIpValid()) {
            return 'Access denied';
        } elseif (!$this->toolkitHelper->isKeyValid($this->getParam('key'))) {
            return 'Key wrong or missing!';
        }

        $oOrder = $this->orderHelper->getOrderByTxid($this->getParam('txid'));
        if (!$oOrder) {
            return 'Order not found';
        }

        if ($this->getParam('txaction') != 'appointed' && empty($this->transactionStatus->getAppointedIdByTxid($this->getParam('txid')))) {
            return 'Appointed status has to be handled first!';
        }

        $lockKey = 'payone_order_status_lock_' . $oOrder->getId();
        if ($this->cache->load($lockKey)) {
            return 'Another status is being handled at the moment.';
        }

        // Lock transaction status handling for this order for 50 seconds to not handle multiple statuses at the same time
        $this->cache->save('1', $lockKey, [], 50);

        try {
            if ($this->getParam('txaction') == 'appointed' && $oOrder->getStatus() == 'canceled') {
                // order was canceled in checkout, probably due to browser-back-button usage -> create a new order for incoming payment
                $oOrder = $this->substituteOrder->createSubstituteOrder($oOrder, false);
            }

            $this->logTransactionStatus($oOrder, $this->getPostArray(), true);

            $this->transactionStatusHandler->handle($oOrder, $this->getPostArray());
        } catch (\Exception $e) {
            $this->cache->remove($lockKey);
            throw $e;
        }

        return 'TSOK';
    }

    /**
     * Executing TransactionStatus handling
     *
     * @return \Magento\Framework\Controller\Result\Raw
filter:
    dependency_paths: ["lib/*"]
     */
    public function execute()
    {
        $sOutput = $this->handleTransactionStatus();

        $oResultRaw = $this->resultRawFactory->create();
        $oResultRaw->setContents($sOutput);
        
        return $oResultRaw;
    }
}


1			<?php
2
3			/**
4			* PAYONE Magento 2 Connector is free software: you can redistribute it and/or modify
5			* it under the terms of the GNU Lesser General Public License as published by
6			* the Free Software Foundation, either version 3 of the License, or
7			* (at your option) any later version.
8			*
9			* PAYONE Magento 2 Connector is distributed in the hope that it will be useful,
10			* but WITHOUT ANY WARRANTY; without even the implied warranty of
11			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12			* GNU Lesser General Public License for more details.
13			*
14			* You should have received a copy of the GNU Lesser General Public License
15			* along with PAYONE Magento 2 Connector. If not, see <http://www.gnu.org/licenses/>.
16			*
17			* PHP version 5
18			*
19			* @category Payone
20			* @package Payone_Magento2_Plugin
21			* @author FATCHIP GmbH <[email protected]>
22			* @copyright 2003 - 2016 Payone GmbH
23			* @license <http://www.gnu.org/licenses/> GNU Lesser General Public License
24			* @link http://www.payone.de
25			*/
26
27			namespace Payone\Core\Controller\Transactionstatus;
28
29			use Magento\Sales\Model\Order;
			0 ignored issues – show Bug introduced 2018-03-28 06:55 UTC by Report Bug Copy Issue Report The type `Magento\Sales\Model\Order` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
30
31			/**
32			* TransactionStatus receiver
33			*/
34			class Index extends \Payone\Core\Controller\ExternalAction
35			{
36			/**
37			* TransactionStatus model
38			*
39			* @var \Payone\Core\Model\ResourceModel\TransactionStatus
40			*/
41			protected $transactionStatus;
42
43			/**
44			* PAYONE toolkit helper
45			*
46			* @var \Payone\Core\Helper\Toolkit
47			*/
48			protected $toolkitHelper;
49
50			/**
51			* PAYONE environment helper
52			*
53			* @var \Payone\Core\Helper\Environment
54			*/
55			protected $environmentHelper;
56
57			/**
58			* PAYONE order helper
59			*
60			* @var \Payone\Core\Helper\Order
61			*/
62			protected $orderHelper;
63
64			/**
65			* TransactionStatus handler
66			*
67			* @var \Payone\Core\Model\Handler\TransactionStatus
68			*/
69			protected $transactionStatusHandler;
70
71			/**
72			* Result factory for file-download
73			*
74			* @var \Magento\Framework\Controller\Result\RawFactory
			0 ignored issues – show Bug introduced 2018-03-28 06:55 UTC by Report Bug Copy Issue Report The type `Magento\Framework\Controller\Result\RawFactory` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
75			*/
76			protected $resultRawFactory;
77
78			/**
79			* PAYONE substitute order handler
80			*
81			* @var \Payone\Core\Model\Handler\SubstituteOrder
82			*/
83			protected $substituteOrder;
84
85			/**
86			* CacheInterface object
87			*
88			* @var \Magento\Framework\App\CacheInterface
			0 ignored issues – show Bug introduced 2025-07-28 13:20 UTC by Report Bug Copy Issue Report The type `Magento\Framework\App\CacheInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
89			*/
90			protected $cache;
91
92			/**
93			* Constructor
94			*
95			* @param \Magento\Framework\App\Action\Context $context
96			* @param \Magento\Framework\Data\Form\FormKey $formKey
97			* @param \Payone\Core\Model\ResourceModel\TransactionStatus $transactionStatus
98			* @param \Payone\Core\Helper\Toolkit $toolkitHelper
99			* @param \Payone\Core\Helper\Environment $environmentHelper
100			* @param \Payone\Core\Helper\Order $orderHelper
101			* @param \Payone\Core\Model\Handler\TransactionStatus $transactionStatusHandler,
102			* @param \Magento\Framework\Controller\Result\RawFactory $resultRawFactory
103			* @param \Payone\Core\Model\Handler\SubstituteOrder $substituteOrder
104			* @param \Magento\Framework\App\CacheInterface $cache
105			*/
106			public function __construct(
107			\Magento\Framework\App\Action\Context $context,
			0 ignored issues – show Bug introduced 2019-07-04 13:50 UTC by Report Bug Copy Issue Report The type `Magento\Framework\App\Action\Context` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
108			\Magento\Framework\Data\Form\FormKey $formKey,
			0 ignored issues – show Bug introduced 2019-07-04 13:50 UTC by Report Bug Copy Issue Report The type `Magento\Framework\Data\Form\FormKey` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
109			\Payone\Core\Model\ResourceModel\TransactionStatus $transactionStatus,
110			\Payone\Core\Helper\Toolkit $toolkitHelper,
111			\Payone\Core\Helper\Environment $environmentHelper,
112			\Payone\Core\Helper\Order $orderHelper,
113			\Payone\Core\Model\Handler\TransactionStatus $transactionStatusHandler,
114			\Magento\Framework\Controller\Result\RawFactory $resultRawFactory,
115			\Payone\Core\Model\Handler\SubstituteOrder $substituteOrder,
116			\Magento\Framework\App\CacheInterface $cache
117			) {
118			parent::__construct($context, $formKey);
119			$this->transactionStatus = $transactionStatus;
120			$this->toolkitHelper = $toolkitHelper;
121			$this->environmentHelper = $environmentHelper;
122			$this->orderHelper = $orderHelper;
123			$this->transactionStatusHandler = $transactionStatusHandler;
124			$this->resultRawFactory = $resultRawFactory;
125			$this->substituteOrder = $substituteOrder;
126			$this->cache = $cache;
127			}
128
129			/**
130			* Return request parameter value
131			*
132			* @param string $sKey
133			* @return string
134			*/
135			protected function getParam($sKey)
136			{
137			return $this->getRequest()->getParam($sKey, '');
138			}
139
140			/**
141			* Return Post array
142			*
143			* @return array
144			*/
145			protected function getPostArray()
146			{
147			return $this->getRequest()->getPostValue();
148			}
149
150			/**
151			* Write the TransactionStatus to the database
152			*
153			* @param Order $oOrder
154			* @param array $aRequest
155			* @param bool $blWillBeHandled
156			* @return void
157			*/
158			protected function logTransactionStatus(Order $oOrder, $aRequest, $blWillBeHandled)
159			{
160			$this->transactionStatus->addTransactionLogEntry($aRequest, $oOrder, $blWillBeHandled);
161			}
162
163			/**
164			* Main method for executing all needed actions for the incoming TransactionStatus
165			*
166			* @return string
167			*/
168			protected function handleTransactionStatus()
169			{
170			if (!$this->environmentHelper->isRemoteIpValid()) {
171			return 'Access denied';
172			} elseif (!$this->toolkitHelper->isKeyValid($this->getParam('key'))) {
173			return 'Key wrong or missing!';
174			}
175
176			$oOrder = $this->orderHelper->getOrderByTxid($this->getParam('txid'));
177			if (!$oOrder) {
178			return 'Order not found';
179			}
180
181			if ($this->getParam('txaction') != 'appointed' && empty($this->transactionStatus->getAppointedIdByTxid($this->getParam('txid')))) {
182			return 'Appointed status has to be handled first!';
183			}
184
185			$lockKey = 'payone_order_status_lock_' . $oOrder->getId();
186			if ($this->cache->load($lockKey)) {
187			return 'Another status is being handled at the moment.';
188			}
189
190			// Lock transaction status handling for this order for 50 seconds to not handle multiple statuses at the same time
191			$this->cache->save('1', $lockKey, [], 50);
192
193			try {
194			if ($this->getParam('txaction') == 'appointed' && $oOrder->getStatus() == 'canceled') {
195			// order was canceled in checkout, probably due to browser-back-button usage -> create a new order for incoming payment
196			$oOrder = $this->substituteOrder->createSubstituteOrder($oOrder, false);
197			}
198
199			$this->logTransactionStatus($oOrder, $this->getPostArray(), true);
200
201			$this->transactionStatusHandler->handle($oOrder, $this->getPostArray());
202			} catch (\Exception $e) {
203			$this->cache->remove($lockKey);
204			throw $e;
205			}
206
207			return 'TSOK';
208			}
209
210			/**
211			* Executing TransactionStatus handling
212			*
213			* @return \Magento\Framework\Controller\Result\Raw
			0 ignored issues – show Bug introduced 2019-02-27 13:33 UTC by Report Bug Copy Issue Report The type `Magento\Framework\Controller\Result\Raw` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
214			*/
215			public function execute()
216			{
217			$sOutput = $this->handleTransactionStatus();
218
219			$oResultRaw = $this->resultRawFactory->create();
220			$oResultRaw->setContents($sOutput);
221
222			return $oResultRaw;
223			}
224			}
225

PAYONE-GmbH / magento-2

Pull Request — master (#600)

Index::__construct() A

Complexity

Size

Duplication

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like