API Normalise trailing slashes in links

silverstripe · Jan 17, 2023 · 8b15a90 · 8b15a90
1 parent 50cefa6
commit 8b15a90
Show file tree

Hide file tree

Showing 14 changed files with 47 additions and 45 deletions.
diff --git a/client/dist/js/SilverStripeNavigator.js b/client/dist/js/SilverStripeNavigator.js
diff --git a/client/dist/js/TinyMCE_sslink-anchor.js b/client/dist/js/TinyMCE_sslink-anchor.js
diff --git a/client/dist/js/TinyMCE_sslink-internal.js b/client/dist/js/TinyMCE_sslink-internal.js
diff --git a/client/dist/js/bundle.js b/client/dist/js/bundle.js
diff --git a/client/src/legacy/CMSMain.Tree.js b/client/src/legacy/CMSMain.Tree.js
@@ -1,6 +1,7 @@
 import $ from 'jquery';
 import i18n from 'i18n';
 import reactConfirm from 'reactstrap-confirm';
+import { joinUrlPaths } from 'lib/urls';
 
 $.entwine('ss.tree', function($) {
   $('.cms-tree').entwine({
@@ -45,7 +46,7 @@ $.entwine('ss.tree', function($) {
       });
 
       const baseUrl = $('base').attr('href') || ''; // Edge17 and IE11 require absolute paths
-      window.location.assign(baseUrl + urlWithParams);
+      window.location.assign(joinUrlPaths(baseUrl, urlWithParams));
     },
 
     getTreeConfig: function() {

diff --git a/client/src/legacy/CMSMain.js b/client/src/legacy/CMSMain.js
@@ -1,4 +1,5 @@
 import $ from 'jquery';
+import { joinUrlPaths } from 'lib/urls';
 
 /**
  * Behaviour for the CMS Content Toolbar.
@@ -81,7 +82,7 @@ $.entwine('ss', function ($) {
       localStorage.setItem('ss.pages-view-type', viewType);
       if(isContentViewInSidebar && viewType === VIEW_TYPE_LIST) {
         const baseUrl = $('base').attr('href') || '';  // Edge17 and IE11 need absolute path
-        window.location.assign(baseUrl + $contentView.data('url-listviewroot'));
+        window.location.assign(joinUrlPaths(baseUrl, $contentView.data('url-listviewroot')));
 
         return;
       }

diff --git a/code/Controllers/ModelAsController.php b/code/Controllers/ModelAsController.php
@@ -84,7 +84,7 @@ public function handleRequest(HTTPRequest $request): HTTPResponse
         // If the database has not yet been created, redirect to the build page.
         /** @skipUpgrade */
         if (!DB::is_active() || !ClassInfo::hasTable('SiteTree')) {
-            $this->getResponse()->redirect(Director::absoluteBaseURL() . 'dev/build?returnURL=' . (isset($_GET['url']) ? urlencode($_GET['url']) : null));
+            $this->getResponse()->redirect(Controller::join_links(Director::absoluteBaseURL(), 'dev/build?returnURL=' . (isset($_GET['url']) ? urlencode($_GET['url']) : null)));
             $this->popCurrent();
 
             return $this->getResponse();

diff --git a/code/Controllers/OldPageRedirector.php b/code/Controllers/OldPageRedirector.php
@@ -102,7 +102,7 @@ public static function find_old_page($params, $parent = null, $redirect = false)
                     // No valid page found.
                     if ($redirect) {
                         // If we had some redirect to be done, lets do it. imagine /foo/action -> /bar/action, we still want this redirect to happen if action isn't a page
-                        return $page->Link() . implode('/', $params);
+                        return Controller::join_links($page->Link(), implode('/', $params));
                     }
                 }
             } else {

diff --git a/code/Forms/SiteTreeURLSegmentField.php b/code/Forms/SiteTreeURLSegmentField.php
@@ -134,7 +134,7 @@ public function setURLPrefix($url)
      */
     public function getURLPrefix()
     {
-        return $this->urlPrefix;
+        return rtrim($this->urlPrefix ?? '', '/') . '/';
     }
 
     public function getURLSuffix()

diff --git a/code/Model/SiteTree.php b/code/Model/SiteTree.php
@@ -697,7 +697,7 @@ public function RelativeLink($action = null)
             $action = null;
         }
 
-        $link = Controller::join_links($base, '/', $action);
+        $link = Controller::join_links($base, $action);
 
         $this->extend('updateRelativeLink', $link, $base, $action);