Changeset 9376

Timestamp:

01/19/2015 08:33:06 PM (12 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;
         }