[4.x] Include alt field in shallowly augmented assets (#10013)

statamic · May 7, 2024 · a7f97df · a7f97df
1 parent 7dabbac
commit a7f97df
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/src/Assets/Asset.php b/src/Assets/Asset.php
@@ -1063,7 +1063,7 @@ public function defaultAugmentedArrayKeys()
 
     public function shallowAugmentedArrayKeys()
     {
-        return ['id', 'url', 'permalink', 'api_url'];
+        return ['id', 'url', 'permalink', 'api_url', 'alt'];
     }
 
     protected function defaultAugmentedRelations()

diff --git a/tests/Fieldtypes/AssetsTest.php b/tests/Fieldtypes/AssetsTest.php
@@ -69,6 +69,20 @@ public function it_augments_to_a_single_asset_when_max_files_is_one()
     /** @test */
     public function it_shallow_augments_to_a_collection_of_assets()
     {
+        AssetContainer::find('test')
+            ->queryAssets()
+            ->where('path', 'foo/one.txt')
+            ->first()
+            ->set('alt', 'Alt text for one')
+            ->save();
+
+        AssetContainer::find('test')
+            ->queryAssets()
+            ->where('path', 'bar/two.txt')
+            ->first()
+            ->set('alt', 'Alt text for two')
+            ->save();
+
         $augmented = $this->fieldtype()->shallowAugment(['foo/one.txt', 'bar/two.txt', 'unknown.txt']);
 
         $this->assertInstanceOf(Collection::class, $augmented);
@@ -79,19 +93,28 @@ public function it_shallow_augments_to_a_collection_of_assets()
                 'url' => '/assets/foo/one.txt',
                 'permalink' => 'http://localhost/assets/foo/one.txt',
                 'api_url' => 'http://localhost/api/assets/test/foo/one.txt',
+                'alt' => 'Alt text for one',
             ],
             [
                 'id' => 'test::bar/two.txt',
                 'url' => '/assets/bar/two.txt',
                 'permalink' => 'http://localhost/assets/bar/two.txt',
                 'api_url' => 'http://localhost/api/assets/test/bar/two.txt',
+                'alt' => 'Alt text for two',
             ],
         ], $augmented->toArray());
     }
 
     /** @test */
     public function it_shallow_augments_to_a_single_asset_when_max_files_is_one()
     {
+        AssetContainer::find('test')
+            ->queryAssets()
+            ->where('path', 'foo/one.txt')
+            ->first()
+            ->set('alt', 'Alt text for one')
+            ->save();
+
         $augmented = $this->fieldtype(['max_files' => 1])->shallowAugment(['foo/one.txt']);
 
         $this->assertInstanceOf(AugmentedCollection::class, $augmented);
@@ -100,6 +123,7 @@ public function it_shallow_augments_to_a_single_asset_when_max_files_is_one()
             'url' => '/assets/foo/one.txt',
             'permalink' => 'http://localhost/assets/foo/one.txt',
             'api_url' => 'http://localhost/api/assets/test/foo/one.txt',
+            'alt' => 'Alt text for one',
         ], $augmented->toArray());
     }