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

Timestamp:

03/27/2014 06:20:37 PM (11 years ago)

Author:

boonebgorges

Message:

Don't allow a user to favorite a single activity item more than once.

Fixes #5478

File:

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

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

@@ -407 +407 @@
         $user_id = bp_loggedin_user_id();
 
-    // Update the user's personal favorites
-    $my_favs   = bp_get_user_meta( $user_id, 'bp_favorite_activities', true );
+    $my_favs = bp_get_user_meta( $user_id, 'bp_favorite_activities', true );
+    if ( empty( $my_favs ) || ! is_array( $my_favs ) ) {
+        $my_favs = array();
+    }
+
+    // Bail if the user has already favorited this activity item
+    if ( in_array( $activity_id, $my_favs ) ) {
+        return false;
+    }
+
+    // Add to user's favorites
     $my_favs[] = $activity_id;