I could be wrong, but when I first looked at this, I remembered a time when maybe this plugin didn't take OGNL parameters? If that's the case, this is probably just a remnant. You might want to switch around the logic a little bit in your patch so that height/weight can be set as OGNL params, and then you pull the values out as Longs when you set them on the rendered chart... (or however you want to do it).
You can probably check your new patch by adding a getMyHeight and getMyWidth to an action and see if you can use them as OGNL properties to set on the result (i.e. - <param name="height">${myHeight}</param> I can't remember all of the conversion rules, but I'm pretty sure that test was added when the params were made OGNL aware. -Wes On Tue, Aug 25, 2009 at 1:47 PM, Security Management<list-subscripti...@secmgmt.com> wrote: > OK, going through the tests for the jfreechart plugin, I see this: > > result.setHeight("${myHeight}"); > result.setWidth("${myWidth}"); > > ValueStack stack = ActionContext.getContext().getValueStack(); > stack.set("myHeight", 250); > stack.set("myWidth", 150); > > I get that it's testing that the result is picking the values up off of the > value stack. > > However, the methods in the plugin setWidth(Long x) take Longs, so this does > not even compile. > > How is that? > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org > For additional commands, e-mail: dev-h...@struts.apache.org > > -- Wes Wannemacher Head Engineer, WanTii, Inc. Need Training? Struts, Spring, Maven, Tomcat... Ask me for a quote! --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org