From d79d07175eb5481fe227e863584e2e77c747998c Mon Sep 17 00:00:00 2001 From: Davo Smith Date: Tue, 15 Oct 2013 14:30:33 +0100 Subject: [PATCH] MDL-35402 forms - prevent crash when multiple rules defined on an editor element --- lib/formslib.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/formslib.php b/lib/formslib.php index f282a3c8c3f..71e213a8015 100644 --- a/lib/formslib.php +++ b/lib/formslib.php @@ -2135,8 +2135,9 @@ class MoodleQuickForm extends HTML_QuickForm_DHTMLRulesTableless { } } //for editor element, [text] is appended to the name. + $fullelementname = $elementName; if ($element->getType() == 'editor') { - $elementName .= '[text]'; + $fullelementname .= '[text]'; //Add format to rule as moodleform check which format is supported by browser //it is not set anywhere... So small hack to make sure we pass it down to quickform if (is_null($rule['format'])) { @@ -2144,8 +2145,8 @@ class MoodleQuickForm extends HTML_QuickForm_DHTMLRulesTableless { } } // Fix for bug displaying errors for elements in a group - $test[$elementName][0][] = $registry->getValidationScript($element, $elementName, $rule); - $test[$elementName][1]=$element; + $test[$fullelementname][0][] = $registry->getValidationScript($element, $fullelementname, $rule); + $test[$fullelementname][1]=$element; //end of fix } }