Merge pull request #2369 from nextcloud/backport/2368/stable29

Author: skjnldsv

js/viewer-main.mjs

@@ -425,7 +425,7 @@ var b4=Object.defineProperty;var y4=(e,r,a)=>r in e?b4(e,r,{enumerable:!0,config
  * You should have received a copy of the GNU Affero General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  *
- */function Pj(e,r,a){const n="".concat(e.dirname,"/").concat(e.basename).replace(/^\/\//,"/"),i=[window.OCP.Files.Router.name,{...window.OCP.Files.Router.params},{...window.OCP.Files.Router.query},!0],s=()=>window.OCP.Files.Router.goToRoute(...i);ah(e,r,a),OCA.Viewer.open({path:n,onPrev:ah,onNext:ah,onClose:s})}function ah(e,r,a){window.OCP.Files.Router.goToRoute(null,{view:r.id,fileid:e.fileid},{dir:a,openfile:!0},!0)}/**
+ */function Pj(e,r,a){const n="".concat(e.dirname,"/").concat(e.basename).replace(/^\/\//,"/"),i=[window.OCP.Files.Router.name,{...window.OCP.Files.Router.params},{...window.OCP.Files.Router.query},!0],s=()=>{delete i[2].openfile,window.OCP.Files.Router.goToRoute(...i)};ah(e,r,a),OCA.Viewer.open({path:n,onPrev:ah,onNext:ah,onClose:s})}function ah(e,r,a){window.OCP.Files.Router.goToRoute(null,{view:r.id,fileid:e.fileid},{dir:a,openfile:!0},!0)}/**
  * @copyright Copyright (c) 2020 Azul <azul@riseup.net>
  *
  * @author Azul <azul@riseup.net>

js/viewer-main.mjs.map

@@ -34,7 +34,11 @@ export default function(node, view, dir) {
 		{ ...window.OCP.Files.Router.query },
 		true,
 	]
-	const onClose = () => window.OCP.Files.Router.goToRoute(...oldRoute)
+	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)
+	}
 	pushToHistory(node, view, dir)
 	OCA.Viewer.open({ path, onPrev: pushToHistory, onNext: pushToHistory, onClose })
 }