Changeset 5658 for trunk/src/includes/users/functions.php

Timestamp:

03/24/2015 01:32:20 PM (11 years ago)

Author:

johnjamesjacoby

Message:

API: Introduce bbp_redirect()

• Wraps wp_safe_redirect() & exit() calls
• Prevents errors from potentially empty wp_get_referrer() results
• Replace wp_safe_redirect() usages with bbp_redirect()

Fixes #2778.

File:

• trunk/src/includes/users/functions.php (modified) (13 diffs)

trunk/src/includes/users/functions.php

@@ -23 +23 @@
  * @uses home_url() To get the home url
  * @uses esc_url() To escape the url
- * @uses wp_safe_redirect() To redirect
+ * @uses bbp_redirect() To redirect
  */
 function bbp_redirect_login( $url = '', $raw_url = '', $user = '' ) {
@@ -395 +395 @@
  * @uses bbp_get_favorites_link() To get the favorites page link
  * @uses bbp_get_topic_permalink() To get the topic permalink
- * @uses wp_safe_redirect() To redirect to the url
+ * @uses bbp_redirect() To redirect to the url
  */
 function bbp_favorites_handler( $action = '' ) {
@@ -471 +471 @@
         }
 
-        wp_safe_redirect( $redirect );
-
-        // For good measure
-        exit();
+        bbp_redirect( $redirect );
 
     // Fail! Handle errors
@@ -1116 +1113 @@
  * @uses bbp_is_subscription() To check if it's the subscription page
  * @uses bbp_get_forum_permalink() To get the forum permalink
- * @uses wp_safe_redirect() To redirect to the url
+ * @uses bbp_redirect() To redirect to the url
  */
 function bbp_forum_subscriptions_handler( $action = '' ) {
@@ -1192 +1189 @@
         }
 
-        wp_safe_redirect( $redirect );
-
-        // For good measure
-        exit();
+        bbp_redirect( $redirect );
 
     // Fail! Handle errors
@@ -1222 +1216 @@
  * @uses bbp_is_subscription() To check if it's the subscription page
  * @uses bbp_get_topic_permalink() To get the topic permalink
- * @uses wp_safe_redirect() To redirect to the url
+ * @uses bbp_redirect() To redirect to the url
  */
 function bbp_subscriptions_handler( $action = '' ) {
@@ -1298 +1292 @@
         }
 
-        wp_safe_redirect( $redirect );
-
-        // For good measure
-        exit();
+        bbp_redirect( $redirect );
 
     // Fail! Handle errors
@@ -1327 +1318 @@
  * @uses delete_option() To delete the displayed user's email id option
  * @uses bbp_get_user_profile_edit_url() To get the edit profile url
- * @uses wp_safe_redirect() To redirect to the url
+ * @uses bbp_redirect() To redirect to the url
  * @uses bbp_verify_nonce_request() To verify the nonce and check the request
  * @uses current_user_can() To check if the current user can edit the user
@@ -1371 +1362 @@
             delete_option( $user_id . '_new_email' );
 
-            wp_safe_redirect( add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $user_id ) ) );
-            exit();
+            bbp_redirect( add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $user_id ) ) );
         }
 
@@ -1378 +1368 @@
     } elseif ( is_multisite() && bbp_is_user_home_edit() && ! empty( $_GET['dismiss'] ) && ( $user_id . '_new_email' === $_GET['dismiss'] ) ) {
         delete_option( $user_id . '_new_email' );
-        wp_safe_redirect( add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $user_id ) ) );
-        exit();
+        bbp_redirect( add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $user_id ) ) );
     }
 
@@ -1420 +1409 @@
         $redirect = add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $edit_user ) );
 
-        wp_safe_redirect( $redirect );
-        exit;
+        bbp_redirect( $redirect );
     }
 }
@@ -1704 +1692 @@
  * @uses current_user_can()
  * @uses bbp_get_displayed_user_id()
- * @uses wp_safe_redirect()
+ * @uses bbp_redirect()
  * @uses bbp_get_user_profile_url()
  */
@@ -1744 +1732 @@
 
     // Redirect
-    wp_safe_redirect( $redirect_to );
-    exit();
+    bbp_redirect( $redirect_to );
 }