Fixed missing assets for stale aliases.

salsadigitalauorg · Mar 31, 2020 · 36477dc · 36477dc
1 parent 80994f9
commit 36477dc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Asset.php b/src/Asset.php
@@ -181,6 +181,7 @@ public static function load($id) {
     $values = Database::getConnection()->select('minisite_asset', 'ma')
       ->fields('ma')
       ->condition('id', $id)
+      ->orderBy('id', 'DESC')
       ->execute()
       ->fetchAssoc();
 
@@ -202,6 +203,7 @@ public static function loadByUri($uri) {
     $values = Database::getConnection()->select('minisite_asset', 'ma')
       ->fields('ma')
       ->condition('source', $uri)
+      ->orderBy('id', 'DESC')
       ->execute()
       ->fetchAssoc();
 
@@ -223,6 +225,7 @@ public static function loadByAlias($alias) {
     $values = Database::getConnection()->select('minisite_asset', 'ma')
       ->fields('ma')
       ->condition('alias', $alias)
+      ->orderBy('id', 'DESC')
       ->execute()
       ->fetchAssoc();
 
@@ -239,6 +242,7 @@ public static function loadByAlias($alias) {
   public static function loadAll() {
     $values = Database::getConnection()->select('minisite_asset', 'ma')
       ->fields('ma')
+      ->orderBy('id', 'DESC')
       ->execute()
       ->fetchAllAssoc('id', \PDO::FETCH_ASSOC);
 

diff --git a/tests/src/Functional/AssetTest.php b/tests/src/Functional/AssetTest.php
@@ -271,9 +271,9 @@ public function testLoadAll() {
     $loaded = Asset::loadAll();
 
     $this->assertEquals(3, count($loaded));
-    $this->assertEquals($asset1->id(), $loaded[0]->id());
+    $this->assertEquals($asset3->id(), $loaded[0]->id());
     $this->assertEquals($asset2->id(), $loaded[1]->id());
-    $this->assertEquals($asset3->id(), $loaded[2]->id());
+    $this->assertEquals($asset1->id(), $loaded[2]->id());
   }
 
 }