-
Notifications
You must be signed in to change notification settings - Fork 241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed #018337: REST index missing module repository activation #72
base: master
Are you sure you want to change the base?
Conversation
On 2nd thought, would it work for you @nfrp, if you included the module list setup directly in your controller, as to not enforce the coupling between |
I think the eZModule activation is needed since it appears to be quite common (I had this issue a couple of times on custom REST controllers and the problem is quite confusing...) |
@oms @lolautruche i also believe that making content creation from REST transparent should help here. This patch makes it transparent. The patch itself is maybe not the best way to do it, but on the principle i think the feature is important. |
After some though I think I would vote -1 myself. rest is ment to be used with public api eventually, so when it is this patch will be removed thus breaking all code that relies on it, better to break early during development then break during upgrade possible on prod. |
Now that we moved ahead on creating the PHP API, was there any move on the subject ? In other words, can we make available a temporary solution for people to use the REST api in W mode ? Cheers guys, |
See comment above, if people want to use the old "api" then they it is safer that they do as much old api init code in their own code as possible as that init code will not be part of rest api 2.0 witch uses public api. |
Ok for the low-quality workaround. I would recommend updating the various tutorials with this instruction. It cost me some debug time to figure-out what went wrong, we should help other developers not waste this time. The old API will still be used for some time until the brand new PHP + REST API is released & stable, and people will want to make mobile applications in the meantime. |
No description provided.