Getnet\SubSellerMagento\Cron\SynchronizeCreate

Complexity

Total Complexity

3

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	execute()	0	13	2

<?php
/**
 * Copyright © Getnet. All rights reserved.
 *
 * @author    Bruno Elisei <brunoelisei@o2ti.com>
 * See LICENSE for license details.
 */

namespace Getnet\SubSellerMagento\Cron;

use Getnet\SubSellerMagento\Api\SubSellerRepositoryInterface;
use Getnet\SubSellerMagento\Logger\Logger;
use Getnet\SubSellerMagento\Model\Console\Command\Synchronize\Create;
use Magento\Framework\Api\SearchCriteriaBuilder;

/**
 * Class Synchronize Create Sub Seller on Getnet.
 */
class SynchronizeCreate
{
    /**
     * @var Logger
     */
    protected $logger;

    /**
     * @var Create
     */
    protected $create;

    /**
     * @var SubSellerRepositoryInterface
     */
    protected $subSellerRepository;

    /**
     * @var SearchCriteriaBuilder
     */
    protected $searchCriteria;

    /**
     * Constructor.
     *
     * @param Logger                       $logger
     * @param Create                       $create
     * @param SubSellerRepositoryInterface $subSellerRepository
     * @param SearchCriteriaBuilder        $searchCriteria
     */
    public function __construct(
        Logger $logger,
        Create $create,
        SubSellerRepositoryInterface $subSellerRepository,
        SearchCriteriaBuilder $searchCriteria
    ) {
        $this->logger = $logger;
        $this->create = $create;
        $this->subSellerRepository = $subSellerRepository;
        $this->searchCriteria = $searchCriteria;
    }

    /**
     * Execute the cron.
     *
     * @return void
     */
    public function execute()
    {
        $this->logger->info('Cronjob Sync Create Seller');

        $searchCriteria = $this->searchCriteria->addFilter('status', 1)->create();
        $subSellers = $this->subSellerRepository->getList($searchCriteria);

        foreach ($subSellers->getItems() as $subSeller) {
            $this->logger->info(sprintf('Cronjob Sync Create Seller id %s.', $subSeller->getId()));
            $this->create->create($subSeller->getId());

It seems like $subSeller->getId() can also be of type null; however, parameter $subSellerId of Getnet\SubSellerMagento\Model\Console\Command\Synchronize\Create::create() does only seem to accept integer, maybe add an additional type check?

        }

        $this->logger->info('Cronjob Sync Create Seller is done.');
    }
}