Jetspeed 2
  1. Jetspeed 2
  2. JS2-711

Support JSP decorators as well as Velocity

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.2
    • Fix Version/s: 2.1.2
    • Component/s: Decorations/Themes
    • Labels:
      None

      Description

      Support JSP decorators as well as Velocity, will require putting JPT (PowerTool) into JSP request attribute, and dispatching to a JSP instead of Velocity servlet

        Activity

        Hide
        David Sean Taylor added a comment -

        patches from Mohan KR applied
        We now support JSP templates with the tigris decorations only
        If someone wants to convert over the other demos, that would be helpful
        Thanks to Mohan for a big contribution for his first experience with Jetspeed!

        Show
        David Sean Taylor added a comment - patches from Mohan KR applied We now support JSP templates with the tigris decorations only If someone wants to convert over the other demos, that would be helpful Thanks to Mohan for a big contribution for his first experience with Jetspeed!
        Hide
        Ate Douma added a comment -

        We also need to provide jsp based decorator template replacements for:

        • columns/layout-help.vm
        • maximized/layout.vm
        • solo/layout.vm
        • tcolumns/layout.vm
        Show
        Ate Douma added a comment - We also need to provide jsp based decorator template replacements for: columns/layout-help.vm maximized/layout.vm solo/layout.vm tcolumns/layout.vm
        Hide
        Ate Douma added a comment -

        With the latest patch provided by Mohan KR which added JSP support for layout-help, solo and maximized decorators.
        I think this now completes this feature and I'm happy with it to incorporate it in the 2.1.1 release.
        But please note: the JSP decorators are very new, and only available yet for the tigris theme, and should still be considered somewhat experimental.

        What Mohan has provided so far works already very well though (great job Mohan!).

        The JSP decorators can be enabled by modifying (properly commented) settings in:

        • decorators/layout/tigris/decorator.properties
        • decorators/portlet/tigris/decorator.properties
        • WEB-INF/templates/layout/html/columns/layout.properties
        • WEB-INF/templates/layout/html/maximized/layout.properties
        • WEB-INF/templates/layout/html/solo/layout.properties

        Note: you need to change the settings in all of the above properties files, and then restart jetspeed.

        Also note: for the tcolumns template there isn't a JSP replacement (yet), but that isn't much used anyway AFAIK.
        Just to be sure: don't switch your page layout portlet(s) from columns to tcolumns, otherwise your page will be broken.

        Show
        Ate Douma added a comment - With the latest patch provided by Mohan KR which added JSP support for layout-help, solo and maximized decorators. I think this now completes this feature and I'm happy with it to incorporate it in the 2.1.1 release. But please note: the JSP decorators are very new, and only available yet for the tigris theme, and should still be considered somewhat experimental. What Mohan has provided so far works already very well though (great job Mohan!). The JSP decorators can be enabled by modifying (properly commented) settings in: decorators/layout/tigris/decorator.properties decorators/portlet/tigris/decorator.properties WEB-INF/templates/layout/html/columns/layout.properties WEB-INF/templates/layout/html/maximized/layout.properties WEB-INF/templates/layout/html/solo/layout.properties Note: you need to change the settings in all of the above properties files, and then restart jetspeed. Also note: for the tcolumns template there isn't a JSP replacement (yet), but that isn't much used anyway AFAIK. Just to be sure: don't switch your page layout portlet(s) from columns to tcolumns, otherwise your page will be broken.

          People

          • Assignee:
            Ate Douma
            Reporter:
            David Sean Taylor
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development