Packaging gaps: en-US language not declared, undeclared stub dirs, manifest/SQL version drift #107

New Issue

2026-06-29T14:20:31Z

jmiller commented

2026-06-29 14:20:31 +00:00

Packaging / manifest gaps in `com_mokoog/mokoog.xml`

en-US language not packaged — <files folder="language"> declares only <folder>en-GB</folder>, but language/en-US/ exists on disk and won't be installed. Either declare en-US or drop the on-disk files. (Same check should be applied to the three plugins.)
Undeclared stub directories — src/Field/ and src/Service/ exist on disk (empty, only index.html) but aren't declared in the manifest <folder> list — dead scaffolding. Either declare them (if intended) or delete.
Manifest vs SQL version drift — manifest <version>01.05.00</version> but sql/updates/mysql/ contains 01.05.01.sql (higher than the manifest version). Joomla records schema version from the highest update filename, so the recorded schema/version is inconsistent. (No-op file today, but should be reconciled — either bump the manifest or rename the SQL file to match an actual release.)

Note

SQL schema parity itself is fine — install.mysql.sql contains the full final column set and the incremental updates reconcile to the same shape (verified). This issue is about manifest/packaging hygiene only.

## Packaging / manifest gaps in `com_mokoog/mokoog.xml` 1. **`en-US` language not packaged** — `<files folder="language">` declares only `<folder>en-GB</folder>`, but `language/en-US/` exists on disk and won't be installed. Either declare `en-US` or drop the on-disk files. (Same check should be applied to the three plugins.) 2. **Undeclared stub directories** — `src/Field/` and `src/Service/` exist on disk (empty, only `index.html`) but aren't declared in the manifest `<folder>` list — dead scaffolding. Either declare them (if intended) or delete. 3. **Manifest vs SQL version drift** — manifest `<version>01.05.00</version>` but `sql/updates/mysql/` contains `01.05.01.sql` (higher than the manifest version). Joomla records schema version from the highest update filename, so the recorded schema/version is inconsistent. (No-op file today, but should be reconciled — either bump the manifest or rename the SQL file to match an actual release.) ## Note SQL schema parity itself is fine — `install.mysql.sql` contains the full final column set and the incremental updates reconcile to the same shape (verified). This issue is about manifest/packaging hygiene only.

jmiller added this to the Code Quality & Testing milestone 2026-06-29 14:20:31 +00:00

jmiller added the bug label 2026-06-29 14:20:31 +00:00

jmiller commented

2026-06-29 14:20:34 +00:00

Branch created: feature/107-packaging-gaps-en-us-language-not-declar

git fetch origin
git checkout feature/107-packaging-gaps-en-us-language-not-declar

Branch created: [`feature/107-packaging-gaps-en-us-language-not-declar`](https://git.mokoconsulting.tech/MokoConsulting/MokoSuiteOpenGraph/src/branch/feature/107-packaging-gaps-en-us-language-not-declar) ```bash git fetch origin git checkout feature/107-packaging-gaps-en-us-language-not-declar ```

jmiller referenced this issue from a commit

2026-06-29 15:26:05 +00:00

fix: features & quality batch (#95, #103, #104, #107)

jmiller referenced this issue

2026-06-29 15:26:28 +00:00

fix: features & quality — access.xml/config.xml, CSV import UI, dead-code/leak, packaging (#95 #103 #104 #107) #111

jmiller referenced this issue from a commit

2026-06-29 15:26:42 +00:00

Merge pull request 'fix: features & quality — access.xml/config.xml, CSV import UI, dead-code/leak, packaging (#95 #103 #104 #107)' (#111) from fix/features-quality-batch into dev

jmiller commented

2026-06-29 15:27:17 +00:00

Fixed in PR #111 (merged to dev). Declared language/en-US in the component manifest (was on disk but never installed) and removed the undeclared empty stub dirs src/Field/src/Service. The manifest/SQL version-number drift is managed by the release CI (auto-bump), so leaving that to the workflow. Closing.

Fixed in PR #111 (merged to `dev`). Declared `language/en-US` in the component manifest (was on disk but never installed) and removed the undeclared empty stub dirs `src/Field`/`src/Service`. The manifest/SQL version-number drift is managed by the release CI (auto-bump), so leaving that to the workflow. Closing.

jmiller closed this issue

2026-06-29 15:27:21 +00:00

jmiller referenced this issue

2026-06-29 16:00:44 +00:00

docs: CHANGELOG + README for the current development cycle #114

jmiller referenced this issue

2026-06-29 16:26:51 +00:00

Release: promote dev → main (dashboard, edit UI, ACL, security & forward-compat) #115

Sign in to join this conversation.