Bug fix #133

Author: nyamsprod

components/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 All Notable changes to `League\Uri\Components` will be documented in this file
 
+## [Next](https://github.com/thephpleague/uri-components/compare/7.4.0...master) - TBD
+
+### Added
+
+- None
+
+### Fixed
+
+- Fix package to avoid PHP8.4 deprecation warnings
+- Bug Fix `removeEmptyPairs` [#133](https://github.com/thephpleague/uri-src/issues/133)
+
+### Deprecated
+
+- None
+
+### Removed
+
+- None
+
 ## [7.4.0](https://github.com/thephpleague/uri-components/compare/7.3.0...7.4.0) - 2023-12-01
 
 ### Added

components/UriModifier.php

@@ -65,7 +65,7 @@ public static function removePairs(Psr7UriInterface|UriInterface $uri, string ..
      */
     public static function removeEmptyPairs(Psr7UriInterface|UriInterface $uri): Psr7UriInterface|UriInterface
     {
-        return Modifier::from($uri)->removeQueryPairsByKey()->getUri();
+        return Modifier::from($uri)->removeEmptyQueryPairs()->getUri();
     }
 
     /**

components/UriModifierTest.php

@@ -0,0 +1,29 @@
+<?php
+
+/**
+ * League.Uri (https://uri.thephpleague.com)
+ *
+ * (c) Ignace Nyamagana Butera <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace League\Uri;
+
+use PHPUnit\Framework\TestCase;
+
+final class UriModifierTest extends TestCase
+{
+    /** @test */
+    public function it_will_remove_empty_pairs_fix_issue_133(): void
+    {
+        $removeEmptyPairs = fn (string $str): ?string => UriModifier::removeEmptyPairs(Http::createFromString($str))->getQuery(); /* @phpstan-ignore-line */
+
+        self::assertSame('', $removeEmptyPairs('https://a.b/c?d='));
+        self::assertSame('', $removeEmptyPairs('https://a.b/c?=d'));
+        self::assertSame('', $removeEmptyPairs('https://a.b/c?='));
+    }
+}