ICEfaces
  1. ICEfaces
  2. ICE-7584

ACE components triggering double-submits when used with nested ace:ajax

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC1
    • Fix Version/s: 3.0.RC2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      IF 3, ACE components
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      The following components have their ace:ajax behaviour handling written such that it makes the non ace:ajax-style component request, followed by an extraneous request to trigger the ace:ajax execute/render and listener, when one would expect those render/execute targets to be the targets of the previous request and the ace:ajax listener to be activated during the same lifecycle as the previous request.

      This behaviour was originally specced this way, but subsequent investigation has shown this to be incorrect. The expected behaviour is that the ace:ajax submit will take precedence in cases where the same event would trigger both an event listener attribute on the component and a defined event ace:ajax tag. This is consistent with standard f:ajax tag behaviour as well.

      Thus, the following components need changes so that the defined ace:ajax behaviour submits the params that the component request normally does.

      ace:accordion
      ace:droppable
      ace:progressBar
      ace:resizable
      ace:tooltip

        Activity

        Hide
        Nils Lundquist added a comment -

        All fixes made and briefly tested on showcase. More rigorous testing could be done, but I don't have access to existing tests to base them off of, and the change should be uniformly safe, testing would serve only to catch possible typos.

        Show
        Nils Lundquist added a comment - All fixes made and briefly tested on showcase. More rigorous testing could be done, but I don't have access to existing tests to base them off of, and the change should be uniformly safe, testing would serve only to catch possible typos.
        Hide
        Nils Lundquist added a comment - - edited

        Changes made in r26911, r26910, r26909 and r26908

        Show
        Nils Lundquist added a comment - - edited Changes made in r26911, r26910, r26909 and r26908
        Hide
        Nils Lundquist added a comment -

        Issue extends to more components than originally stated.

        Fixed linkbutton and pushbutton in r26924

        Show
        Nils Lundquist added a comment - Issue extends to more components than originally stated. Fixed linkbutton and pushbutton in r26924
        Hide
        Nils Lundquist added a comment -

        Can't find any other effected usages.

        Show
        Nils Lundquist added a comment - Can't find any other effected usages.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Nils Lundquist
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: