Merge pull request #21 from creative-commoners/pulls/1.0/ask-first

FIX Return default history controller when no request is available in injector
silverstripe · Jun 12, 2018 · 984a84b · 984a84b
2 parents 1f314fb + b9cce22
commit 984a84b
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/Controllers/HistoryControllerFactory.php b/src/Controllers/HistoryControllerFactory.php
@@ -20,6 +20,10 @@ class HistoryControllerFactory implements Factory
 
     public function create($service, array $params = array())
     {
+        // If no request is available yet, return the default controller
+        if (!Injector::inst()->has(HTTPRequest::class)) {
+            return new CMSPageHistoryController();
+        }
         $request = Injector::inst()->get(HTTPRequest::class);
         $id = $request->param('ID');