fix: log invio fatture da bulk

Author: Pek5892

modules/fatture/bulk.php

@@ -168,8 +168,8 @@
                 $fattura_elettronica = new FatturaElettronica($fattura->id);
 
                 // Verifica che la fattura sia in stato corretto per l'invio
-                // Accetta sia 'GEN' che NULL/vuoto (fatture appena generate)
-                if (!empty($fattura->codice_stato_fe) && $fattura->codice_stato_fe != 'GEN') {
+                // Accetta 'GEN' (generata), NULL/vuoto (appena generate), 'ERR' (trasmissione fallita)
+                if (!empty($fattura->codice_stato_fe) && $fattura->codice_stato_fe != 'GEN' && $fattura->codice_stato_fe != 'ERR') {
                     $skipped[] = $fattura->numero_esterno.' (stato: '.$fattura->codice_stato_fe.')';
                     continue;
                 }

plugins/exportFE/src/Interaction.php

@@ -73,8 +73,20 @@ public static function sendInvoice($id_record)
             ]);
             $body = static::responseBody($response);
 
+            // Validazione della risposta
+            if (empty($body) || !isset($body['status'])) {
+                logger()->error('Risposta API non valida per fattura '.$fattura->numero_esterno.': '.json_encode($body));
+
+                return [
+                    'code' => 500,
+                    'message' => tr('Risposta non valida dal server'),
+                ];
+            }
+
+            $status_code = (int) $body['status'];
+
             // Aggiornamento dello stato in base alla risposta
-            if ($body['status'] == 200 || $body['status'] == 301) {
+            if ($status_code == 200 || $status_code == 301) {
                 // Invio riuscito
                 database()->update('co_documenti', [
                     'codice_stato_fe' => 'WAIT',
@@ -91,7 +103,7 @@ public static function sendInvoice($id_record)
             }
 
             return [
-                'code' => $body['status'],
+                'code' => $status_code,
                 'message' => $body['message'] ?? tr('Risposta non valida dal server'),
             ];
         } catch (\UnexpectedValueException $e) {
@@ -120,9 +132,19 @@ public static function getInvoiceRecepits($id_record)
             ]);
             $body = static::responseBody($response);
 
+            // Validazione della risposta
+            if (empty($body) || !isset($body['status'])) {
+                logger()->error('Risposta API non valida per ricevute fattura ID '.$id_record.': '.json_encode($body));
+
+                return [
+                    'code' => 500,
+                    'message' => tr('Risposta non valida dal server'),
+                ];
+            }
+
             return [
-                'code' => $body['status'],
-                'results' => $body['results'],
+                'code' => (int) $body['status'],
+                'results' => $body['results'] ?? [],
             ];
         } catch (\UnexpectedValueException) {
         }

plugins/importFE/src/Interaction.php

@@ -57,8 +57,8 @@ public static function getRemoteList()
             $response = static::request('POST', 'fatture_da_importare');
             $body = static::responseBody($response);
 
-            if ($body['status'] == '200') {
-                $list = $body['results'];
+            if (!empty($body) && isset($body['status']) && (int) $body['status'] == 200) {
+                $list = $body['results'] ?? [];
             }
         }
 
@@ -119,8 +119,10 @@ public static function processInvoice($filename)
         $body = static::responseBody($response);
 
         $message = '';
-        if ($body['status'] != '200') {
-            $message = $body['status'].' - '.$body['message'];
+        if (empty($body) || !isset($body['status']) || (int) $body['status'] != 200) {
+            $status = $body['status'] ?? 'unknown';
+            $msg = $body['message'] ?? 'Errore sconosciuto';
+            $message = $status.' - '.$msg;
         }
 
         return $message;

plugins/receiptFE/src/Interaction.php

@@ -57,8 +57,8 @@ public static function getRemoteList()
             $response = static::request('POST', 'notifiche_da_importare');
             $body = static::responseBody($response);
 
-            if ($body['status'] == '200') {
-                $results = $body['results'];
+            if (!empty($body) && isset($body['status']) && (int) $body['status'] == 200) {
+                $results = $body['results'] ?? [];
 
                 foreach ($results as $result) {
                     $list[] = [
@@ -124,8 +124,10 @@ public static function processReceipt($filename)
         $body = static::responseBody($response);
 
         $result = true;
-        if ($body['status'] != '200') {
-            $result = $body['status'].' - '.$body['message'];
+        if (empty($body) || !isset($body['status']) || (int) $body['status'] != 200) {
+            $status = $body['status'] ?? 'unknown';
+            $msg = $body['message'] ?? 'Errore sconosciuto';
+            $result = $status.' - '.$msg;
         }
 
         return $result;