portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 24701] New: - GenricMVCAction.java navigation doesn't work
Date Fri, 14 Nov 2003 09:52:54 GMT
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: jetspeed-dev@jakarta.apache.org
        ReportedBy: A.Herz@FreeGroup.de


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: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


Mime
View raw message