X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=src%2FWallabag%2FCoreBundle%2FHelper%2FRuleBasedTagger.php;h=b490e2090b4a95d5f263b1d1c5f9d17be8cdc9d3;hb=bccf7a70bc22171904a5bb82d282e9c956684abc;hp=3f9953c0995b8422de0cd78b0e5b39f82a28af8e;hpb=625acf335298186b4ff983f9321900d1238e854b;p=github%2Fwallabag%2Fwallabag.git

diff --git a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
index 3f9953c0..b490e209 100644
--- a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
+++ b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
@@ -3,7 +3,6 @@
 namespace Wallabag\CoreBundle\Helper;
 
 use RulerZ\RulerZ;
-
 use Wallabag\CoreBundle\Entity\Entry;
 use Wallabag\CoreBundle\Entity\Tag;
 use Wallabag\CoreBundle\Repository\EntryRepository;
@@ -18,15 +17,15 @@ class RuleBasedTagger
 
     public function __construct(RulerZ $rulerz, TagRepository $tagRepository, EntryRepository $entryRepository)
     {
-        $this->rulerz          = $rulerz;
-        $this->tagRepository   = $tagRepository;
+        $this->rulerz = $rulerz;
+        $this->tagRepository = $tagRepository;
         $this->entryRepository = $entryRepository;
     }
 
     /**
      * Add tags from rules defined by the user.
      *
-     * @param Entry $entry Entry to tag.
+     * @param Entry $entry Entry to tag
      */
     public function tag(Entry $entry)
     {
@@ -38,7 +37,7 @@ class RuleBasedTagger
             }
 
             foreach ($rule->getTags() as $label) {
-                $tag = $this->getTag($entry->getUser(), $label);
+                $tag = $this->getTag($label);
 
                 $entry->addTag($tag);
             }
@@ -50,23 +49,28 @@ class RuleBasedTagger
      *
      * @param User $user
      *
-     * @return array<Entry> A list of modified entries.
+     * @return array<Entry> A list of modified entries
      */
     public function tagAllForUser(User $user)
     {
-        $rules   = $this->getRulesForUser($user);
-        $entries = array();
+        $rules = $this->getRulesForUser($user);
+        $entries = [];
+        $tagsCache = [];
 
         foreach ($rules as $rule) {
-            $qb      = $this->entryRepository->getBuilderForAllByUser($user->getId());
+            $qb = $this->entryRepository->getBuilderForAllByUser($user->getId());
             $entries = $this->rulerz->filter($qb, $rule->getRule());
 
             foreach ($entries as $entry) {
                 foreach ($rule->getTags() as $label) {
-                    $tag = $this->getTag($user, $label);
+                    // avoid new tag duplicate by manually caching them
+                    if (!isset($tagsCache[$label])) {
+                        $tagsCache[$label] = $this->getTag($label);
+                    }
+
+                    $tag = $tagsCache[$label];
 
                     $entry->addTag($tag);
-                    $entries[] = $entry;
                 }
             }
         }
@@ -75,19 +79,18 @@ class RuleBasedTagger
     }
 
     /**
-     * Fetch a tag for a user.
+     * Fetch a tag.
      *
-     * @param User   $user
-     * @param string $label The tag's label.
+     * @param string $label The tag's label
      *
      * @return Tag
      */
-    private function getTag(User $user, $label)
+    private function getTag($label)
     {
-        $tag = $this->tagRepository->findOneByLabelAndUserId($label, $user->getId());
+        $tag = $this->tagRepository->findOneByLabel($label);
 
         if (!$tag) {
-            $tag = new Tag($user);
+            $tag = new Tag();
             $tag->setLabel($label);
         }