Closed wilsonge closed 9 years ago
I think this is ready to merge, but before I do it, can you restore the plugin code? For the moment we still need to keep the plugin working otherwise the admin component doesn't work. We can remove it again at a later stage.
Thanks @wilsonge.
What will be the workflow now?
I mean, for the tests before I did:
Now that is an app what would be the workflow for the tests?
This is an initial aproach to new MVC administrator component for webservices. Any comments very welcome. I think i ll need some help integrating the layouts to new MVC.
https://github.com/fastslack/webservices/tree/newmvc/component/admin
It looks like a decent start - I'm assuming a lot of the display controller comes from Patchtester. One of your biggest issues right now is that the database connection needs to come from the one defined in the config of the webservices application and not from the CMS (https://github.com/fastslack/webservices/blob/newmvc/component/admin/Webservices/Controller/DisplayController.php#L115).
That was one of the reasons I had to hack things up in the existing component (https://github.com/joomla-projects/webservices/blob/develop/component/admin/models/webservice.php#L86-L97)
Nice! thanks for the information. What do you think is the best way to replace the database connection configuration?
The main issue that i'm having right now is that i have not any model class for forms like JModelForm. Do you know any example using forms and new mvc?
I'm pretty sure there aren't any - largely because the frameworks's form package just never really worked. What exactly isn't working - I think the code in the JModelForm package for dealing with forms is pretty ok
Here a new PR for new MVC move, any comments welcome: https://github.com/joomla-projects/webservices/pull/22
The problem with JModelForm is the getState compatibility that cannot allow to extend it. That class still depending to legacy.
This was a continuation of the work done in #13
This improves the abstraction between the soap and HAL classes by making a common
Joomla\Webservices\Webservices
class and attempts to try and break the integrations with Joomla out into theJoomla\Webservices\Integrations
namespace so we can make future compatibility with Joomla 4 "simpler"Large Item Todos:
JApplicationCms::isAdmin
or theJPATH_SITE
/JPATH_ADMIN
constantsSpecific Todos: