updateOriginUrl: add behavior when diff is fragment and query

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
2018-10-22 23:08:58 +02:00
parent e07fadea76
commit fc040c749d
2 changed files with 9 additions and 0 deletions
--- a/src/Wallabag/CoreBundle/Helper/ContentProxy.php
+++ b/src/Wallabag/CoreBundle/Helper/ContentProxy.php
@ -344,6 +344,7 @@ class ContentProxy
                    break;
                case ['fragment']:
                case ['query']:
+                case ['fragment', 'query']:
                    // noop
                    break;
                default:
--- a/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php
+++ b/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php
@ -801,6 +801,14 @@ class ContentProxyTest extends TestCase
                null,
                'example.org',
            ],
+            'fragment and query string in fetched content' => [
+                'https://example.org/hello',
+                null,
+                'https://example.org/hello?foo#world',
+                'https://example.org/hello',
+                null,
+                'example.org',
+            ]
        ];
    }