gandalf/secubox-openwrt
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(client-guardian): correct API module imports in logs and alerts views

Browse Source 
- Changed from L.require('client-guardian.api') to 'require client-guardian/api as API'
- Updated all api. references to API. in both files
- Resolves "api.getLogs is not a function" and "api.getAlerts is not a function" errors

The L.require() pattern doesn't work properly with async module loading in LuCI.
Using the proper 'require' directive ensures the module is loaded correctly.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
CyberMind-FR 2025-12-31 10:56:53 +01:00
parent 829aa1d399
commit c8b261ea5a
2 changed files with 9 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
luci-app-client-guardian/htdocs/luci-static/resources/view/client-guardian/alerts.js
View File

@ -6,14 +6,13 @@
'require uci'; 'require uci';
'require rpc'; 'require rpc';
'require ui'; 'require ui';
'require client-guardian/api as API';
var api = L.require('client-guardian.api');
return view.extend({ return view.extend({
load: function() { load: function() {
return Promise.all([ return Promise.all([
api.getAlerts(), API.getAlerts(),
api.getLogs(20, null) API.getLogs(20, null)
]); ]);
}, },
@ -216,7 +215,7 @@ return view.extend({
E('div', { 'class': 'spinning' }) E('div', { 'class': 'spinning' })
]); ]);
api.sendTestAlert('email').then(function(result) { API.sendTestAlert('email').then(function(result) {
ui.hideModal(); ui.hideModal();
if (result.success) { if (result.success) {
ui.addNotification(null, E('p', {}, '✅ Email de test envoyé avec succès!'), 'success'); ui.addNotification(null, E('p', {}, '✅ Email de test envoyé avec succès!'), 'success');
@ -232,7 +231,7 @@ return view.extend({
E('div', { 'class': 'spinning' }) E('div', { 'class': 'spinning' })
]); ]);
api.sendTestAlert('sms').then(function(result) { API.sendTestAlert('sms').then(function(result) {
ui.hideModal(); ui.hideModal();
if (result.success) { if (result.success) {
ui.addNotification(null, E('p', {}, '✅ SMS de test envoyé avec succès!'), 'success'); ui.addNotification(null, E('p', {}, '✅ SMS de test envoyé avec succès!'), 'success');

9
luci-app-client-guardian/htdocs/luci-static/resources/view/client-guardian/logs.js
View File

@ -4,14 +4,13 @@
'require dom'; 'require dom';
'require poll'; 'require poll';
'require ui'; 'require ui';
'require client-guardian/api as API';
var api = L.require('client-guardian.api');
return view.extend({ return view.extend({
refreshInterval: 5000, refreshInterval: 5000,
load: function() { load: function() {
return api.getLogs(100, null); return API.getLogs(100, null);
}, },
render: function(data) { render: function(data) {
@ -170,7 +169,7 @@ return view.extend({
E('div', { 'class': 'spinning' }) E('div', { 'class': 'spinning' })
]); ]);
api.getLogs(limit, null).then(function(data) { API.getLogs(limit, null).then(function(data) {
var container = document.getElementById('logs-container'); var container = document.getElementById('logs-container');
var logs = data.logs || []; var logs = data.logs || [];
@ -192,7 +191,7 @@ return view.extend({
var self = this; var self = this;
var limit = parseInt(document.getElementById('filter-limit')?.value || 100); var limit = parseInt(document.getElementById('filter-limit')?.value || 100);
return api.getLogs(limit, null).then(function(data) { return API.getLogs(limit, null).then(function(data) {
var container = document.getElementById('logs-container'); var container = document.getElementById('logs-container');
if (!container) return; if (!container) return;