From 4b26ed6a838b3c20b75916aa667d9a83a13afc16 Mon Sep 17 00:00:00 2001
From: Damian Mooyman <damian@silverstripe.com>
Date: Mon, 4 Sep 2017 15:37:47 +1200
Subject: [PATCH] BUG Prevent treeview loading repeatedly on each page edit
 form Fixes #1949

---
 code/Controllers/CMSMain.php | 20 ++------------------
 1 file changed, 2 insertions(+), 18 deletions(-)

diff --git a/code/Controllers/CMSMain.php b/code/Controllers/CMSMain.php
index af5466e660..ae804f4ed5 100644
--- a/code/Controllers/CMSMain.php
+++ b/code/Controllers/CMSMain.php
@@ -138,7 +138,6 @@ class CMSMain extends LeftAndMain implements CurrentPageIdentifier, PermissionPr
 
     private static $url_handlers = [
         'EditForm/$ID' => 'EditForm',
-        'treeview/$ID' => 'treeview',
         'listview/$ParentID' => 'listview',
     ];
 
@@ -330,13 +329,7 @@ public function LinkListViewChildren($parentID)
      */
     public function LinkTreeViewDeferred()
     {
-        $link = $this->Link('treeview');
-        // Ensure selected page is encoded into URL
-        $selectedID = $this->currentPageID();
-        if ($selectedID) {
-            $link = Controller::join_links($link, $selectedID);
-        }
-        return $this->LinkWithSearch($link);
+        return $this->Link('treeview');
     }
 
     public function LinkPageEdit($id = null)
@@ -547,11 +540,6 @@ public function getTreeNodeClasses(SiteTree $node)
         // Get classes from object
         $classes = $node->CMSTreeClasses();
 
-        // Flag as current
-        if ($this->isCurrentPage($node)) {
-            $classes .= ' current';
-        }
-
         // Get status flag classes
         $flags = $node->getStatusFlags();
         if ($flags) {
@@ -1345,14 +1333,10 @@ protected function getArchiveWarningMessage($record)
      * This method exclusively handles deferred ajax requests to render the
      * pages tree deferred handler (no pjax-fragment)
      *
-     * @param HTTPRequest $request
      * @return string HTML
      */
-    public function treeview($request)
+    public function treeview()
     {
-        // Ensure selected page ID is highlighted
-        $pageID = $request->param('ID') ?: 0;
-        $this->setCurrentPageID($pageID);
         return $this->renderWith($this->getTemplatesWithSuffix('_TreeView'));
     }