ICEfaces
  1. ICEfaces
  2. ICE-9542

showcase - dataTable - Grouping demo - row expansion issues when applying filters

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA
    • Fix Version/s: EE-3.3.0.GA_P02, 4.0
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      EE ICEfaces 3.3.0.GA_P01 rev# 37461/ EE ICEfaces 3.3.0.GA Release code
      Server: Tomcat7
      Browsers: all
    • Assignee Priority:
      P2

      Description

      In showcase -> Grouping: if applying a filter on the "Weight" column, the expansion of the rows does not appear to work properly.
      The automated test is successful because it does not test row expansion combined with filtering.

      Issue1: unable to expand a row for certain applied filters
      1. Enter '4' in 'Weight' filtering input text -> two rows will be rendered.
      2. Try to expand each row -> only first row can be expanded.
      Removing followed by re-applying the filter will render the second row being expanded.

      Issue2: first row duplication
      1. Enter '1' for 'Weight' filtering text -> 15 rows rendered.
      2. Try to expand first row (ID=4) -> this row will be expanded but it is rendered twice, each instance calculating different values for the Cost Per Pound.
      3. Trying to expand a different row will still duplicate the first row, and the row fails to expand.
      Removing the filter and then re-entering the filter removes the duplicated row.

      Row expansion is functional on this demo if no filters are applied.

        Activity

        Hide
        Arturo Zambrano added a comment -

        This issue seems to only occur with Mojarra and not with Myfaces.

        Show
        Arturo Zambrano added a comment - This issue seems to only occur with Mojarra and not with Myfaces.
        Hide
        Arturo Zambrano added a comment - - edited

        The problem starts at revision 31485 (ICE-7952), with a fix for a MyFaces compatibility issue in the showcase app code. Using the latest revision in either the 4.0 trunk or 3.3 EE branch, the problem is fixed by changing the scope of the class DataTableBindings.java in the showcase from @RequestScoped to @CustomScoped(value="#{window}"). However, this might cause issues with MyFaces.

        Show
        Arturo Zambrano added a comment - - edited The problem starts at revision 31485 ( ICE-7952 ), with a fix for a MyFaces compatibility issue in the showcase app code. Using the latest revision in either the 4.0 trunk or 3.3 EE branch, the problem is fixed by changing the scope of the class DataTableBindings.java in the showcase from @RequestScoped to @CustomScoped(value="#{window}"). However, this might cause issues with MyFaces.
        Hide
        Arturo Zambrano added a comment -

        Committed fix to 4.0 trunk at revision 40966 and to 3.3 EE maintenance branch at revision 40967. The fix is in the showcase app code. Modified the ace:dataTable Grouping demo to use a DataTable binding either in a request-scoped bean or in a window-scoped bean, depending whether MyFaces is being used or not. This modification only affects the Grouping demo.

        Tested this demo, using MyFaces and no issues were found related to this fix.

        Show
        Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 40966 and to 3.3 EE maintenance branch at revision 40967. The fix is in the showcase app code. Modified the ace:dataTable Grouping demo to use a DataTable binding either in a request-scoped bean or in a window-scoped bean, depending whether MyFaces is being used or not. This modification only affects the Grouping demo. Tested this demo, using MyFaces and no issues were found related to this fix.
        Hide
        Carmen Cristurean added a comment -

        On Grouping demo, rows cannot be expanded anymore (EE-3.3.0-maintenance rev. 40984).

        Show
        Carmen Cristurean added a comment - On Grouping demo, rows cannot be expanded anymore (EE-3.3.0-maintenance rev. 40984).
        Hide
        Arturo Zambrano added a comment -

        Reverted previous fixes at revisions 40987 and 40988.

        Show
        Arturo Zambrano added a comment - Reverted previous fixes at revisions 40987 and 40988.
        Hide
        Arturo Zambrano added a comment -

        Committed new fix at revisions 40989 and 40990. This fix is in the ace:dataTable code and consists in reapplying filtering and sorting to the data model at render phase when using Mojarra, in order to solve inconsistency issues.

        The root of the problem was that after sending a filter request, the subsequent requests weren't based on the filtered data set, but on the original, unfiltered one, and that was causing unpredictable, odd behaviour when interacting with the table.

        Show
        Arturo Zambrano added a comment - Committed new fix at revisions 40989 and 40990. This fix is in the ace:dataTable code and consists in reapplying filtering and sorting to the data model at render phase when using Mojarra, in order to solve inconsistency issues. The root of the problem was that after sending a filter request, the subsequent requests weren't based on the filtered data set, but on the original, unfiltered one, and that was causing unpredictable, odd behaviour when interacting with the table.
        Hide
        Carmen Cristurean added a comment - - edited

        Verified with EE-3.3.0-maintenance branch/ ICEfaces4 trunk rev. 40990 (Mojarra and MyFaces) in FF28, Chrome34, IE10, and all issues above appear to be resolved. Also, none of the issues from ICE-9946 could be reproduced anymore.

        Show
        Carmen Cristurean added a comment - - edited Verified with EE-3.3.0-maintenance branch/ ICEfaces4 trunk rev. 40990 (Mojarra and MyFaces) in FF28, Chrome34, IE10, and all issues above appear to be resolved. Also, none of the issues from ICE-9946 could be reproduced anymore.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Carmen Cristurean
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: