Support sluggable generic webservices - Githubissues

resthub / resthub-spring-stack

RESThub Spring stack

http://resthub.org/spring-stack.html

Other

121 stars 66 forks source link

Support sluggable generic webservices #118

Closed sdeleuze closed 12 years ago

sdeleuze commented 12 years ago

Please review carefully this pullrequest, since it introduces some limited but important changes. The goal is to be able to implement sluggable webservices with RESThub.

I provide some unit tests in order to show how to use it.

The 2 mains changes I had to do :

A slug can be updated, so I had to remove the resourceId equality in update() method of controlers. We made here a specific assumption not true in use case with fields other than id as identifier
Since I removed this check, getResourceById() was not used anymore so, I removed it.

Please suggest better solution if you are not OK with these changes.

bclozel commented 12 years ago

I'm OK with this. We're definitely headed the right way (towards DTOs, real REST APIs and Hypermedia).

Merging!