jon * wrote:
> 

> org.apache.turbine.util.TurbineUser.java
> org.apache.turbine.util.TurbineUserPeer.java
> org.apache.turbine.util.User.java
> org.apache.turbine.util.access.UserFactory.java
> org.apache.turbine.util.access.VisitorRolePeer.java
> 
> I propose that we make a new package in the org.apache.turbine.util package
> and call it "user" or "visitor" (please vote) and then place the above files
> into that package.

 +1 on org.apache.turbine.util.visitor (matches DB table)

> In fact, after thinking about this further, I think that we should rename
> the "access" package to "security" since that is more of what the files
> (roles/permissions/ACL/logonfailedexception) in there are providing.

 +1 

While we're at it, how about moving the SecurityCheck class from the
util package to the security package?

> Now, even more thinking, I propose to rename "LogonFailedException" to be
> "LoginFailedException" to be more consistent with the "Login" screen and
> action names.

 +1 makes sense.

Dammit, when am I going to get the opportunity to -1 something from you
gurus!? 

dave


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Problems?:           [EMAIL PROTECTED]

Reply via email to