Changeset 50156 for trunk/tests/phpunit/tests/url.php

Timestamp:

02/02/2021 07:01:18 PM (4 years ago)

Author:

SergeyBiryukov

Message:

General: Remove admin and login exceptions for https in get_home_url().

Previously, get_home_url() would automatically switch to https if the current request is already https, but would only do so on the front end.

This addresses the inconsistent behavior of returning different values in the admin and on the frontend.

Follow-up to [12598], [21937], [24844].

Props herregroen, mukesh27.
Fixes #52421.

File:

• trunk/tests/phpunit/tests/url.php (modified) (2 diffs)

trunk/tests/phpunit/tests/url.php

@@ -204 +204 @@
         // Pretend to be in the site admin.
         set_current_screen( 'dashboard' );
-        $home = get_option( 'home' );
-
-        // home_url() should return http when in the admin.
-        $_SERVER['HTTPS'] = 'on';
-        $this->assertSame( $home, home_url() );
+        $home       = get_option( 'home' );
+        $home_https = str_replace( 'http://', 'https://', $home );
+
+        // is_ssl() should determine the scheme in the admin.
+        $_SERVER['HTTPS'] = 'on';
+        $this->assertSame( $home_https, home_url() );
 
         $_SERVER['HTTPS'] = 'off';
         $this->assertSame( $home, home_url() );
 
-        // If not in the admin, is_ssl() should determine the scheme.
+        // is_ssl() should determine the scheme on front end too.
         set_current_screen( 'front' );
         $this->assertSame( $home, home_url() );
-        $_SERVER['HTTPS'] = 'on';
-        $home             = str_replace( 'http://', 'https://', $home );
-        $this->assertSame( $home, home_url() );
+
+        $_SERVER['HTTPS'] = 'on';
+        $this->assertSame( $home_https, home_url() );
 
         // Test with https in home.
@@ -252 +253 @@
         // Pretend to be in the site admin.
         set_current_screen( 'dashboard' );
-        $home = network_home_url();
-
-        // home_url() should return http when in the admin.
+        $home       = network_home_url();
+        $home_https = str_replace( 'http://', 'https://', $home );
+
+        // is_ssl() should determine the scheme in the admin.
         $this->assertSame( 0, strpos( $home, 'http://' ) );
         $_SERVER['HTTPS'] = 'on';
-        $this->assertSame( $home, network_home_url() );
+        $this->assertSame( $home_https, network_home_url() );
 
         $_SERVER['HTTPS'] = 'off';
         $this->assertSame( $home, network_home_url() );
 
-        // If not in the admin, is_ssl() should determine the scheme.
+        // is_ssl() should determine the scheme on front end too.
         set_current_screen( 'front' );
         $this->assertSame( $home, network_home_url() );
         $_SERVER['HTTPS'] = 'on';
-        $home             = str_replace( 'http://', 'https://', $home );
-        $this->assertSame( $home, network_home_url() );
+        $this->assertSame( $home_https, network_home_url() );
 
         $GLOBALS['current_screen'] = $screen;