Author: fanningpj
Date: Tue Mar 22 14:10:02 2022
New Revision: 1899133
URL: http://svn.apache.org/viewvc?rev=1899133&view=rev
Log:
[bug-65973] partial fix for removing hyperlinks that span multiple cells
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java?rev=1899133&r1=1899132&r2=1899133&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
Tue Mar 22 14:10:02 2022
@@ -3088,14 +3088,14 @@ public class XSSFSheet extends POIXMLDoc
//TODO handle case where hyperlink ref spans many rows
(https://bz.apache.org/bugzilla/show_bug.cgi?id=65973)
//but where only some rows are being removed and others are
not (range will need to be modified)
if (range.getFirstRow() == range.getLastRow() &&
rowsToRemoveSet.contains(range.getFirstRow())) {
- hyperlinks.remove(link);
+ removeHyperlink(link);
} else if (range.getFirstRow() != range.getLastRow()) {
boolean toRemove = true;
for (int i = range.getFirstRow(); i <= range.getLastRow()
&& toRemove; i++) {
toRemove = rowsToRemoveSet.contains(i);
}
if (toRemove) {
- hyperlinks.remove(link);
+ removeHyperlink(link);
}
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
