Changeset 52187 for trunk/src/wp-includes/functions.php

Timestamp:

11/16/2021 06:48:47 PM (3 years ago)

Author:

hellofromTonya

Message:

HTTP API: Remove empty ? when only anchor remains in add_query_arg().

If after processing through add_query_arg() a ?# remains, this commit removes the unnecessary and unused ? character as there are no query args in the URL.

Includes tests.

Follow-up to [1823], [5193], [5999], [6005].

Props benjaminanakenam, sabernhardt, costdev, hellofromTonya.
Fixes #44499.

File:

• trunk/src/wp-includes/functions.php (modified) (1 diff)

trunk/src/wp-includes/functions.php

@@ -1170 +1170 @@
     $ret = $protocol . $base . $ret . $frag;
     $ret = rtrim( $ret, '?' );
+    $ret = str_replace( '?#', '#', $ret );
     return $ret;
 }