Mathielen\DataImport\ItemConverter\ContextMergeConverter

Complexity

Total Complexity

7

Size/Duplication

Total Lines	39
Duplicated Lines	41.03 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

56.25%

Importance

Changes	4
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	7	7	1
A	onImportPrepare()	9	9	3
A	onImportFinish()	0	5	1
A	convert()	0	8	2

<?php
namespace Mathielen\DataImport\ItemConverter;

use Ddeboer\DataImport\ItemConverter\ItemConverterInterface;
use Mathielen\DataImport\Event\ImportProcessEvent;
use Mathielen\ImportEngine\Import\Import;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

class ContextMergeConverter implements ItemConverterInterface, EventSubscriberInterface
{

    private $currentContext;

    public static function getSubscribedEvents()

This method seems to be duplicated in your project.

    {
        return array(
            ImportProcessEvent::AFTER_PREPARE => array('onImportPrepare', 0),
            ImportProcessEvent::AFTER_FINISH => array('onImportFinish', 0),
        );
    }

    public function onImportPrepare(ImportProcessEvent $event)

This method seems to be duplicated in your project.

    {
        //merge converter only works if context is an array
        if (!($event->getContext() instanceof Import) || !is_array($event->getContext()->getRun()->getContext())) {
            return;
        }

        $this->currentContext = $event->getContext()->getRun()->getContext();
    }

    public function onImportFinish(ImportProcessEvent $event, $eventName, EventDispatcherInterface $eventDispatcher)

The parameter $event is not used and could be removed.

The parameter $eventName is not used and could be removed.

    {
        //remove the subscriber when its done
        $eventDispatcher->removeSubscriber($this);
    }

    public function convert($input)
    {
        if (!$this->currentContext) {
            return $input;
        }

        return array_merge($this->currentContext, $input);
    }

}