Hey Dan,
so our positions arn't that far as our discussion in our PR's :-) You
are simply concerned about the work and our resources, isn't it?
I thought "why having a resources-rest-bundle and just have GET only" I
mean REST isn't complete by fullfill all CRUD methods, but i really love
the idea of resources as they a that near to the way REST thinks
resources (location, state,..), but REST without any state transfer does
not sound complete. You proposed to support the following functions on a
resource:
- move
- delete
- rename
- get
for the beginning. And yea i can live with that, as it is the beginning
of a state transfer. Move and Rename would change the path and "local
name" of a resource and the underlying payload. The only thing we should
do is to speak about it, means to explain why a REST api on resources
wouldn't change the payload, maybe it would just propose other urls to
do so.
But to do the minimal we should open a minimal door in the
RepositoryInterface of the resource component, means completely implment
the EditInterface. What to use and how to use it would be a discussion
when using that door from the resource-rest-bundle.
To complete the REST picture i would propose to have a API, maybe by the
help of SyliusRestBundle (+ for Deserialisation, QueryBuilder), to work
on content documents. Maybe by example in ContentBundle only. David and
I are on a good way to support method aware routing for our dynamic
router. So i do not think the use the static router of Sylius (maybe in
a chain). So i would propose a enhancer or equal to fill route
parameters with the values Sylius would need them.
Why content-bundle? Yea i really love the idea of the api endpoint based
on the routes of a content document. That would make frontend-editing
more easy as the routing is easy and the client application can think
the way a user would go through the HTML frontend.
greets max