ICEfaces
  1. ICEfaces
  2. ICE-10577

ace:dataTable - Issues with filterEvent="enter"

          Details

            Description

            Issue #1: When using filterEvent="enter" filtering should only happen on the enter key press. This is working fine when typing in filter input but when removing text via the backspace key, a filter event is fired for each back space key press.

            Issue #2: When using multiple filter fields, only the field that has the enter key pressed on will do the filtering, the other field is cleared. To test with attach test case:
             - Enter in 'a' in the first column.
             - Enter in '1' in the second column.
             - Click enter, the first column field is cleared and filtering happens for just the second column.

            Issue #3: The enter filter event isn't working on the ICEfaces 4 and ICEfaces EE 3 trunk code. It is clearing out the field and not doing any filtering.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows the issue.

              Steps:

              • Load welcomeICEfaces.jsf
              • Enter in some text into a filter input field.
              • Press the enter key, filtered data is shown.
              • Use the backspace key to delete text, another filter event is called.
              Show
              Arran Mccullough added a comment - Attached test case that shows the issue. Steps: Load welcomeICEfaces.jsf Enter in some text into a filter input field. Press the enter key, filtered data is shown. Use the backspace key to delete text, another filter event is called.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case13330Example2.war [ 18787 ]
              Attachment Case13330Example2.zip [ 18788 ]
              Arran Mccullough made changes -
              Summary ace:dataTable - Filter event fired for other kets whenb using filterEvent="enter" ace:dataTable - Filter event fired for other keys when using filterEvent="enter"
              Arran Mccullough made changes -
              Summary ace:dataTable - Filter event fired for other keys when using filterEvent="enter" ace:dataTable - Isssues with filterEvent="enter"
              Description When using filterEvent="enter" filtering should only happen on the enter key press. This is working fine when typing in filter input but when removing text via the backspace key, a filter event is fired for each back space key press. Issue #1: When using filterEvent="enter" filtering should only happen on the enter key press. This is working fine when typing in filter input but when removing text via the backspace key, a filter event is fired for each back space key press.

              Issue #2: When using multiple filter fields, only the field that has the enter key pressed on will do the filtering, the other field is cleared. To test with attach test case:
               - Enter in 'a' in the first column.
               - Enter in '1' in the second column.
               - Click enter, the first column field is cleared and filtering happens for just the second column.

              Issue #3: The enter filter event isn't working on the ICEfaces 4 and ICEfaces EE 3 trunk code. It is clearing out the field and not doing any filtering.
              Arran Mccullough made changes -
              Summary ace:dataTable - Isssues with filterEvent="enter" ace:dataTable - Issues with filterEvent="enter"
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-4.0.0.GA [ 11171 ]
              Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
              Assignee Priority P1 [ 10010 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #44314 Thu Mar 26 14:09:55 MDT 2015 art.zambrano ICE-10577 added submitOnEnter=disabled to the filter fields, in order to prevent the bridge from handling the filtering request; modified the FilterState class to save the filter values of all columns in the table, not just the one that submitted the filtering request
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/FilterState.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableHeadRenderer.java
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix to the 4.0 trunk at revision 44314, to the 4.0 EE tag at revision 44315, and to the 3.3 EE maintenance branch at revision 44316.

              The fix consists in adding submitOnEnter=disabled to the filter fields, in order to prevent the bridge from handling the filtering request, instead of the table. This solves issues #1 and #3 above. Also, modified the FilterState class to save the filter values of all columns in the table, not just the one that submitted the filtering request. This solves issue #2.

              Show
              Arturo Zambrano added a comment - Committed fix to the 4.0 trunk at revision 44314, to the 4.0 EE tag at revision 44315, and to the 3.3 EE maintenance branch at revision 44316. The fix consists in adding submitOnEnter=disabled to the filter fields, in order to prevent the bridge from handling the filtering request, instead of the table. This solves issues #1 and #3 above. Also, modified the FilterState class to save the filter values of all columns in the table, not just the one that submitted the filtering request. This solves issue #2.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified fixes with the attached test case on EE-4.0 Jenkins Build#3 in IE11, FF34, Chrome41.

              Show
              Carmen Cristurean added a comment - Verified fixes with the attached test case on EE-4.0 Jenkins Build#3 in IE11, FF34, Chrome41.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Fixed Javascript errors above in the 3.3 EE maintenance branch at revision 44347.

              Show
              Arturo Zambrano added a comment - Fixed Javascript errors above in the 3.3 EE maintenance branch at revision 44347.
              Ken Fyten made changes -
              Fix Version/s 4.1 [ 11375 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: