ENH Take advantage of auto-scaffolded form fields (#581)

silverstripe · Jun 26, 2024 · d1acb20 · d1acb20
1 parent ec2bc71
commit d1acb20
Showing 1 changed file with 5 additions and 10 deletions.
diff --git a/src/Model/Subsite.php b/src/Model/Subsite.php
@@ -35,6 +35,7 @@
 use SilverStripe\Versioned\Versioned;
 use UnexpectedValueException;
 use SilverStripe\Core\ClassInfo;
+use SilverStripe\Forms\GridField\GridFieldAddExistingAutocompleter;
 
 /**
  * A dynamically created subsite. SiteTree objects can now belong to a subsite.
@@ -666,16 +667,10 @@ public function canEdit($member = false)
     public function getCMSFields()
     {
         $this->beforeUpdateCMSFields(function (FieldList $fields) {
-            if ($this->exists()) {
-                // Add a GridField for domains to a new tab if the subsite has already been created
-                $fields->addFieldsToTab('Root.Domains', [
-                    GridField::create(
-                        'Domains',
-                        '',
-                        $this->Domains(),
-                        GridFieldConfig_RecordEditor::create(10)
-                    )
-                ]);
+            $domainsGridField = $fields->dataFieldByName('Domains');
+            if ($domainsGridField instanceof GridField) {
+                $domainsGridField->setTitle('');
+                $domainsGridField->getConfig()->removeComponentsByType(GridFieldAddExistingAutocompleter::class);
             }
 
             // Remove the default scaffolded blacklist field, we replace it with a checkbox set field