ICEfaces
  1. ICEfaces
  2. ICE-8714

Showcase: ace:gMap - Directions (deltaSubmit specific failure)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 3.2.BETA1
    • Fix Version/s: EE-3.2.0.GA, 3.3
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Icefaces3/trunk revision# 31809
    • Affects:
      Compatibility/Configuration

      Description


      1) Load showcase sample application configured with org.icefaces.deltaSubmit = true.
      2) Navigate to ace:gMap - Directions page
      3) Type some new directions into the field and submit
      The map and the direction list are not updated.

        Activity

        Hide
        Mircea Toma added a comment -

        The issue caused by the MenuRenderer who is rendering the option tags directly to the output stream. Because of that DeltaSubmitPhaseListener cannot calculate the previous parameters off the 'select' elements.

        Modified DOMResponseWriter.write() method to try to parse the passed in string since some renderers write directly to the outputstream. Also changed the calculation of previous parameters for select elements to pick the first option in case none of the options are marked as selected thus matching browser's behaviour.

        Show
        Mircea Toma added a comment - The issue caused by the MenuRenderer who is rendering the option tags directly to the output stream. Because of that DeltaSubmitPhaseListener cannot calculate the previous parameters off the 'select' elements. Modified DOMResponseWriter.write() method to try to parse the passed in string since some renderers write directly to the outputstream. Also changed the calculation of previous parameters for select elements to pick the first option in case none of the options are marked as selected thus matching browser's behaviour.
        Hide
        Ken Fyten added a comment -

        It has been determined that adding this much complexity and potential performance hit simple to support the poorly implemented h:selectOneMenu component when in deltaSubmit mode doesn't make much sense.

        Instead, we will document that the h:selectOneMenu component isn't supported in deltaSubmit=true mode, and offer alternatives.

        Show
        Ken Fyten added a comment - It has been determined that adding this much complexity and potential performance hit simple to support the poorly implemented h:selectOneMenu component when in deltaSubmit mode doesn't make much sense. Instead, we will document that the h:selectOneMenu component isn't supported in deltaSubmit=true mode, and offer alternatives.
        Hide
        Mircea Toma added a comment -

        Reverted changes made to DOMresponseWriter.write method, the method will not try to parse the paseed in string anymore.

        Show
        Mircea Toma added a comment - Reverted changes made to DOMresponseWriter.write method, the method will not try to parse the paseed in string anymore.
        Hide
        Ken Fyten added a comment -

        As this issue is a result of none-standard rendering technique employed by the h:selectOneMenu component, and the solution is too complex, inefficient, and fragile to justify given that the issue only affects h:selectOneMenu when used in deltaSubmit=true mode, we will not fix this issue.

        Work-arounds are to use another component such as ace:list or ice:selectOneMenu instead. This has been documented as a Known Issue in the 3.2 release notes.

        We will also not modify the gmap demo at this time to accommodate.

        Show
        Ken Fyten added a comment - As this issue is a result of none-standard rendering technique employed by the h:selectOneMenu component, and the solution is too complex, inefficient, and fragile to justify given that the issue only affects h:selectOneMenu when used in deltaSubmit=true mode, we will not fix this issue. Work-arounds are to use another component such as ace:list or ice:selectOneMenu instead. This has been documented as a Known Issue in the 3.2 release notes. We will also not modify the gmap demo at this time to accommodate.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: