Merge pull request #1234 from superdesk/swp-2213

Rule processor should process article on update, same as article publish
superdesk · Jun 12, 2023 · 7ac89b0 · 7ac89b0
2 parents e526a0b + d225245
commit 7ac89b0
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/src/SWP/Bundle/ContentBundle/Controller/ContentPushController.php b/src/SWP/Bundle/ContentBundle/Controller/ContentPushController.php
@@ -49,16 +49,16 @@ class ContentPushController extends AbstractController {
   private PackageRepository $packageRepository;//swp.repository.package
   private FileProviderInterface $fileProvider;
 
-  /**
-   * @param EventDispatcherInterface $eventDispatcher
-   * @param FormFactoryInterface $formFactory
-   * @param MessageBusInterface $messageBus
-   * @param DataTransformerInterface $dataTransformer
-   * @param MediaManagerInterface $mediaManager
-   * @param EntityManagerInterface $entityManager
-   * @param PackageRepositoryInterface $packageRepository
-   * @param FileProviderInterface $fileProvider
-   */
+    /**
+     * @param EventDispatcherInterface $eventDispatcher
+     * @param FormFactoryInterface $formFactory
+     * @param MessageBusInterface $messageBus
+     * @param DataTransformerInterface $dataTransformer
+     * @param MediaManagerInterface $mediaManager
+     * @param EntityManagerInterface $entityManager
+     * @param PackageRepository $packageRepository
+     * @param FileProviderInterface $fileProvider
+     */
   public function __construct(EventDispatcherInterface $eventDispatcher, FormFactoryInterface $formFactory,
                               MessageBusInterface      $messageBus,
                               DataTransformerInterface $dataTransformer, MediaManagerInterface $mediaManager,

diff --git a/src/SWP/Bundle/CoreBundle/EventSubscriber/ProcessArticleRulesSubscriber.php b/src/SWP/Bundle/CoreBundle/EventSubscriber/ProcessArticleRulesSubscriber.php
@@ -70,7 +70,7 @@ public function processRules(ArticleEvent $event)
         $article = $event->getArticle();
         $count = $this->publishDestinationProvider->countDestinations($package);
 
-        if (0 < $count || $article->isPublished()) {
+        if (0 < $count) {
             return;
         }