XOOPS / XoopsCore

Core Framework for next version of XOOPS CMS: 2.6.0
https://xoops.org
138 stars 81 forks source link

Controller - proof of concept #543

Closed alain91 closed 5 years ago

alain91 commented 7 years ago

As I look for notifications services, I want to show a proposal to implement Controller in Xoops

The aim is to allow generic behaviour and to make easy to add functionalities like container/service management

Please, have a look

BTW, I modified theme.tpl with missing css files

alain91 commented 7 years ago

There could be another solution : add the new features like container manager into Xoops class

geekwright commented 7 years ago

We definitely will be needing our /index.php endpoint to take on an expanded role. I've tried to look over your suggestion, but have been very pressed for time. I like the idea, and am looking forward to discussing it in more detail soon.

A note on the theme template change -- something else is wrong. The "missing" css files should be supplied as managed assets, largely directed by the theme's theme_onload.php script. Changing the template in that way is not correct.