Changeset 4367 for trunk/bp-activity/bp-activity-functions.php

Timestamp:

05/15/2011 10:40:03 PM (15 years ago)

Author:

boonebgorges

Message:

Reconfigures the way that new activity mentions are stored and incremented, so that counts more accurately reflect new and deleted activity items. Fixes #2082

File:

• trunk/bp-activity/bp-activity-functions.php (modified) (4 diffs)

trunk/bp-activity/bp-activity-functions.php

@@ -16 +16 @@
  * @since 1.3
  *
- * @param $content The content of the activity, usually found in $activity->content
+ * @param str $content The content of the activity, usually found in $activity->content
  * @return array $usernames Array of the found usernames that match existing users
  */
@@ -31 +31 @@
 
 /**
- * Reduces new mention count for mentioned users when activity items are deleted
+ * Resets a user's unread mentions list and count
  *
  * @package BuddyPress Activity
  * @since 1.3
  *
- * @param $activity_id The unique id for the activity item
- */
-function bp_activity_reduce_mention_count( $activity_id ) {
+ * @param int $user_id The id of the user whose unread mentions are being reset
+ */
+function bp_activity_clear_new_mentions( $user_id ) {
+        delete_user_meta( $user_id, 'bp_new_mention_count' );
+        delete_user_meta( $user_id, 'bp_new_mentions' );
+}
+
+/**
+ * Adjusts new mention count for mentioned users when activity items are deleted or created
+ *
+ * @package BuddyPress Activity
+ * @since 1.3
+ *
+ * @param int $activity_id The unique id for the activity item
+ */
+function bp_activity_adjust_mention_count( $activity_id, $action = 'add' ) {
         $activity = new BP_Activity_Activity( $activity_id );
 
@@ -51 +64 @@
                                 continue;
 
-                        // Decrease the number of new @ mentions for the user
+                        // Adjust the mention list and count for the member
                         $new_mention_count = (int)get_user_meta( $user_id, 'bp_new_mention_count', true );
-                        update_user_meta( $user_id, 'bp_new_mention_count', $new_mention_count - 1 );
+                        if ( !$new_mentions = get_user_meta( $user_id, 'bp_new_mentions', true ) )
+                                $new_mentions = array();
+                                
+                        switch ( $action ) {
+                                case 'delete' :
+                                        $key = array_search( $activity_id, $new_mentions );
+                                        if ( $key !== false ) {
+                                                var_dump( $new_mentions );
+                                                var_dump( $activity_id );
+                                                unset( $new_mentions[$key] );
+                                        }
+                                        break;
+                                
+                                case 'add' :
+                                default :
+                                        if ( !in_array( $activity_id, $new_mentions ) ) {
+                                                $new_mentions[] = (int)$activity_id;
+                                        }
+                                        break;
+                        }
+                        
+                        // Get an updated mention count                 
+                        $new_mention_count = count( $new_mentions );
+                        
+                        // Resave the user_meta
+                        update_user_meta( $user_id, 'bp_new_mention_count', $new_mention_count );
+                        update_user_meta( $user_id, 'bp_new_mentions', $new_mentions );
                 }
         }
 }
-add_action( 'bp_activity_action_delete_activity', 'bp_activity_reduce_mention_count' );
 
 /**
@@ -758 +796 @@
         $args = wp_parse_args( $args, $defaults );
 
+        // Adjust the new mention count of any mentioned member
+        bp_activity_adjust_mention_count( $args['id'], 'delete' );
+
         if ( !$activity_ids_deleted = BP_Activity_Activity::delete( $args ) )
                 return false;