Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-3291

Update sitemesh-plugin to use version 2.4.2. (CLONE, CATCHALL).

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.1.8
    • Fix Version/s: 2.2.1
    • Component/s: Plugin - SiteMesh
    • Labels:
      None
    • Environment:

      JDK 1.5+, Struts 2.1.6+ & SiteMesh 2.4+

    • Flags:
      Patch

      Description

      Catch-all bug for the following bugs: WW-3116 (marked as resolved, not fixed), WW-1128 (should now work on 2.4 and lower), WW-3275 (very good description),

      Detailed description of this issue can be found in the following discussion thread.
      http://www.nabble.com/Struts-2.1.8-%2B-SiteMesh-2.4.2-%2B-FreeMarkerPageFilter-Broken-td25721587.html

      I am attaching a patch that updates the struts2-sitemesh plugin to use the more current SiteMeshPageFilter.java class. The old code was deprecated and deleted from the sitemesh with the 2.4.2 release!

      The new files are as follows. I am testing the freemarker extensively, and will test the velocity a bit later. Please test both and let me know if there are any problems.

      FreeMarkerMapper2DecoratorSelector.java OldDecorator2NewStrutsDecorator.java VelocityMapper2DecoratorSelector.java
      FreeMarkerPageFilter.java OldDecorator2NewStrutsFreemarkerDecorator.java VelocityPageFilter.java
      NoneDecoratorMapper.java OldDecorator2NewStrutsVelocityDecorator.java

        Attachments

        1. WW-3291.diff
          41 kB
          Christian Wolfgang Stone
        2. WW-3291+WW-3296.diff
          103 kB
          Musachy Barroso

          Issue Links

            Activity

              People

              • Assignee:
                lukaszlenart Lukasz Lenart
                Reporter:
                xtianstone Christian Wolfgang Stone
              • Votes:
                2 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: