ICEfaces
  1. ICEfaces
  2. ICE-6717

<f:ajax disabled="true"> does not work correctly

    Details

      Description

      <h:commandButton ...>
        <f:ajax disabled="true">
      </h:commandButton>

      works as long as no input field has had the focus or no ajax-enabled button is clicked.

      See the attached example:
      'h:commandButton without Ajax' works in the beginning but not anymore after a click in one of the input fields or '...with Ajax' buttons.

      This happens with the nightly builds from March 29th as well as the one from March 22th

        Activity

        Hide
        Stefan Niederhauser added a comment -

        Example app to demonstrate the issue

        Show
        Stefan Niederhauser added a comment - Example app to demonstrate the issue
        Hide
        Adrian Gygax added a comment -

        I just verifed this bug and it happens also with ICEfaces 2.0.1 final.

        One scenario to reproduce the bug (Tested with Firefox 4):
        1. Open in browser: http://localhost:8080/jsf20-web-0.0.1-SNAPSHOT/pages/iceWithoutAjax.xhtml
        2. Click "h:commandButton without Ajax" once. This is a normal POST as expected
        3. Click "h:commandButton without Ajax" a second time. Now, this is an Ajax POST which is not what we want as we specified <f:ajax disabled="true"/>

        Show
        Adrian Gygax added a comment - I just verifed this bug and it happens also with ICEfaces 2.0.1 final. One scenario to reproduce the bug (Tested with Firefox 4): 1. Open in browser: http://localhost:8080/jsf20-web-0.0.1-SNAPSHOT/pages/iceWithoutAjax.xhtml 2. Click "h:commandButton without Ajax" once. This is a normal POST as expected 3. Click "h:commandButton without Ajax" a second time. Now, this is an Ajax POST which is not what we want as we specified <f:ajax disabled="true"/>
        Hide
        Ted Goddard added a comment -

        Is this a regression? (Have you observed it working prior to March 22?)

        Show
        Ted Goddard added a comment - Is this a regression? (Have you observed it working prior to March 22?)
        Hide
        Stefan Niederhauser added a comment -

        I did not try it with earlier versions because there were other problems

        Show
        Stefan Niederhauser added a comment - I did not try it with earlier versions because there were other problems
        Hide
        Stefan Niederhauser added a comment -

        just tried it... it happens also with 2.0.0

        Show
        Stefan Niederhauser added a comment - just tried it... it happens also with 2.0.0
        Hide
        Ted Goddard added a comment -

        This should likely be targeted for the next EE release.

        Show
        Ted Goddard added a comment - This should likely be targeted for the next EE release.
        Hide
        Ted Goddard added a comment -

        This is partially a regression from ICE-6674 from the code change for ICE-6327. The basic problem is that IE and Firefox handle calling the "native submit" of a form differently from WebKit browsers. IE and Firefox are not including the submitting element.

        Show
        Ted Goddard added a comment - This is partially a regression from ICE-6674 from the code change for ICE-6327 . The basic problem is that IE and Firefox handle calling the "native submit" of a form differently from WebKit browsers. IE and Firefox are not including the submitting element.
        Hide
        Ted Goddard added a comment -

        Any component that can submit a form is a candidate for <f:ajax disabled="true">. In other words, complete testing would require component-showcase or a set of component regression tests to be modified so that each input component included <f:ajax disabled="true/false">.

        Show
        Ted Goddard added a comment - Any component that can submit a form is a candidate for <f:ajax disabled="true">. In other words, complete testing would require component-showcase or a set of component regression tests to be modified so that each input component included <f:ajax disabled="true/false">.
        Hide
        Ken Fyten added a comment -

        Resolved pending confirmation by QA.

        Show
        Ken Fyten added a comment - Resolved pending confirmation by QA.
        Hide
        Ted Goddard added a comment -

        Enhancement added so that if <h:form> is marked <f:ajax disabled="true"> the ice.captureSubmit() feature will be disabled as well. This prevents Ajax from being enabled on components that themselves do not directly support <f:ajax>.

        Show
        Ted Goddard added a comment - Enhancement added so that if <h:form> is marked <f:ajax disabled="true"> the ice.captureSubmit() feature will be disabled as well. This prevents Ajax from being enabled on components that themselves do not directly support <f:ajax>.
        Hide
        Ted Goddard added a comment -

        Test case for <f:ajax> and <ui:include> problem here:

        http://java.net/jira/browse/JAVASERVERFACES-2041

        This problem is unique to ui:include.

        Show
        Ted Goddard added a comment - Test case for <f:ajax> and <ui:include> problem here: http://java.net/jira/browse/JAVASERVERFACES-2041 This problem is unique to ui:include.

          People

          • Assignee:
            Mandeep Hayher
            Reporter:
            Stefan Niederhauser
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: