On Jan 11, 2013, at 10:01 AM, Sebastien Blanc <scm.blanc@gmail.com> wrote:

Hi All,
In the scope of https://issues.jboss.org/browse/AEROGEAR-803 , we have a method to retrieve an user (based on its id). If the user don't exist it returns null.

There were some suggestions to change that into :
- throwing a runtime exception (seems a bit brutal)

This does seem brutal, unless there is some security issues with not doing this

i think it should be the client dev's responsibility to check if the return value exists.

- or returning an empty user (can be confusing)

What is your opinion about this ?

Seb

ps: the method https://gist.github.com/4510879

_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev