Handle https:// when manipulating 'home'

[ryan]

This:

preg_replace('|https?://[^/]+|i', '', get_settings('home') . '/' )

Instead of this:

preg_replace('|http://[^/]+|i', '', get_settings('home') . '/' )

[ryan]

​http://wordpress.org/pipermail/cvs_wordpress.org/2004-June/000092.html

[techboyg5]

First Trac ticket?

[underwer]

#55323 was marked as a duplicate.

The untrailingslashit function in formatting.php passes the $value param directly into the PHP rtrim function, which only takes Strings. Anything else trows a "Deprecated" warning (example below).

In my personal case, Yoast SEO was the culprit in the stacktrace, but this untrailingslashit should at least validate and/or cast the $value param to String with strval before calling rtrim.

Incoming updated version:

function untrailingslashit( $value ) {
        return rtrim( strval( $value ), '/\\' ); // <-- Cast $value to String with strval()
}

Previous unvalidated version:

function untrailingslashit( $value ) {
        return rtrim( $value, '/\\' );
}

Fixes these types of errors:
Deprecated: rtrim(): Passing null to parameter #1 ($string) of type string is deprecated in /wp-includes/formatting.php on line 2819

Trac ticket: 63379 Fix for deprecated rtrim passing null to parameter

Fixes errors like the following:
Deprecated: rtrim(): Passing null to parameter 1 ($string) of type string is deprecated in /wp-includes/formatting.php on line 2819

The untrailingslashit function in formatting.php passes the $value param directly into the PHP rtrim function, which only takes Strings. Anything else trows a "Deprecated" warning (example below).

In my personal case, Yoast SEO was the culprit in the stacktrace, but this untrailingslashit should at least validate and/or cast the $value param to String with strval before calling rtrim.

Incoming updated version:

function untrailingslashit( $value ) {
        return rtrim( strval( $value ), '/\\' ); // <-- Cast $value to String with strval()
}

Previous unvalidated version:

function untrailingslashit( $value ) {
        return rtrim( $value, '/\\' );
}

Fixes these types of errors:
Deprecated: rtrim(): Passing null to parameter #1 ($string) of type string is deprecated in /wp-includes/formatting.php on line 2819

Trac ticket: 63379 Fix for deprecated rtrim passing null to parameter

[arsproejcta]

preg_replace('|https?:[^{/]+|i', , get_settings('home') . '/');}

✅ Explanation of the Fix

https?: → matches both ​http:// and ​https://

[^{/]+ → matches the domain name portion (anything up to the next /)}

The rest of the function logic remains the same, ensuring compatibility with both secure and non-secure WordPress installations.