Changeset 13898

Timestamp:

06/02/2024 05:55:08 PM (7 months ago)

Author:

espellcaste

Message:

Use wp_parse_url instead of parse_url to parse URLs.

Props imath
Closes ​https://github.com/buddypress/buddypress/pull/307
See #7228

Location:

trunk

Files:

• src/bp-core/bp-core-functions.php (modified) (1 diff)
• src/bp-core/classes/class-bp-media-extractor.php (modified) (2 diffs)
• src/bp-templates/bp-nouveau/buddypress-functions.php (modified) (1 diff)
• src/bp-templates/bp-nouveau/includes/messages/functions.php (modified) (1 diff)
• src/bp-xprofile/classes/class-bp-xprofile-field-type-telephone.php (modified) (1 diff)
• tests/phpunit/testcases/core/functions/bpVerifyNonceRequest.php (modified) (1 diff)

trunk/src/bp-core/bp-core-functions.php

@@ -2398 +2398 @@
     // Parse home_url() into pieces to remove query-strings, strange characters,
     // and other funny things that plugins might to do to it.
-    $parsed_home = parse_url( home_url( '/', ( is_ssl() ? 'https' : 'http' ) ) );
+    $parsed_home = wp_parse_url( home_url( '/', ( is_ssl() ? 'https' : 'http' ) ) );
 
     // Maybe include the port, if it's included in home_url().

trunk/src/bp-core/classes/class-bp-media-extractor.php

@@ -631 +631 @@
             }
 
-            $path = untrailingslashit( parse_url( $audio['attributes'][ $src_param ], PHP_URL_PATH ) );
+            $path = untrailingslashit( wp_parse_url( $audio['attributes'][ $src_param ], PHP_URL_PATH ) );
 
             foreach ( $audio_types as $extension ) {
@@ -728 +728 @@
             }
 
-            $path = untrailingslashit( parse_url( $video['attributes'][ $src_param ], PHP_URL_PATH ) );
+            $path = untrailingslashit( wp_parse_url( $video['attributes'][ $src_param ], PHP_URL_PATH ) );
 
             foreach ( $video_types as $extension ) {

trunk/src/bp-templates/bp-nouveau/buddypress-functions.php

@@ -706 +706 @@
         }
 
-        $uri = parse_url( $path );
+        $uri = wp_parse_url( $path );
 
         if ( false === strpos( $uri['path'], 'customize.php' ) ) {

trunk/src/bp-templates/bp-nouveau/includes/messages/functions.php

@@ -158 +158 @@
             'more' => __( '(and %d others)', 'buddypress' ),
         ),
-        'rootUrl'           => parse_url( $root_url, PHP_URL_PATH ),
+        'rootUrl'           => wp_parse_url( $root_url, PHP_URL_PATH ),
         'supportedRoutes'   => $routes,
     );

trunk/src/bp-xprofile/classes/class-bp-xprofile-field-type-telephone.php

@@ -140 +140 @@
     public static function display_filter( $field_value, $field_id = '' ) {
         $url   = wp_strip_all_tags( $field_value );
-        $parts = parse_url( $url );
+        $parts = wp_parse_url( $url );
 
         // Add the tel:// protocol to the field value.

trunk/tests/phpunit/testcases/core/functions/bpVerifyNonceRequest.php

@@ -75 +75 @@
      */
     public function add_port_and_subdirectory_to_home_url( $url, $path, $scheme ) {
-        $home = parse_url( get_option( 'home' ) );
+        $home = wp_parse_url( get_option( 'home' ) );
         $home_path = isset( $home['path'] ) ? $home['path'] : '';
         return $scheme . '://' . $home['host'] . ':80' . $home_path . '/wordpress' . $path;