Merge pull request #1042 from creative-commoners/pulls/4.10/remove-field

FIX Remove TopPageID from editable fields
silverstripe · Mar 3, 2023 · 6a71b65 · 6a71b65
2 parents 00840f6 + 758741f
commit 6a71b65
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/TopPage/DataExtension.php b/src/TopPage/DataExtension.php
@@ -12,6 +12,7 @@
 use SilverStripe\ORM\ValidationException;
 use SilverStripe\Versioned\Versioned;
 use SilverStripe\View\ViewableData;
+use SilverStripe\Forms\FieldList;
 
 /**
  * Class DataExtension
@@ -255,6 +256,11 @@ public function withFixedTopPage(int $topPageID, callable $callback)
         }
     }
 
+    public function updateCMSFields(FieldList $fields)
+    {
+        $fields->removeByName('TopPageID');
+    }
+
     /**
      * Get the ID of a page which is currently set as the fixed top page
      *

diff --git a/src/TopPage/FluentExtension.php b/src/TopPage/FluentExtension.php
@@ -8,6 +8,7 @@
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\Queries\SQLUpdate;
 use TractorCow\Fluent\State\FluentState;
+use SilverStripe\Forms\FieldList;
 
 /**
  * Class FluentExtension
@@ -29,6 +30,12 @@ class FluentExtension extends DataExtension
         'TopPageLocale' => 'Varchar',
     ];
 
+    public function updateCMSFields(FieldList $fields)
+    {
+        $fields->removeByName('TopPageID');
+        $fields->removeByName('TopPageLocale');
+    }
+
     /*
      * @inheritdoc
      */