MDL-56127 forms: multiple select dependencies fixed
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Mon, 26 Sep 2016 21:16:15 +0000 (23:16 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Thu, 29 Sep 2016 14:53:08 +0000 (16:53 +0200)
This is a fix for a regressions introduced by MDL-54778. It
seems that escaped all our tests because there isn't any case
of such a dependency in core, but Daniele Cordella was facing
it within his surveypro module.

lib/form/form.js

index 43e209c..38e518b 100644 (file)
@@ -497,6 +497,7 @@ if (typeof M.form.dependencyManager === 'undefined') {
                     // Multiple selects can have one or more value assigned. A pipe (|) is used as a value separator
                     // when multiple values have to be selected at the same time.
                     values = value.split('|');
+                    selected = [];
                     this.get('options').each(function() {
                         if (this.get('selected')) {
                             selected[selected.length] = this.get('value');