Changeset 11356 for trunk/src/bp-activity/bp-activity-template.php

Timestamp:

12/29/2016 07:38:58 PM (8 years ago)

Author:

boonebgorges

Message:

Discard activity filter values that are invalid for the current context.

Activity filters are stored in a cookie that persists across
activity pages. But some filter values are not valid in all contexts,
such as new_member (valid in the sitewide feed, but not in a group).
This changeset ensures that filter values sent via AJAX are discarded
if they aren't valid in the current context.

Props lakrisgubben.
Fixes #4062.

File:

• trunk/src/bp-activity/bp-activity-template.php (modified) (1 diff)

trunk/src/bp-activity/bp-activity-template.php

@@ -3801 +3801 @@
      */
     function bp_get_activity_show_filters( $context = '' ) {
-        // Set default context based on current page.
-        if ( empty( $context ) ) {
-
-            // On member pages, default to 'member', unless this
-            // is a user's Groups activity.
-            if ( bp_is_user() ) {
-                if ( bp_is_active( 'groups' ) && bp_is_current_action( bp_get_groups_slug() ) ) {
-                    $context = 'member_groups';
-                } else {
-                    $context = 'member';
-                }
-
-            // On individual group pages, default to 'group'.
-            } elseif ( bp_is_active( 'groups' ) && bp_is_group() ) {
-                $context = 'group';
-
-            // 'activity' everywhere else.
-            } else {
-                $context = 'activity';
+        $filters = array();
+        $actions = bp_activity_get_actions_for_context( $context );
+        foreach ( $actions as $action ) {
+            // Friends activity collapses two filters into one.
+            if ( in_array( $action['key'], array( 'friendship_accepted', 'friendship_created' ) ) ) {
+                $action['key'] = 'friendship_accepted,friendship_created';
             }
-        }
-
-        $filters = array();
-
-        // Walk through the registered actions, and prepare an the
-        // select box options.
-        foreach ( bp_activity_get_actions() as $actions ) {
-            foreach ( $actions as $action ) {
-                if ( ! in_array( $context, (array) $action['context'] ) ) {
-                    continue;
-                }
-
-                // Friends activity collapses two filters into one.
-                if ( in_array( $action['key'], array( 'friendship_accepted', 'friendship_created' ) ) ) {
-                    $action['key'] = 'friendship_accepted,friendship_created';
-                }
-
-                $filters[ $action['key'] ] = $action['label'];
-            }
+
+            $filters[ $action['key'] ] = $action['label'];
         }