CaptureSplitOmsCommand::execute() - Code Metrics - Inspection of "ECO-2802: Implementated split and nonsplit capture..." - spryker-eco/crefo-pay - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — feature/eco-2295/master (#7)

by Aleksey

created 2019-07-15 15:07 UTC

CaptureSplitOmsCommand::execute() A

↳ Parent: CaptureSplitOmsCommand

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

/**
 * MIT License
 * For full license information, please view the LICENSE file that was distributed with this source code.
 */

namespace SprykerEco\Zed\CrefoPay\Communication\Oms\Command;

use Orm\Zed\Sales\Persistence\SpySalesOrderItem;
filter:
    dependency_paths: ["lib/*"]
use Spryker\Zed\Oms\Business\Util\ReadOnlyArrayObject;
use SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface;
use SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface;

class CaptureSplitOmsCommand implements CrefoPayOmsCommandByItemInterface
{
    /**
     * @var \SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface
     */
    protected $mapper;

    /**
     * @var \SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface
     */
    protected $facade;

    /**
     * @param \SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface $mapper
     * @param \SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface $facade
     */
    public function __construct(
        CrefoPayOmsMapperInterface $mapper,
        CrefoPayFacadeInterface $facade
    ) {
        $this->mapper = $mapper;
        $this->facade = $facade;
    }

    /**
     * @param \Orm\Zed\Sales\Persistence\SpySalesOrderItem $salesOrderItem
     * @param \Spryker\Zed\Oms\Business\Util\ReadOnlyArrayObject $data
     *
     * @return void
     */
    public function execute(SpySalesOrderItem $salesOrderItem, ReadOnlyArrayObject $data): void
    {
        $orderTransfer = $this->mapper
            ->mapSpySalesOrderToOrderTransfer($salesOrderItem->getOrder());

        $this->facade
            ->executeCaptureOmsCommand($orderTransfer, [$salesOrderItem->getIdSalesOrderItem()]);
    }
}


1			<?php
2
3			/**
4			* MIT License
5			* For full license information, please view the LICENSE file that was distributed with this source code.
6			*/
7
8			namespace SprykerEco\Zed\CrefoPay\Communication\Oms\Command;
9
10			use Orm\Zed\Sales\Persistence\SpySalesOrderItem;
			0 ignored issues – show Bug introduced 2019-05-21 16:22 UTC by Report Bug Copy Issue Report The type `Orm\Zed\Sales\Persistence\SpySalesOrderItem` 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 Spryker\Zed\Oms\Business\Util\ReadOnlyArrayObject;
12			use SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface;
13			use SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface;
14
15			class CaptureSplitOmsCommand implements CrefoPayOmsCommandByItemInterface
16			{
17			/**
18			* @var \SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface
19			*/
20			protected $mapper;
21
22			/**
23			* @var \SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface
24			*/
25			protected $facade;
26
27			/**
28			* @param \SprykerEco\Zed\CrefoPay\Communication\Oms\CrefoPayOmsMapperInterface $mapper
29			* @param \SprykerEco\Zed\CrefoPay\Business\CrefoPayFacadeInterface $facade
30			*/
31			public function __construct(
32			CrefoPayOmsMapperInterface $mapper,
33			CrefoPayFacadeInterface $facade
34			) {
35			$this->mapper = $mapper;
36			$this->facade = $facade;
37			}
38
39			/**
40			* @param \Orm\Zed\Sales\Persistence\SpySalesOrderItem $salesOrderItem
41			* @param \Spryker\Zed\Oms\Business\Util\ReadOnlyArrayObject $data
42			*
43			* @return void
44			*/
45			public function execute(SpySalesOrderItem $salesOrderItem, ReadOnlyArrayObject $data): void
46			{
47			$orderTransfer = $this->mapper
48			->mapSpySalesOrderToOrderTransfer($salesOrderItem->getOrder());
49
50			$this->facade
51			->executeCaptureOmsCommand($orderTransfer, [$salesOrderItem->getIdSalesOrderItem()]);
52			}
53			}
54

spryker-eco / crefo-pay

Pull Request — feature/eco-2295/master (#7)

CaptureSplitOmsCommand::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like