> -----Original Message----- > From: Jason Pyeron [mailto:jpye...@pdinc.us] > Sent: Saturday, May 21, 2011 15:10 > To: 'Struts Developers List' > Subject: Making progress on ww-3628 > > It looks like the servleturlrenderer had a change in business > logic, with the introduction of ActionInvocation.getProxy() > > Adding in a line like > > final String method = urlComponent.getMethod() != null ? > urlComponent.getMethod() : ai.getProxy().getMethod(); > > And using the method name in the call to > determineActionURL(..., method, ...) > > Functionally works, but has a new side effect. > > /context/ -> /context/!execute > /context/action -> /context/action!execute > > In the past, if the method was not specified, it would still > not be specified.
Fixed. > > So my question is there a better (or more prefered way) to > track if the method was specified at runtime or was defaulted > due to configuration? > <snip/> > > ActionProxy.isMethodSpecified() > > > > > This would allow a line like: > > final String method = urlComponent.getMethod() != null || > !ai.getProxy().isMethodSpecified() ? urlComponent.getMethod() : > ai.getProxy().getMethod(); > This is the path I took. Patch it assached to the jira ticket [https://issues.apache.org/jira/browse/WW-3628]. BTW what is the purpose for com.opensymphony.xwork2.mock.MockActionProxy? P.s. sandbox access would have made this easier... I had to patch 2.1.8 and HEAD -- -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- - - - Jason Pyeron PD Inc. http://www.pdinc.us - - Principal Consultant 10 West 24th Street #100 - - +1 (443) 269-1555 x333 Baltimore, Maryland 21218 - - - -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- This message is copyright PD Inc, subject to license 20080407P00. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
