846 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
mokogitea-actions[bot]	4027c727d9	chore(version): pre-release bump to 02.58.07-dev [skip ci]	2026-07-06 01:36:07 +00:00
jmiller	b729aa5052	Merge pull request 'feat: prune remote backups on retention + consolidate backup helpers (#229, #230)' (#231) from fix/retention-remote-shared-helper into dev	2026-07-06 01:35:45 +00:00
mokogitea-actions[bot]	7a829b6eca	chore(version): pre-release bump to 02.58.06-dev [skip ci]	2026-07-06 01:34:49 +00:00
jmiller	447f7b572e	feat: prune remote backups on retention + consolidate backup helpers ... Retention (RetentionManager and the plugin's hourly cleanup) only ever deleted local files; remote archives grew unbounded. Add an idempotent delete() to RemoteUploaderInterface + all four uploaders and have RetentionManager remove each pruned archive from the profile's enabled remotes (best-effort; failures logged). The shared restore.php is left in place since every backup overwrites it. (#229) Consolidate duplicated plumbing (#230): - New RemoteUploaderFactory replaces the createUploaderFromParams copy duplicated in BackupEngine and SteppedBackupEngine. - RetentionManager becomes the single retention authority (global-default fallback + pruneOrphans()); the system plugin delegates to it and its duplicate doCleanup()/deleteBackupRecord() logic is removed. - Backend controller, API controller and legacy cli/mokosuitebackup.php now run through the shared BackupRunner instead of BackupEngine directly. Refs #229 #230 Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 20:34:04 -05:00
mokogitea-actions[bot]	f56c1bfbd1	chore(version): pre-release bump to 02.58.05-dev [skip ci]	2026-07-06 01:17:22 +00:00
jmiller	aeb9d18ccb	Merge pull request 'fix: restore missing str_replace subject arg in MokoRestore standalone gen (#226)' (#228) from fix/mokorestore-str-replace-args into dev	2026-07-06 01:17:01 +00:00
mokogitea-actions[bot]	0a51ed75da	chore(version): pre-release bump to 02.58.04-dev [skip ci]	2026-07-06 01:16:02 +00:00
jmiller	0ae9bfd93a	fix: restore missing subject arg in MokoRestore str_replace (standalone gen) ... MokoRestore::generateStandaloneScript() rewrites the 'Backup Archive' pre-check via str_replace(ORIG, REPL) but was missing the third $php subject argument, so every standalone-mode backup fatally errored with 'str_replace() expects at least 3 arguments, 2 given' at the 'Generating standalone restore.php...' step. The archive still finalized and uploaded, but no restore.php was produced -- the underlying cause of #226 (restore script missing on the remote). Refs #226 Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 20:15:30 -05:00
mokogitea-actions[bot]	1c0a87ca4a	chore(version): pre-release bump to 02.58.03-dev [skip ci]	2026-07-06 00:40:33 +00:00
jmiller	2cb8527c25	Merge pull request 'fix(engine): check + log restore-script remote upload (fixes #226)' (#227) from fix/restore-script-upload-result into dev	2026-07-06 00:40:18 +00:00
mokogitea-actions[bot]	35febb8854	chore(version): pre-release bump to 02.58.02-dev [skip ci]	2026-07-06 00:39:57 +00:00
jmiller	bf91611b97	fix(engine): check + log restore-script remote upload (no longer silent) ... The standalone restore script was uploaded to each remote with the result discarded and nothing logged, so a failed second upload left the restore script missing from the remote with no warning while the archive still recorded 'complete'. Capture the result: log success, and on failure set uploadFailed (-> warning status) and log the reason. Fixes #226 Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 19:39:37 -05:00
mokogitea-actions[bot]	9bc0fc8ad8	chore(version): pre-release bump to 02.58.01-dev [skip ci]	2026-07-06 00:33:03 +00:00
jmiller	e75a58709b	Merge pull request 'chore(sync): cascade main -> dev' (#225) from main into dev	2026-07-06 00:32:12 +00:00
gitea-actions[bot]	8be1e332c6	chore(release): build 02.58.00 [skip ci] stable	2026-07-06 00:32:07 +00:00
jmiller	ba91f74edc	Merge pull request 'release: promote dev → main (installer + admin-menu + UI fixes)' (#224) from dev into main	2026-07-06 00:30:29 +00:00
mokogitea-actions[bot]	387bfbdeda	chore(version): pre-release bump to 02.57.13-dev [skip ci]	2026-07-06 00:17:00 +00:00
jmiller	f7d8d82a4e	Merge pull request 'refactor(ui): modal padding via Bootstrap container, drop msbackup class' (#223) from fix/modal-container-padding into dev	2026-07-06 00:16:33 +00:00
mokogitea-actions[bot]	9e5a942800	chore(version): pre-release bump to 02.57.12-dev [skip ci]	2026-07-06 00:16:10 +00:00
jmiller	dad3ce2db3	refactor(ui): modal padding via Bootstrap container, drop msbackup class ... Per feedback, wrap the destination modal content in a Bootstrap .container-fluid and give that a 1rem frame in the WAM stylesheet, instead of a custom .msbackup class on .modal-content. Inner header/body/footer drop their own horizontal padding so the frame isn't doubled. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 19:15:49 -05:00
mokogitea-actions[bot]	8325afb03a	chore(version): pre-release bump to 02.57.11-dev [skip ci]	2026-07-06 00:06:23 +00:00
jmiller	662918db38	Merge pull request 'feat(ui): SFTP key file upload + msbackup modal padding frame' (#222) from fix/modal-msbackup-class into dev	2026-07-06 00:04:47 +00:00
jmiller	1ab99daaba	feat(ui): SFTP key file upload; msbackup modal padding frame ... - Add/Edit Destination modal: SFTP key can now be UPLOADED as a file (read into the key textarea client-side via FileReader) instead of copy/paste; pasting still works. - Give .modal-content a "msbackup" class with a 1rem padding frame — Bootstrap's .modal-content has no padding of its own, which made the modal feel cramped. Styling lives in the WAM stylesheet (media/.../admin.css), scoped to .msbackup. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 19:04:05 -05:00
mokogitea-actions[bot]	c84dcf2864	chore(version): pre-release bump to 02.57.10-dev [skip ci]	2026-07-05 23:59:02 +00:00
jmiller	e1f5afb301	Merge pull request 'fix(plugin): pre-update backup notifies on success, not only failure' (#221) from fix/preupdate-notification into dev	2026-07-05 23:58:21 +00:00
mokogitea-actions[bot]	dfd4e77ade	chore(version): pre-release bump to 02.57.09-dev [skip ci]	2026-07-05 23:58:13 +00:00
jmiller	1a1c81550c	fix(plugin): pre-update backup notifies on success too, not only failure ... runPreActionBackup() only enqueued a message when the backup failed, so a successful pre-update backup produced no admin notification at all. Now it surfaces every outcome via the BackupRunner status: complete -> success, warning (archive made but remote upload failed) -> warning, fail -> error. Refs #192 Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 18:57:42 -05:00
mokogitea-actions[bot]	9571b3759a	chore(version): pre-release bump to 02.57.08-dev [skip ci]	2026-07-05 23:49:26 +00:00
jmiller	115ccfb069	Merge pull request 'fix(ui): WAM-loaded CSS, per-destination keep-local, tidy destination modal' (#220) from fix/remote-modal-cleanup into dev	2026-07-05 23:49:04 +00:00
mokogitea-actions[bot]	2d538d3372	chore(version): pre-release bump to 02.57.07-dev [skip ci]	2026-07-05 23:48:51 +00:00
jmiller	97e8d0cbe9	fix(ui): WAM-loaded CSS, per-destination keep-local, tidy destination modal ... - Move component admin CSS to the Joomla Web Asset Manager: new media/com_mokosuitebackup/css/admin.css + media/joomla.asset.json, a <media> entry in the manifest, and useStyle('com_mokosuitebackup.admin') in the profile edit view. Removes the inline <style> block (no more inline CSS). - Remove the profile-level "Keep local copy" field/"Remote" fieldset from forms/profile.xml — it is now per remote destination (modal remoteKeepLocal). - Clean up the Add/Edit Destination modal padding/margins (in admin.css). Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 18:48:33 -05:00
mokogitea-actions[bot]	8d5b33a74d	chore(version): pre-release bump to 02.57.06-dev [skip ci]	2026-07-05 23:32:04 +00:00
jmiller	77bfaff8cd	Merge pull request 'fix(installer): self-heal admin menu + alias collision + short-constant label' (#219) from fix/admin-menu-selfheal into dev	2026-07-05 23:31:41 +00:00
mokogitea-actions[bot]	1c420b2277	chore(version): pre-release bump to 02.57.05-dev [skip ci]	2026-07-05 23:31:24 +00:00
jmiller	2af2641712	fix(installer): self-heal admin menu; fix alias collision; short-constant label ... Fallout of the duplicate-component element bug: - Preflight now runs removeOrphanedComponent() BEFORE the component install recreates its menu, so the orphan's "Backup" menu (alias "backup") no longer collides ("The alias backup is already being used by the Backup menu item"). - ensureSubmenuItems() recreates the top-level "Backup" menu when it is missing (via new createTopMenu()) instead of bailing out — so a failed install that deleted the parent menu no longer leaves Backup gone from the admin menu (which also broke the cPanel module and MokoSuiteClient component detection). - The top-level menu label now uses the short constant COM_MOKOJOOMBACKUP_SHORT (manifest <menu> + enforced/normalized in the script). Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 18:31:06 -05:00
mokogitea-actions[bot]	a3c7183374	chore(version): pre-release bump to 02.57.04-dev [skip ci]	2026-07-05 23:15:31 +00:00
jmiller	4f91d1e432	Merge pull request 'chore(sync): cascade main -> dev' (#212) from main into dev	2026-07-05 23:03:57 +00:00
jmiller	604412f82f	Merge chore/normalize-docs: normalize doc/config refs	2026-07-05 22:50:52 +00:00
jmiller	94ff15c128	chore(docs): normalize refs — MokoStandards->org wiki, moko-platform->mokocli, residual MokoGitea/MokoSuite ... Non-workflow docs/config only (CLAUDE.md, README, branch-protection, ISSUE_TEMPLATE/config.yml, ci-issue-reporter). Workflow files, manifest schema, and Makefile install paths left for the forge-rebrand gate. Authored-by: Moko Consulting	2026-07-05 22:50:50 +00:00
mokogitea-actions[bot]	a92f423767	chore(version): pre-release bump to 02.57.02-dev [skip ci]	2026-07-05 22:44:13 +00:00
jmiller	ba2cda1385	Merge pull request 'fix(installer): drop legacy remote columns in PHP, not PREPARE-based SQL' (#218) from fix/sql-prepared-statement-protocol into dev	2026-07-05 22:43:53 +00:00
mokogitea-actions[bot]	72aeea6955	chore(version): pre-release bump to 02.57.01-dev [skip ci]	2026-07-05 22:41:55 +00:00
jmiller	8a85370e32	fix(installer): drop legacy remote columns in PHP, not PREPARE-based SQL ... 02.56.01.sql used PREPARE/EXECUTE/DEALLOCATE to conditionally drop the 26 legacy remote_storage/ftp_*/sftp_*/gdrive_*/s3_* columns portably. Joomla's installer rejects those statements on MySQL 8 (error 1295: "This command is not supported in the prepared statement protocol yet"), aborting the whole component install when updating from an old schema. Neutralize 02.56.01.sql (now a no-op comment) and move the purge into Pkg_MokoSuiteBackupInstallerScript::dropLegacyRemoteColumns(): gate on INFORMATION_SCHEMA, then issue one plain ALTER dropping only the columns that still exist — portable on MariaDB and MySQL 8, idempotent, non-fatal. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 17:41:26 -05:00
gitea-actions[bot]	ea91b49e50	chore(release): build 02.57.00 [skip ci]	2026-07-05 22:30:31 +00:00
jmiller	99d8e954bd	Merge pull request 'feat: BackupRunner helper — unify backup run/status/notification' (#214) from feat/backup-runner-helper into main	2026-07-05 22:29:38 +00:00
jmiller	28ca110d7b	docs(changelog): add BackupRunner entry (#214) ... Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 17:29:28 -05:00
jmiller	307067580c	feat: BackupRunner helper — unify backup run/status/notification ... Every non-interactive backup trigger used a slightly different path, so final status and notification gating behaved inconsistently depending on what started the backup (this is why pre-update backups did not gate notifications the way "Backup Now" does). Add src/Service/BackupRunner.php as the single synchronous entry point. It delegates to BackupEngine (which already sends notifications gated by the profile's notify_on_success/notify_on_failure and applies retention — so there is exactly ONE notification layer), and normalises the result, including the real complete/warning/fail status the engine records but did not return. BackupRunner self-loads BackupEngine so callers require one file. Routed through BackupRunner: - plg_system runPreActionBackup() (preaction) + web-cron handler (webcron) - plg_content triggerAutoBackup() (pre-install) - plg_task scheduled task run - plg_console RunCommand (cli) The dashboard AJAX/stepped "Backup Now" flow is unchanged (it is the interactive equivalent). The installer progress popup is deferred — the installer request cannot drive the AJAX modal (TODO(#196) noted in code). php -l clean on all changed files. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 17:28:59 -05:00
jmiller	7f5f041588	Merge pull request 'fix(release): idempotent changelog promote (no duplicate version headers)' (#217) from fix/changelog-promote-idempotent into main	2026-07-05 22:28:19 +00:00
jmiller	129be7ef9d	fix(release): idempotent changelog promote + dedupe 02.56.11 header ... Guard the promote so it only inserts a version header when absent (stops the empty duplicate headers each build was appending), and remove the duplicate 02.56.11 header the non-idempotent promote just created. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-05 17:28:08 -05:00
jmiller	14392de078	Merge pull request 'docs(changelog): backfill 02.56.07 + 02.56.08 (shipped with empty changelogs)' (#215) from fix/changelog-backfill into main	2026-07-05 22:26:17 +00:00