From 63a002cdf0d9826dcad22ae419fc651a2bb04caf Mon Sep 17 00:00:00 2001
From: Pek5892 <pek5892@gmail.com>
Date: Tue, 7 May 2024 14:35:53 +0200
Subject: [PATCH] Fix minore

---
 actions.php                            |  2 +-
 modules/categorie_articoli/actions.php |  2 +-
 modules/categorie_impianti/actions.php |  2 +-
 modules/interventi/actions.php         | 18 +++++++++---------
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/actions.php b/actions.php
index 38d9e21cb..a750c422c 100755
--- a/actions.php
+++ b/actions.php
@@ -489,7 +489,7 @@ if ($structure->permission == 'rw') {
         ($include_file = $structure->filepath('actions.php')) ? include $include_file : null;
 
         // 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);
 
             $query = 'SELECT `id`, `html_name` AS `title` FROM `zz_fields` WHERE '.$custom_where;
diff --git a/modules/categorie_articoli/actions.php b/modules/categorie_articoli/actions.php
index 144f9a45e..abb592cce 100755
--- a/modules/categorie_articoli/actions.php
+++ b/modules/categorie_articoli/actions.php
@@ -40,7 +40,7 @@ switch (filter('op')) {
         }
 
         // Redirect alla categoria se si sta modificando una sottocategoria
-        if ($id_original != null) {
+        if (!empty($id_original)) {
             $database->commitTransaction();
             redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
             exit;
diff --git a/modules/categorie_impianti/actions.php b/modules/categorie_impianti/actions.php
index b90ca3400..3649a5398 100755
--- a/modules/categorie_impianti/actions.php
+++ b/modules/categorie_impianti/actions.php
@@ -45,7 +45,7 @@ switch (filter('op')) {
         }
 
         // Redirect alla categoria se si sta modificando una sottocategoria
-        if ($id_original != null) {
+        if (!empty($id_original)) {
             $database->commitTransaction();
             redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
             exit;
diff --git a/modules/interventi/actions.php b/modules/interventi/actions.php
index f4e24d537..c4f8dbe2f 100644
--- a/modules/interventi/actions.php
+++ b/modules/interventi/actions.php
@@ -89,7 +89,7 @@ switch (post('op')) {
             $tags_presenti[] = $tag['id_tag'];
         }
 
-        $tags = post('tags');
+        $tags = post('tags') ?: [];
         $tags_presenti = [];
 
         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 = [];
 
         foreach ($tecnici_assegnati_array as $tecnico_assegnato) {
@@ -504,7 +504,7 @@ switch (post('op')) {
         break;
 
     case 'manage_articolo':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $articolo = Articolo::find(post('idriga'));
         } else {
             $originale = ArticoloOriginale::find(post('idarticolo'));
@@ -533,7 +533,7 @@ switch (post('op')) {
 
         $articolo->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             flash()->info(tr('Articolo modificato!'));
         } else {
             flash()->info(tr('Articolo aggiunto!'));
@@ -555,7 +555,7 @@ switch (post('op')) {
         break;
 
     case 'manage_sconto':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $sconto = Sconto::find(post('idriga'));
         } else {
             $sconto = Sconto::build($intervento);
@@ -566,7 +566,7 @@ switch (post('op')) {
         $sconto->note = post('note');
         $sconto->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             flash()->info(tr('Sconto/maggiorazione modificato!'));
         } else {
             flash()->info(tr('Sconto/maggiorazione aggiunto!'));
@@ -575,7 +575,7 @@ switch (post('op')) {
         break;
 
     case 'manage_riga':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $riga = Riga::find(post('idriga'));
         } else {
             $riga = Riga::build($intervento);
@@ -596,7 +596,7 @@ switch (post('op')) {
 
         $riga->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))){
             flash()->info(tr('Riga modificata!'));
         } else {
             flash()->info(tr('Riga aggiunta!'));
@@ -1233,7 +1233,7 @@ switch (post('op')) {
         $numero_totale = 0;
 
         foreach ($righe as $riga) {
-            if ($riga['id'] != null) {
+            if (!empty($riga['id'])) {
                 $articolo = Articolo::find($riga['id']);
             }