Re: [struts] Dynamic Results

2008-02-15 Thread stanlick
I like this idea *more*! I'll write the JIRA on this plug-in and include a suitable fix for the glaring oversights. On Fri, Feb 15, 2008 at 10:30 AM, Dale Newfield <[EMAIL PROTECTED]> wrote: > [EMAIL PROTECTED] wrote: > > I think if dynamic parameters are supported in the mappings, they should >

Re: [struts] Dynamic Results

2008-02-15 Thread Dale Newfield
Dale Newfield wrote: It sounds like a reasonable set of patches could be made to xwork instead. If parse were made a result attribute (rather than a parameter), then the framework could be modified to conditionally do the parsing before setting values on the result (rather than the result its

Re: [struts] Dynamic Results

2008-02-15 Thread Dale Newfield
[EMAIL PROTECTED] wrote: I think if dynamic parameters are supported in the mappings, they should be supported whether the sample of documentation utilizes them or not. Do you agree that bundled plug-ins should pass a baseline compatibility test? It sounds like a reasonable set of patches coul

Re: [struts] Dynamic Results

2008-02-14 Thread Dave Newton
--- [EMAIL PROTECTED] wrote: > The ChartAction extends ActionSupport. Wouldn't the > action-mapping-OGNL-parsing be looking for height/width getters on the > ChartAction? I think we're all talking past each other here. It's the responsibility of the *result* to parse OGNL in its parameters: the

Re: [struts] Dynamic Results

2008-02-14 Thread stanlick
Hey Dale -- The ChartAction extends ActionSupport. Wouldn't the action-mapping-OGNL-parsing be looking for height/width getters on the ChartAction? Scott On Thu, Feb 14, 2008 at 4:39 PM, Dale Newfield <[EMAIL PROTECTED]> wrote: > [EMAIL PROTECTED] wrote: > > I am puzzled at why S2 is not execu

Re: [struts] Dynamic Results

2008-02-14 Thread Dale Newfield
[EMAIL PROTECTED] wrote: I am puzzled at why S2 is not executing getHeight() on my ChartAction and substituting the height and weight into these params. This is done by StrutsResultSupport. I would guess that whatever class is implementing your chart result does not extend/leverage that capab