diff --git a/src/Wallabag/CoreBundle/Helper/ContentProxy.php b/src/Wallabag/CoreBundle/Helper/ContentProxy.php index a93f4a2d6..74130be8e 100644 --- a/src/Wallabag/CoreBundle/Helper/ContentProxy.php +++ b/src/Wallabag/CoreBundle/Helper/ContentProxy.php @@ -374,8 +374,6 @@ class ContentProxy $entry->setUrl($url); break; case ['fragment']: - case ['query']: - case ['fragment', 'query']: // noop break; default: diff --git a/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php b/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php index a60aec5b4..3dd9273c8 100644 --- a/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php +++ b/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php @@ -777,20 +777,12 @@ class ContentProxyTest extends TestCase null, 'example.com', ], - 'no query string in fetched content' => [ - 'https://example.org/hello?world=1', - null, - 'https://example.org/hello', - 'https://example.org/hello?world=1', - null, - 'example.org', - ], 'query string in fetched content' => [ 'https://example.org/hello', null, 'https://example.org/hello?world=1', + 'https://example.org/hello?world=1', 'https://example.org/hello', - null, 'example.org', ], 'fragment in fetched content' => [ @@ -805,8 +797,8 @@ class ContentProxyTest extends TestCase 'https://example.org/hello', null, 'https://example.org/hello?foo#world', + 'https://example.org/hello?foo#world', 'https://example.org/hello', - null, 'example.org', ], 'different path and query string in fetch content' => [