From 3c9ed9a20933b9f51df5fb45765ce9dd31f84b46 Mon Sep 17 00:00:00 2001
From: uhensler <urs.hensler@frentix.com>
Date: Wed, 27 Mar 2019 09:27:44 +0100
Subject: [PATCH] OO-3937: Building the URL of Collabora Office the right way

---
 .../wopi/collabora/ui/CollaboraEditorUrlBuilder.java        | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/olat/modules/wopi/collabora/ui/CollaboraEditorUrlBuilder.java b/src/main/java/org/olat/modules/wopi/collabora/ui/CollaboraEditorUrlBuilder.java
index 666a3401502..bd11a6a45ee 100644
--- a/src/main/java/org/olat/modules/wopi/collabora/ui/CollaboraEditorUrlBuilder.java
+++ b/src/main/java/org/olat/modules/wopi/collabora/ui/CollaboraEditorUrlBuilder.java
@@ -99,16 +99,16 @@ class CollaboraEditorUrlBuilder {
 		}
 		
 		if (StringHelper.containsNonWhitespace(permission)) {
-			url.append("?permission=").append(permission);
+			url.append("&permission=").append(permission);
 		}
 		
 		if (closeButton) {
 			// Apparently that parameter is ignored. The close button is always visible!
-			url.append("?closebutton=1");
+			url.append("&closebutton=1");
 		}
 		
 		if (debug) {
-			url.append("?debug=1");
+			url.append("&debug=1");
 		}
 		
 		return url.toString();
-- 
GitLab