Skip to Content.
Sympa Menu

grouper-users - RE: [grouper-users] WS replaceAllExisting behavior

Subject: Grouper Users - Open Discussion List

List archive

RE: [grouper-users] WS replaceAllExisting behavior


Chronological Thread 
  • From: Chris Hyzer <>
  • To: Colin Hudler <>, Grouper Users Mailing List <>
  • Subject: RE: [grouper-users] WS replaceAllExisting behavior
  • Date: Fri, 23 Jan 2009 01:34:44 -0500
  • Accept-language: en-US
  • Acceptlanguage: en-US

This is fixed and tested in the GROUPER_1_4 branch of grouper-ws which will
be in the 1.4.1 release next week.

Sorry about the problem.

Regards,
Chris

> -----Original Message-----
> From: Chris Hyzer
> [mailto:]
> Sent: Thursday, January 22, 2009 5:23 PM
> To: Colin Hudler; Grouper Users Mailing List
> Subject: RE: [grouper-users] WS replaceAllExisting behavior
>
> Colin, thanks for the detailed report of the problem it saved time.
>
> Yes, this looks like a bug. My unit tests were checking the return
> code, but didn’t take the extra step of seeing that the members were
> still members. I changed the unit test so it now fails, and I see what
> the issue is... it seems if you add a Subject object to a Set, and
> then see if the subject is in Set, it will always return false since
> this Subject doesn’t override the equals()/hashcode() methods. So I
> will change it so it doesn’t use a set... will be fixed in 1.4.1 which
> will come out soon.
>
> Thanks!
> Chris
>
> > -----Original Message-----
> > From: Colin Hudler
> > [mailto:]
> > Sent: Wednesday, January 21, 2009 3:03 PM
> > To: Grouper Users Mailing List
> > Subject: [grouper-users] WS replaceAllExisting behavior
> >
> > Greetings,
> >
> > My question is about this short grouper session, where I set
> > replaceAllExisting to 'T', and it removes members from my list that
> > were already in the group, even though they are in the PUT xml.
> >
> > Please let me know if that is expected behavior. Please see my
> > attached session, where I hope the problem is made clear.



Archive powered by MHonArc 2.6.16.

Top of Page