Re: [corona-dev] Bug 166245: Manage Team Members in PCC via Web Services

[Marcin Okraszewski <marcin.okraszewski@xxxxxxxxxxxxxxxx>]

My believe is to have ability to add, remove and modify team members. 
There are two possible ways for this:
1. Current: Go through team repository adapter. It is possible to do 
those operations now.
2. There are methods joinProject() and unjoinProject() in project 
container manager. Not used, but they are there :(

Please correct me, if I'm wrong. They deliver almost the same 
functionality. But there are use cases for both of them:

The first way:
1) Open project container.
2) Find team member repository descriptor.
3) Take team repository adapter.
4) Change, add, remove team members.
5) Close project container.

The second way (what I think might be an intention):
1) Invoke joinProject() / unjoinProject() on the project container 
manager to add / remove team member.

Note - in the second scenario user might add himself, even if he is not 
a team member. I think this is the main difference - you do not have to 
have project container opened, to manage the team. Don't know if we 
really want it. If not, I would just leave the 1st. approach and remove 
second option.

Marcin


O'Flynn, Dennis wrote:
> See the comment below from Marcin re: this issue.
>
> ------- Comment #4 from marcin.okraszewski@xxxxxxxxxxxxx 2007-04-17 
> 04:34 -------
>
> Dennis,
>
> Just one question. What was the intention of this bug? Was it project 
> container
>
> proxy methods or via team member repository WS?
>
> */[O'Flynn, Dennis] Can you propose a use-case (or two) that will 
> define how each would be used? /*
>
> I've noticed that there are methods "joinProject()" and 
> "unjoinProject()". Did
>
> you wanted to have "right-click" a container and select "join 
> project"? Or was
>
> it accessing directly team repository to add a project. Note, that the 
> second
>
> approach requires to have a container opened. So only other team 
> member can add
>
> you. If it would be done through project container manager, then you 
> could do
>
> it yourself, even on closed container...
>
> */[O'Flynn, Dennis] I’m not familiar with these methods. I did a brief 
> search and did not find where these methods are used. These methods 
> probably should be deprecated and removed./*
>
> */ /*
>
> */I think it would be best to continue the user actions of open/close 
> a PC and not switch to join/unjoin since they are not used./*
>
> --
>
> Configure bugzilla e-mail: 
> https://bugs.eclipse.org/bugs/userprefs.cgi?tab=email
>
> ------- You are receiving this mail because: -------
>
> You reported the bug, or are watching the reporter.
>
>
> The contents of this e-mail are intended for the named addressee only. 
> It contains information that may be confidential. Unless you are the 
> named addressee or an authorized designee, you may not copy or use it, 
> or disclose it to anyone else. If you received it in error please 
> notify us immediately and then destroy it.
> ------------------------------------------------------------------------
>
> _______________________________________________
> corona-dev mailing list
> corona-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/corona-dev
>   
The contents of this e-mail are intended for the named addressee only. It contains information that may be confidential. Unless you are the named addressee or an authorized designee, you may not copy or use it, or disclose it to anyone else. If you received it in error please notify us immediately and then destroy it.