Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-1776

Changing template renderers in order to use include instead of forward

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 3.5 RC1
    • 3.1 M3
    • core
    • None

      The jsp template renderer actually uses a forward in order to render jsps.
      This is the primary problem for which the magnolia filter can't be mapped with "forward" dispatcher in web.xml and could cause other problems when integrating magnolia with other MVC frameworks.

      We will change it in 3.1 in order to include jsps instead of forwaring to them.

      The simple change is as following:

      RequestDispatcher rd = request.getRequestDispatcher(requestReceiver);

      • rd.forward(request, response);
        + rd.include(request, response);

      Note that this could require updating any custom filter mapping (not the magnolia one) you may have mapped to *.jsp only, when upgrading a custom application based on a previous version of magnolia.

        Acceptance criteria

              fgiust Fabrizio Giustina
              fgiust Fabrizio Giustina
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Task DoD