Skip to content

Commit 1482bac

Browse files
Pek5892Pek5892
committed
refactor: uniformate icone Aggiornamenti
1 parent b6d4751 commit 1482bac

4 files changed

Lines changed: 77 additions & 30 deletions

File tree

‎include/init/requirements.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -496,7 +496,7 @@
496496
}
497497

498498
$card_color = $general_status ? 'success' : 'danger';
499-
$card_icon = $general_status ? 'fa-check-circle' : 'fa-exclamation-circle';
499+
$card_icon = $general_status ? 'fa-check-circle' : 'fa-warning';
500500

501501
echo '
502502
<div class="card card-outline card-'.$card_color.' requirements-card mb-3 collapsable collapsed-card">

‎modules/aggiornamenti/actions.php‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,21 @@
284284
$user_name = $user ? $user['username'] : null;
285285
}
286286

287-
$results[] = [
287+
$result_item = [
288288
'id' => $key,
289289
'class' => $controllo,
290290
'name' => $nome_controllo,
291291
'last_execution' => $operation ? $operation['created_at'] : null,
292292
'last_user' => $user_name,
293293
];
294+
295+
// Aggiungi le date di filtro per il controllo DatiFattureElettroniche
296+
if ($controllo === DatiFattureElettroniche::class) {
297+
$result_item['period_start'] = $_SESSION['period_start'] ?? null;
298+
$result_item['period_end'] = $_SESSION['period_end'] ?? null;
299+
}
300+
301+
$results[] = $result_item;
294302
}
295303

296304
echo json_encode($results, JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES);

‎modules/aggiornamenti/controlli.php‎

Lines changed: 52 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ function avviaControlli(button) {
146146
dataType: "JSON",
147147
data: {
148148
id_module: globals.id_module,
149-
op: "controlli-disponibili",
149+
op: "controlli-ultima-esecuzione",
150150
},
151151
success: async function(controlli) {
152152
// Ripristino pulsante
@@ -257,6 +257,11 @@ function avviaControllo(controllo) {
257257
titleElement.addClass("requirements-card-title-success");
258258
cardElement.addClass("card-success");
259259
260+
// Aggiorna icona a check-circle
261+
let iconElement = titleElement.find(".requirements-icon");
262+
iconElement.removeClass("fa-info-circle fa-exclamation-circle fa-times-circle");
263+
iconElement.addClass("fa-check-circle");
264+
260265
// Pulisci il body e mostra il messaggio di successo
261266
bodyElement.html(`<p class="text-muted">'.tr('Nessun problema rilevato').'</p>`);
262267
@@ -286,7 +291,16 @@ function avviaControllo(controllo) {
286291
titleElement.addClass(titleColorCls);
287292
cardElement.addClass(cardColorCls);
288293
289-
// Ordina per gravità: warning prima di info
294+
// Aggiorna icona in base al colore
295+
let iconElement = titleElement.find(".requirements-icon");
296+
iconElement.removeClass("fa-info-circle fa-exclamation-circle fa-warning fa-times-circle");
297+
if (orphanRecords.length > 0) {
298+
iconElement.addClass("fa-exclamation-circle");
299+
} else {
300+
iconElement.addClass("fa-info-circle");
301+
}
302+
303+
// Ordina per gravita: warning prima di info
290304
if (orphanRecords.length > 0) {
291305
titleElement.append(` <span class="badge badge-warning ml-2">${orphanRecords.length}</span>`);
292306
}
@@ -327,15 +341,22 @@ function avviaControllo(controllo) {
327341
});
328342
});
329343
330-
// Aggiungi le nuove classi in base al tipo di badge più grave
344+
// Aggiungi le nuove classi in base al tipo di badge piu grave
345+
let iconElement = titleElement.find(".requirements-icon");
346+
iconElement.removeClass("fa-info-circle fa-exclamation-circle fa-warning fa-times-circle");
347+
331348
if (hasDanger) {
332349
headerElement.addClass("requirements-card-header-danger");
333350
titleElement.addClass("requirements-card-title-danger");
334351
cardElement.addClass("card-danger");
352+
iconElement.addClass("fa-times-circle");
335353
} else if (hasWarning) {
336354
headerElement.addClass("requirements-card-header-warning");
337355
titleElement.addClass("requirements-card-title-warning");
338356
cardElement.addClass("card-warning");
357+
iconElement.addClass("fa-exclamation-circle");
358+
} else {
359+
iconElement.addClass("fa-info-circle");
339360
}
340361
341362
// Mostra le badge per tipo di avviso
@@ -348,20 +369,36 @@ function avviaControllo(controllo) {
348369
if (infoCount > 0) {
349370
titleElement.append(` <span class="badge badge-info ml-2">${infoCount}</span>`);
350371
}
372+
373+
// Aggiungi le date di filtro in testo grigio
374+
if (controllo["period_start"] && controllo["period_end"]) {
375+
let dataInizio = new Date(controllo["period_start"]);
376+
let dataFine = new Date(controllo["period_end"]);
377+
let dataInizioFormattata = dataInizio.toLocaleDateString("it-IT");
378+
let dataFineFormattata = dataFine.toLocaleDateString("it-IT");
379+
titleElement.append(` <span style="color: #999; font-size: 0.9rem; margin-left: 10px;">${dataInizioFormattata} - ${dataFineFormattata}</span>`);
380+
}
351381
} else {
352-
// Se ci sono avvisi, determina il tipo di badge più grave
382+
// Se ci sono avvisi, determina il tipo di badge piu grave
353383
let hasDanger = records.some(r => r.type === "danger");
354384
let hasWarning = records.some(r => r.type === "warning");
355385
356386
// Aggiungi le nuove classi in base al tipo di badge
387+
let iconElement = titleElement.find(".requirements-icon");
388+
iconElement.removeClass("fa-info-circle fa-exclamation-circle fa-warning fa-times-circle");
389+
357390
if (hasDanger) {
358391
headerElement.addClass("requirements-card-header-danger");
359392
titleElement.addClass("requirements-card-title-danger");
360393
cardElement.addClass("card-danger");
394+
iconElement.addClass("fa-times-circle");
361395
} else if (hasWarning) {
362396
headerElement.addClass("requirements-card-header-warning");
363397
titleElement.addClass("requirements-card-title-warning");
364398
cardElement.addClass("card-warning");
399+
iconElement.addClass("fa-exclamation-circle");
400+
} else {
401+
iconElement.addClass("fa-info-circle");
365402
}
366403
367404
// Per altri controlli, mostra il contatore di errori
@@ -824,7 +861,7 @@ function initcard(controllo, success, records) {
824861
cssClass = "card-outline card-danger";
825862
headerClass = "requirements-card-header requirements-card-header-danger";
826863
titleClass = "requirements-card-title requirements-card-title-danger";
827-
icon = "exclamation-circle";
864+
icon = "times-circle";
828865
}
829866
830867
// Usa i colori determinati sopra
@@ -836,6 +873,7 @@ function initcard(controllo, success, records) {
836873
let cardColorClass = "card-info";
837874
let headerColorClass = "requirements-card-header-info";
838875
let titleColorClass = "requirements-card-title-info";
876+
let finalIcon = icon;
839877
840878
if (records.length > 0) {
841879
// Determina il colore più grave
@@ -865,10 +903,14 @@ function initcard(controllo, success, records) {
865903
cardColorClass = "card-danger";
866904
headerColorClass = "requirements-card-header-danger";
867905
titleColorClass = "requirements-card-title-danger";
906+
finalIcon = "times-circle";
868907
} else if (hasWarning) {
869908
cardColorClass = "card-warning";
870909
headerColorClass = "requirements-card-header-warning";
871910
titleColorClass = "requirements-card-title-warning";
911+
finalIcon = "warning";
912+
} else {
913+
finalIcon = "info-circle";
872914
}
873915
}
874916
@@ -881,7 +923,7 @@ function initcard(controllo, success, records) {
881923
let card = `<div class="card ` + finalCssClass + `" id="controllo-` + controllo["id"] + `" data-controllo-name="` + controllo["name"] + `" data-controllo-class="` + controllo["class"] + `">
882924
<div class="card-header with-border ` + finalHeaderClass + `">
883925
<h3 class="card-title ` + finalTitleClass + `">
884-
<i class="fa fa-` + icon + ` mr-2 requirements-icon"></i>` + controllo["name"];
926+
<i class="fa fa-` + finalIcon + ` mr-2 requirements-icon"></i>` + controllo["name"];
885927
886928
// Aggiungi badge inline per il controllo IntegritaFile
887929
if (controllo["class"] === "Modules\\\\Aggiornamenti\\\\Controlli\\\\IntegritaFile" && !success && records.length > 0) {
@@ -910,6 +952,8 @@ function initcard(controllo, success, records) {
910952
card += ` <span class="badge badge-danger ml-2">${records.length}</span>`;
911953
}
912954
955+
// Le date di filtro per il controllo DatiFattureElettroniche verranno aggiunte dopo esecuzione
956+
913957
card += `</h3>
914958
<div class="card-tools pull-right" style="display: flex; align-items: center; gap: 10px;">`;
915959
@@ -1253,7 +1297,7 @@ function eseguiAzioneGlobale(buttonElement) {
12531297
12541298
let isInfo = controlliInfo.includes(controlloClass);
12551299
let alertClass = isInfo ? "alert-info" : "alert-warning";
1256-
let iconClass = isInfo ? "fa-info-circle text-info" : "fa-exclamation-triangle text-warning";
1300+
let iconClass = isInfo ? "fa-info-circle text-info" : "fa-exclamation-circle text-warning";
12571301
12581302
// Genera la lista delle operazioni
12591303
let operazioniHtml = "";
@@ -1396,7 +1440,7 @@ function eseguiRisoluzioneGlobale(button, controlloId, controlloClass, successCa
13961440
13971441
let errorHtml = `
13981442
<div class="alert alert-danger">
1399-
<h4><i class="fa fa-exclamation-triangle"></i> '.tr('Errore durante la risoluzione').'</h4>
1443+
<h4><i class="fa fa-times-circle"></i> '.tr('Errore durante la risoluzione').'</h4>
14001444
<p>'.tr('Si è verificato un errore').': ${errorMessage}</p>
14011445
</div>
14021446
`;

‎modules/aggiornamenti/edit.php‎

Lines changed: 15 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ function highlightDifferences($current, $expected) {
263263
$has_any_errors = !empty($custom) || $has_file_errors || $has_table_errors || $has_view_errors || $has_module_errors || $has_field_errors;
264264

265265
$customizations_card_class = $has_any_errors ? 'card-warning' : 'card-success';
266-
$customizations_icon = $has_any_errors ? 'fa-exclamation-triangle' : 'fa-check';
266+
$customizations_icon = $has_any_errors ? 'fa-warning' : 'fa-check-circle';
267267
$customizations_title = $has_any_errors ? tr('Personalizzazioni Rilevate') : tr('Personalizzazioni');
268268

269269
echo '
@@ -284,9 +284,9 @@ function highlightDifferences($current, $expected) {
284284
$modified_files_count = count($checksum_errors_grouped);
285285
$custom_files_count = count($custom_files);
286286

287-
// Determina il colore in base all'avviso più grave
287+
// Determina il colore in base all'avviso piu grave
288288
$file_card_color = 'success';
289-
$file_icon = 'fa-check';
289+
$file_icon = 'fa-check-circle';
290290
if ($modified_files_count > 0) {
291291
$file_card_color = 'warning';
292292
$file_icon = 'fa-warning';
@@ -370,7 +370,7 @@ function highlightDifferences($current, $expected) {
370370
// Card Tabelle
371371
$table_count = count($tables);
372372
$table_card_color = $table_count > 0 ? 'info' : 'success';
373-
$table_icon = $table_count > 0 ? 'fa-info-circle' : 'fa-check';
373+
$table_icon = $table_count > 0 ? 'fa-info-circle' : 'fa-check-circle';
374374

375375
echo '
376376
<div class="card card-outline card-'.$table_card_color.' requirements-card mb-3 collapsable collapsed-card">
@@ -411,29 +411,25 @@ function highlightDifferences($current, $expected) {
411411
// Conta gli avvisi per tipo
412412
$view_warning_count = 0;
413413
$view_info_count = 0;
414-
$view_dark_count = 0;
415414

416415
if ($has_view_data_issues) {
417416
foreach ($custom_views_not_standard as $view) {
418417
match ($view['reason']) {
419418
'Vista aggiuntiva' => $view_info_count++,
420-
'Vista mancante' => $view_dark_count++,
419+
'Vista mancante' => $view_info_count++,
421420
'Query modificata' => $view_warning_count++,
422421
'Modulo non previsto' => $view_info_count++,
423422
default => null,
424423
};
425424
}
426425
}
427426

428-
// Determina il colore della card in base all'avviso più grave
427+
// Determina il colore della card in base all'avviso piu grave
429428
$view_card_color = 'success';
430-
$view_icon = 'fa-check';
429+
$view_icon = 'fa-check-circle';
431430
if ($view_warning_count > 0 || $views_file_missing) {
432431
$view_card_color = 'warning';
433432
$view_icon = 'fa-warning';
434-
} elseif ($view_dark_count > 0) {
435-
$view_card_color = 'dark';
436-
$view_icon = 'fa-check';
437433
} elseif ($view_info_count > 0) {
438434
$view_card_color = 'info';
439435
$view_icon = 'fa-info-circle';
@@ -446,7 +442,6 @@ function highlightDifferences($current, $expected) {
446442
<i class="fa '.$view_icon.' mr-2 requirements-icon"></i>
447443
'.tr('Viste personalizzate').'
448444
'.($view_warning_count > 0 ? '<span class="badge badge-warning ml-2">'.$view_warning_count.'</span>' : '').'
449-
'.($view_dark_count > 0 ? '<span class="badge badge-dark ml-2">'.$view_dark_count.'</span>' : '').'
450445
'.($view_info_count > 0 ? '<span class="badge badge-info ml-2">'.$view_info_count.'</span>' : '').'
451446
</h3>
452447
<div class="card-tools pull-right">
@@ -475,7 +470,7 @@ function highlightDifferences($current, $expected) {
475470
foreach ($custom_views_not_standard as $index => $view) {
476471
$badge_class = match ($view['reason']) {
477472
'Vista aggiuntiva' => 'badge-info',
478-
'Vista mancante' => 'badge-dark',
473+
'Vista mancante' => 'badge-info',
479474
'Query modificata' => 'badge-warning',
480475
'Modulo non previsto' => 'badge-info',
481476
default => 'badge-secondary',
@@ -531,7 +526,7 @@ function highlightDifferences($current, $expected) {
531526
} elseif ($views_file_missing) {
532527
echo '
533528
<div class="alert alert-warning alert-database">
534-
<i class="fa fa-warning"></i> '.tr('Impossibile effettuare il controllo delle viste in assenza del file _FILE_', [
529+
<i class="fa fa-exclamation-circle"></i> '.tr('Impossibile effettuare il controllo delle viste in assenza del file _FILE_', [
535530
'_FILE_' => '<b>views.json</b>',
536531
]).'.
537532
</div>';
@@ -564,9 +559,9 @@ function highlightDifferences($current, $expected) {
564559
}
565560
}
566561

567-
// Determina il colore della card in base all'avviso più grave
562+
// Determina il colore della card in base all'avviso piu grave
568563
$module_card_color = 'success';
569-
$module_icon = 'fa-check';
564+
$module_icon = 'fa-check-circle';
570565
if ($module_warning_count > 0 || $modules_file_missing) {
571566
$module_card_color = 'warning';
572567
$module_icon = 'fa-warning';
@@ -661,7 +656,7 @@ function highlightDifferences($current, $expected) {
661656
} elseif ($modules_file_missing) {
662657
echo '
663658
<div class="alert alert-warning alert-database">
664-
<i class="fa fa-warning"></i> '.tr('Impossibile effettuare il controllo dei moduli in assenza del file _FILE_', [
659+
<i class="fa fa-exclamation-circle"></i> '.tr('Impossibile effettuare il controllo dei moduli in assenza del file _FILE_', [
665660
'_FILE_' => '<b>modules.json</b>',
666661
]).'.
667662
</div>';
@@ -789,16 +784,16 @@ function highlightDifferences($current, $expected) {
789784
// Determina il colore in base all'avviso più grave
790785
if ($database_danger_count > 0) {
791786
$database_card_color = 'danger';
792-
$database_icon = 'fa-warning';
787+
$database_icon = 'fa-times-circle';
793788
} elseif ($database_warning_count > 0 || $database_file_missing) {
794789
$database_card_color = 'warning';
795-
$database_icon = 'fa-warning';
790+
$database_icon = 'fa-exclamation-circle';
796791
} elseif ($database_info_count > 0) {
797792
$database_card_color = 'info';
798793
$database_icon = 'fa-info-circle';
799794
} else {
800795
$database_card_color = 'success';
801-
$database_icon = 'fa-check';
796+
$database_icon = 'fa-check-circle';
802797
}
803798

804799

0 commit comments

Comments
 (0)