MokoConsulting/MokoSuiteOpenGraph
1
0
Fork 0
Code Issues 3 Pull Requests Releases 4 Wiki Activity

No TagsController for admin list publish/unpublish/delete operations #48

New Issue
Closed
opened 2026-06-21 14:54:34 +00:00 by jmiller · 1 comment
jmiller commented 2026-06-21 14:54:34 +00:00
Owner
Copy Link
Copy Source

Problem

The admin list view has toolbar buttons for delete, publish, and unpublish, but no TagsController extends AdminController exists to handle these tasks. Only DisplayController, BatchController, and ImportExportController are registered.

Clicking "Delete" or toggling publish status in the admin list will produce a Joomla task routing error.

Fix

Create source/packages/com_mokoog/src/Controller/TagsController.php extending Joomla\CMS\MVC\Controller\AdminController with:

  • Proper CSRF validation
  • ACL checks for core.delete and core.edit.state
  • getModel() override returning TagModel

Files

  • New: source/packages/com_mokoog/src/Controller/TagsController.php
  • Related: source/packages/com_mokoog/src/View/Tags/HtmlView.php (toolbar setup)
  • Related: source/packages/com_mokoog/tmpl/tags/default.php (list template)
## Problem The admin list view has toolbar buttons for delete, publish, and unpublish, but no `TagsController extends AdminController` exists to handle these tasks. Only `DisplayController`, `BatchController`, and `ImportExportController` are registered. Clicking "Delete" or toggling publish status in the admin list will produce a Joomla task routing error. ## Fix Create `source/packages/com_mokoog/src/Controller/TagsController.php` extending `Joomla\CMS\MVC\Controller\AdminController` with: - Proper CSRF validation - ACL checks for `core.delete` and `core.edit.state` - `getModel()` override returning `TagModel` ## Files - New: `source/packages/com_mokoog/src/Controller/TagsController.php` - Related: `source/packages/com_mokoog/src/View/Tags/HtmlView.php` (toolbar setup) - Related: `source/packages/com_mokoog/tmpl/tags/default.php` (list template)
jmiller added the bugproduction-readiness labels 2026-06-21 14:54:34 +00:00
jmiller commented 2026-06-21 14:54:46 +00:00
Author
Owner
Copy Link
Copy Source

Branch created: feature/48-no-tagscontroller-for-admin-list-publish

git fetch origin
git checkout feature/48-no-tagscontroller-for-admin-list-publish
Branch created: [`feature/48-no-tagscontroller-for-admin-list-publish`](https://git.mokoconsulting.tech/MokoConsulting/MokoSuiteOpenGraph/src/branch/feature/48-no-tagscontroller-for-admin-list-publish) ```bash git fetch origin git checkout feature/48-no-tagscontroller-for-admin-list-publish ```
Sign in to join this conversation.
Labels
Clear labels
bug
Something is broken
 documentation
Documentation improvements
 enhancement
production-readiness
Required before v1.0 release
 security
Security vulnerability or hardening
 testing
Test coverage and QA
No labels bug production-readiness
Priority Medium
Type Feature
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/MokoSuiteOpenGraph#48
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?