fix(config): saveAll crash when saving modules
saveModules() used event.target, which is undefined when called from saveAll. Replaced with direct checkbox lookups and null checks.
This commit is contained in:
@@ -964,14 +964,17 @@ const Config = (() => {
|
|||||||
}
|
}
|
||||||
|
|
||||||
async function saveModules() {
|
async function saveModules() {
|
||||||
var btn = event.target;
|
var cbWa = document.getElementById('cfg-module-whatsapp');
|
||||||
if (btn) { btn.disabled = true; btn.textContent = 'Guardando...'; }
|
var cbMp = document.getElementById('cfg-module-marketplace');
|
||||||
|
var cbMeli = document.getElementById('cfg-module-meli');
|
||||||
|
var cbCat = document.getElementById('cfg-module-catalog');
|
||||||
|
if (!cbWa && !cbMp && !cbMeli && !cbCat) return;
|
||||||
try {
|
try {
|
||||||
var data = {
|
var data = {
|
||||||
whatsapp: document.getElementById('cfg-module-whatsapp').checked,
|
whatsapp: cbWa ? cbWa.checked : true,
|
||||||
marketplace: document.getElementById('cfg-module-marketplace').checked,
|
marketplace: cbMp ? cbMp.checked : true,
|
||||||
meli: document.getElementById('cfg-module-meli').checked,
|
meli: cbMeli ? cbMeli.checked : true,
|
||||||
catalog: document.getElementById('cfg-module-catalog').checked,
|
catalog: cbCat ? cbCat.checked : true,
|
||||||
};
|
};
|
||||||
var res = await fetch(API + '/modules', {
|
var res = await fetch(API + '/modules', {
|
||||||
method: 'PUT',
|
method: 'PUT',
|
||||||
@@ -986,8 +989,6 @@ const Config = (() => {
|
|||||||
toast('Módulos actualizados');
|
toast('Módulos actualizados');
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
toast(e.message, 'error');
|
toast(e.message, 'error');
|
||||||
} finally {
|
|
||||||
if (btn) { btn.disabled = false; btn.textContent = 'Guardar módulos'; }
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -928,7 +928,7 @@
|
|||||||
<script src="/pos/static/js/pos-utils.js?v=33" defer></script>
|
<script src="/pos/static/js/pos-utils.js?v=33" defer></script>
|
||||||
<script src="/pos/static/js/sidebar.js" defer></script>
|
<script src="/pos/static/js/sidebar.js" defer></script>
|
||||||
<script src="/pos/static/js/kiosk.js" defer></script>
|
<script src="/pos/static/js/kiosk.js" defer></script>
|
||||||
<script src="/pos/static/js/config.js?v=34" defer></script>
|
<script src="/pos/static/js/config.js?v=35" defer></script>
|
||||||
<script src="/pos/static/js/sync-engine.js" defer></script>
|
<script src="/pos/static/js/sync-engine.js" defer></script>
|
||||||
<script>if('serviceWorker' in navigator){navigator.serviceWorker.register('/pos/sw.js',{scope:'/pos/'});}</script>
|
<script>if('serviceWorker' in navigator){navigator.serviceWorker.register('/pos/sw.js',{scope:'/pos/'});}</script>
|
||||||
<script src="/pos/static/js/pwa-install.js" defer></script>
|
<script src="/pos/static/js/pwa-install.js" defer></script>
|
||||||
|
|||||||
Reference in New Issue
Block a user