ICEfaces
  1. ICEfaces
  2. ICE-5941

Remove reliance on resetHiddenFields() in compat components

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha3
    • Fix Version/s: 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      compat

      Description

      Certain components create hidden fields, either via server rendering, or via javascript DOM manipulation, which have a name but no id. These hidden input fields are special, intended for sending one-off messages back to the server. They rely on resetHiddenFields() clearing the special hidden input field values, so that the one-off messages are not re-sent in subsequent form submits.

      It should be possible to find all of these, by searching for "hidden" in the source code, so that they can be enumerated, and hopefully removed, thus removing the need for resetHiddenFields() itself. Likely, uses can be replaced with parameters to iceSubmit() and iceSubmitPartial(). An alternate strategy is depicted in ICE-5796, the parent task, for removing resetHiddenFields() itself.
      1. screenshot-01.png
        84 kB
      2. screenshot-02.png
        83 kB
      3. screenshot-03.png
        181 kB
      4. screenshot-04.png
        188 kB
      5. screenshot-05.png
        274 kB

        Issue Links

          Activity

            People

            • Assignee:
              yip.ng
              Reporter:
              Mark Collette
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: