From 83edfba240f0ab643b59355fa7663fe55eef6693 Mon Sep 17 00:00:00 2001 From: "Eloy Lafuente (stronk7)" Date: Mon, 26 Sep 2016 23:16:15 +0200 Subject: [PATCH] MDL-56127 forms: multiple select dependencies fixed 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 | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/form/form.js b/lib/form/form.js index 43e209c394a..38e518bef44 100644 --- a/lib/form/form.js +++ b/lib/form/form.js @@ -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');