From 058aef3bfb560bd9acabd23b5d3e03f26ca5609d Mon Sep 17 00:00:00 2001 From: PJ Fanning Date: Tue, 22 Mar 2022 14:10:02 +0000 Subject: [PATCH] [bug-65973] partial fix for removing hyperlinks that span multiple cells git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1899133 13f79535-47bb-0310-9956-ffa450edef68 --- .../main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java index 50e66f2f99..f0ee66b39c 100644 --- a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java +++ b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java @@ -3088,14 +3088,14 @@ public class XSSFSheet extends POIXMLDocumentPart implements Sheet, OoxmlSheetEx //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); } } } -- 2.39.5