Merge pull request 'fix(installer): extension update auto-continues after pre-update backup' (#269) from fix/extension-refire into dev

source/packages/plg_system_mokosuitebackup/media/js/installer-backup.js

@@ -131,20 +131,34 @@
 				if (box) { box.checked = true; }
 			});
 
-			/* Joomla's list-check reads boxchecked — set it so the re-fire passes. */
+			/* Restore boxchecked AND fire its `change` event. Joomla's toolbar web
+			   component (joomla-toolbar-button) keeps a list-selection button
+			   DISABLED until boxchecked changes via a change event — setting the
+			   value alone leaves the button disabled, so executeTask() would no-op
+			   and the update never submits (it just returns to the list). */
 			var bc = form.querySelector('input[name="boxchecked"]');
-			if (bc) { bc.value = (data.cids || []).length; }
+			if (bc) {
+				bc.value = (data.cids || []).length;
+				bc.dispatchEvent(new Event('change', { bubbles: true }));
+			}
 		}
 
 		window.__msbResuming = true;
 
-		var btn = document.querySelector('[task="' + data.task + '"]');
+		/* Re-fire the toolbar action. The `task` attribute is on the OUTER
+		   <joomla-toolbar-button>, but its click handler is bound to the INNER
+		   <button>/<a>, so click that inner element (clicking the wrapper does
+		   nothing). Fall back to Joomla.submitbutton if the button isn't found. */
+		window.setTimeout(function () {
+			var wc    = document.querySelector('[task="' + data.task + '"]');
+			var inner = wc ? (wc.querySelector('button, a') || wc) : null;
 
-		if (btn) {
-			window.setTimeout(function () { btn.click(); }, 300);
-		} else if (window.Joomla && typeof Joomla.submitbutton === 'function') {
-			Joomla.submitbutton(data.task);
-		}
+			if (inner) {
+				inner.click();
+			} else if (window.Joomla && typeof Joomla.submitbutton === 'function') {
+				Joomla.submitbutton(data.task);
+			}
+		}, 300);
 	}
 
 	if (document.readyState === 'loading') {