Merge branch 'ent-6634-Bug-borrado-net-scan-scripts' into 'develop'

fixed wrong error message: recon task is not necessarily linked to script

See merge request artica/pandorafms!3601
This commit is contained in:
Daniel Rodriguez 2021-01-25 09:52:47 +01:00
commit 7a9d928a98
1 changed files with 9 additions and 16 deletions

View File

@ -350,28 +350,21 @@ class ManageNetScanScripts extends Wizard
['id_recon_script' => $id_script]
);
$result_dlt2 = db_process_sql_delete(
'trecon_task',
['id_recon_script' => $id_script]
);
if (!$result_dlt) {
$result = [
'error' => 1,
'msg' => __('Problem deleting Net scan Scripts'),
];
} else {
$result_dlt2 = db_process_sql_delete(
'trecon_task',
['id_recon_script' => $id_script]
);
if (!$result_dlt2) {
$result = [
'error' => 1,
'msg' => __('Problem deleting Net scan Scripts'),
];
} else {
$result = [
'error' => 0,
'msg' => __('Deleted successfully'),
];
}
$result = [
'error' => 0,
'msg' => __('Deleted successfully'),
];
}
return $result;