Skip to content

Commit a88484e

Browse files
committed
ui: factor out nipa_select_add_option()
Signed-off-by: Jakub Kicinski <[email protected]>
1 parent 6e1044c commit a88484e

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

ui/nipa.js

+10-8
Original file line numberDiff line numberDiff line change
@@ -108,16 +108,21 @@ function nipa_filters_set_from_url()
108108
nipa_input_set_from_url("fl-pw");
109109
}
110110

111+
function nipa_select_add_option(select_elem, show_str, value)
112+
{
113+
const opt = document.createElement('option');
114+
opt.value = value;
115+
opt.innerHTML = show_str;
116+
select_elem.appendChild(opt);
117+
}
118+
111119
function nipa_filter_add_options(data_raw, elem_id, field)
112120
{
113121
var elem = document.getElementById(elem_id);
114122
var values = new Set();
115123

116124
// Re-create "all"
117-
const opt = document.createElement('option');
118-
opt.value = "";
119-
opt.innerHTML = "-- all --";
120-
elem.appendChild(opt);
125+
nipa_select_add_option(elem, "-- all --", "");
121126

122127
// Create the dynamic entries
123128
$.each(data_raw, function(i, v) {
@@ -127,10 +132,7 @@ function nipa_filter_add_options(data_raw, elem_id, field)
127132
values.add(v);
128133
});
129134
for (const value of values) {
130-
const opt = document.createElement('option');
131-
opt.value = value;
132-
opt.innerHTML = value;
133-
elem.appendChild(opt);
135+
nipa_select_add_option(elem, value, value);
134136
}
135137
}
136138

0 commit comments

Comments
 (0)