[ https://issues.apache.org/struts/browse/WW-2871?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44956#action_44956 ]
Nicolas Raynaud commented on WW-2871: ------------------------------------- Ok, I will check as soon as possible. To be clear, I made a mistake on the issue version field, I was actually using an older one. For now I have to convert my project, it seems, I will report after that. > avoid fetching twice the model in ModelDrivenInterceptor > --------------------------------------------------------- > > Key: WW-2871 > URL: https://issues.apache.org/struts/browse/WW-2871 > Project: Struts 2 > Issue Type: Improvement > Components: Core Interceptors > Affects Versions: 2.1.2 > Reporter: Nicolas Raynaud > Priority: Minor > > the ModelDrivenInterceptor fetche twice the model by default, this is really > anoying when we back the getModel on a database action. > here is the code I propose : > public class ModelDrivenInterceptor extends AbstractInterceptor { > public String intercept(final ActionInvocation invocation) throws > Exception { > final Object action = invocation.getAction(); > if (action instanceof ModelDriven) { > final ModelDriven<?> modelDriven = (ModelDriven<?>) action; > //avoid fetching the model twice. > final Object model = modelDriven.getModel(); > if (model != null) > invocation.getStack().push(model); > } > return invocation.invoke(); > } > } -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.