Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-16 18:56:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

updated settings module (v2) to fix multiple selects and checkboxes using name instead of id, which isn't always guaranteed to be set.

Browse Source
This commit is contained in:
Julian Lam
2015-01-27 14:47:39 -05:00
parent d096a639d6
commit a3cadff1d3
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
public/src/modules/settings.js
View File

@@ -482,14 +482,14 @@ define('settings', function () {
formEl.find('input[type="checkbox"]').each(function (idx, inputEl) { formEl.find('input[type="checkbox"]').each(function (idx, inputEl) {
inputEl = $(inputEl); inputEl = $(inputEl);
if (!inputEl.is(':checked')) { if (!inputEl.is(':checked')) {
values[inputEl.attr('id')] = 'off'; values[inputEl.attr('name')] = 'off';
} }
}); });
// Normalizing value of multiple selects // Normalizing value of multiple selects
formEl.find('select[multiple]').each(function(idx, selectEl) { formEl.find('select[multiple]').each(function(idx, selectEl) {
selectEl = $(selectEl); selectEl = $(selectEl);
values[selectEl.attr('id')] = JSON.stringify(selectEl.val()); values[selectEl.attr('name')] = JSON.stringify(selectEl.val());
}); });
socket.emit('admin.settings.set', { socket.emit('admin.settings.set', {