Updated bulk action js to pass the checkboxe names into the get array this allows the use of multiple checkbox name arrays to be passed at once instead of just selected_ids had to update each bulk model from selected_ids to to the array that was passed. This was important so we could mix files and documents together

js/ajax_modal.js

@@ -2,31 +2,18 @@
 $(document).on('click', '.ajax-modal', function (e) {
   e.preventDefault();
 
-  const $trigger = $(this);
-  let modalUrl = $trigger.data('modal-url');
+  const $trigger  = $(this);
+
+  // Prefer data-modal-url, fallback to href
+  let modalUrl = $trigger.data('modal-url') || $trigger.attr('href') || '#';
   const modalSize = $trigger.data('modal-size') || 'md';
-  const modalId = 'ajaxModal_' + new Date().getTime();
+  const modalId   = 'ajaxModal_' + Date.now();
 
-  // -------- Optional bulk mode (activated via data-bulk="true") --------
-  if ($trigger.data('bulk') === true || $trigger.data('bulk') === 'true') {
-    const selector = $trigger.data('bulk-selector') || '.bulk-select:checked';
-    const param    = $trigger.data('bulk-param') || 'selected_ids[]';
-
-    const ids = Array.from(document.querySelectorAll(selector)).map(cb => cb.value);
-
-    if (!ids.length) {
-      alert('Please select at least one item.');
-      return; // abort opening modal
-    }
-
-    // Merge ids into existing query string safely
-    const urlObj = new URL(modalUrl, window.location.href);
-    ids.forEach(id => urlObj.searchParams.append(param, id));
-
-    // Preserve path + updated query (avoid absolute origin for relative AJAX)
-    modalUrl = urlObj.pathname + (urlObj.search ? '?' + urlObj.searchParams.toString() : '');
+  // If no usable URL, bail
+  if (!modalUrl || modalUrl === '#') {
+    console.warn('ajax-modal: No modal URL found on trigger:', this);
+    return;
   }
-  // --------------------------------------------------------------------
 
   // Show loading spinner while fetching content
   const loadingSpinner = `