Changeset 13898

Timestamp:

06/02/2024 05:55:08 PM (2 years 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;