diff options
| -rw-r--r-- | plugins/managesieve/Changelog | 2 | ||||
| -rw-r--r-- | plugins/managesieve/managesieve.js | 13 |
2 files changed, 9 insertions, 6 deletions
diff --git a/plugins/managesieve/Changelog b/plugins/managesieve/Changelog index 3f4f154b2..16d7ca8bd 100644 --- a/plugins/managesieve/Changelog +++ b/plugins/managesieve/Changelog @@ -1,3 +1,5 @@ +- Fix filters set label after activation + * version 2.4 [2010-04-01] ----------------------------------------------------------- - Fixed bug in DIGEST-MD5 authentication (http://pear.php.net/bugs/bug.php?id=17285) diff --git a/plugins/managesieve/managesieve.js b/plugins/managesieve/managesieve.js index f93162da3..549afd077 100644 --- a/plugins/managesieve/managesieve.js +++ b/plugins/managesieve/managesieve.js @@ -438,14 +438,15 @@ if (window.rcmail) { if (!this.gui_objects.filtersetslist || !name) return false; - var opts = this.gui_objects.filtersetslist.getElementsByTagName('option'); - var regx = new RegExp(RegExp.escape(' (' + this.get_label('managesieve.active') + ')')); + var opts = this.gui_objects.filtersetslist.getElementsByTagName('option'), + label = ' (' + this.get_label('managesieve.active') + ')', + regx = new RegExp(RegExp.escape(label)); - for (var x=1; x<opts.length; x++) + for (var x=0; x<opts.length; x++) if (opts[x].value != name && opts[x].innerHTML.match(regx)) - opts[x].innerHTML = opts[x].innerHTML.replace(regx, ''); - else if (opts[x].value == name) - opts[x].innerHTML = opts[x].innerHTML + ' (' + this.get_label('managesieve.active') + ')'; + opts[x].innerHTML = opts[x].innerHTML.replace(regx, ''); + else if (opts[x].value == name) + opts[x].innerHTML = opts[x].innerHTML + label; }; // Set delete |