[aerogear-dev] Question about retrieving an AeroGearUser ?

Bruno Oliveira bruno at abstractj.org
Mon Jan 14 03:09:16 EST 2013


Just go with RuntimeException and we change it if necessary. 


-- 
"The measure of a man is what he does with power" - Plato
-
@abstractj
-
Volenti Nihil Difficile



On Monday, January 14, 2013 at 5:45 AM, Sebastien Blanc wrote:

> 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 at qmx.me (mailto:qmx at qmx.me)> wrote:
> > 
> > On 11/01/2013, at 13:01, Sebastien Blanc <scm.blanc at gmail.com (mailto:scm.blanc at 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 at lists.jboss.org (mailto:aerogear-dev at lists.jboss.org)
> > > https://lists.jboss.org/mailman/listinfo/aerogear-dev
> > 
> > 
> > _______________________________________________
> > aerogear-dev mailing list
> > aerogear-dev at lists.jboss.org (mailto:aerogear-dev at lists.jboss.org)
> > https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org (mailto:aerogear-dev at lists.jboss.org)
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> 


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130114/6432f81c/attachment.html 


More information about the aerogear-dev mailing list