Changeset 4551 for trunk/includes/forums/functions.php

Timestamp:

12/06/2012 10:29:35 AM (12 years ago)

Author:

johnjamesjacoby

Message:

Introduce bbp_post_request() and bbp_get_request() sub-actions:

• Update actions and sub-actions to use these new handlers.
• Update new/edit forum/topic/reply handlers to use these.
• Update phpdoc as needed.
• Remove duplicate 'bp_actions' request hooks from BuddyPress group forums.
• Does not include admin-side requests.

File:

• trunk/includes/forums/functions.php (modified) (4 diffs)

trunk/includes/forums/functions.php

@@ -78 +78 @@
  * Handles the front end forum submission
  *
+ * @param string $action The requested action to compare this function to
  * @uses bbPress:errors::add() To log various error messages
  * @uses bbp_verify_nonce_request() To verify the nonce and check the request
@@ -107 +108 @@
  *                                              messages
  */
-function bbp_new_forum_handler() {
-
-    // Bail if not a POST action
-    if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
-        return;
+function bbp_new_forum_handler( $action = '' ) {
 
     // Bail if action is not bbp-new-forum
-    if ( empty( $_POST['action'] ) || ( 'bbp-new-forum' !== $_POST['action'] ) )
+    if ( 'bbp-new-forum' !== $action )
         return;
 
@@ -339 +336 @@
  * Handles the front end edit forum submission
  *
+ * @param string $action The requested action to compare this function to
  * @uses bbPress:errors::add() To log various error messages
  * @uses bbp_get_forum() To get the forum
@@ -368 +366 @@
  *                                              messages
  */
-function bbp_edit_forum_handler() {
-
-    // Bail if not a POST action
-    if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
-        return;
+function bbp_edit_forum_handler( $action = '' ) {
 
     // Bail if action is not bbp-edit-forum
-    if ( empty( $_POST['action'] ) || ( 'bbp-edit-forum' !== $_POST['action'] ) )
+    if ( 'bbp-edit-forum' !== $action )
         return;