Ok,
We have a majority for the Exception, so go! go! go! for the runtime Exception , shall I create a new Exception in ag-security ? Suggestions for the name ?
Seb


On Fri, Jan 11, 2013 at 5:57 PM, Douglas Campos <qmx@qmx.me> wrote:

On 11/01/2013, at 13:01, 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)
+1
> - or returning an empty user (can be confusing)
-1
>
> 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


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