Skip to content
● Critical PHP security fixes are out: update PHP on your Grav sites. What to do →

Plugin Integration with the admin plugin

Started by Steven Spungin 9 years ago · 0 replies · 907 views
9 years ago

I recently spent some time writing plugins that integrated with grav-core and the admin plugin. I found it really frustrating that the admin plugin hijacked many services and deviated from the general architecture. In particular:

  1. Pages under plugin/admin/pages do not get put into folders like pages in user/pages

  2. Additional (generic) events are fired by admin and seem to be better located in core. (such as onPageNotFound and onAdminTwigTemplatePaths)

  3. Multiple forms defined front-matter pages under /admin/pages are not recognized.

So I fixed or worked around these issues, and now raise this design question up for discussion:

There are circular dependencies between admin and grav core. How does everyone feel about the dependency issue? Should it be refactored?

If I spend time migrating my fixes for a PR, should they be in admin, or grav-core?

Suggested topics

Topic Participants Replies Views Activity
Plugins · by DrPen, 2 days ago
2 79 56 minutes ago
Plugins · by renedebaaij, 1 week ago
1 134 3 days ago
Plugins · by Thomas, 6 days ago
1 144 5 days ago
Plugins · by complanar, 4 weeks ago
6 471 3 weeks ago
Plugins · by BenLaKnet, 3 weeks ago
1 294 3 weeks ago