Update location to handle pre-existing trailing slashes

kategengler · kategengler · commit 947d41919795 · 2024-12-17T14:40:59.000-05:00
diff --git a/app/locations/trailing-history.js b/app/locations/trailing-history.js
@@ -14,6 +14,8 @@ export default HistoryLocation.extend({
 
 export function formatURL(url) {
   let modifiedURL = new URL(url, 'http://example.com');
-  modifiedURL.pathname += '/';
+  if (!modifiedURL.pathname.endsWith('/')) {
+    modifiedURL.pathname += '/';
+  }
   return `${modifiedURL.pathname}${modifiedURL.search}${modifiedURL.hash}`;
 }

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,8 @@ export default HistoryLocation.extend({`
`14`	`14`
`15`	`15`	`export function formatURL(url) {`
`16`	`16`	`let modifiedURL = new URL(url, 'http://example.com');`
`17`		`- modifiedURL.pathname += '/';`
	`17`	`+ if (!modifiedURL.pathname.endsWith('/')) {`
	`18`	`+ modifiedURL.pathname += '/';`
	`19`	`+ }`
`18`	`20`	return `${modifiedURL.pathname}${modifiedURL.search}${modifiedURL.hash}`;
`19`	`21`	`}`