MokoConsulting/MokoSuiteCross
1
0
Fork 0
Code Issues 2 Pull Requests Releases 5 Wiki Activity

fix: Joomla 6 event type compatibility (hotfix) #208

Merged
jmiller merged 1 commits from fix/joomla6-event-hotfix into main 2026-06-28 18:48:25 +00:00
Conversation 0 Commits 1 Files Changed 1
+2 -2
jmiller commented 2026-06-28 18:48:12 +00:00
Owner
Copy Link
Copy Source

Summary

Hotfix cherry-picked from #207 (already merged to dev).

  • Remove strict type hints on onContentAfterSave() and onContentChangeState() in the content plugin
  • Joomla 6 dispatches Model\AfterSaveEvent instead of Content\AfterSaveEvent, causing a fatal TypeError on article save
  • Untyped $event parameters accept both Joomla 5 and 6 event objects

Test plan

  • Save an article on Joomla 6 -- no fatal error
  • Verify cross-posting triggers on article publish
## Summary Hotfix cherry-picked from #207 (already merged to dev). - Remove strict type hints on `onContentAfterSave()` and `onContentChangeState()` in the content plugin - Joomla 6 dispatches `Model\AfterSaveEvent` instead of `Content\AfterSaveEvent`, causing a fatal TypeError on article save - Untyped `$event` parameters accept both Joomla 5 and 6 event objects ## Test plan - [ ] Save an article on Joomla 6 -- no fatal error - [ ] Verify cross-posting triggers on article publish
jmiller added 1 commit 2026-06-28 18:48:12 +00:00
fix: remove Joomla 5-only event type hints for Joomla 6 compatibility
Universal: PR Check / Branch Policy (pull_request) Failing after 2s
Details
Joomla: Extension CI / Release Readiness Check (pull_request) Failing after 5s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 5s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 6s
Details
Generic: Repo Health / Access control (pull_request) Successful in 1s
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Branch Cleanup / Delete merged branch (pull_request) Failing after 2s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 19s
Details
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Failing after 5s
Details
Universal: Build & Release / Promote to RC (pull_request) Has been skipped
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 42s
Details
Universal: Build & Release / Build & Release Pipeline (pull_request) Successful in 18s
Details
Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Successful in 5m40s
Details
Joomla: Extension CI / Lint & Validate (pull_request) Failing after 5m55s
Details
Generic: Project CI / Tests (pull_request) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.2) (pull_request) Has been cancelled
Details
Joomla: Extension CI / Tests (PHP 8.3) (pull_request) Has been cancelled
Details
Joomla: Extension CI / PHPStan Analysis (pull_request) Has been cancelled
Details
Joomla: Extension CI / Build RC Pre-Release (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: Scripts Governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Repository Health (pull_request) Has been cancelled
Details
18e84fbcfe 
Joomla 6 dispatches Model\AfterSaveEvent instead of Content\AfterSaveEvent.
Remove strict type hints on onContentAfterSave() and onContentChangeState()
to accept both Joomla 5 and 6 event objects.

Authored-by: Moko Consulting
jmiller merged commit fceb3d5bf5 into main 2026-06-28 18:48:25 +00:00
jmiller deleted branch fix/joomla6-event-hotfix 2026-06-28 18:48:25 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Priority -
Type -
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/MokoSuiteCross#208
Delete Branch "fix/joomla6-event-hotfix"

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?