ShopwarePlugins\Connect\Subscribers\Supplier

Complexity

Total Complexity

8

Size/Duplication

Total Lines	88
Duplicated Lines	39.77 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getSubscribedEvents()	0	6	1
A	extentBackendSupplier()	21	21	3
A	markConnectSuppliers()	0	14	2
A	getConnectSuppliers()	14	14	1

<?php
/**
 * (c) shopware AG <info@shopware.com>
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ShopwarePlugins\Connect\Subscribers;

use Doctrine\DBAL\Connection;
use Enlight\Event\SubscriberInterface;

class Supplier implements SubscriberInterface
{
    /**
     * @var Connection
     */
    private $connection;

    /**
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
    }

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return [
            'Enlight_Controller_Action_PostDispatch_Backend_Supplier' => 'extentBackendSupplier',
        ];
    }

    /**
     * @param \Enlight_Event_EventArgs $args
     */
    public function extentBackendSupplier(\Enlight_Event_EventArgs $args)

This method seems to be duplicated in your project.

    {
        /** @var $subject \Enlight_Controller_Action */
        $subject = $args->getSubject();
        $request = $subject->Request();

        switch ($request->getActionName()) {
            case 'load':
                $subject->View()->extendsTemplate(
                    'backend/supplier/list.js'
                );
                break;
            case 'getSuppliers':
                $subject->View()->data = $this->markConnectSuppliers(
                    $subject->View()->data
                );
                break;
            default:
                break;
        }
    }

    /**
     * @param array $suppliers
     * @return array
     */
    protected function markConnectSuppliers($suppliers)
    {
        $supplierIds = array_map(function ($row) {
            return $row['id'];
        }, $suppliers);

        $connectSuppliers = $this->getConnectSuppliers($supplierIds);

        foreach ($suppliers as $index => $supplier) {
            $suppliers[$index]['isConnect'] = in_array($supplier['id'], $connectSuppliers);
        }

        return $suppliers;
    }

    /**
     * @param array $supplierIds
     * @return array
     */
    protected function getConnectSuppliers($supplierIds)

This method seems to be duplicated in your project.

    {
        /** @var \Doctrine\DBAL\Connection $conn */
        $builder = $this->connection->createQueryBuilder();
        $builder->select('supplierID')
            ->from('s_articles_supplier_attributes', 'sa')
            ->where('sa.supplierID IN (:supplierIds)')
            ->andWhere('sa.connect_is_remote = 1')
            ->setParameter('supplierIds', $supplierIds, \Doctrine\DBAL\Connection::PARAM_INT_ARRAY);

        return array_map(function ($item) {
            return $item['supplierID'];
        }, $builder->execute()->fetchAll(\PDO::FETCH_ASSOC));
    }
}