Grouper Users - Open Discussion List

[Chris Hyzer <>]

> So I understand, what do you think the problem is between lines 199-200 ?
> ---
> Line 199 is:
> nameAttribute.setValues(GrouperUtil.toList(new String[] { name }));
> I don't quite understand why you're using GrouperUtil.toList( ) instead of
> Java Util classess to construct a list?

Because there is more than one way to do things, and if there is no bug in 
it, or performance problem, or maintainability problem etc, then lets focus 
on working on more important things :)  FYI we have a lot of Grouper and 
Jakarta etc utility methods that are null safe / shortcuts and are used 
throughout the code.

> The variable "attributes" in line 261, INSTEAD OF the variable in line 257 ,
> is the value to be used by Shibboleth.
> I'm not quite sure why you wrote the code like this. 
> But I wonder if you could insert the code for logging before line 261 to
> check if the return value for function resolve() is correct?
> 

Its an anonymous inner class that passes the value to the outer part which 
returns it... looks fine to me.  Do you think you can add log statements 
where you see fit, rebuild, and let us know how it goes?  :)

Thanks,
Chris