[Yanel-dev] Re: Access Control User Interface

[Josias Thöny]

Hi Paloma,

On Mon, 2007-01-29 at 09:51 +0100, Paloma Gomez wrote:
> Hi Josias,
> 
> I have been assigned the creation of the access control user interface.
> Michi suggested that I implement a YanelUserResource.java that would be
> responsible for creating, editing and deleting users.

I have the feeling that we lack a concept for cms use-cases and user
interface. We have a lot of usecases like:
- manage users
- create new documents
- ...
These usecases could share some common functionality, like
form-/parameter handling or gui generation (just like the
usecase-framework in Lenya). I wonder if we could have something like a
generic UsecaseResource for that. But for the moment it may be ok to
just create a resource for user management.


> 
> We have been discussing if all the implementation should be done within
> YanelUserResource.java or I should split it so the user resource is
> responsible for the interaction with the user and other classes are
> actually responsible for creating/modifying/deleting users.

+1 for splitting.

> 
> I suggested that I add a UserManager.java and a User.java class to the
> security package to perform these tasks (it would be a schema similar to
> Lenya's). Michi answered we need to think about it and suggested that I
> ask you for your opinion.

The proposed API from the bugzilla entry looks good to me.
The question is how to merge that with the current IdentityManager.
I think that the Identity is some kind of abstraction of the following
items:
- user
- machine
- ip range
- world
So maybe we could keep Identity.java as a super-class or an interface.
But I'm not sure which methods to put into Identity.java.

Further the IdentityManager still could be the entry point for yanel.
Maybe we could add the methods:
getUserManager()
getGroupManager()
or would it be easier to forget about UserManager/GroupManager and put
all methods to manage users/groups into IdentityManager?

WDYT?

josias

> 
> You can read the complete discussion at
> 
> http://bugzilla.wyona.com/cgi-bin/bugzilla/show_bug.cgi?id=5116
> 
> where there is also a preliminar version that works partially.
> 
> Thanks,
> 
> Paloma
> 
> 
> 
>