uPortal IRC Logs-2011-08-18

Owned by Apereo Infrastructure
Last updated: Aug 18, 2011Version comment
3 min read

[13:17:41 CDT(-0500)] <athena> found some more issues w/ the entity selector with smart ldap
[13:17:55 CDT(-0500)] <athena> at least for this server, the smart ldap groups have commas in the name, because it's using the fully distinguished name
[13:18:11 CDT(-0500)] <athena> and spring webflow thinks the parameter w/ commas is multiple groups
[13:18:15 CDT(-0500)] <athena> not quite sure how to get around that
[13:18:57 CDT(-0500)] <EricDalquist> I wonder if you can change the delimiter that webflow uses
[13:20:02 CDT(-0500)] <athena> yeah, though dunno what we'd use
[13:20:10 CDT(-0500)] <athena> and honestly, it shouldn't be doing that
[13:20:20 CDT(-0500)] <athena> what we really want is for it not to split stuff up
[13:20:25 CDT(-0500)] <EricDalquist> yeah
[13:20:31 CDT(-0500)] <EricDalquist> so on the url you have a param:
[13:20:32 CDT(-0500)] <athena> and only send through multiple values when there are genuinely multiple parameters
[13:20:34 CDT(-0500)] <EricDalquist> foo=a,b,c
[13:20:36 CDT(-0500)] <athena> yep
[13:20:43 CDT(-0500)] <athena> wonder if adding brackets will fix it
[13:20:43 CDT(-0500)] <EricDalquist> and instead of in java seeing "foo" and "a,b,c"
[13:20:48 CDT(-0500)] <athena> foo[]=a,b,c
[13:20:57 CDT(-0500)] <EricDalquist> you get "foo" and ["a", "b", "c"]
[13:21:00 CDT(-0500)] <athena> yep
[13:21:09 CDT(-0500)] <athena> probably because the argument to the method is a list
[13:21:11 CDT(-0500)] <EricDalquist> and it isn't jquery or something else doing it?
[13:21:13 CDT(-0500)] <athena> nope
[13:21:21 CDT(-0500)] <EricDalquist> oh
[13:21:23 CDT(-0500)] <EricDalquist> yeah
[13:21:24 CDT(-0500)] <EricDalquist> hrm
[13:21:31 CDT(-0500)] <athena> but that supports multi-select
[13:21:36 CDT(-0500)] <EricDalquist> thats probably SpEL's fault
[13:21:50 CDT(-0500)] <EricDalquist> brb
[13:21:51 CDT(-0500)] <athena> so sometimes we'll see foo[]=a,b,c&foo[]=d
[13:34:39 CDT(-0500)] <EricDalquist> what does the line in the flow that calls the method look like?
[13:49:10 CDT(-0500)] <athena> <set name="flowScope.groups" value="groupListHelper.getEntityBeans(requestParameters.groups)"/>
[13:49:14 CDT(-0500)] <athena> sorry, missed that (smile)
[13:49:29 CDT(-0500)] <EricDalquist> and getEntityBeans is a String[]?
[13:49:39 CDT(-0500)] <athena> List<String>, i believe
[13:49:44 CDT(-0500)] <athena> yep
[13:50:10 CDT(-0500)] <EricDalquist> Try changing it to String[]
[13:50:15 CDT(-0500)] <EricDalquist> see if that changes anything
[14:05:51 CDT(-0500)] <EricDalquist> athena: http://static.springsource.org/spring/docs/3.0.x/reference/expressions.html
[14:05:51 CDT(-0500)] <EricDalquist> look at section 6.5.3
[14:05:56 CDT(-0500)] <EricDalquist> not sure how best to fix it
[14:06:06 CDT(-0500)] <EricDalquist> you could try making the method param an Object
[14:06:09 CDT(-0500)] <EricDalquist> then set a break point
[14:06:11 CDT(-0500)] <athena> hmm
[14:06:11 CDT(-0500)] <athena> yeah
[14:06:12 CDT(-0500)] <EricDalquist> and see what you get
[14:06:14 CDT(-0500)] <athena> i'll try some stuff
[14:06:20 CDT(-0500)] <EricDalquist> just to see where the split happens
[14:06:23 CDT(-0500)] <athena> migh need a custom binder, dunno
[14:06:39 CDT(-0500)] <EricDalquist> yeah I don't think it is the binder
[14:06:42 CDT(-0500)] <EricDalquist> I think its SpEL
[14:06:52 CDT(-0500)] <athena> ahhh
[14:06:53 CDT(-0500)] <athena> gotcha
[14:07:19 CDT(-0500)] <athena> you know, i wonder if it behaves differently if more than one parameter is submitted
[14:07:21 CDT(-0500)] <athena> will have to try that
[14:08:08 CDT(-0500)] <athena> if it does we could probably check and use a different method if there's only one param
[14:08:18 CDT(-0500)] <EricDalquist> yeah
[14:08:26 CDT(-0500)] <athena> will have to play around with it
[14:08:37 CDT(-0500)] <athena> getting a demo instance configured w/ rc2
[14:08:39 CDT(-0500)] <athena> going pretty well so far
[14:08:50 CDT(-0500)] <athena> have it configured against our ldap for persondir and smart ldap
[14:08:54 CDT(-0500)] <athena> that's all i've really found so far
[14:09:05 CDT(-0500)] <athena> always nice to be able to test it in a more "real" environment
[14:11:14 CDT(-0500)] <EricDalquist> yes