fix(Url): root/docId separator in Url util

Move pathname handling into the correct branch and ensure a ':' is
inserted between root and newDocId when appropriate

Author: UnschooledGamer

src/utils/Url.js

@@ -85,22 +85,23 @@ export default {
 					} else if (!rootCondition === !newDocIdCondition) {
 						root += "/";
 					}
-					// if pathname is undefined, meaning a docId/volume (e.g :primary:)
-					// has not been detected, so no newDocId's ":" will be added.
-					if(!pathname) {
-						// Ensure proper path separator between root and newDocId
-						let separator = '';
-						if (root.endsWith('/') && newDocId.startsWith('/')) {
-							// Both have separator, strip one from newDocId
-							newDocId = newDocId.slice(1);
-						} else if (!root.endsWith('/') && !newDocId.startsWith('/')) {
-							// Neither has separator, add one
-							separator = '/';
-						}
-						return `${contentUri.rootUri}::${root}${separator}${newDocId}${query}`;
-					}
 					return `${contentUri.rootUri}::${root}${newDocId}${query}`;
 				}
+
+				// if pathname is undefined, meaning a docId/volume (e.g :primary:)
+				// has not been detected, so no newDocId's ":" will be added.
+				if (!pathname) {
+					// Ensure proper path separator between root and newDocId
+					let separator = "";
+					if (root.endsWith("/") && newDocId.startsWith("/")) {
+						// Both have separator, strip one from newDocId
+						newDocId = newDocId.slice(1);
+					} else if (!root.endsWith("/") && !newDocId.startsWith("/")) {
+						// Neither has separator, add one
+						separator = "/";
+					}
+					return `${contentUri.rootUri}::${root}${separator}${newDocId}${query}`;
+				}
 				return `${contentUri.rootUri}::${root}:${newDocId}${query}`;
 			} catch (error) {
 				return null;