Jetspeed 2
  1. Jetspeed 2
  2. JS2-977

Remove jsf-demo and jpetstore from demo and installer

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.0
    • Fix Version/s: 2.2.0
    • Component/s: Deployment, Installer
    • Labels:
      None

      Description

      As the JSF and JPetstore portlets no longer work in the 2.2.0 trunk version (bridges are broken), they should not be used right now in jetspeed demo deployment and installer

        Activity

        Hide
        Jeroen Hoffman added a comment -

        Attached a patch that

        • removes jsf and jpetstore dependencies from jetspeed-demo and installer.
        • sets correct dependencies to webcontent and apa apps in installer
        • removes references to jpetstore and jsf portlets in psml pages
        • also renames demo::IFramePortlet to webcontent::IFramePortlet in psmls (sorry, out of scope but fixed it as I saw it)
        Show
        Jeroen Hoffman added a comment - Attached a patch that removes jsf and jpetstore dependencies from jetspeed-demo and installer. sets correct dependencies to webcontent and apa apps in installer removes references to jpetstore and jsf portlets in psml pages also renames demo::IFramePortlet to webcontent::IFramePortlet in psmls (sorry, out of scope but fixed it as I saw it)
        Hide
        Ate Douma added a comment -

        Small correction: the JSF demo still works on the current trunk.
        But this "exciting" demo doesn't really add much value so removing it from the demo deployment and installer is desired indeed.

        Show
        Ate Douma added a comment - Small correction: the JSF demo still works on the current trunk. But this "exciting" demo doesn't really add much value so removing it from the demo deployment and installer is desired indeed.
        Hide
        Ate Douma added a comment -

        Thanks for the patch Jeroen.

        Vivek: can you please review and apply when valid?

        Show
        Ate Douma added a comment - Thanks for the patch Jeroen. Vivek: can you please review and apply when valid?
        Hide
        Vivek Kumar added a comment -

        Patch applied

        Show
        Vivek Kumar added a comment - Patch applied

          People

          • Assignee:
            Vivek Kumar
            Reporter:
            Jeroen Hoffman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development