Ticket #6504: 6504.diff

src/bp-templates/bp-legacy/buddypress-functions.php

@@ -134 +134 @@
                 }
 
                 /** Ajax **************************************************************/
+                //these ajax actions don't require users to be logged in
+                $non_privileged_actions = array(
 
-                $actions = array(
-
                         // Directory filters
                         'blogs_filter'    => 'bp_legacy_theme_object_template_loader',
                         'forums_filter'   => 'bp_legacy_theme_object_template_loader',
                         'groups_filter'   => 'bp_legacy_theme_object_template_loader',
                         'members_filter'  => 'bp_legacy_theme_object_template_loader',
+                        
+                        // Activity
+                        'activity_get_older_updates'  => 'bp_legacy_theme_activity_template_loader',
+                        'activity_widget_filter'      => 'bp_legacy_theme_activity_template_loader',
+                        'get_single_activity_content' => 'bp_legacy_theme_get_single_activity_content',
+                        
+                );
+                
+
+
+                /**
+                 * Register all of these AJAX handlers
+                 *
+                 * The "wp_ajax_" action is used for logged in users, and "wp_ajax_nopriv_"
+                 * executes for users that aren't logged in. This is for backpat with BP <1.6.
+                 */
+                foreach( $non_privileged_actions as $name => $function ) {
+                        //we need these actions for both the logged in as well as logged out users
+                        add_action( 'wp_ajax_'        . $name, $function );
+                        add_action( 'wp_ajax_nopriv_' . $name, $function );
+                }
+                
+                //only for logged in users
+                $actions = array(
+
                         'messages_filter' => 'bp_legacy_theme_messages_template_loader',
                         'invite_filter'   => 'bp_legacy_theme_invite_template_loader',
                         'requests_filter' => 'bp_legacy_theme_requests_template_loader',
@@ -152 +177 @@
                         'reject_friendship' => 'bp_legacy_theme_ajax_reject_friendship',
 
                         // Activity
-                        'activity_get_older_updates'  => 'bp_legacy_theme_activity_template_loader',
                         'activity_mark_fav'           => 'bp_legacy_theme_mark_activity_favorite',
                         'activity_mark_unfav'         => 'bp_legacy_theme_unmark_activity_favorite',
-                        'activity_widget_filter'      => 'bp_legacy_theme_activity_template_loader',
                         'delete_activity'             => 'bp_legacy_theme_delete_activity',
                         'delete_activity_comment'     => 'bp_legacy_theme_delete_activity_comment',
-                        'get_single_activity_content' => 'bp_legacy_theme_get_single_activity_content',
                         'new_activity_comment'        => 'bp_legacy_theme_new_activity_comment',
                         'post_update'                 => 'bp_legacy_theme_post_update',
                         'bp_spam_activity'            => 'bp_legacy_theme_spam_activity',
@@ -181 +203 @@
                 if ( bp_is_active( 'messages', 'star' ) ) {
                         $actions['messages_star'] = 'bp_legacy_theme_ajax_messages_star_handler';
                 }
-
-                /**
-                 * Register all of these AJAX handlers
-                 *
-                 * The "wp_ajax_" action is used for logged in users, and "wp_ajax_nopriv_"
-                 * executes for users that aren't logged in. This is for backpat with BP <1.6.
-                 */
+                
                 foreach( $actions as $name => $function ) {
                         add_action( 'wp_ajax_'        . $name, $function );
-                        add_action( 'wp_ajax_nopriv_' . $name, $function );
                 }
 
                 add_filter( 'bp_ajax_querystring', 'bp_legacy_theme_ajax_querystring', 10, 2 );