ICEfaces
  1. ICEfaces
  2. ICE-9894

ace:dataExporter - Exported column order not correct after tableConfigPanel changes

    Details

    • Assignee Priority:
      P2

      Description

      An ace:dataTable uses an ace:columnGroup tag. The rows that use the colspan have the ace:excludeFromExport tag so that they are now exported. Initial export of the columns show the correct order. Upon changing the order of columns via the ace:tableConfigPanel the columns are displayed correctly on the page but are still in the previous state on export. The following warning message is logged:

      WARNING: Header column ordering list size does not match number of defined columns in table masterTable.

      Tested using CSV export.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case. The icefaces-*.jar files need to be added back into war file to run.

        Steps:

        • Load welcomeICEfaces.jsf
        • If an export is done now, the export reflects the current order.
        • Open the tableConfigPanel and change the order of some columns.
        • Upon save, the column order looks correct on the page.
        • Once an export is done, the order in the export does not reflect what is on the page.
        Show
        Arran Mccullough added a comment - Attached test case. The icefaces-*.jar files need to be added back into war file to run. Steps: Load welcomeICEfaces.jsf If an export is done now, the export reflects the current order. Open the tableConfigPanel and change the order of some columns. Upon save, the column order looks correct on the page. Once an export is done, the order in the export does not reflect what is on the page.
        Hide
        Arturo Zambrano added a comment -

        r40360: Committed fix to 4.0 trunk. Overloaded getRows() method to add mode for including all columns in a columnGroup, even if they are excluded from rendering/exporting, for purposes of determining correct column ordering.

        Show
        Arturo Zambrano added a comment - r40360: Committed fix to 4.0 trunk. Overloaded getRows() method to add mode for including all columns in a columnGroup, even if they are excluded from rendering/exporting, for purposes of determining correct column ordering.
        Hide
        Arturo Zambrano added a comment -

        r40361: Committed fix to 3.3 EE maintenance branch.

        Show
        Arturo Zambrano added a comment - r40361: Committed fix to 3.3 EE maintenance branch.
        Hide
        Liana Munroe added a comment -

        Confirmed fixed Icefaces 4, ee-3.3.0.GA maintenance, Williams tableConfigEnh, revision #40362, Tomcat 7, all browsers.

        Show
        Liana Munroe added a comment - Confirmed fixed Icefaces 4, ee-3.3.0.GA maintenance, Williams tableConfigEnh, revision #40362, Tomcat 7, all browsers.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: