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

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              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.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case12694Example.war [ 16708 ]
              Attachment Case12694Example.zip [ 16709 ]
              Ken Fyten made changes -
              Summary ace:dataExporter - Exported colomn order not correct after tableConfigPanel changes ace:dataExporter - Exported column order not correct after tableConfigPanel changes
              Ken Fyten made changes -
              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.               		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.
              Ken Fyten made changes -
              Fix Version/s 4.0.BETA [ 10770 ]
              Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Ken Fyten made changes -
              Assignee Priority P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #40360 Mon Mar 17 17:48:38 MDT 2014 art.zambrano ICE-9894 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
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/OuterTableCSVExporter.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/PDFExporter.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/ExcelExporter.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/CSVExporter.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/Exporter.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dataexporter/XMLExporter.java
              Hide
              Permalink
              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
              Permalink
              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.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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.
              Ken Fyten made changes -
              Affects Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: