struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Security Management" <>
Subject RE: 2.1.8?
Date Tue, 25 Aug 2009 18:06:09 GMT
OK, I'll take a stab at it, and if I get stuck, let the list know.  I'm
trying to learn as I go, so it might take awhile...

>-----Original Message-----
>From: Wes Wannemacher []
>Sent: Tuesday, August 25, 2009 2:01 PM
>To: Struts Developers List
>Subject: Re: 2.1.8?
>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.
>On Tue, Aug 25, 2009 at 1:47 PM, Security
>Management<> 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:
>> For additional commands, e-mail:
>Wes Wannemacher
>Head Engineer, WanTii, Inc.
>Need Training? Struts, Spring, Maven, Tomcat...
>Ask me for a quote!
>To unsubscribe, e-mail:
>For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message