ICEfaces
  1. ICEfaces
  2. ICE-4425

applyBrowserDOMChanges affecting all forms

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: 1.8.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      All

      Description

      If you have more than one form in a view, and you submit one of the forms, then the applyBrowserDOMChanges algorithm is not just applied to that one form, but to every form in the view. This is incorrect. In particular, the way html forms handle checkboxes is to not include them in the submitted form if they're unchecked. When applyBrowserDOMChanges processes all forms, it falsely identifies checkboxes as unchecked, in the unsubmitted forms.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18808 Mon Apr 27 14:27:45 MDT 2009 mark.collette ICE-4425 : applyBrowserDOMChanges affecting all forms
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/BridgeFacesContext.java
        Mark Collette created issue -
        Mark Collette made changes -
        Field Original Value New Value
        Assignee Mark Collette [ mark.collette ]
        Arran Mccullough made changes -
        Salesforce Case [50070000008L7vx]
        Hide
        Mark Collette added a comment -

        Now applyBrowserDOMChanges finds the submitted form, and only processes the input fields within it.

        Subversion 18808
        icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java

        Show
        Mark Collette added a comment - Now applyBrowserDOMChanges finds the submitted form, and only processes the input fields within it. Subversion 18808 icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java
        Mark Collette made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 1.8.1 [ 10170 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: