Changeset 58097 for trunk/src/wp-admin/includes/network.php

Timestamp:

05/04/2024 07:23:31 PM (7 months ago)

Author:

johnbillion

Message:

Bootstrap/Load: Add support for custom ports in multisite site addresses.

This allows a Multisite network to use an address that includes a port name, such as example.com:1234, and adds support for this to the local development environment too. You can now run a Multisite installation on the local development environment, for example at localhost:8889.

This also fixes some bugs with running a single site installation on a port, and updates the testing infrastructure so that the whole test suite runs both with and without a port number.

Props djzone, scribu, nacin, ipstenu, F J Kaiser, jeremyfelt, johnjamesjacoby, spacedmonkey, PerS, Clorith, Blackbam, enrico.sorcinelli, Jules Colle, obliviousharmony, desrosj, johnbillion

Fixes #21077, #52088

File:

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

trunk/src/wp-admin/includes/network.php

@@ -149 +149 @@
     }
 
-    $hostname  = get_clean_basedomain();
-    $has_ports = strstr( $hostname, ':' );
-    if ( ( false !== $has_ports && ! in_array( $has_ports, array( ':80', ':443' ), true ) ) ) {
-        wp_admin_notice(
-            '<strong>' . __( 'Error:' ) . '</strong> ' . __( 'You cannot install a network of sites with your server address.' ),
-            array(
-                'additional_classes' => array( 'error' ),
-            )
-        );
-
-        echo '<p>' . sprintf(
-            /* translators: %s: Port number. */
-            __( 'You cannot use port numbers such as %s.' ),
-            '<code>' . $has_ports . '</code>'
-        ) . '</p>';
-        echo '<a href="' . esc_url( admin_url() ) . '">' . __( 'Go to Dashboard' ) . '</a>';
-        echo '</div>';
-        require_once ABSPATH . 'wp-admin/admin-footer.php';
-        die();
-    }
+    // Strip standard port from hostname.
+    $hostname = preg_replace( '/(?::80|:443)$/', '', get_clean_basedomain() );
 
     echo '<form method="post">';