HermanPeeren
commented
9 years ago OK I see, you are right! The properties are added in JTable, by getting the database columns with getFields() (and the getters and setters in the deprecated JObject). Putting them explicitly there doesn't save the roundtrip to the db, so indeed, it is no use to declare those variables explicitly as long as we use JTable.
It is probably more a matter that I'm used to making things as explicit as possible (and in Doctrine ORM db-schemas are generated by the mapping, on one spot for all supported databases, not this reversed order we use in Joomla and Nooku).
An other example of making things explicit: I always put all used references to other classes in a use clause on top (and then don't use the FQN in the code itself); in that way I can easily see what the dependencies are and namespaces can be easily changed if necessary, without going through the rest of the code. In your index.php for instance I would have: use Joomla\Webservices\Service\ConfigurationProvider on top and then registerServiceProvider(new ConfigurationProvider) in the code. Dependencies are the sauce of the spaghetti-code. I also have some concerns about the use of that service container and even put some question marks at some autoloading practices. Convenient, but so was global $mainframe...
wilsonge
fastslack
I haven't tested everything fully but I think this should get the basis of the component working.
Note that the code used to get the component working with the app at best should be described as hacky and it needs a COMPLETE REWORK really. However I think it does work so we can actually run tests and stuff
Note that the webservice table is still created in the CMS however it is UNUSED - it will use the one you need to set up as part of the application.
To install the component simply change the path in
components/admin/webservices.phpof the$applicationPathvariable to point to the root of your webservices app and then zip everything up inside the GIT_ROOT/components directory and install through Joomla as usual