Merge pull request #2232 from nextcloud/backport/2229/stable14

[stable14] fix(softUndeleteEntry): Only reset index of bookmarks if it is indeed…

Author: marcelklehr

lib/Db/TreeMapper.php

@@ -542,20 +542,21 @@ public function softUndeleteEntry(string $type, int $id, ?int $folderId = null):
 			return;
 		}
 
-		// set entry as not deleted
-		// has to come last to not break findByAncestorFolder
-		$qb = $this->db->getQueryBuilder();
-		$qb
-			->update('bookmarks_tree')
-			->set('soft_deleted_at', $qb->createNamedParameter(null, IQueryBuilder::PARAM_DATE))
-			->where($qb->expr()->eq('id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)))
-			->andWhere($qb->expr()->eq('type', $qb->createNamedParameter($type, IQueryBuilder::PARAM_STR)));
-		if ($folderId !== null) {
-			$qb->set('index', $qb->createNamedParameter($this->countChildren($folderId)));
-			$qb->andWhere($qb->expr()->eq('parent_folder', $qb->createNamedParameter($folderId, IQueryBuilder::PARAM_INT)));
+		if ($this->isEntrySoftDeleted($type, $id, $folderId)) {
+			// set entry as not deleted
+			// has to come last to not break findByAncestorFolder
+			$qb = $this->db->getQueryBuilder();
+			$qb
+				->update('bookmarks_tree')
+				->set('soft_deleted_at', $qb->createNamedParameter(null, IQueryBuilder::PARAM_DATE))
+				->where($qb->expr()->eq('id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)))
+				->andWhere($qb->expr()->eq('type', $qb->createNamedParameter($type, IQueryBuilder::PARAM_STR)));
+			if ($folderId !== null) {
+				$qb->set('index', $qb->createNamedParameter($this->countChildren($folderId)));
+				$qb->andWhere($qb->expr()->eq('parent_folder', $qb->createNamedParameter($folderId, IQueryBuilder::PARAM_INT)));
+			}
+			$qb->executeStatement();
 		}
-		$qb->executeStatement();
-
 	}
 
 	/**