You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2016/07/18 15:41:47 UTC
[Bug 59881] New: Fix for blank cells in headers in DB in D*
functions
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Bug ID: 59881
Summary: Fix for blank cells in headers in DB in D* functions
Product: POI
Version: 3.15-dev
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P2
Component: SS Common
Assignee: dev@poi.apache.org
Reporter: patrick.zimmermann@haltec.de
Created attachment 34052
--> https://bz.apache.org/bugzilla/attachment.cgi?id=34052&action=edit
dstar_empty_header_fix.patch
The D* functions errored out when the database headers contained blank cells.
Excel 2013/Win7 doesn't (I didn't try with other Excel versions).
This patch fixes that.
Test included.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] [PATCH] Fix for blank cells in headers in DB in D*
functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Patrick Zimmermann <pa...@haltec.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|Fix for blank cells in |[PATCH] Fix for blank cells
|headers in DB in D* |in headers in DB in D*
|functions |functions
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] Fix for blank cells in headers in DB in D* functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Patrick Zimmermann <pa...@haltec.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |PatchAvailable
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] [PATCH] Fix for blank cells in headers in DB in D*
functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
--- Comment #5 from Javen O'Neal <on...@apache.org> ---
(In reply to Patrick Zimmermann from comment #0)
> Excel 2013/Win7 doesn't [error out] (I didn't try with other Excel versions).
Formula evaluation does not error out in LibreOffice 5.0.3.2, either.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] [PATCH] Fix for blank cells in headers in DB in D*
functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #4 from Javen O'Neal <on...@apache.org> ---
I see the added test in DGet.xls now
Row 102:
> empty cells
> =DGET(G103:I104,"Three",K103:L104)
> don't fail with empty cells in DB headers
Applied in r1753346. Thanks for the patch! This will be included in POI 3.15
beta3.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] [PATCH] Fix for blank cells in headers in DB in D*
functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Patrick Zimmermann <pa...@haltec.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |NEW
--- Comment #3 from Patrick Zimmermann <pa...@haltec.de> ---
The respective test is in the DGet.xls file. That file should replace
test-data/spreadsheet/DGet.xls
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] [PATCH] Fix for blank cells in headers in DB in D*
functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |NEEDINFO
--- Comment #2 from Javen O'Neal <on...@apache.org> ---
(In reply to Patrick Zimmermann from comment #0)
> Test included.
Could you re-upload your patch? I did not see a unit test for this.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59881] Fix for blank cells in headers in DB in D* functions
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59881
--- Comment #1 from Patrick Zimmermann <pa...@haltec.de> ---
Created attachment 34053
--> https://bz.apache.org/bugzilla/attachment.cgi?id=34053&action=edit
DGet.xls
Updated test xls file.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org