resthub / resthub-spring-stack

RESThub Spring stack
http://resthub.org/spring-stack.html
Other
121 stars 66 forks source link

Fix issue #122 : Add resolver for JPA exceptions to resthub-jpa #124

Closed sdeleuze closed 12 years ago

bmeurant commented 12 years ago

I am not convinced that we must implement this in resthub-jpa artifact and then have to systematically include spring-mvc as dependency of resthub-jpa : what a bout a jar exe project ?

In fact I am convinced of the fact that I disagree :-)

sdeleuze commented 12 years ago

You should notice the true in pom.xml, it is just used for compilation and does not add springmvc dependency.

The only other solution is to add hibernate as an optional dependency to resthub-web-server with the same principle. I prefer the solution proposed in my pull request.

Knowing that, I am sure you will agree, isn't it ? ;-)

bmeurant commented 12 years ago

Ok. I didn't see the optional parameter ... Nevertheless, I think I would prefer adding an optional dependency to jpa/hibernate in resthub-web-server to not break encapsulation principles and layers architectures and dependencies