ICEfaces
  1. ICEfaces
  2. ICE-6470

ice:commandButton Double Post with SingleSubmit

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0-EE-Beta1, 2.0.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      JSF 2.1 (Composite or Enterprise Components)

      Description

      After adding icecore:singleSubmit tag to the composite-component-showcase, I noticed clicking on ice:commandButtons in the compat-composite-component source resulted in two posts to the server.

      The first post @this, @all, the second post @all @all. This does not happen with ice:commandLink which has one @all @all post.

        Issue Links

          Activity

          Brad Kroeger created issue -
          Brad Kroeger made changes -
          Field Original Value New Value
          Assignee Ken Fyten [ ken.fyten ]
          Hide
          Brad Kroeger added a comment -

          I am also seeing the double post with h:commandButton

          Show
          Brad Kroeger added a comment - I am also seeing the double post with h:commandButton
          Brad Kroeger made changes -
          Link This issue blocks IPCK-202 [ IPCK-202 ]
          Carlo Guglielmin made changes -
          Link This issue blocks IPCK-206 [ IPCK-206 ]
          Carlo Guglielmin made changes -
          Link This issue blocks IPCK-208 [ IPCK-208 ]
          Carlo Guglielmin made changes -
          Comment [ The double POST with singleSubmit is also seen with the dualList, or more specifically ice:selectManyListbox. Ref http://jira.icefaces.org/browse/IPCK-202 ]
          Hide
          Carlo Guglielmin added a comment -

          Double POST with singleSubmit also present in IPCK-202, IPCK-206, and IPCK-208.

          Show
          Carlo Guglielmin added a comment - Double POST with singleSubmit also present in IPCK-202 , IPCK-206 , and IPCK-208 .
          Ted Goddard made changes -
          Salesforce Case []
          Fix Version/s 2.0.1 [ 10255 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P1
          Assignee Ken Fyten [ ken.fyten ] Ted Goddard [ ted.goddard ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 2.0.0-EE-Beta1 [ 10250 ]
          Hide
          Ted Goddard added a comment -

          Further investigation indicates that the strange browser behavior is due to the interaction of the UI blocker with extra singleSubmit calls. The UI blocker is behaving correctly, but if it is incorrectly triggered by singleSubmit incorrectly handling a blur event, then the subsequent onclick will be discarded.

          Both "image" and "select-multiple" were allowing singleSubmit to be triggered via their onblur events. The candidate fix is to discard blur events from these elements.

          Show
          Ted Goddard added a comment - Further investigation indicates that the strange browser behavior is due to the interaction of the UI blocker with extra singleSubmit calls. The UI blocker is behaving correctly, but if it is incorrectly triggered by singleSubmit incorrectly handling a blur event, then the subsequent onclick will be discarded. Both "image" and "select-multiple" were allowing singleSubmit to be triggered via their onblur events. The candidate fix is to discard blur events from these elements.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #23867 Tue Feb 01 10:29:03 MST 2011 ted.goddard discarding image and select-many onblur for singleSubmit (ICE-6470)
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/application.js
          Hide
          Ted Goddard added a comment - - edited

          Fix is checked in. Assigning to Carlo for verification from the ICEfaces 2 trunk. Please also verify the related JIRAs.

          Show
          Ted Goddard added a comment - - edited Fix is checked in. Assigning to Carlo for verification from the ICEfaces 2 trunk. Please also verify the related JIRAs.
          Ted Goddard made changes -
          Assignee Ted Goddard [ ted.goddard ] Carlo Guglielmin [ carlo.guglielmin ]
          Hide
          Carlo Guglielmin added a comment -

          Rebuilding and testing with the fix against the 3 related bugs shows all duplicate POSTS have been eliminated. Looks fixed to me, thanks Ted.

          Show
          Carlo Guglielmin added a comment - Rebuilding and testing with the fix against the 3 related bugs shows all duplicate POSTS have been eliminated. Looks fixed to me, thanks Ted.
          Carlo Guglielmin made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Nils Lundquist made changes -
          Link This issue blocks ICE-6536 [ ICE-6536 ]
          Ken Fyten made changes -
          Assignee Priority P1
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Carlo Guglielmin
              Reporter:
              Brad Kroeger
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: