MokoConsulting/MokoSuiteBackup
1
0
Fork 0
Code Issues 29 Pull Requests 1 Releases 2 Wiki Activity

fix: pre-update backup fires on Joomla Update (match layout=update) #243

Merged
jmiller merged 1 commits from fix/preupdate-layout-update into dev 2026-07-12 20:27:34 +00:00
Conversation 0 Commits 1 Files Changed 2
+16 -8
jmiller commented 2026-07-12 20:24:31 +00:00
Owner
Copy Link
Copy Source

The Install-the-update button posts to option=com_joomlaupdate&layout=update (only layout=update, no view/task), so the redirect never fired. Now matches layout=update as well as view=update and update.install. Refs #242

https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

The Install-the-update button posts to option=com_joomlaupdate&layout=update (only layout=update, no view/task), so the redirect never fired. Now matches layout=update as well as view=update and update.install. Refs #242 https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
jmiller added 1 commit 2026-07-12 20:27:28 +00:00
fix: pre-update backup fires on Joomla Update (match layout=update)
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Failing after 3s
Details
Generic: Project CI / Lint & Validate (pull_request) Has been cancelled
Details
Universal: PR Check / Branch Policy (pull_request) Has been cancelled
Details
Universal: PR Check / Require Docs Update (pull_request) Has been cancelled
Details
Universal: PR Check / Wiki Update Reminder (pull_request) Has been cancelled
Details
Universal: PR Check / Secret Scan (pull_request) Has been cancelled
Details
Universal: PR Check / Validate PR (pull_request) Has been cancelled
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Has been cancelled
Details
Branch Cleanup / Delete merged branch (pull_request) Has been cancelled
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been cancelled
Details
Generic: Project CI / Tests (pull_request) Has been cancelled
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
86381a5319 
The 'Install the update' button posts to
option=com_joomlaupdate&layout=update (Joomla's confirm/updating page),
carrying only layout=update -- no view, no task. The redirect matched
view=update / update.install, so it never fired on the Joomla Update page.

Match layout=update (plus view=update and legacy update.install), and
return the browser to the same layout flagged is_backed_up=1.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
jmiller force-pushed fix/preupdate-layout-update from 10816235c1 to 86381a5319 2026-07-12 20:27:28 +00:00 Compare
jmiller merged commit efc1e19b27 into dev 2026-07-12 20:27:34 +00:00
jmiller deleted branch fix/preupdate-layout-update 2026-07-12 20:27:34 +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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteBackup#243
Delete Branch "fix/preupdate-layout-update"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?