issue #8: add extra form validation

dmitriim · Sep 22, 2024 · adbcba6 · adbcba6
1 parent 14ce980
commit adbcba6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/classes/preset_form.php b/classes/preset_form.php
@@ -43,6 +43,8 @@ protected function definition() {
         $this->_form->addRule('name', get_string('required'), 'required');
         $this->_form->setType('name', PARAM_TEXT);
 
+        $this->_form->addElement('static', 'error', '');
+
         foreach ($this->get_field_types() as $type) {
             $methodname = 'get_'  . $type . '_options';
             $this->_form->addElement(
@@ -119,6 +121,18 @@ protected function get_tags_options(): array {
      */
     public function validation($data, $files) {
         $errors = parent::validation($data, $files);
+
+        $empty = 0;
+        foreach ($this->get_field_types() as $type) {
+            if (empty($data[$type])) {
+                $empty++;
+            }
+        }
+
+        if ($empty == count($this->get_field_types())) {
+            $errors['error'] = get_string('mustselectentities', 'tool_enrolprofile');
+        }
+
         return $errors;
     }
 

diff --git a/lang/en/tool_enrolprofile.php b/lang/en/tool_enrolprofile.php
@@ -44,3 +44,4 @@
 $string['fieldtype:categories'] = 'Categories';
 $string['fieldtype:courses'] = 'Courses';
 $string['fieldtype:tags'] = 'Tags';
+$string['mustselectentities'] = 'You must select at least one of the following entities';