Merge pull request #23 from creative-commoners/pulls/1.0/url-priority

FIX Lower URL priority so controller is not auto loaded over CMSPageHistoryController
silverstripe · Jun 17, 2018 · 8b54dcd · 8b54dcd
2 parents 984a84b + 848bade
commit 8b54dcd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/Controllers/CMSPageHistoryViewerController.php b/src/Controllers/CMSPageHistoryViewerController.php
@@ -24,7 +24,7 @@ class CMSPageHistoryViewerController extends CMSMain
 
     private static $url_rule = '/$Action/$ID/$VersionID/$OtherVersionID';
 
-    private static $url_priority = 42;
+    private static $url_priority = 41;
 
     private static $required_permission_codes = 'CMS_ACCESS_CMSMain';
 

diff --git a/tests/Behat/features/revert-to-a-version.feature b/tests/Behat/features/revert-to-a-version.feature
@@ -6,6 +6,8 @@ Feature: Revert to a version
 
   Background:
     Given a "page" "Home" with "Content"="Initial version"
+    And I have a config file "enable-historyviewer.yml"
+
     Given I am logged in with "ADMIN" permissions
     And I go to "/admin/pages"
     And I click on "Home" in the tree