FIX Tighten routing rule for userforms ping action

silverstripe · Aug 16, 2023 · 00a25dc · 00a25dc
1 parent 60717e5
commit 00a25dc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/_config/routes.yml b/_config/routes.yml
@@ -4,4 +4,4 @@ After: framework/routes#coreroutes
 ---
 SilverStripe\Control\Director:
   rules:
-    UserDefinedFormController//$Action: SilverStripe\UserForms\Control\UserDefinedFormController
+    UserDefinedFormController//ping: SilverStripe\UserForms\Control\UserDefinedFormController
diff --git a/code/Form/UserForm.php b/code/Form/UserForm.php
@@ -173,6 +173,7 @@ public function getRequiredFields()
             ->Fields()
             ->filter('Required', true)
             ->column('Name');
+
         $requiredNames = array_merge($requiredNames, $this->getEmailRecipientRequiredFields());
         $required = UserFormsRequiredFields::create($requiredNames);
         $this->extend('updateRequiredFields', $required);