ComputopPayPalExpressPaymentHandler::addPaymentToQuote() - Code Metrics - Inspection of "CC-14473: Fixed notifications while redirecting fa..." - spryker-eco/computop - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#44)

by Michael

created 2021-12-16 13:24 UTC

addPaymentToQuote() A

↳ Parent: ComputopPayPalExpressPaymentHandler

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	5
dl	0
loc	11
rs	10
c	1
b	0
f	0
cc	2
nc	2
nop	2

<?php

/**
 * MIT License
 * Use of this software requires acceptance of the Evaluation License Agreement. See LICENSE file.
 */

namespace SprykerEco\Yves\Computop\Handler\PostPlace;

use Generated\Shared\Transfer\ComputopPayPalExpressPaymentTransfer;
filter:
    dependency_paths: ["lib/*"]
use Generated\Shared\Transfer\QuoteTransfer;
filter:
    dependency_paths: ["lib/*"]
use Spryker\Shared\Kernel\Transfer\AbstractTransfer;

class ComputopPayPalExpressPaymentHandler extends AbstractPostPlacePaymentHandler
{
    /**
     * @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer
     * @param \Spryker\Shared\Kernel\Transfer\AbstractTransfer $responseTransfer
     *
     * @return \Generated\Shared\Transfer\QuoteTransfer
     */
    protected function addPaymentToQuote(QuoteTransfer $quoteTransfer, AbstractTransfer $responseTransfer): QuoteTransfer
    {
        if ($quoteTransfer->getPaymentOrFail()->getComputopPayPalExpress() === null) {
            $quoteTransfer->getPaymentOrFail()->setComputopPayPalExpress(new ComputopPayPalExpressPaymentTransfer());
        }

        $quoteTransfer->getPaymentOrFail()->getComputopPayPalExpressOrFail()->setPayPalExpressInitResponse(
            $responseTransfer,
        );

        return $quoteTransfer;
    }

    /**
     * @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer
     *
     * @return \Generated\Shared\Transfer\QuoteTransfer
     */
    protected function saveInitResponse(QuoteTransfer $quoteTransfer): QuoteTransfer
    {
        return $this->computopClient->savePayPalExpressInitResponse($quoteTransfer);
    }
}


1			<?php
2
3			/**
4			* MIT License
5			* Use of this software requires acceptance of the Evaluation License Agreement. See LICENSE file.
6			*/
7
8			namespace SprykerEco\Yves\Computop\Handler\PostPlace;
9
10			use Generated\Shared\Transfer\ComputopPayPalExpressPaymentTransfer;
			0 ignored issues – show Bug introduced 2021-05-10 09:09 UTC by Report Bug Copy Issue Report The type `Generated\Shared\Transfe...lExpressPaymentTransfer` 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...
11			use Generated\Shared\Transfer\QuoteTransfer;
			0 ignored issues – show Bug introduced 2021-10-15 07:47 UTC by Report Bug Copy Issue Report The type `Generated\Shared\Transfer\QuoteTransfer` 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...
12			use Spryker\Shared\Kernel\Transfer\AbstractTransfer;
13
14			class ComputopPayPalExpressPaymentHandler extends AbstractPostPlacePaymentHandler
15			{
16			/**
17			* @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer
18			* @param \Spryker\Shared\Kernel\Transfer\AbstractTransfer $responseTransfer
19			*
20			* @return \Generated\Shared\Transfer\QuoteTransfer
21			*/
22			protected function addPaymentToQuote(QuoteTransfer $quoteTransfer, AbstractTransfer $responseTransfer): QuoteTransfer
23			{
24			if ($quoteTransfer->getPaymentOrFail()->getComputopPayPalExpress() === null) {
25			$quoteTransfer->getPaymentOrFail()->setComputopPayPalExpress(new ComputopPayPalExpressPaymentTransfer());
26			}
27
28			$quoteTransfer->getPaymentOrFail()->getComputopPayPalExpressOrFail()->setPayPalExpressInitResponse(
29			$responseTransfer,
30			);
31
32			return $quoteTransfer;
33			}
34
35			/**
36			* @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer
37			*
38			* @return \Generated\Shared\Transfer\QuoteTransfer
39			*/
40			protected function saveInitResponse(QuoteTransfer $quoteTransfer): QuoteTransfer
41			{
42			return $this->computopClient->savePayPalExpressInitResponse($quoteTransfer);
43			}
44			}
45

spryker-eco / computop

Pull Request — master (#44)

addPaymentToQuote() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like