Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-27 17:16:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: relax selectors for client-side form validation so that all form elements are checked

Browse Source
This commit is contained in:
Julian Lam
2022-11-24 20:17:36 -05:00
parent 0e319a58cb
commit 43e7c98824
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
public/src/admin/settings.js
View File

@@ -66,7 +66,7 @@ define('admin/settings', ['uploader', 'mousetrap', 'hooks', 'alerts', 'settings'
saveBtn.off('click').on('click', function (e) { saveBtn.off('click').on('click', function (e) {
e.preventDefault(); e.preventDefault();
const ok = settings.check(document.querySelectorAll('#content [data-field][pattern]')); const ok = settings.check(document.querySelectorAll('#content [data-field]'));
if (!ok) { if (!ok) {
return; return;
} }

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

@@ -517,7 +517,8 @@ define('settings', ['hooks', 'alerts'], function (hooks, alerts) {
save: function (hash, formEl, callback) { save: function (hash, formEl, callback) {
formEl = $(formEl); formEl = $(formEl);
const controls = formEl.get(0).querySelectorAll('input[name][pattern]'); const controls = formEl.get(0).elements;
console.log(controls);
const ok = Settings.check(controls); const ok = Settings.check(controls);
if (!ok) { if (!ok) {
return; return;