Bug 59684 - CellReference.separateRefParts(String reference) Fails to Parse Row References Correctly
Summary: CellReference.separateRefParts(String reference) Fails to Parse Row Reference...
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: SS Common (show other bugs)
Version: 3.14-FINAL
Hardware: PC Linux
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-09 15:59 UTC by Sean Smitz
Modified: 2016-06-10 11:56 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Smitz 2016-06-09 15:59:15 UTC
I have a sheet with a named range composed of a set of rows (HOME!$169:$454). When creating the AreaReference the following is passed to CellReference.separateRefParts(String reference="HOME!$169"). The CellRef is then set to the following values: _rowIndex = 169, _colIndex = -1, _sheetName = "HOME", _isRowAbs = false; _isColAbs = true; I believe _isRowAbs should be 'true' and _isColAbs should be 'false'. This happens because the '$' from reference gets parsed into parts.colRef instead of being parsed into parts.rowRef with the digits for the row number.
Comment 1 Javen O'Neal 2016-06-10 11:56:06 UTC
Thanks for reporting this bug and giving some hints on where the problem was. I wrote a unit test per comment 0 and fixed the behavior of CellReference(String) to be able to read entire-row reference strings.

Applied patch in r1747687 and changelog in r1747688. Will be fixed in POI 3.15 beta 2.