Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.8.1
-
Component/s: ICE-Components
-
Labels:None
-
Environment:-
Description
When on page > 1 in a paginated dataTable, the dataExporter (ignorePagination=false) will add empty rows to the spreadsheet accounting for entries in previous pages. For example, on page 1 it will correctly export the current page data with no extra empty rows. If you paginate to page 2 or 3, you will see the current page exported but also a large amount of empty rows which make up the previous pages. Attached is a screenshot.
Activity
| Field | Original Value | New Value |
|---|---|---|
| Attachment | screenshot-1.jpg [ 11910 ] |
| Summary |
dataExporter has issues exporting current page with ignorePagination set to false |
dataExporter has issues exporting current page with ignorePagination set to false |
| Salesforce Case | [] | |
| Description | When on a page > 1 in a paginated dataTable, the dataExporter (ignorePagination=false) will add empty rows to the spreadsheet accounting for entries in previous pages. For example, on page 1 it will correctly export the current page data with no extra space. If you paginate to page 2 or 3, you will see the current page exported but also a large amount of empty rows which make up the previous pages. Attached is a screenshot. | When on page > 1 in a paginated dataTable, the dataExporter (ignorePagination=false) will add empty rows to the spreadsheet accounting for entries in previous pages. For example, on page 1 it will correctly export the current page data with no extra empty rows. If you paginate to page 2 or 3, you will see the current page exported but also a large amount of empty rows which make up the previous pages. Attached is a screenshot. |
| Salesforce Case | [50070000009f00F] |
| Fix Version/s | 1.8.2 [ 10190 ] | |
| Assignee Priority | P1 | |
| Assignee | Adnan Durrani [ adnan.durrani ] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #19156 | Thu Aug 13 10:06:02 MDT 2009 | adnan.durrani | Fix for |
| Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/dataexporter/DataExporter.java
|
| Status | Open [ 1 ] | In Progress [ 3 ] |
Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\dataexporter\DataExporter.java
Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\dataexporter\DataExporter.java
Completed: At revision: 19156
Show
Adnan Durrani
added a comment - Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\dataexporter\DataExporter.java
Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\dataexporter\DataExporter.java
Completed: At revision: 19156
| Status | In Progress [ 3 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Fix Version/s | 1.8.2-RC1 [ 10210 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
| Assignee Priority | P1 |
It's question about OutputHandler for Excel.
DataExporter provide rowIndex, which used by ExcelOutputHandler.
Way to fix this IMO next:
(file ExcelOutputHandler.java, changed lines market with ///!)
...
private Integer startRow = null; ///!
...
public void writeCell(Object output, int col, int row) {
if(startRow == null) startRow = new Integer(row); ///!
WritableCell cell = null;
if( output instanceof String )
{ cell = new Label(col, row + 1 - startRow.intValue(), (String)output); ///! }else if( output instanceof Double )
{ cell = new Number(col, row + 1 - startRow.intValue(), ((Double)output).doubleValue()); ///! }try
{ sheet.addCell(cell); }catch(WriteException e)
{ System.out.println("Could not write excel cell"); e.printStackTrace(); }}