fix(installer): self-heal orphaned com_component-mokosuitebackup on update #216

Merged

jmiller merged 2 commits from fix/orphan-component-cleanup into main 2026-07-05 22:23:09 +00:00

Conversation 1 Commits 2 Files Changed 15

+122 -12

jmiller commented 2026-07-05 22:12:10 +00:00

Owner

Copy Link

Copy Source

Follow-up to #213. The manifest fix stops new duplicate registrations; this reconciles already-affected sites (like suite.dev) automatically on the next update — no manual DB surgery, no SSH.

What it does

Pkg_MokoSuiteBackupInstallerScript::postflight() now calls removeOrphanedComponent(), which finds any component whose element matches %component%mokosuitebackup% (explicitly excluding com_mokosuitebackup) and removes:

• its #__menu admin menu items,
• its #__schemas rows,
• its #__update_sites_extensions mapping,
• the #__extensions record itself,
• the stray administrator/components/com_component-mokosuitebackup/ folder (native recursive delete — no Joomla Folder namespace dependency).

Idempotent (no-op when clean) and wrapped in try/catch so it can never block an install/update. Runs on install + update. php -l clean.

Result for suite.dev

After the corrected package (≥ this build) is installed, the real com_mokosuitebackup (id 951) updates to current and the bogus com_component-mokosuitebackup (id 1218) + its folder/menu/schema rows are removed — the admin then loads the current component (working remote-destinations tab, no legacy dropdowns).

Note: the changelog [Unreleased] entry here will lightly overlap #215 (changelog backfill) — trivial rebase if that merges first.

https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

Follow-up to #213. The manifest fix stops *new* duplicate registrations; this **reconciles already-affected sites** (like suite.dev) automatically on the next update — no manual DB surgery, no SSH. ## What it does `Pkg_MokoSuiteBackupInstallerScript::postflight()` now calls `removeOrphanedComponent()`, which finds any component whose `element` matches `%component%mokosuitebackup%` (explicitly excluding `com_mokosuitebackup`) and removes: - its `#__menu` admin menu items, - its `#__schemas` rows, - its `#__update_sites_extensions` mapping, - the `#__extensions` record itself, - the stray `administrator/components/com_component-mokosuitebackup/` folder (native recursive delete — no Joomla `Folder` namespace dependency). Idempotent (no-op when clean) and wrapped in try/catch so it can never block an install/update. Runs on install + update. `php -l` clean. ## Result for suite.dev After the corrected package (≥ this build) is installed, the real `com_mokosuitebackup` (id 951) updates to current **and** the bogus `com_component-mokosuitebackup` (id 1218) + its folder/menu/schema rows are removed — the admin then loads the current component (working remote-destinations tab, no legacy dropdowns). Note: the changelog `[Unreleased]` entry here will lightly overlap #215 (changelog backfill) — trivial rebase if that merges first. https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

jmiller added 1 commit 2026-07-05 22:12:11 +00:00

fix(installer): self-heal orphaned com_component-mokosuitebackup on update ... 1029e40908

The old "Type - Name" component <name> made Joomla register a duplicate
component under element com_component-mokosuitebackup (its own extension row,
admin menu, schema rows and administrator/ folder), orphaning the real
com_mokosuitebackup on the old version. The manifest fix (#213) prevents new
occurrences; this reconciles already-affected sites automatically.

Package postflight now removes any component whose element matches
%component%mokosuitebackup% (excluding com_mokosuitebackup): deletes its
#__menu, #__schemas and #__update_sites_extensions rows, the #__extensions
record, and the stray admin folder. Idempotent and non-fatal.

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

jmiller commented 2026-07-05 22:12:32 +00:00

Author

Owner

Copy Link

Copy Source

Wiki reminder: docs are wiki-first -- if this PR changes behavior, usage, config, or standards, please update the repo wiki before/after merge. (non-blocking)

<!-- wiki-reminder --> **Wiki reminder:** docs are wiki-first -- if this PR changes behavior, usage, config, or standards, please update the repo wiki before/after merge. _(non-blocking)_

jmiller added 1 commit 2026-07-05 22:14:25 +00:00

chore(version): pre-release bump to 02.56.11-dev [skip ci] 39d94a31d6

jmiller merged commit 2688ea352b into main 2026-07-05 22:23:09 +00:00

jmiller deleted branch fix/orphan-component-cleanup 2026-07-05 22:23:09 +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#216