From 857f0865010284d7179c0a1c462050d2cf7f1391 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 3 Dec 2019 00:05:19 +0000
Subject: [PATCH] Fix ?via= args in SpecPermalinkConstructor.js

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 src/utils/permalinks/SpecPermalinkConstructor.js | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/utils/permalinks/SpecPermalinkConstructor.js b/src/utils/permalinks/SpecPermalinkConstructor.js
index 1c80ff8975..d8b3633bb3 100644
--- a/src/utils/permalinks/SpecPermalinkConstructor.js
+++ b/src/utils/permalinks/SpecPermalinkConstructor.js
@@ -73,18 +73,16 @@ export default class SpecPermalinkConstructor extends PermalinkConstructor {
             // Probably a group, no further parsing needed.
             return PermalinkParts.forGroup(entity);
         } else if (entity[0] === '#' || entity[0] === '!') {
-            if (parts.length === 1) {
-                return PermalinkParts.forRoom(entity, []);
+            if (parts.length === 1) { // room without event permalink
+                const [roomId, query=""] = entity.split("?");
+                const via = query.split(/&?via=/g).filter(p => !!p);
+                return PermalinkParts.forRoom(roomId, via);
             }
 
             // rejoin the rest because v3 events can have slashes (annoyingly)
             const eventIdAndQuery = parts.length > 1 ? parts.slice(1).join('/') : "";
-            const secondaryParts = eventIdAndQuery.split("?");
-
-            const eventId = secondaryParts[0];
-            const query = secondaryParts.length > 1 ? secondaryParts[1] : "";
-
-            const via = query.split("via=").filter(p => !!p);
+            const [eventId, query=""] = eventIdAndQuery.split("?");
+            const via = query.split(/&?via=/g).filter(p => !!p);
 
             return PermalinkParts.forEvent(entity, eventId, via);
         } else {