Ticket #44499: 44499.1.diff

src/wp-includes/functions.php

@@ -1169 +1169 @@
         $ret = preg_replace( '#=(&|$)#', '$1', $ret );
         $ret = $protocol . $base . $ret . $frag;
         $ret = rtrim( $ret, '?' );
+        $ret = str_replace( '?#', '#', $ret );
         return $ret;
 }
 

tests/phpunit/tests/functions.php

@@ -665 +665 @@
                 $this->assertSame( 'foo=bar&1=2', $url );
         }
 
+        /**
+         * Tests that add_query_arg removes the question mark when
+         * a parameter is set to false.
+         *
+         * @ticket 44499
+         *
+         * @group add_query_arg
+         *
+         * @covers ::add_query_arg
+         */
+        public function test_add_query_arg_removes_question_mark() {
+                $url      = 'http://example.org?param=value#anchor';
+                $expected = 'http://example.org#anchor';
+                $actual   = add_query_arg( 'param', false, $url );
+                $this->assertSame( $expected, $actual );
+        }
+
         /**
          * @ticket 21594
          */