Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.8
-
Fix Version/s: 1.8.3, 1.8.2-EE-GA_P02, 2.0.0
-
Component/s: ICE-Components
-
Labels:None
-
Environment:IE7, Firefox 3, Windows Vista
Description
ice:dataExporter component does not work with ice:dataTable which uses ice:columns for dynamic column definition
Reason: method renderToHandler in class DataExporterRenderer does only supports ice:column and not ice:columns
Reason: method renderToHandler in class DataExporterRenderer does only supports ice:column and not ice:columns
Made extensive modifications to the outputting code to handle iterating over UIColumns, which was exacerbated by the UIColumns having an unspecified number rows, requiring actual iteration to determine where the includeColumns indexes would end up being.
Testing should involve having a mix of several ice:columns interspersed with several ice:column objects. There should be two main scenarios: one with includeColumns specified, using out of sequence columns indexes which correspond to both ice:column and ice:columns, and the other scenario without includeColumns specified. That should cover most of the code paths. Testing with Excel is easier than CSV, since CSV intentionally omits outputting the column headers.
icefaces trunk
Subversion 23073
icefaces 2 trunk
Subversion 23074