holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use set_select_options() to set options read from template to make sure everything is done the same

Browse Source
This commit is contained in:
Nathan Gray 2012-05-02 19:45:35 +00:00
parent 0d5c3a5231
commit 2a0df92594
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
etemplate/js/et2_widget_selectbox.js
View File

@ -337,12 +337,12 @@ var et2_selectbox = et2_inputWidget.extend({
var options = et2_directChildrenByTagName(_node, "options");
for (var i = 0; i < options.length; i++)
{
this._appendOptionElement(
et2_readAttrWithDefault(options[i], "value", options[i].textContent),
options[i].textContent,
et2_readAttrWithDefault(options[i], "title", "")
);
this.options.select_options[et2_readAttrWithDefault(options[i], "value", options[i].textContent)] = {
"label": options[i].textContent,
"title": et2_readAttrWithDefault(options[i], "title", "")
};
}
this.set_select_options(this.options.select_options);
},
set_value: function(_value) {
@ -430,7 +430,7 @@ var et2_selectbox = et2_inputWidget.extend({
this.multiOptions.empty();
}
// Re-add empty, it's usually not there
if(this.options.empty_label && typeof _options[''] == 'undefined')
if(this.options.empty_label)
{
_options[''] = this.options.empty_label;
}