Feature Requests item #1438745, was opened at 2006-02-25 20:16
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684978&aid=1438745&group_id=119783

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 5
Submitted By: Gili Tzabari (cowwoc)
Assigned to: Nobody/Anonymous (nobody)
Summary: Replace class Roles with Java5 constructs

Initial Comment:
Since Wicket plans to migrate to Java5 (and in fact
most of the roles package uses Java5 code, doesn't it?)
I suggest dumping
wicket.authorization.strategies.role.Roles in favor of:

1) enum Roles (defined by the user)

For example:

enum Roles
{
  USER,
  ADMIN
}

2) Set<Roles> to replace Wicket's Roles class

Set already defines contains() and containsAll() which
fully cover all the functionality provided by the
Wicket Roles class -- but in a type-safe manner.

I would recommend simply showing an example of this in
the wicket-auth-roles-examples but not having any
formal code in wicket-auth-roles. I think that Wicket
should not predefine roles such as USER, ADMIN as this
is very subjective and belongs completely in the user
code. For example, I have:

enum Roles
{
  GUEST,
  BASIC_USER,
  PREMIUM_USER,
  ADMINISTRATOR
}

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684978&aid=1438745&group_id=119783


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to