Rafal Krzewski <[EMAIL PROTECTED]> writes:

> Java Apache CVS Development wrote:
> > 
> >   User: jonbolt
> >   Date: 01/01/21 19:00:56
> > 
> >   Modified:    src/java/org/apache/turbine/om/security TurbineUser.java
> >                         User.java
> >   Log:
> >   ok, i think that i'm happier now with this definition. rafal, is this cool with 
>you?
> > 
> 
> Is there any reason not to have setConfirmed(boolean) to accompany
> boolean isConfirmed()?
> It can be done just fine with setConfirmed(String) but
> setConfirmed(true) just looks
> better :-).

Is there ever a case where we actually want to unconfirm someone?

It seems like the usual case will just be confirmation.  If so, perhaps
this method should be overloaded to pass a positive confirmation value
when no parameters are used.
-- 

Daniel Rall <[EMAIL PROTECTED]>


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to