From 2b4af3ce289926a3a41ccced22a7eb4227a59d59 Mon Sep 17 00:00:00 2001
From: Jonathan Miller <jmiller@mokoconsulting.tech>
Date: Tue, 14 Jul 2026 10:11:32 -0500
Subject: [PATCH] @ fix(installer): extension update now auto-continues after
 pre-update backup

Two bugs in the client-side resume path made the extension update return
to the selection screen (requiring a manual re-select + re-submit) instead
of continuing after the backup:

1. The re-fire clicked the OUTER <joomla-toolbar-button> (which carries the
   task attr), but the web component binds its click handler to the INNER
   <button>. Clicking the wrapper did nothing. Now clicks the inner element.
2. list-selection toolbar buttons start DISABLED and only enable when
   boxchecked fires a `change` event. We set boxchecked.value but never
   dispatched change, so the button stayed disabled and executeTask() no-opd.
---
 .../media/js/installer-backup.js              | 30 ++++++++++++++-----
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/source/packages/plg_system_mokosuitebackup/media/js/installer-backup.js b/source/packages/plg_system_mokosuitebackup/media/js/installer-backup.js
index e3f5ac4a..cb64a736 100644
--- a/source/packages/plg_system_mokosuitebackup/media/js/installer-backup.js
+++ b/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') {