Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product\ValidatorTrait

Complexity

Total Complexity

21

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	validateMainImage()	0	10	4
A	validateAttributes()	0	14	4
A	validate()	0	13	3
A	validateObjects()	0	5	2
B	validateSkuInfoList()	0	17	7
A	processResponse()	0	6	1

<?php
declare(strict_types=1);

/**
 * Domain Entity Class
 * @author Max Demian <ticaje@filetea.me>
 */

namespace Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product;

use Ticaje\Contract\Patterns\Interfaces\Decorator\DecoratorInterface;
use Ticaje\AeSdk\Infrastructure\Interfaces\Provider\Request\RequestDtoInterface;

/**
 * Class ValidatorTrait
 * @package Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product
 */
trait ValidatorTrait
{
    /**
     * @inheritDoc
     */
    public function validate(RequestDtoInterface $request): DecoratorInterface
    {
        $requestContent = $request->getContent();
        $params = json_decode($requestContent); // This might be subject to a volatile strategy
        foreach (self::ARTIFACTS as $component) {

The constant Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product\ValidatorTrait::ARTIFACTS was not found. Maybe you did not declare it correctly or list all dependencies?

            $function = "validate{$component}";
            $validComponent = $this->$function($params);
            if (!$validComponent) {
                return $this->response;
            }
        }
        $this->processResponse(1, "Request is Valid", $requestContent);
        return $this->response;
    }

    /**
     * @param $params
     * @return bool
     */
    private function validateObjects($params)
    {
        $result = is_object($params);
        $result ?: $this->processResponse(0, "Request must be an object");
        return $result;
    }

    /**
     * @param $params
     * @return bool
     */
    private function validateMainImage($params)
    {
        $result = (function () use ($params) {
            $mainImageArray = $params->main_image_urls_list ?? null;
            return $mainImageArray
                && is_array($mainImageArray)
                && 6 > count($mainImageArray);
        })();
        $result ?: $this->processResponse(0, "Incorrect Images compliance");
        return $result;
    }

    /**
     * @param $params
     * @return bool
     */
    private function validateSkuInfoList($params)
    {
        $result = (function ($params) {
            $list = $params->sku_info_list ?? null;
            return $list
                && is_array($list)
                && count($list) > 0
                && (function ($list) {
                    $missing = [];
                    foreach (static::SKU_INFO_LIST_REQUIRED_ATTRIBUTES as $property) {

The constant Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product\ValidatorTrait::SKU_INFO_LIST_REQUIRED_ATTRIBUTES was not found. Maybe you did not declare it correctly or list all dependencies?

                        property_exists($list[0], $property) ?: array_push($missing, $property);
                    }
                    return empty($missing);
                })($list);
        })($params);
        $result ?: $this->processResponse(0, "Incorrect Info List compliance");
        return $result;
    }

    /**
     * @param $params
     * @return bool
     */
    private function validateAttributes($params)
    {
        $result = (function ($params) use (&$missing) {
            $missing = [];
            foreach (static::REQUIRED_ATTRIBUTES as $property) {

The constant Ticaje\AliexpressConsumer\Domain\Entity\Request\Validator\Product\ValidatorTrait::REQUIRED_ATTRIBUTES was not found. Maybe you did not declare it correctly or list all dependencies?

                property_exists($params, $property) ?: array_push($missing, $property);
            }
            return empty($missing);
        })($params);
        $result ?: (function ($missing) {
            $missingAttributes = implode(', ', $missing);
            $this->processResponse(0, "Missing parameters {$missingAttributes}");
        })($missing);
        return $result;
    }

    /**
     * @param $success
     * @param $message
     * @param null $result

Are you sure the doc-type for parameter $result is correct as it would always require null to be passed?

     * Kind of mixing concerns yeh
     */
    private function processResponse($success = true, $message, $result = null)
    {
        $this->response
            ->setSuccess((bool)$success)
            ->setRequest($result)
            ->setMessage($message);
    }
}