Merge branch 'ent-8179-Mensaje-anti-hacking-al-intentar-eliminar-modulos' into 'develop'

minor fix

See merge request artica/pandorafms!4562
This commit is contained in:
Daniel Rodriguez 2021-12-23 12:16:54 +00:00
commit 7f78a5b514
1 changed files with 30 additions and 20 deletions

View File

@ -253,6 +253,7 @@ $module_action = (string) get_parameter('module_action');
if ($module_action === 'delete') {
$id_agent_modules_delete = (array) get_parameter('id_delete');
$print_result_msg = true;
$count_correct_delete_modules = 0;
foreach ($id_agent_modules_delete as $id_agent_module_del) {
$id_grupo = (int) agents_get_agent_group($id_agente);
@ -268,12 +269,19 @@ if ($module_action === 'delete') {
}
if ($id_agent_module_del < 1) {
db_pandora_audit(
'HACK Attempt',
'Expected variable from form is not correct'
if (count($id_agent_modules_delete) === 1) {
ui_print_error_message(
__('No modules selected')
);
die(__('Nice try buddy'));
exit;
$print_result_msg = false;
} else {
ui_print_error_message(
__('There was a problem completing the operation')
);
}
continue;
}
enterprise_include_once('include/functions_config_agents.php');
@ -421,6 +429,7 @@ if ($module_action === 'delete') {
}
}
if ($print_result_msg === true) {
$count_modules_to_delete = count($id_agent_modules_delete);
if ($count_correct_delete_modules == 0) {
ui_print_error_message(
@ -442,6 +451,7 @@ if ($module_action === 'delete') {
);
}
}
}
} else if ($module_action === 'disable') {
$id_agent_modules_disable = (array) get_parameter('id_delete');