MokoConsulting/moko-platform
1
0
Fork 0
Code Issues 9 Pull Requests Actions 64 Packages Releases 4 Wiki Activity

fix(joomla6): update webservices plugins for Joomla 6 typed event API #164

New Issue
Closed
opened 2026-05-26 18:33:36 +00:00 by jmiller · 4 comments
jmiller commented 2026-05-26 18:33:36 +00:00
Owner
Copy Link
Copy Source

Summary

Joomla 6 changed the event handler system. Plugins using SubscriberInterface now receive a typed event object as the first argument instead of pass-by-reference parameters. This breaks all plg_webservices_* plugins across MokoWaaS repos.

Breaking Change

Old (Joomla 4/5):

public function onBeforeApiRoute(&$router): void

New (Joomla 6):

use Joomla\CMS\Event\Application\BeforeApiRouteEvent;

public function onBeforeApiRoute(BeforeApiRouteEvent $event): void
{
    $router = $event->getRouter();
}

Error

Call to undefined method BeforeApiRouteEvent::createCRUDRoutes()

Scope

  • MokoWaaS (tracked as MokoWaaS#48)
  • Any Joomla repo with webservices plugins
  • May need a drift check / validator to detect old-style event handlers

Additional

Also discovered: api/templates/system/index.php must exist for Joomla 6.1 CMSApplication::isValidTemplate(). Verify in packaging/deployment.

Acceptance Criteria

  • Update MokoWaaSApi.php to use BeforeApiRouteEvent typed parameter
  • Verify api/templates/system/ is included in packages
  • Consider adding a Joomla 6 compat validator to moko-platform
## Summary Joomla 6 changed the event handler system. Plugins using `SubscriberInterface` now receive a typed event object as the first argument instead of pass-by-reference parameters. This breaks all `plg_webservices_*` plugins across MokoWaaS repos. ## Breaking Change Old (Joomla 4/5): ```php public function onBeforeApiRoute(&$router): void ``` New (Joomla 6): ```php use Joomla\CMS\Event\Application\BeforeApiRouteEvent; public function onBeforeApiRoute(BeforeApiRouteEvent $event): void { $router = $event->getRouter(); } ``` ## Error ``` Call to undefined method BeforeApiRouteEvent::createCRUDRoutes() ``` ## Scope - MokoWaaS (tracked as MokoWaaS#48) - Any Joomla repo with webservices plugins - May need a drift check / validator to detect old-style event handlers ## Additional Also discovered: `api/templates/system/index.php` must exist for Joomla 6.1 CMSApplication::isValidTemplate(). Verify in packaging/deployment. ## Acceptance Criteria - [ ] Update MokoWaaSApi.php to use BeforeApiRouteEvent typed parameter - [ ] Verify api/templates/system/ is included in packages - [ ] Consider adding a Joomla 6 compat validator to moko-platform
jmiller added the joomlaphppriority: critical labels 2026-05-26 18:33:36 +00:00
jmiller commented 2026-05-26 18:33:40 +00:00
Author
Owner
Copy Link
Copy Source

Branch created: feature/164-fix-joomla6-update-webservices-plugins-f

git fetch origin
git checkout feature/164-fix-joomla6-update-webservices-plugins-f
Branch created: [`feature/164-fix-joomla6-update-webservices-plugins-f`](https://git.mokoconsulting.tech/MokoConsulting/moko-platform/src/branch/feature/164-fix-joomla6-update-webservices-plugins-f) ```bash git fetch origin git checkout feature/164-fix-joomla6-update-webservices-plugins-f ```
jmiller commented 2026-05-26 18:33:54 +00:00
Author
Owner
Copy Link
Copy Source

Source issue: MokoConsulting/MokoWaaS#48
Hotfixed on live server 2026-05-26. Proper fix pending in source repo.

Source issue: MokoConsulting/MokoWaaS#48 Hotfixed on live server 2026-05-26. Proper fix pending in source repo.
jmiller commented 2026-05-26 18:34:56 +00:00
Author
Owner
Copy Link
Copy Source

Related issues:

**Related issues:** - MokoConsulting/MokoWaaS#48 (source — hotfixed on live) - MokoConsulting/moko-platform#164 (tracking) - MokoConsulting/Template-Generic#32 - MokoConsulting/Template-Joomla#13
jmiller commented 2026-05-26 19:42:06 +00:00
Author
Owner
Copy Link
Copy Source

Proof

The Joomla 6 typed event fix was applied directly in MokoWaaS (commit 3f20ad9, merged as a68e90d). MokoWaaS#48 is closed.

A Joomla 6 compatibility validator for CI would be a separate feature request if needed.

## Proof The Joomla 6 typed event fix was applied directly in MokoWaaS (commit 3f20ad9, merged as a68e90d). MokoWaaS#48 is closed. A Joomla 6 compatibility validator for CI would be a separate feature request if needed.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
automation
Automated processes or scripts
 breaking-change
Breaking API or functionality change
 build
Build system changes
 ci-cd
CI/CD pipeline changes
 config
Configuration file changes
 css
CSS/styling changes
 dependencies
Dependency updates
 deploy-failure
Automated deploy failure tracking
 docker
Docker configuration changes
 documentation
Documentation changes
 dolibarr
Dolibarr module or extension
 generic
Generic project or library
 good first issue
Good for newcomers
 health-check
Repository health check results
 health: excellent
Health score 90-100
 health: fair
Health score 50-69
 health: good
Health score 70-89
 health: poor
Health score below 50
 help wanted
Extra attention needed
 html
HTML template changes
 javascript
JavaScript code changes
 joomla
Joomla extension or component
 mokostandards
MokoStandards compliance
 needs-review
Awaiting code review
 php
PHP code changes
 priority: critical
Critical priority, must be addressed immediately
 priority: high
High priority
 priority: low
Low priority
 priority: medium
Medium priority
 push-failure
File push failure requiring attention
 python
Python code changes
 security
Security-related changes
 size/l
Large change (101-300 lines)
 size/m
Medium change (31-100 lines)
 size/s
Small change (11-30 lines)
 size/xl
Extra large change (301-1000 lines)
 size/xs
Extra small change (1-10 lines)
 size/xxl
Extremely large change (1000+ lines)
 standards-drift
Repository drifted from MokoStandards
 standards-update
MokoStandards sync update
 status: blocked
Blocked by another issue or dependency
 status: in-progress
Currently being worked on
 status: needs-review
Awaiting code review
 status: on-hold
Temporarily on hold
 status: pending
Pending action or decision
 status: wontfix
This will not be worked on
 sync-failure
Bulk sync failure requiring attention
 sync-report
Bulk sync run report
 template-validation-failure
Template workflow validation failure
 tests
Test suite changes
 type: bug
Something isn't working
 type: chore
Maintenance tasks
 type: enhancement
Enhancement to existing feature
 type: feature
New feature or request
 type: refactor
Code refactoring
 type: version
Version-related change
 typescript
TypeScript code changes
 version
Version bump or release
 version-drift
Version mismatch detected
 work-in-progress
Work in progress, not ready for merge
bug
Something is not working
 chore
Maintenance and housekeeping
 documentation
Documentation improvements
 enhancement
Improvement to existing functionality
 feature
New feature or request
 pending: dependency
Blocked by another issue or external dependency
 pending: deployment
Tested and approved, awaiting deployment to production
 pending: design
Needs UI/UX or architecture design before implementation
 pending: documentation
Feature works, needs documentation/wiki update
 pending: feedback
Awaiting feedback or decision from stakeholder
 pending: review
Implementation complete, awaiting code review
 pending: testing
Feature implemented but not yet tested
 priority: critical
Must fix immediately
 priority: high
Should fix soon
 priority: low
Nice to have
 priority: medium
Fix when convenient
 refactor
Code restructuring without behavior change
 roadmap
Planned feature or enhancement tracked on the roadmap
 scope: client
Client-specific work
 scope: dolibarr
Dolibarr modules and customizations
 scope: infrastructure
Server, CI, backups, monitoring
 scope: joomla
Joomla templates and extensions
 scope: waas
MokoWaaS platform
 security
Security vulnerability or hardening
 status: blocked
Waiting on external dependency
 status: duplicate
Duplicate of another issue
 status: in-progress
Being worked on
 status: needs-review
Ready for review
 status: wontfix
Will not be addressed
No labels joomla php priority: critical
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/moko-platform#164
Delete Branch "%!s()"

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?