Release: dev → main (02.59.x) #253

Merged

jmiller merged 19 commits from dev into main 2026-07-12 23:33:28 +00:00

Conversation 0 Commits 19 Files Changed 29

+199 -94

jmiller commented 2026-07-12 23:25:18 +00:00

Owner

Copy Link

Copy Source

Release dev → main (02.59.06-dev).

Since 02.59.00

• Pre-action backup now fires for every extension update/uninstall, not just the first in a 10-minute window. Replaced the coarse 600s session throttle (which disabled the interceptor + server fallback for all extensions after any one backup — even after updating MokoSuiteBackup itself or a core update) with one backup per Update action: per-request dedupe + a one-shot action-keyed skip flag; core update decoupled to its own key.
• Full-screen backup screen: black backdrop, and all screen CSS moved into the Joomla Web Asset Manager (media/css/runbackup.css + joomla.asset.json) instead of an inline <style> block.
• Earlier review fixes carried from the last cycle (return-URL backslash open-redirect, btoa UTF-8 safety, snapshot duplicate-mode non-transactional honesty).

https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

Release **dev → main** (02.59.06-dev). ## Since 02.59.00 - **Pre-action backup now fires for every extension update/uninstall**, not just the first in a 10-minute window. Replaced the coarse 600s session throttle (which disabled the interceptor + server fallback for all extensions after any one backup — even after updating MokoSuiteBackup itself or a core update) with **one backup per Update action**: per-request dedupe + a one-shot action-keyed skip flag; core update decoupled to its own key. - Full-screen backup screen: **black backdrop**, and all screen CSS moved into the **Joomla Web Asset Manager** (media/css/runbackup.css + joomla.asset.json) instead of an inline <style> block. - Earlier review fixes carried from the last cycle (return-URL backslash open-redirect, btoa UTF-8 safety, snapshot duplicate-mode non-transactional honesty). https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

jmiller added 16 commits 2026-07-12 23:25:18 +00:00

@ ... 3eb2562abd

fix: pre-action backup fires for every extension, not one per 10 min

Root cause of "pre-update backup only fires on the backup extension":
a coarse 600s session throttle. Whenever ANY pre-action backup ran
(including updating MokoSuiteBackup itself, or a core Joomla update), the
key mokosuitebackup.preaction_backup_before_update was armed for 10
minutes. onBeforeCompileHead read it as `recentBackup` and DISABLED the
client-side interceptor for all extensions, and runPreActionBackup was
throttled by the same window. So after updating one extension, every
other extension update silently skipped its backup for 10 minutes.

Replace the time window with "one backup per Update action":
- runPreActionBackup dedupes per-request (a batch update fires the event
  once per extension in a single request -> back up once) via instance
  flags, and consumes a ONE-SHOT session skip flag for the re-fired
  client update/uninstall (skip once, next distinct action backs up).
- ajax.preupdateAck sets the one-shot flag for the SPECIFIC action
  (update|uninstall), passed from installer-backup.js via &msb_action and
  surfaced by Runbackup HtmlView -> CFG.action. A core update passes no
  action, so it never suppresses an extension backup.
- onBeforeCompileHead drops the recentBackup time-suppression (the
  re-fire loop is already guarded client-side by window.__msbResuming).
- The core Joomla update path uses its own key
  (mokosuitebackup.core_update_backed_up), decoupled from extensions.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
@

chore(version): pre-release bump to 02.59.01-dev [skip ci] 2df4b280eb

Merge pull request 'fix: pre-action backup fires for every extension (not one per 10 min)' (#250) from fix/preaction-per-action-backup into dev f02c4d4c9d

chore(version): pre-release bump to 02.59.02-dev [skip ci] d9d6ee9181

@ ... 8a783aa25a

style(runbackup): black backdrop on the full-screen backup screen

Set html/body background to #000 so the full-screen pre-update / Backup
Now progress screen runs on a black backdrop (the card + Atum-coloured
progress bar sit on top).

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
@

chore(version): pre-release bump to 02.59.03-dev [skip ci] d1ccb8d419

Merge pull request 'style(runbackup): black backdrop on the full-screen backup screen' (#251) from fix/runbackup-black-bg into dev 14cc00f93f

chore(version): pre-release bump to 02.59.04-dev [skip ci] 8237f85056

@ ... d658019007

refactor(runbackup): move screen CSS into the Web Asset Manager

All full-screen backup CSS now loads through Joomla WAM instead of an
inline <style> block (and the redundant inline style="width:0"):

- New media/css/runbackup.css holds the whole stylesheet.
- Registered as asset com_mokosuitebackup.runbackup in joomla.asset.json.
- tmpl/runbackup/default.php loads it via
  getDocument()->getWebAssetManager()->useStyle(com_mokosuitebackup.runbackup),
  matching the existing com_mokosuitebackup.admin pattern.

The css folder already ships via the manifest <folder>css</folder>, so no
manifest change is needed.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
@

chore(version): pre-release bump to 02.59.05-dev [skip ci] 059cbfe058

Merge pull request 'refactor(runbackup): move screen CSS into the Web Asset Manager' (#252) from fix/runbackup-css-wam into dev 25991c474b

chore(version): pre-release bump to 02.59.06-dev [skip ci] 9ee800b151

@ ... 1e6d9ba078

chore: back-merge main into dev (release 02.59.00 version stamps)

Reconciles main (released 02.59.00) into dev ahead of the next release.
All conflicts were VERSION-stamp comments/tags in manifests, SECURITY.md
and issue-branch.yml — kept dev's current 02.59.05 line. No functional
changes. Clears the dev -> main merge conflict for the release PR.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
@

Merge remote-tracking branch 'origin/dev' into dev 444cfa3fdb

chore(version): auto-bump patch 02.59.07-dev [skip ci] 1c5c511c06

chore(version): pre-release bump to 02.59.08-dev [skip ci] 0562d3effd

jmiller added 2 commits 2026-07-12 23:33:01 +00:00

@ ... f62b1ec72e

docs(ajax): correct preupdateAck docblock to one-shot skip semantics

The docblock still described the removed 10-minute throttle ("arms BOTH
keys"). It now sets a one-shot, action-keyed skip flag consumed once by
the system plugin. Comment-only; no behaviour change.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
@

Merge pull request 'docs(ajax): correct preupdateAck docblock' (#254) from docs/preupdateack-docblock into dev a60f7ac032

jmiller added 1 commit 2026-07-12 23:33:16 +00:00

chore(version): pre-release bump to 02.59.09-dev [skip ci] 0c845fe807

jmiller merged commit 89f1869120 into main 2026-07-12 23:33:28 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

component: admin

Admin UI and views

component: api

REST API

component: engine

Backup/restore engine

component: remote

Remote storage (FTP/GDrive)

component: scheduler

Scheduled tasks

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No labels

Priority -

Type -

Milestone

No items

No Milestone

Assignees

Clear assignees

jmiller (Jonathan Miller)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteBackup#253