MokoConsulting/MokoGIT
1
0
Fork 0
Code Issues 146 Pull Requests Releases 96 Wiki Activity

fix: derive Joomla element name with correct lowercase + type prefix #636

Closed
jmiller wants to merge 0 commits from fix into main
pull from: fix
merge into: :main
Conversation 0 Commits - Files Changed -
+10 -5
jmiller commented 2026-06-18 13:34:25 +00:00
Owner
Copy Link
Copy Source

Summary

  • Lowercase and strip hyphens in AutoElementName() so element names match what Joomla stores in #__extensions.element (e.g. pkg_mokosuitebackup instead of pkg_MokoSuiteBackup)
  • Remove incorrect plg_ prefix for plugins — Joomla plugins have no element prefix; the folder column determines the plugin group

Problem

The Joomla update server was emitting <element>pkg_MokoSuiteBackup</element> but Joomla's updater matches by element + type + client_id, and the installed extension has pkg_mokosuitebackup in #__extensions. The case mismatch made updates invisible.

Test plan

  • Verify AutoElementName() returns pkg_mokosuitebackup for Name=MokoSuiteBackup, PackageType=package
  • Verify plugins return just lowername with no prefix
  • Verify explicit element_name override still takes precedence
  • Verify hyphened names like Moko-Suite produce pkg_mokosuite

Fixes #635

## Summary - **Lowercase and strip hyphens** in `AutoElementName()` so element names match what Joomla stores in `#__extensions.element` (e.g. `pkg_mokosuitebackup` instead of `pkg_MokoSuiteBackup`) - **Remove incorrect `plg_` prefix** for plugins — Joomla plugins have no element prefix; the `folder` column determines the plugin group ## Problem The Joomla update server was emitting `<element>pkg_MokoSuiteBackup</element>` but Joomla's updater matches by `element + type + client_id`, and the installed extension has `pkg_mokosuitebackup` in `#__extensions`. The case mismatch made updates invisible. ## Test plan - [ ] Verify `AutoElementName()` returns `pkg_mokosuitebackup` for Name=`MokoSuiteBackup`, PackageType=`package` - [ ] Verify plugins return just `lowername` with no prefix - [ ] Verify explicit `element_name` override still takes precedence - [ ] Verify hyphened names like `Moko-Suite` produce `pkg_mokosuite` Fixes #635
jmiller added 2 commits 2026-06-18 13:34:26 +00:00
chore(version): pre-release bump to 06.13.03-dev [skip ci] 0a0d7b704d
fix: derive Joomla element name with correct lowercase + type prefix
Branch Policy Check / Verify merge target (pull_request) Successful in 2s
Details
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 10s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 1s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 32s
Details
Generic: Repo Health / Site Health (push) Has been skipped
Details
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 2s
Details
Universal: Secret Scanning / Gitleaks Secret Scan (pull_request) Successful in 41s
Details
PR RC Release / Build RC Release (pull_request) Failing after 34s
Details
Universal: Auto Version Bump / Version Bump (push) Successful in 4s
Details
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Successful in 1m46s
Details
Universal: Build & Release / Promote to RC (pull_request) Has been skipped
Details
Universal: Build & Release / Build & Release Pipeline (pull_request) Has been skipped
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
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
Generic: Repo Health / Report Issues (pull_request) Has been cancelled
Details
Generic: Repo Health / Scripts governance (push) Has been cancelled
Details
Generic: Repo Health / Repository health (push) Has been cancelled
Details
Generic: Repo Health / Report Issues (push) Has been cancelled
Details
73a1320d72 
AutoElementName() was using the manifest Name field verbatim, producing
element names like "pkg_MokoSuiteBackup" instead of "pkg_mokosuitebackup".
Joomla's updater matches by element+type+client_id in #__extensions, so
the case mismatch made updates invisible.

Changes:
- Lowercase name and strip hyphens in AutoElementName()
- Remove incorrect "plg_" prefix for plugins (Joomla plugins have no
  element prefix; the folder column determines the plugin group)

Fixes #635
jmiller closed this pull request 2026-06-18 13:35:08 +00:00
jmiller deleted branch fix 2026-06-18 13:35:09 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
breaking-change
Breaking API or behavior change
 ci-cd
CI/CD pipeline changes
 config
Configuration changes
 dependencies
Dependency updates
 deploy-failure
Deployment failed
 docker
Docker/container changes
 documentation
Documentation changes
 feature: wiki
Wiki system enhancements
 good first issue
Good for newcomers
 health-check
Repo health check result
 help wanted
Extra attention needed
 mokostandards
Related to MokoStandards framework
 push-failure
Git push operation failed
 security
Security vulnerability or hardening
 size/l
200-500 lines changed
 size/m
50-200 lines changed
 size/s
10-50 lines changed
 size/xl
500+ lines changed
 size/xs
< 10 lines changed
 standards-drift
Deviates from MokoStandards
 standards-update
MokoStandards compliance update
 sync-failure
Sync or mirror failed
 tech-debt
Technical debt and TODO/FIXME items
 upstream
upstream
Inherited from upstream Gitea
 work-in-progress
Draft or incomplete work
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) moko-deploy (Moko Deploy Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGIT#636
Delete Branch "fix"

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?