Changeset 6026 for trunk/src/includes/replies/functions.php

Timestamp:

05/19/2016 07:35:41 PM (9 years ago)

Author:

johnjamesjacoby

Message:

Statuses, Types, Visibilities: Pass the forum, topic, or reply ID into helper functions.

These ID's pass additional context into respective filters, useful for fine-grained control over UI elements.

This commit normalizes the way these functions work across all of bbPress where applicable, by ensuring all similar functions accept an ID and assume 0 by default. Note that there are currently 2 locations where no ID exists to pass context into, which is by design.

Fixes #2954.

File:

• trunk/src/includes/replies/functions.php (modified) (2 diffs)

trunk/src/includes/replies/functions.php

@@ -1696 +1696 @@
  * @since 2.6.0 bbPress (r5399)
  *
+ * @param int $reply_id   Optional. Reply id.
+ *
  * @return array
  */
-function bbp_get_reply_statuses() {
+function bbp_get_reply_statuses( $reply_id = 0 ) {
     return apply_filters( 'bbp_get_reply_statuses', array(
         bbp_get_public_status_id()  => _x( 'Publish', 'Publish the reply',     'bbpress' ),
@@ -1704 +1706 @@
         bbp_get_trash_status_id()   => _x( 'Trash',   'Trash the reply',       'bbpress' ),
         bbp_get_pending_status_id() => _x( 'Pending', 'Mark reply as pending', 'bbpress' ),
-    ) );
+    ), $reply_id );
 }