added array_unique to avoid duplicated entries (#26)

sulu · Feb 5, 2018 · fdbf942 · fdbf942
1 parent 77e66d9
commit fdbf942
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/GoneSubscriber/GoneDocumentSubscriber.php b/GoneSubscriber/GoneDocumentSubscriber.php
@@ -118,8 +118,6 @@ public function createRedirects(RemoveEvent $event)
     }
 
     /**
-     * @param BasePageDocument $docum ent
-     *
      * @return array
      */
     protected function getUrls(BasePageDocument $document)
@@ -165,7 +163,7 @@ protected function getUrls(BasePageDocument $document)
             $url = parse_url($url, PHP_URL_PATH);
         }
 
-        return $urls;
+        return array_unique($urls);
     }
 
     /**

diff --git a/Tests/Unit/GoneSubscriber/GoneDocumentSubscriberTest.php b/Tests/Unit/GoneSubscriber/GoneDocumentSubscriberTest.php
@@ -101,7 +101,7 @@ protected function setUp()
         $this->webspaceManager = $this->prophesize(WebspaceManager::class);
         $this->webspaceManager->findWebspaceByKey('example')->willReturn($this->webspace->reveal());
         $this->webspaceManager->findUrlsByResourceLocator('/article', 'test', 'en')
-            ->willReturn(['http://{host}/en/article']);
+            ->willReturn(['http://{host}/en/article', 'http://sulu.io/en/article']);
         $this->webspaceManager->findUrlsByResourceLocator('/article1', 'test', 'en')
             ->willReturn(['http://{host}/en/article1']);
         $this->webspaceManager->findUrlsByResourceLocator('/article2', 'test', 'en')