Fixing row and column tracking when there are blank cells

[lsergio]

Fixed cell row and column index tracking when there are blank cells at the beginning of the row, like:

<row>
			<c r="B2" t="s">
				<v>136316</v>
			</c>
			<c t="s">
				<v>136317</v>
			</c>
			<c t="b">
				<v>1</v>
			</c>
			<c t="b">
				<v>1</v>
			</c>
			<c t="s">
				<v>136318</v>
			</c>
			<c t="s">
				<v>265</v>
			</c>
			<c t="s">
				<v>136319</v>
			</c>
			<c r="J2" s="2">
				<v>45411.095648645831</v>
			</c>
		</row>

The A2 cells is blank the the lib loses track of the indexes.

[rzymek]

Thank you!
Could you add a test that would verify the fix (that is it would fail before your changes)?

[rzymek]

👍

[rzymek]

Now we have both trackedRowIndex field and parameter.
Shouldn't this.trackedRowIndex just be increated in this loop?

[lsergio]

Yeah, that's an option. I added the rowIndex parameter to match the existing colIndex parameter instead of getting one piece of info from the parameter and the other from the attribute

[lsergio]

Thank you! Could you add a test that would verify the fix (that is it would fail before your changes)?

I added a new test case for the RowSpliterator class. I could not create a test that processes the spreadsheet where I found the issue because it contains customer sensitive data, and I didn't manage to create another one - in all my attempts all cells had explicit addresses in the xml content.