SWP\Bundle\CoreBundle\EventSubscriber\HandleArticleChangeSubscriber

Complexity

Total Complexity

8

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getSubscribedEvents()	0	8	1
A	processArticle()	0	5	1
A	updateRoute()	0	10	2
A	refreshLists()	0	12	3

<?php

declare(strict_types=1);

/*
 * This file is part of the Superdesk Web Publisher Core Bundle.
 *
 * Copyright 2018 Sourcefabric z.ú. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2018 Sourcefabric z.ú
 * @license http://www.superdesk.org/license
 */

namespace SWP\Bundle\CoreBundle\EventSubscriber;

use Doctrine\ORM\EntityManagerInterface;
use SWP\Bundle\ContentBundle\ArticleEvents;
use SWP\Bundle\ContentBundle\Event\ArticleEvent;
use SWP\Bundle\ContentBundle\Model\ArticleInterface;
use SWP\Bundle\CoreBundle\Model\ContentListItemInterface;
use SWP\Bundle\CoreBundle\Repository\ContentListItemRepositoryInterface;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

class HandleArticleChangeSubscriber implements EventSubscriberInterface
{
    private $contentListItemRepository;

    private $manager;

    public function __construct(ContentListItemRepositoryInterface $contentListItemRepository, EntityManagerInterface $manager)
    {
        $this->contentListItemRepository = $contentListItemRepository;
        $this->manager = $manager;
    }

    public static function getSubscribedEvents()
    {
        return [
            ArticleEvents::POST_UPDATE => 'processArticle',
            ArticleEvents::POST_CREATE => 'processArticle',
            ArticleEvents::POST_UNPUBLISH => 'processArticle',
        ];
    }

    public function processArticle(ArticleEvent $event)
    {
        $this->refreshLists($event->getArticle());
        $this->updateRoute($event->getArticle());
    }

    private function updateRoute(ArticleInterface $article)
    {
        $route = $article->getRoute();
        if (null === $route) {
            return;
        }

        $route->setArticlesUpdatedAt(new \DateTime());
        $this->manager->flush();
    }

    private function refreshLists(ArticleInterface $article)
    {
        $contentListItems = $this->contentListItemRepository->findItemsByArticle($article);

$article of type object<SWP\Bundle\ContentBundle\Model\ArticleInterface> is not a sub-type of object<SWP\Bundle\CoreBundle\Model\ArticleInterface>. It seems like you assume a child interface of the interface SWP\Bundle\ContentBundle\Model\ArticleInterface to be always present.

        /** @var ContentListItemInterface $item */
        foreach ($contentListItems as $item) {
            if (!$article->isPublished()) {
                $this->manager->remove($item);
            }
            $item->getContentList()->setUpdatedAt(new \DateTime('now'));
        }
        $this->manager->flush();
    }
}