Changeset 10515 for trunk/src/bp-activity/bp-activity-functions.php

Timestamp:

02/05/2016 03:37:27 AM (9 years ago)

Author:

boonebgorges

Message:

Sort activity actions when they're added, instead of when they're fetched.

Sorting activity actions can be an expensive process, so should not be done
any more frequently than necessary. And it's only necessary to sort when a
new action has been added. So instead of sorting in bp_activity_get_actions(),
we now do it in bp_activity_set_action().

Props boonebgorges, imath.
Fixes #6865.

File:

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

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

@@ -385 +385 @@
         'position'        => $position,
     ), $component_id, $type, $description, $format_callback, $label, $context );
+
+    // Sort the actions of the affected component.
+    $action_array = (array) $bp->activity->actions->{$component_id};
+    $action_array = bp_sort_by_key( $action_array, 'position', 'num' );
+
+    // Restore keys.
+    $bp->activity->actions->{$component_id} = new stdClass;
+    foreach ( $action_array as $key_ordered ) {
+        $bp->activity->actions->{$component_id}->{$key_ordered['key']} = $key_ordered;
+    }
 
     return true;
@@ -571 +581 @@
                 $post_type->position
             );
-        }
-    }
-
-    // Sort the actions by their position within each component.
-    foreach ( $bp->activity->actions as $component => $actions ) {
-        $actions = (array) $actions;
-        $temp = bp_sort_by_key( $actions, 'position', 'num' );
-
-        // Restore keys.
-        $bp->activity->actions->{$component} = new stdClass;
-        foreach ( $temp as $key_ordered ) {
-            $bp->activity->actions->{$component}->{$key_ordered['key']} = $key_ordered;
         }
     }