From 66c29cc8b63972bbf1f1b2d9f13f556bd6d847d1 Mon Sep 17 00:00:00 2001
From: Armin Schrenk <armin.schrenk@skymatic.de>
Date: Wed, 17 Jul 2024 22:51:11 +0200
Subject: [PATCH] improve entry removal tolerance against reformatted document

---
 .../linux/quickaccess/DolphinPlaces.java        | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/cryptomator/linux/quickaccess/DolphinPlaces.java b/src/main/java/org/cryptomator/linux/quickaccess/DolphinPlaces.java
index 462f638..4d0ca15 100644
--- a/src/main/java/org/cryptomator/linux/quickaccess/DolphinPlaces.java
+++ b/src/main/java/org/cryptomator/linux/quickaccess/DolphinPlaces.java
@@ -20,6 +20,7 @@
 import java.nio.file.Path;
 import java.nio.file.StandardCopyOption;
 import java.nio.file.StandardOpenOption;
+import java.util.List;
 import java.util.UUID;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.locks.Lock;
@@ -122,9 +123,8 @@ public void remove() throws QuickAccessServiceException {
 				//validate
 				xmlValidator.validate(new StreamSource(new StringReader(placesContent)));
 				//modify
-				var placesContentPart1 = placesContent.substring(0, idIndex);
-				int openingTagIndex = placesContentPart1.lastIndexOf("<bookmark href=");
-				var contentToWrite1 = placesContentPart1.substring(0, openingTagIndex).stripTrailing();
+				int openingTagIndex = indexOfEntryOpeningTag(placesContent, idIndex);
+				var contentToWrite1 = placesContent.substring(0, openingTagIndex).stripTrailing();
 
 				int closingTagEndIndex = placesContent.indexOf('>', placesContent.indexOf("</bookmark", idIndex));
 				var part2Tmp = placesContent.substring(closingTagEndIndex + 1).split("\\v+", 2); //removing leading vertical whitespaces, but no indentation
@@ -146,6 +146,17 @@ public void remove() throws QuickAccessServiceException {
 		}
 	}
 
+	private static int indexOfEntryOpeningTag(String placesContent, int idIndex) {
+		var xmlWhitespaceChars = List.of(' ', '\t', '\n');
+		for (char c : xmlWhitespaceChars) {
+			int idx = placesContent.lastIndexOf("<bookmark" + c, idIndex);
+			if (idx != -1) {
+				return idx;
+			}
+		}
+		throw new IllegalStateException("File " + PLACES_FILE + " is valid xbel file, but does not contain opening bookmark tag.");
+	}
+
 	@CheckAvailability
 	public static boolean isSupported() {
 		try {