Merge pull request #12649 from jdalsem/finished-upgrade-sort

feat(upgrades): completed upgrades are sorted by completion time

engine/classes/Elgg/UpgradeService.php

@@ -14,6 +14,7 @@
 use React\Promise\Promise;
 use RuntimeException;
 use Throwable;
+use Elgg\Database\Clauses\EntitySortByClause;
 
 /**
  * Upgrade service for Elgg
@@ -489,7 +490,7 @@ public function getPendingUpgrades($async = true) {
 	}
 
 	/**
-	 * Get completed (async) upgrades
+	 * Get completed (async) upgrades ordered by recently completed first
 	 *
 	 * @param bool $async Include async upgrades
 	 *
@@ -498,6 +499,11 @@ public function getPendingUpgrades($async = true) {
 	public function getCompletedUpgrades($async = true) {
 		$completed = [];
 
+		$order_by_completed_time = new EntitySortByClause();
+		$order_by_completed_time->direction = 'DESC';
+		$order_by_completed_time->property = 'completed_time';
+		$order_by_completed_time->property_type = 'private_setting';
+
 		$upgrades = elgg_get_entities([
 			'type' => 'object',
 			'subtype' => 'elgg_upgrade',
@@ -506,6 +512,7 @@ public function getCompletedUpgrades($async = true) {
 				'name' => 'is_completed',
 				'value' => true,
 			],
+			'order_by' => $order_by_completed_time,
 			'limit' => false,
 			'batch' => true,
 		]);