Merge pull request #1483 from craftcms/bugfix/1481-assets-field-filen…

…ame-dont-import Bugfix/1481 assets field filename dont import
craftcms · Jul 18, 2024 · 0263669 · 0263669
2 parents 890e001 + d4f4075
commit 0263669
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/fields/Assets.php b/src/fields/Assets.php
@@ -118,8 +118,10 @@ public function parseField(): mixed
         $base64ToUpload = [];
 
         $filenamesFromFeed = $upload ? DataHelper::fetchArrayValue($this->feedData, $this->fieldInfo, 'options.filenameNode') : null;
-        // see https://github.com/craftcms/feed-me/issues/1471
-        $filenamesFromFeed = array_splice($filenamesFromFeed, $nodeKey, count($value));
+        if ($filenamesFromFeed) {
+            // see https://github.com/craftcms/feed-me/issues/1471
+            $filenamesFromFeed = array_splice($filenamesFromFeed, $nodeKey, count($value));
+        }
 
         // Fire an 'onAssetFilename' event
         $event = new AssetFilenameEvent([

diff --git a/src/templates/_includes/fields/assets.html b/src/templates/_includes/fields/assets.html
@@ -75,7 +75,7 @@
                 label: 'Use this filename for assets created from URL:'|t('feed-me'),
                 name: 'options[filenameNode]',
                 value: hash_get(feed.fieldMapping, optionsPath ~ '.filenameNode') ?: '',
-                options: feedData,
+                options: feedData|filter(option => option.value != 'usedefault'),
                 class: 'selectize fullwidth',
             }) }}
         </div>