Changeset 9376

Timestamp:

01/19/2015 08:33:06 PM (10 years ago)

Author:

johnjamesjacoby

Message:

In BP_Activity_Activity::total_favorite_count() invert the return order so 0 is the default at the end of the method. Also remove inline conditional assignment.

File:

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

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

@@ -1607 +1607 @@
      */
     public static function total_favorite_count( $user_id ) {
-        if ( !$favorite_activity_entries = bp_get_user_meta( $user_id, 'bp_favorite_activities', true ) )
-            return 0;
-
-        return count( maybe_unserialize( $favorite_activity_entries ) );
+
+        // Get activities from user meta
+        $favorite_activity_entries = bp_get_user_meta( $user_id, 'bp_favorite_activities', true );
+        if ( ! empty( $favorite_activity_entries ) ) {
+            return count( maybe_unserialize( $favorite_activity_entries ) );
+        }
+
+        // No favorites
+        return 0;
     }