Fix minore

This commit is contained in:
Pek5892 2024-05-07 14:35:53 +02:00
parent 1dc8343e0b
commit 63a002cdf0
4 changed files with 12 additions and 12 deletions

View File

@ -489,7 +489,7 @@ if ($structure->permission == 'rw') {
($include_file = $structure->filepath('actions.php')) ? include $include_file : null; ($include_file = $structure->filepath('actions.php')) ? include $include_file : null;
// Operazioni generiche per i campi personalizzati // Operazioni generiche per i campi personalizzati
if (post('op') != null) { if (!empty(post('op'))) {
$custom_where = !empty($id_plugin) ? '`id_plugin` = '.prepare($id_plugin) : '`id_module` = '.prepare($id_module); $custom_where = !empty($id_plugin) ? '`id_plugin` = '.prepare($id_plugin) : '`id_module` = '.prepare($id_module);
$query = 'SELECT `id`, `html_name` AS `title` FROM `zz_fields` WHERE '.$custom_where; $query = 'SELECT `id`, `html_name` AS `title` FROM `zz_fields` WHERE '.$custom_where;

View File

@ -40,7 +40,7 @@ switch (filter('op')) {
} }
// Redirect alla categoria se si sta modificando una sottocategoria // Redirect alla categoria se si sta modificando una sottocategoria
if ($id_original != null) { if (!empty($id_original)) {
$database->commitTransaction(); $database->commitTransaction();
redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record)); redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
exit; exit;

View File

@ -45,7 +45,7 @@ switch (filter('op')) {
} }
// Redirect alla categoria se si sta modificando una sottocategoria // Redirect alla categoria se si sta modificando una sottocategoria
if ($id_original != null) { if (!empty($id_original)) {
$database->commitTransaction(); $database->commitTransaction();
redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record)); redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
exit; exit;

View File

@ -89,7 +89,7 @@ switch (post('op')) {
$tags_presenti[] = $tag['id_tag']; $tags_presenti[] = $tag['id_tag'];
} }
$tags = post('tags'); $tags = post('tags') ?: [];
$tags_presenti = []; $tags_presenti = [];
foreach ($tags as $tag) { foreach ($tags as $tag) {
@ -126,7 +126,7 @@ switch (post('op')) {
} }
} }
$tecnici_assegnati_array [] = post('tecnici_assegnati'); $tecnici_assegnati_array = post('tecnici_assegnati') ?: [];
$tecnici_assegnati = []; $tecnici_assegnati = [];
foreach ($tecnici_assegnati_array as $tecnico_assegnato) { foreach ($tecnici_assegnati_array as $tecnico_assegnato) {
@ -504,7 +504,7 @@ switch (post('op')) {
break; break;
case 'manage_articolo': case 'manage_articolo':
if (post('idriga') != null) { if (!empty(post('idriga'))) {
$articolo = Articolo::find(post('idriga')); $articolo = Articolo::find(post('idriga'));
} else { } else {
$originale = ArticoloOriginale::find(post('idarticolo')); $originale = ArticoloOriginale::find(post('idarticolo'));
@ -533,7 +533,7 @@ switch (post('op')) {
$articolo->save(); $articolo->save();
if (post('idriga') != null) { if (!empty(post('idriga'))) {
flash()->info(tr('Articolo modificato!')); flash()->info(tr('Articolo modificato!'));
} else { } else {
flash()->info(tr('Articolo aggiunto!')); flash()->info(tr('Articolo aggiunto!'));
@ -555,7 +555,7 @@ switch (post('op')) {
break; break;
case 'manage_sconto': case 'manage_sconto':
if (post('idriga') != null) { if (!empty(post('idriga'))) {
$sconto = Sconto::find(post('idriga')); $sconto = Sconto::find(post('idriga'));
} else { } else {
$sconto = Sconto::build($intervento); $sconto = Sconto::build($intervento);
@ -566,7 +566,7 @@ switch (post('op')) {
$sconto->note = post('note'); $sconto->note = post('note');
$sconto->save(); $sconto->save();
if (post('idriga') != null) { if (!empty(post('idriga'))) {
flash()->info(tr('Sconto/maggiorazione modificato!')); flash()->info(tr('Sconto/maggiorazione modificato!'));
} else { } else {
flash()->info(tr('Sconto/maggiorazione aggiunto!')); flash()->info(tr('Sconto/maggiorazione aggiunto!'));
@ -575,7 +575,7 @@ switch (post('op')) {
break; break;
case 'manage_riga': case 'manage_riga':
if (post('idriga') != null) { if (!empty(post('idriga'))) {
$riga = Riga::find(post('idriga')); $riga = Riga::find(post('idriga'));
} else { } else {
$riga = Riga::build($intervento); $riga = Riga::build($intervento);
@ -596,7 +596,7 @@ switch (post('op')) {
$riga->save(); $riga->save();
if (post('idriga') != null) { if (!empty(post('idriga'))){
flash()->info(tr('Riga modificata!')); flash()->info(tr('Riga modificata!'));
} else { } else {
flash()->info(tr('Riga aggiunta!')); flash()->info(tr('Riga aggiunta!'));
@ -1233,7 +1233,7 @@ switch (post('op')) {
$numero_totale = 0; $numero_totale = 0;
foreach ($righe as $riga) { foreach ($righe as $riga) {
if ($riga['id'] != null) { if (!empty($riga['id'])) {
$articolo = Articolo::find($riga['id']); $articolo = Articolo::find($riga['id']);
} }