Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #602 (closed)

Opened 12 years ago

Closed 7 years ago

Pojos Interface getUserDetails method

Reported by: jburel Owned by: jburel
Priority: minor Milestone: Unscheduled
Component: Services Version: 3.0-Beta1
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

Currently the method getUserDetails from IPojos service
returns an Experimenter Data object, user.
user.getGroups().getExperimenters() returns null.

To support collaborative work, user.getGroups().getExperimenters() should
return a collection of Experimenter Data objects.

Note that for those Experimenter Data objects, the method object.getGroups().getExperimenters() can return null. I am not even sure that object.getGroups() needs to return a non-null collection.

Attachments (1)

pojos.patch (742 bytes) - added by jmoore 12 years ago.
Initial changes to support this.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by jburel

the method user.getGroups() should probably only return the real groups
i.e. remove from the list the group 'user' and 'default'

comment:2 Changed 12 years ago by jburel

After discussion with Chris, I realised that I misunderstood the requirement.
So this ticket doesn't need to be considered for now.

comment:3 Changed 12 years ago by jmoore

  • Milestone changed from 3.0-Beta2 to Future
  • Priority changed from major to minor
  • Version changed from 3.0-M3 to 3.0-Beta1

Changed 12 years ago by jmoore

Initial changes to support this.

comment:4 Changed 11 years ago by jmoore

  • Owner changed from jmoore to jburel

And this ticket, Jean-Marie?

comment:5 Changed 11 years ago by jburel

I will have to rewrite some code to test that ticket.
I think we should remove the method from IPojos and only handle user via IAdmin.
When the method was implemented, we did not have the IAdmin service.
for a basic user, user.getGroups() method still returns the default, system, user groups.

comment:6 Changed 11 years ago by jmoore

Ok. That sounds like a good plan. Should I remove in 4.0? or wait later in the 4.x series?

comment:7 Changed 11 years ago by jburel

It can be removed now. I have modified the code in insight already

comment:8 Changed 7 years ago by jburel

  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.80585 sec.)

We're Hiring!