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 Improvement
    • Status: Closed
    • Priority: Minor 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)".

        Issue Links

          Activity

          Woonsan Ko created issue -
          Woonsan Ko made changes -
          Field Original Value New Value
          Link This issue blocks JS2-634 [ JS2-634 ]
          Woonsan Ko made changes -
          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.

          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 public doEdit() method.

          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 public doEdit() method.

          Woonsan Ko made changes -
          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 public doEdit() method.

          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)".
          Woonsan Ko made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ate Douma made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development