SprykerEco\Zed\CrefoPayApi\Business\Builder\Request\AbstractRequestBuilder

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	removeRedundantParams()	0	16	5
A	buildRequestPayload()	0	6	1

<?php

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

namespace SprykerEco\Zed\CrefoPayApi\Business\Builder\Request;

use ArrayObject;
use Generated\Shared\Transfer\CrefoPayApiRequestTransfer;

The type Generated\Shared\Transfer\CrefoPayApiRequestTransfer was not found. Maybe you did not declare it correctly or list all dependencies?

use SprykerEco\Zed\CrefoPayApi\Business\Validator\Request\CrefoPayApiRequestValidatorInterface;

abstract class AbstractRequestBuilder implements CrefoPayApiRequestBuilderInterface
{
    /**
     * @var \SprykerEco\Zed\CrefoPayApi\Business\Validator\Request\CrefoPayApiRequestValidatorInterface
     */
    protected $validator;

    /**
     * @param \Generated\Shared\Transfer\CrefoPayApiRequestTransfer $requestTransfer
     *
     * @return array
     */
    abstract protected function convertRequestTransferToArray(CrefoPayApiRequestTransfer $requestTransfer): array;

    /**
     * @param \SprykerEco\Zed\CrefoPayApi\Business\Validator\Request\CrefoPayApiRequestValidatorInterface $validator
     */
    public function __construct(CrefoPayApiRequestValidatorInterface $validator)
    {
        $this->validator = $validator;
    }

    /**
     * @param \Generated\Shared\Transfer\CrefoPayApiRequestTransfer $requestTransfer
     *
     * @return array
     */
    public function buildRequestPayload(CrefoPayApiRequestTransfer $requestTransfer): array
    {
        $this->validator->validate($requestTransfer);
        $requestPayload = $this->convertRequestTransferToArray($requestTransfer);

        return $this->removeRedundantParams($requestPayload);
    }

    /**
     * @param array $data
     *
     * @return array
     */
    protected function removeRedundantParams(array $data): array
    {
        $data = array_filter($data, function ($item) {
            if ($item instanceof ArrayObject) {
                return $item->count() !== 0;
            }
            return !empty($item);
        });

        foreach ($data as $key => $value) {
            if (is_array($value) || $value instanceof ArrayObject) {
                $data[$key] = $this->removeRedundantParams($value);

It seems like $value can also be of type ArrayObject; however, parameter $data of SprykerEco\Zed\CrefoPayApi\Business\Builder\Request\AbstractRequestBuilder::removeRedundantParams() does only seem to accept array, maybe add an additional type check?

            }
        }

        return $data;
    }
}