Changeset 4655

Timestamp:

12/25/2012 08:00:36 AM (12 years ago)

Author:

johnjamesjacoby

Message:

Introduce bbp_get_redirect_to() to handle getting the request variable. Use this new function where appropriate.

Location:

trunk/includes

Files:

• common/functions.php (modified) (1 diff)
• common/template-tags.php (modified) (1 diff)
• forums/functions.php (modified) (2 diffs)
• replies/functions.php (modified) (3 diffs)
• topics/functions.php (modified) (2 diffs)

trunk/includes/common/functions.php

@@ -228 +228 @@
 
 /** Misc **********************************************************************/
+
+/**
+ * Return the unescaped redirect_to request value
+ *
+ * @bbPress (r4655)
+ *
+ * @return string The URL to redirect to, if set
+ */
+function bbp_get_redirect_to() {
+    $retval = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+
+    return apply_filters( 'bbp_get_redirect_to', $retval );
+}
 
 /**

trunk/includes/common/template-tags.php

@@ -1114 +1114 @@
  * Output hidden request URI field for user forms.
  *
- * The referer link is the current Request URI from the server super global. The
- * input name is 'redirect_to', in case you wanted to check manually.
+ * The referer link is the current Request URI from the server super global. To
+ * check the field manually, use bbp_get_redirect_to().
  *
  * @since bbPress (r2815)

trunk/includes/forums/functions.php

@@ -297 +297 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to  = bbp_get_redirect_to();
 
         // Get the forum URL
@@ -549 +549 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // View all?

trunk/includes/replies/functions.php

@@ -319 +319 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // Get the reply URL
@@ -604 +604 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // Get the reply URL
@@ -1417 +1417 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // Get the reply URL

trunk/includes/topics/functions.php

@@ -344 +344 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // Get the topic URL
@@ -682 +682 @@
 
         // Redirect to
-        $redirect_to = !empty( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '';
+        $redirect_to = bbp_get_redirect_to();
 
         // View all?