ICEfaces
  1. ICEfaces
  2. ICE-5853

Glimmer Regression: ICE-2283 failing with Server Internal Error w/ deltaSubmit enabled

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta1
    • Fix Version/s: 2.0-Beta2, 2.0.0
    • Component/s: None
    • Labels:
      None
    • Environment:
      Tomcat6 & FF3.6

      Description

      Was working fine on Glimmer revision# 21810
      Broken on Glimmer revision# 21816

      Test application available at 'repo\qa\trunk\Regression-Icefaces2\Nightly\ICE-2283'
      1)Load the application
      2)Select some check box
      3)Deselect some check box
      4)Server Internal Error popup is seen when a checkbox is deselected

        Issue Links

          Activity

          Hide
          Deryk Sinotte added a comment -

          This is reproducible with deltaSubmit by itself. I don't think compressDOM is relevant.

          There are a couple of situations where I can add checks for parameters where the String array is not null but is empty (length == 0). However, this only handles the ArrayIndexOutOfBoundsExceptions and causes the example to fail anyway because it looks like it's losing some state.

          Show
          Deryk Sinotte added a comment - This is reproducible with deltaSubmit by itself. I don't think compressDOM is relevant. There are a couple of situations where I can add checks for parameters where the String array is not null but is empty (length == 0). However, this only handles the ArrayIndexOutOfBoundsExceptions and causes the example to fail anyway because it looks like it's losing some state.
          Hide
          Deryk Sinotte added a comment -

          Need to get deltaSubmit working in compat mode for this case to pass when deltaSubmit is on.

          Show
          Deryk Sinotte added a comment - Need to get deltaSubmit working in compat mode for this case to pass when deltaSubmit is on.
          Hide
          Deryk Sinotte added a comment -

          This regression failure is blocked by the more general issue of deltaSubmit in compatibility mode.

          Show
          Deryk Sinotte added a comment - This regression failure is blocked by the more general issue of deltaSubmit in compatibility mode.
          Hide
          Mircea Toma added a comment -

          Remove entry from previous parameter map when the value arrays is empty. Avoid removing submitting element corresponding parameter name from the list of parameters that don't participate in the parameter diffing since iceSubmit/iceSubmitPartial send it twice.
          Delay the parameters calculation on page load to make sure all the form elements are captured during calculation.

          Show
          Mircea Toma added a comment - Remove entry from previous parameter map when the value arrays is empty. Avoid removing submitting element corresponding parameter name from the list of parameters that don't participate in the parameter diffing since iceSubmit/iceSubmitPartial send it twice. Delay the parameters calculation on page load to make sure all the form elements are captured during calculation.
          Hide
          Mircea Toma added a comment -

          Factor out initial parameter calculation into a separate function, ice.calculateInitalParameters(). Renderers belonging to 'javax.faces.Form' family are decorated, a 'script' element is rendered right before the 'form' end tag to make sure ice.calculateInitalParameters executes correctly.

          Show
          Mircea Toma added a comment - Factor out initial parameter calculation into a separate function, ice.calculateInitalParameters(). Renderers belonging to 'javax.faces.Form' family are decorated, a 'script' element is rendered right before the 'form' end tag to make sure ice.calculateInitalParameters executes correctly.

            People

            • Assignee:
              Mircea Toma
              Reporter:
              Mandeep Hayher
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: