Uploaded image for project: 'Jetspeed 2'
  1. Jetspeed 2
  2. JS2-1106

Support auto-switching to edit_defaults even when the portlet doesn't support EDIT mode.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.0
    • Fix Version/s: 2.2.1
    • Component/s: Container
    • Labels:
      None

      Description

      Currently, auto-switching to edit_defaults mode is working in the following condition (See JS2-634 for detail.)

      a) The following property is set to true in /WEB-INF/conf/jetspeed.properties.

      supported.portletmode.autoswitch.edit_defaults=true

      b) A portlet application has custom portlet mode mapping declaration like the following (in portlet.xml):

      <custom-portlet-mode>
      <description>a Custom Edit_defaults Mode</description>
      <portlet-mode>edit_defaults</portlet-mode>
      </custom-portlet-mode>

      c) A portlet must support EDIT mode and it must have public doEdit() method.
      By the way, this portlet must not support edit_defaults mode explicitly in its portlet definition. If it defines edit_defaults mode support explicitly, it means it doesn't need "auto-switching".

      However, with the above conditions, we cannot support auto-switching to edit_defaults mode correctly if a portlet should not support EDIT mode but it should support edit_defaults mode.
      Therefore, I'd like to change the last condition (c) like this:

      c) A portlet must have a public render helper method for edit mode.
      With Portlet 2.0 GenericPortlet class, the helper method can be an annotated one or the defined method, "doEdit(RenderRequest, RenderResponse)".

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                woon_san Woonsan Ko
                Reporter:
                woon_san Woonsan Ko
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: