DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24701>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24701 GenricMVCAction.java navigation doesn't work Summary: GenricMVCAction.java navigation doesn't work Product: Jetspeed Version: 1.4 Platform: All OS/Version: All Status: NEW Severity: Blocker Priority: Other Component: Portlets AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] The doPerform(...) call in the catch block moved outside the catch block. THE BUG: ======== protected void perform(RunData rundata) throws Exception { Context context = getContext(rundata); if ((context != null) && (rundata.getParameters().getString("action") != null)) { // if context is already defined and Actions defined, events // have already been processed, call doPerform logger.debug("Action detected with action + context"); doPerform(rundata, context); } else { // if context is null, create a new one if (context == null) { logger.debug("Action: building action context"); context = new GenericMVCContext(); rundata.getTemplateInfo().setTemplateContext("VelocityActionContext", context); } try { // process implicit ActionEvent invocation logger.debug("Action: try executing events"); GenericMVCPortlet portlet = (GenericMVCPortlet) context.get("portlet"); if (portlet != null) { // verify this portlet is the one requested by checking the // js_peid request var. If there is no js_peid // do not worry a about verifing. helps with backward compat. if (rundata.getParameters().getString("js_peid") == null || PortletSessionState.isMyRequest(rundata, portlet)) { executeEvents(rundata, context); } else { logger.debug("Action: calling doPerform"); doPerform(rundata, context); } } else { executeEvents(rundata, context); } } catch (NoSuchMethodException e) { // no event selected, process normal context generation logger.debug("Action: calling doPerform"); } doPerform(rundata, context);//<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< } } THE SOLUTION: ============= protected void perform(RunData rundata) throws Exception { Context context = getContext(rundata); if ((context != null) && (rundata.getParameters().getString("action") != null)) { // if context is already defined and Actions defined, events // have already been processed, call doPerform logger.debug("Action detected with action + context"); doPerform(rundata, context); } else { // if context is null, create a new one if (context == null) { logger.debug("Action: building action context"); context = new GenericMVCContext(); rundata.getTemplateInfo().setTemplateContext("VelocityActionContext", context); } try { // process implicit ActionEvent invocation logger.debug("Action: try executing events"); GenericMVCPortlet portlet = (GenericMVCPortlet) context.get("portlet"); if (portlet != null) { // verify this portlet is the one requested by checking the // js_peid request var. If there is no js_peid // do not worry a about verifing. helps with backward compat. if (rundata.getParameters().getString("js_peid") == null || PortletSessionState.isMyRequest(rundata, portlet)) { executeEvents(rundata, context); } else { logger.debug("Action: calling doPerform"); doPerform(rundata, context); } } else { executeEvents(rundata, context); } } catch (NoSuchMethodException e) { // no event selected, process normal context generation logger.debug("Action: calling doPerform"); doPerform(rundata, context); //<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< } } } --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
