Merge pull request #2540 from nextcloud/backport/2539/stable28

nextcloud · Sep 20, 2024 · 796114c · 796114c
2 parents 2a5bd07 + c2f0cd8
commit 796114c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 11 deletions.
diff --git a/js/viewer-main.js b/js/viewer-main.js
diff --git a/js/viewer-main.js.map b/js/viewer-main.js.map
diff --git a/src/services/FilesActionHandler.js b/src/services/FilesActionHandler.js
@@ -28,16 +28,11 @@
 export default function(node, view, dir) {
 	// replace potential leading double slashes
 	const path = `${node.dirname}/${node.basename}`.replace(/^\/\//, '/')
-	const oldRoute = [
-		window.OCP.Files.Router.name,
-		{ ...window.OCP.Files.Router.params },
-		{ ...window.OCP.Files.Router.query },
-		true,
-	]
 	const onClose = () => {
 		// This can sometime be called with the openfile set to true already. But we don't want to keep openfile when closing the viewer.
-		delete oldRoute[2].openfile
-		window.OCP.Files.Router.goToRoute(...oldRoute)
+		const newQuery = { ...window.OCP.Files.Router.query }
+		delete newQuery.openfile
+		window.OCP.Files.Router.goToRoute(null, window.OCP.Files.Router.params, newQuery)
 	}
 	pushToHistory(node, view, dir)
 	OCA.Viewer.open({ path, onPrev: pushToHistory, onNext: pushToHistory, onClose })