Merge pull request #16164 from craftcms/bugfix/16151-disable-disallow…

…ed-actions

check if data-id is empty or isn't there at all

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Element indexes now sort by ID by default, for sources that don’t define a default sort option.
 - Fixed a bug where element indexes were sorting by the first sortable attribute alphabetically by default, rather than the first sortable attribute defined by the element type.
+- Fixed a bug where bulk asset actions where shown as available when subfolders were selected, when they shouldn’t have. ([#16151](https://github.com/craftcms/cms/issues/16151))
 
 ## 4.13.2 - 2024-11-19
 

src/web/assets/cp/src/js/ElementActionTrigger.js

@@ -82,8 +82,13 @@ Craft.ElementActionTrigger = Garnish.Base.extend(
         return false;
       }
 
-      if (this.settings.requireId && this.$selectedItems.is('[data-id=""]')) {
-        return false;
+      if (this.settings.requireId) {
+        for (let i = 0; i < this.$selectedItems.length; i++) {
+          const $item = this.$selectedItems.eq(i);
+          if (!Garnish.hasAttr($item, 'data-id') || $item.is('[data-id=""]')) {
+            return false;
+          }
+        }
       }
 
       if (typeof this.settings.validateSelection === 'function') {