fix: allineamento con importFE

Pek5892 · Pek5892 · commit 47c53c69a709 · 2026-04-13T10:12:15.000+02:00
diff --git a/plugins/importFE_ZIP/actions.php b/plugins/importFE_ZIP/actions.php
@@ -510,6 +510,14 @@
         // Riorganizzazione dati ordini per numero di riga
         $dati_ordini = [];
         foreach ($DatiOrdini as $dato) {
+            if (!isset($dato['RiferimentoNumeroLinea']) || !$dato['RiferimentoNumeroLinea']) {
+                $dati_ordini_documento[] = [
+                    'numero' => $dato['IdDocumento'],
+                    'data' => (new Carbon($dato['Data']))->format('d/m/Y'),
+                ];
+                continue;
+            }
+                
             if (is_array($dato['RiferimentoNumeroLinea'])) {
                 foreach ($dato['RiferimentoNumeroLinea'] as $dati => $linea) {
                     foreach ($replaces as $replace) {
@@ -594,6 +602,28 @@
             }
         }
 
+        // Aggiungi gli ordini senza riferimento a riga specifica
+        if (isset($dati_ordini_documento)) {
+            foreach ($dati_ordini_documento as $ordine) {
+                $riferimento_fe[] = tr('Ordine _NUMERO_ del _DATA_',
+                    [
+                        '_NUMERO_' => $ordine['numero'],
+                        '_DATA_' => $ordine['data'],
+                    ]);
+            }
+        }
+
+        // Aggiungi i DDT senza riferimento a riga specifica
+        if (isset($dati_ddt_documento)) {
+            foreach ($dati_ddt_documento as $ddt) {
+                $riferimento_fe[] = tr('DDT _NUMERO_ del _DATA_',
+                    [
+                        '_NUMERO_' => $ddt['numero'],
+                        '_DATA_' => $ddt['data'],
+                    ]);
+            }
+        }
+
         // Iterazione sulle singole righe
         $righe = $fattura->getRighe();
         foreach ($righe as $key => $riga) {
@@ -644,48 +674,50 @@
                 }
             }
 
-            // Se nella fattura elettronica è indicato un DDT cerco quel documento specifico
-            $ddt = $dati_ddt[$numero_linea];
-            $query = "SELECT
-                `dt_righe_ddt`.`id`,
-                `dt_righe_ddt`.`idddt` AS id_documento,
-                `dt_righe_ddt`.`is_descrizione`,
-                `dt_righe_ddt`.`idarticolo`,
-                `dt_righe_ddt`.`is_sconto`, 'ddt' AS ref,
-                CONCAT('DDT num. ', IF(`numero_esterno` != '', `numero_esterno`, `numero`), ' del ', DATE_FORMAT(`data`, '%d/%m/%Y'), ' [', `dt_statiddt_lang`.`title`, ']') AS opzione
-            FROM
-                `dt_righe_ddt`
-                INNER JOIN `dt_ddt` ON `dt_ddt`.`id` = `dt_righe_ddt`.`idddt`
-                INNER JOIN `dt_statiddt` ON `dt_statiddt`.`id` = `dt_ddt`.`idstatoddt`
-                LEFT JOIN `dt_statiddt_lang` ON `dt_statiddt_lang`.`id_record` = `dt_statiddt`.`id` AND `dt_statiddt_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).'
-            WHERE
-                `dt_ddt`.`numero_esterno` = '.prepare($ddt['numero']).' AND
-                YEAR(`dt_ddt`.`data`) = '.prepare($ddt['anno']).' AND
-                `dt_ddt`.`idanagrafica` = '.prepare($anagrafica->id).' AND
-                `dt_righe_ddt`.`qta` > `dt_righe_ddt`.`qta_evasa` AND
-                |where|';
-
-            // Ricerca di righe DDT con stesso Articolo
-            if (!empty($id_articolo)) {
-                $query_articolo = replace($query, [
-                    '|where|' => '`dt_righe_ddt`.`idarticolo` = '.prepare($id_articolo),
-                ]);
+            // Se nella fattura elettronica è indicato un DDT cerco SOLO quel documento specifico
+            if (isset($dati_ddt[$numero_linea])) {
+                $ddt = $dati_ddt[$numero_linea];
+                $query = "SELECT
+                    `dt_righe_ddt`.`id`,
+                    `dt_righe_ddt`.`idddt` AS id_documento,
+                    `dt_righe_ddt`.`is_descrizione`,
+                    `dt_righe_ddt`.`idarticolo`,
+                    `dt_righe_ddt`.`is_sconto`, 'ddt' AS ref,
+                    CONCAT('DDT num. ', IF(`numero_esterno` != '', `numero_esterno`, `numero`), ' del ', DATE_FORMAT(`data`, '%d/%m/%Y'), ' [', `dt_statiddt_lang`.`title`, ']') AS opzione
+                FROM
+                    `dt_righe_ddt`
+                    INNER JOIN `dt_ddt` ON `dt_ddt`.`id` = `dt_righe_ddt`.`idddt`
+                    INNER JOIN `dt_statiddt` ON `dt_statiddt`.`id` = `dt_ddt`.`idstatoddt`
+                    LEFT JOIN `dt_statiddt_lang` ON `dt_statiddt_lang`.`id_record` = `dt_statiddt`.`id` AND `dt_statiddt_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).'
+                WHERE
+                    `dt_ddt`.`numero_esterno` = '.prepare($ddt['numero']).' AND
+                    YEAR(`dt_ddt`.`data`) = '.prepare($ddt['anno']).' AND
+                    `dt_ddt`.`idanagrafica` = '.prepare($anagrafica->id).' AND
+                    `dt_righe_ddt`.`qta` > `dt_righe_ddt`.`qta_evasa` AND
+                    |where|';
 
-                $collegamento = $database->fetchOne($query_articolo);
-            }
+                // Ricerca di righe DDT con stesso Articolo
+                if (!empty($id_articolo)) {
+                    $query_articolo = replace($query, [
+                        '|where|' => '`dt_righe_ddt`.`idarticolo` = '.prepare($id_articolo),
+                    ]);
 
-            // Ricerca di righe DDT per stessa descrizione
-            if (empty($collegamento)) {
-                $query_descrizione = replace($query, [
-                    '|where|' => '`dt_righe_ddt`.`descrizione` = '.prepare($riga['Descrizione']),
-                ]);
+                    $collegamento = $database->fetchOne($query_articolo);
+                }
 
-                $collegamento = $database->fetchOne($query_descrizione);
+                // Ricerca di righe DDT per stessa descrizione
+                if (empty($collegamento)) {
+                    $query_descrizione = replace($query, [
+                        '|where|' => '`dt_righe_ddt`.`descrizione` = '.prepare($riga['Descrizione']),
+                    ]);
+
+                    $collegamento = $database->fetchOne($query_descrizione);
+                }
             }
 
-            // Se nella fattura elettronica NON è indicato un DDT ed è indicato anche un ordine
-            // cerco per quell'ordine
-            if (empty($collegamento)) {
+            // Se nella fattura elettronica NON è indicato un DDT ed è indicato un ordine
+            // cerco SOLO per quell'ordine specifico
+            if (empty($collegamento) && isset($dati_ordini[$numero_linea])) {
                 $ordine = $dati_ordini[$numero_linea];
                 $query = "SELECT
                     `or_righe_ordini`.`id`,
@@ -727,10 +759,10 @@
 
             /*
              * TENTATIVO 2: ricerca solo per articolo o descrizione su documenti
-             * non referenziati nella fattura elettronica
+             * SOLO se nella fattura elettronica NON sono specificati riferimenti per questa riga
              */
-            // Se non ci sono Ordini o DDT cerco per contenuto
-            if (empty($collegamento)) {
+            // Se non ci sono riferimenti specifici nell'XML E non ho trovato collegamenti, cerco per contenuto
+            if (empty($collegamento) && !isset($dati_ddt[$numero_linea]) && !isset($dati_ordini[$numero_linea])) {
                 $match_documento_da_fe = false;
                 $query = "SELECT
                         `dt_righe_ddt`.`id`,
diff --git a/plugins/importFE_ZIP/buttons.php b/plugins/importFE_ZIP/buttons.php
@@ -105,6 +105,11 @@ function compilaRiferimenti(btn) {
                 return;
             }
 
+            // Imposta il flag per indicare che siamo in modalità ricerca automatica
+            if (typeof isAutomaticReferenceSearch !== "undefined") {
+                isAutomaticReferenceSearch = true;
+            }
+
             for (id_riga in response) {
                 data = response[id_riga];
 
@@ -114,6 +119,13 @@ function compilaRiferimenti(btn) {
                 // Impostazione del riferimento
                 impostaRiferimento(id_riga, data.documento, data.riga);
             }
+
+            // Reimposta il flag dopo aver completato la ricerca automatica (con un piccolo delay per gestire operazioni asincrone)
+            setTimeout(function() {
+                if (typeof isAutomaticReferenceSearch !== "undefined") {
+                    isAutomaticReferenceSearch = false;
+                }
+            }, 100);
         },
         error: function(data) {
             swal("'.tr('Errore').'", "'.tr('La ricerca automatica dei riferimenti per le righe non è andata a buon fine').'.", "error");
diff --git a/plugins/importFE_ZIP/generate.php b/plugins/importFE_ZIP/generate.php
@@ -189,7 +189,7 @@ function cleanup(){
                 </h3>
             </div>
             <div class="card-body">
-                <h4>'.$ragione_sociale.' '.(empty($anagrafica) ? '<span class="badge badge-warning">'.tr('Nuova anagrafica').'</span>' : '<small>'.Modules::link('Anagrafiche', $anagrafica->id, '', null, '').'</small>').'</h4>
+                <h4>'.$ragione_sociale.' '.(empty($anagrafica) ? '<span class="badge badge-warning">'.tr('Nuova anagrafica').'</span>' : '<small>'.Modules::link('Anagrafiche', $anagrafica->id, '', '', '').'</small>').'</h4>
                 <hr>
                 <small>
                     '.(!empty($codice_fiscale) ? ('<strong>'.tr('Codice Fiscale').':</strong> '.$codice_fiscale.'<br>') : '').'
@@ -286,28 +286,37 @@ function cleanup(){
     $query = $query_tipo;
 }
 
-$id_tipodocumento = $database->fetchOne($query_tipo)['id'];
+$tipo_doc = $database->fetchOne($query_tipo);
+$id_tipodocumento = $tipo_doc['id'] ?? null;
 
 echo '
         <div class="row">
             <div class="col-md-3">
-                {[ "type": "select", "label": "'.tr('Tipo fattura').'", "name": "id_tipo", "required": 1, "values": "query='.$query.'", "value": "'.($numero_tipo != 1 ? $id_tipodocumento : '').'" ]}
+                {[ "type": "select", "label": "'.tr('Tipo fattura').'", "name": "id_tipo", "required": 1, "values": "query='.$query.'", "value": "'.($numero_tipo != 1 && !empty($id_tipodocumento) ? $id_tipodocumento : '').'" ]}
             </div>';
 
 // Sezionale
-$id_segment = $database->table('co_tipidocumento')->where('id', '=', $id_tipodocumento)->value('id_segment');
+$id_segment = null;
+if (!empty($id_tipodocumento)) {
+    $id_segment = $database->table('co_tipidocumento')->where('id', '=', $id_tipodocumento)->value('id_segment');
+}
 
 echo '
             <div class="col-md-3">
-                {[ "type": "select", "label": "'.tr('Sezionale').'", "name": "id_segment", "required": 1, "ajax-source": "segmenti", "select-options": '.json_encode(['id_module' => $id_module, 'is_fiscale' => 1, 'is_sezionale' => 1, 'for_fe' => 1]).', "value": "'.$id_segment.'" ]}
+                {[ "type": "select", "label": "'.tr('Sezionale').'", "name": "id_segment", "required": 1, "ajax-source": "segmenti", "select-options": '.json_encode(['id_module' => $id_module, 'is_fiscale' => 1, 'is_sezionale' => 1, 'for_fe' => 1]).', "value": "'.($id_segment ?: '').'" ]}
             </div>';
 
 // Data di registrazione
 $data_registrazione = get('data_registrazione');
-$data_registrazione = new Carbon($data_registrazione);
+if (!empty($data_registrazione)) {
+    $data_registrazione = new Carbon($data_registrazione);
+} else {
+    $data_registrazione = null;
+}
+$data_registrazione_value = !empty($data_registrazione) ? $data_registrazione->format('Y-m-d') : $dati_generali['Data'];
 echo '
             <div class="col-md-3">
-                {[ "type": "date", "label": "'.tr('Data di registrazione').'", "name": "data_registrazione", "required": 1, "value": "'.($data_registrazione ?: $dati_generali['Data']).'", "max-date": "-now-", "min-date": "'.$dati_generali['Data'].'" ]}
+                {[ "type": "date", "label": "'.tr('Data di registrazione').'", "name": "data_registrazione", "required": 1, "value": "'.$data_registrazione_value.'", "max-date": "-now-", "min-date": "'.$dati_generali['Data'].'" ]}
             </div>';
 
 if (!empty($anagrafica)) {
@@ -374,6 +383,9 @@ function cleanup(){
 
 // Movimentazioni
 echo '
+            <div class="col-md-3">
+                {[ "type": "checkbox", "label": "'.tr('Movimenta gli articoli').'", "name": "movimentazione", "value": "'.setting('Movimenta magazzino da fatture di vendita').'" ]}
+            </div>
 
             <div class="col-md-3">
                 {[ "type": "checkbox", "label": "'.tr('Creazione automatica articoli').'", "name": "flag_crea_articoli", "value": 0, "help": "'.tr('Nel caso di righe con almeno un nodo \'CodiceArticolo\', il gestionale procede alla creazione dell\'articolo se la riga non risulta assegnata manualmente').'." ]}
@@ -434,24 +446,40 @@ function cleanup(){
 
     // Riorganizzazione dati ordini per numero di riga
     $dati_ordini = [];
+    $dati_ordini_documento = [];
     foreach ($DatiOrdini as $dato) {
         foreach ($dato['RiferimentoNumeroLinea'] as $dati => $linea) {
             $dati_ordini[(int) $linea] = [
                 'numero' => $dato['IdDocumento'],
                 'data' => (new Carbon($dato['Data']))->format('d/m/Y'),
             ];
         }
+
+        if (!$dato['RiferimentoNumeroLinea']) {
+            $dati_ordini_documento[] = [
+                'numero' => $dato['IdDocumento'],
+                'data' => (new Carbon($dato['Data']))->format('d/m/Y'),
+            ];
+        }
     }
 
-    // Riorganizzazione dati ordini per numero di riga
+    // Riorganizzazione dati ddt per numero di riga
     $dati_ddt = [];
+    $dati_ddt_documento = [];
     foreach ($DatiDDT as $dato) {
         foreach ($dato['RiferimentoNumeroLinea'] as $dati => $linea) {
             $dati_ddt[(int) $linea] = [
                 'numero' => $dato['NumeroDDT'],
                 'data' => (new Carbon($dato['DataDDT']))->format('d/m/Y'),
             ];
         }
+
+        if (!$dato['RiferimentoNumeroLinea']) {
+            $dati_ddt_documento[] = [
+                'numero' => $dato['NumeroDDT'],
+                'data' => (new Carbon($dato['DataDDT']))->format('d/m/Y'),
+            ];
+        }
     }
 
     foreach ($righe as $key => $riga) {
@@ -490,21 +518,32 @@ function cleanup(){
         // Prendo il codice articolo dal primo nodo CodiceValore che trovo
         $codice_principale = $codici[0]['CodiceValore'];
         if (!empty($codice_principale)) {
+            if (!empty($anagrafica) && empty($id_articolo)) {
+                $result = $database->fetchOne('SELECT `id_articolo` AS id FROM `mg_fornitore_articolo` WHERE `codice_fornitore` = '.prepare($codice_principale).' AND `id_fornitore` = '.prepare($anagrafica->id));
+                $id_articolo = $result['id'] ?? null;
+                if (empty($id_articolo)) {
+                    $result = $database->fetchOne('SELECT `id_articolo` AS id FROM `mg_fornitore_articolo` WHERE REPLACE(`codice_fornitore`, " ", "") = '.prepare($codice_principale).' AND `id_fornitore` = '.prepare($anagrafica->id));
+                    $id_articolo = $result['id'] ?? null;
+                }
+            }
+
             if (empty($id_articolo)) {
-                $id_articolo = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE `codice` = '.prepare($codice_principale))['id'];
+                $result = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE `codice` = '.prepare($codice_principale));
+                $id_articolo = $result['id'] ?? null;
                 if (empty($id_articolo)) {
-                    $id_articolo = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE REPLACE(`codice`, " ", "") = '.prepare($codice_principale))['id'];
+                    $result = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE REPLACE(`codice`, " ", "") = '.prepare($codice_principale));
+                    $id_articolo = $result['id'] ?? null;
                 }
             }
 
             $idconto_vendita = $database->fetchOne('SELECT `idconto_vendita` FROM `mg_articoli` WHERE `id` = '.prepare($id_articolo))['idconto_vendita'];
         }
 
         $idconto_vendita = $is_autofattura ? setting('Conto per autofattura') : $idconto_vendita;
-        $qta = $riga['Quantita'];
+        $qta = (empty($riga['Quantita']) && (!empty($riga['PrezzoUnitario']) || !empty($riga['Importo'])) ? 1 : $riga['Quantita']);
         $um = $riga['UnitaMisura'];
         $prezzo_unitario = $riga['PrezzoUnitario'] ?: $riga['Importo'];
-        $is_descrizione = empty((float) $riga['Quantita']) && empty((float) $prezzo_unitario);
+        $is_descrizione = empty((float) $qta);
 
         $sconto_unitario = 0;
         $sconti = $riga['ScontoMaggiorazione'] ?: 0;
diff --git a/plugins/importFE_ZIP/init.php b/plugins/importFE_ZIP/init.php
@@ -46,4 +46,20 @@
 
         redirect_url(base_path_osm().'/controller.php?id_module='.$id_module);
     }
+
+    // Se la fattura è già stata importata e siamo in modalità sequenza, passa alla successiva
+    if (!empty($imported) && get('sequence') == 1) {
+        $next_record = $id_record + 1;
+        $next_file = $files[$next_record - 1] ?? null;
+
+        if (!empty($next_file)) {
+            flash()->info(tr('La fattura _NAME_ è già stata importata, passaggio alla successiva...', [
+                '_NAME_' => $record['name'],
+            ]));
+            redirect_url(base_path_osm().'/editor.php?id_module='.$id_module.'&id_plugin='.$id_plugin.'&id_record='.$next_record.'&sequence=1');
+        } else {
+            flash()->info(tr('Tutte le fatture salvate sono state importate!'));
+            redirect_url(base_path_osm().'/controller.php?id_module='.$id_module);
+        }
+    }
 }
diff --git a/plugins/importFE_ZIP/list.php b/plugins/importFE_ZIP/list.php
@@ -23,7 +23,7 @@
 use Carbon\Carbon;
 use Plugins\ImportFE\Interaction;
 
-$list = Interaction::getFileList($list = [], 'Fatture di vendita', 'Importazione FE');
+$list = Interaction::getFileList([], 'Fatture di vendita', 'Importazione FE');
 
 $directory = Plugins\ImportFE\FatturaElettronica::getImportDirectory('Fatture di vendita', 'Importazione FE');
 if (!empty($list)) {
