refactor: associazione articolo in fase di importazione fattura di acquisto

Author: Pek5892

plugins/importFE/actions.php

@@ -428,20 +428,7 @@
                 }
 
                 if (empty($id_articolo)) {
-                    $id_articolo = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE `codice` = '.prepare($codice['CodiceValore']).' AND `deleted_at` IS NULL')['id'];
-
-                    if (empty($id_articolo)) {
-                        $id_articolo = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE REPLACE(`codice`, " ", "") = '.prepare($codice['CodiceValore']).' AND `deleted_at` IS NULL')['id'];
-                    }
-
-                    // Controllo se esistono articoli con barcode corrispondente al codice
-                    if (empty($id_articolo)) {
-                        $id_articolo = $database->fetchOne('SELECT `mg_articoli_barcode`.`idarticolo` FROM `mg_articoli_barcode` INNER JOIN `mg_articoli` ON `mg_articoli`.`id` = `mg_articoli_barcode`.`idarticolo` WHERE `mg_articoli_barcode`.`barcode` = '.prepare($codice['CodiceValore']).' AND `mg_articoli`.`deleted_at` IS NULL')['idarticolo'];
-                    }
-
-                    if (empty($id_articolo)) {
-                        $id_articolo = $database->fetchOne('SELECT `mg_articoli_barcode`.`idarticolo` FROM `mg_articoli_barcode` INNER JOIN `mg_articoli` ON `mg_articoli`.`id` = `mg_articoli_barcode`.`idarticolo` WHERE REPLACE(`mg_articoli_barcode`.`barcode`, " ", "") = '.prepare($codice['CodiceValore']).' AND `mg_articoli`.`deleted_at` IS NULL')['idarticolo'];
-                    }
+                    $id_articolo = $database->fetchOne('SELECT `mg_articoli_barcode`.`idarticolo` FROM `mg_articoli_barcode` INNER JOIN `mg_articoli` ON `mg_articoli`.`id` = `mg_articoli_barcode`.`idarticolo` WHERE REPLACE(`mg_articoli_barcode`.`barcode`, " ", "") = '.prepare($codice['CodiceValore']).' AND `mg_articoli`.`deleted_at` IS NULL')['idarticolo'];
                 }
 
                 if (!empty($id_articolo)) {

plugins/importFE/generate.php

@@ -504,38 +504,38 @@ function cleanup(){
 
         // Visualizzazione codici articoli
         $codici = $riga['CodiceArticolo'] ?: [];
-        $codici = !empty($codici) && !isset($codici[0]) ? [$codici] : $codici;
-
-        $codici_articoli = [];
         $serial = [];
-        $i = 0;
+
+        // Individuazione articolo con codice relativo
+        $id_articolo = null;
+        $codice_principale = null;
+
         foreach ($codici as $codice) {
-            $codici_articoli[] = (($i == 0) ? '<b>' : '').$codice['CodiceValore'].' ('.$codice['CodiceTipo'].')'.(($i == 0) ? '</b>' : '');
-            if (str_contains((string) $codice['CodiceTipo'], 'serial') || str_contains((string) $codice['CodiceTipo'], 'Serial')) {
+            $tipo = strtolower($codice['CodiceTipo']);
+            if (str_starts_with($tipo, 'cod')) {
+                $codice_principale = $codice['CodiceValore'];
+            }
+
+            if (str_starts_with($tipo, 'serial')) {
                 $serial[] = $codice['CodiceValore'];
             }
-            ++$i;
+
+            if (str_starts_with($tipo, 'barcode') || str_starts_with($tipo, 'ean') || str_starts_with($tipo, 'en')) {
+                $barcode = $codice['CodiceValore'];
+            }
         }
 
-        // Individuazione articolo con codice relativo
-        $id_articolo = null;
-        // Prendo il codice articolo dal primo nodo CodiceValore che trovo
-        $codice_principale = $codici[0]['CodiceValore'];
+        // Fallback al primo codice se non trovato
+        if ($codice_principale === null && !empty($codici[0])) {
+            $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)) {
-                $result = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE `codice` = '.prepare($codice_principale));
+                $result = $database->fetchOne('SELECT `id_articolo` AS id FROM `mg_fornitore_articolo` WHERE ((`codice_fornitore` = '.prepare($codice_principale).' OR `barcode_fornitore` = '.prepare($barcode).') AND id_fornitore = '.prepare($anagrafica->id).' AND DELETED_AT IS NULL)');
                 $id_articolo = $result['id'] ?? null;
                 if (empty($id_articolo)) {
-                    $result = $database->fetchOne('SELECT `id` FROM `mg_articoli` WHERE REPLACE(`codice`, " ", "") = '.prepare($codice_principale));
+                    $result = $database->fetchOne('SELECT `id_articolo` AS id FROM `mg_fornitore_articolo` WHERE ((REPLACE(`codice_fornitore`, " ", "") = '.prepare($codice_principale).' OR `barcode_fornitore` = '.prepare($barcode).') AND id_fornitore = '.prepare($anagrafica->id).' AND DELETED_AT IS NULL)');
                     $id_articolo = $result['id'] ?? null;
                 }
             }
@@ -691,7 +691,7 @@ function cleanup(){
                     <div class="card-header">
                         <div class="row">
                             <div class="col-md-5">
-                                {["type": "select", "name": "articoli['.$key.']", "ajax-source": "articoli", "select-options": '.json_encode(['permetti_movimento_a_zero' => 1, 'dir' => 'entrata', 'idanagrafica' => $anagrafica ? $anagrafica->id : '']).', "icon-after": "add|'.Module::where('name', 'Articoli')->first()->id.'|codice='.($codice_principale ? urlencode((string) $codice_principale) : '').'&descrizione='.($riga['Descrizione'] ? urlencode((string) $riga['Descrizione']) : '').'&prezzo_acquisto='.($riga['PrezzoUnitario'] ? urlencode((string) $riga['PrezzoUnitario']) : '').'", "value": "'.$id_articolo.'", "label": "'.tr('Articolo').'","extra": "data-id=\''.$key.'\'" ]}
+                                {["type": "select", "name": "articoli['.$key.']", "ajax-source": "articoli", "select-options": '.json_encode(['permetti_movimento_a_zero' => 1, 'dir' => 'uscita', 'idanagrafica' => $anagrafica ? $anagrafica->id : '']).', "icon-after": "add|'.Module::where('name', 'Articoli')->first()->id.'|codice='.($codice_principale ? urlencode((string) $codice_principale) : '').'&descrizione='.($riga['Descrizione'] ? urlencode((string) $riga['Descrizione']) : '').'&prezzo_acquisto='.($riga['PrezzoUnitario'] ? urlencode((string) $riga['PrezzoUnitario']) : '').'", "value": "'.$id_articolo.'", "label": "'.tr('Articolo').'","extra": "data-id=\''.$key.'\'" ]}
                             </div>
 
                             <div class="col-md-3">

plugins/importFE/src/FatturaOrdinaria.php

@@ -287,33 +287,29 @@ public function saveRighe($articoli, $iva, $conto, $movimentazione = true, $crea
                     }
                 }
 
-                $articolo = ArticoloOriginale::where('codice', $codice)->first();
-
-                if (empty($articolo)) {
-                    $nome_categoria = 'Importazione automatica';
-                    $categoria = Categoria::find((new Categoria())->getByField('title', strtolower($nome_categoria)));
-                    if (empty($categoria)) {
-                        $categoria = Categoria::build();
-                        $categoria->name = $nome_categoria;
-                        $categoria->save();
-                        $categoria->setTranslation('title', $nome_categoria);
-                    }
+                $nome_categoria = 'Importazione automatica';
+                $categoria = Categoria::find((new Categoria())->getByField('title', strtolower($nome_categoria)));
+                if (empty($categoria)) {
+                    $categoria = Categoria::build();
+                    $categoria->name = $nome_categoria;
+                    $categoria->save();
+                    $categoria->setTranslation('title', $nome_categoria);
+                }
 
-                    $articolo = ArticoloOriginale::build($codice, $categoria);
-                    $articolo->setTranslation('title', $riga['Descrizione']);
-                    $articolo->um = $riga['UnitaMisura'];
-                    $articolo->idconto_acquisto = $conto[$key] ?: null;
-                    $articolo->abilita_serial = setting('Serial number abilitato di default');
+                $articolo = ArticoloOriginale::build($codice, $categoria);
+                $articolo->setTranslation('title', $riga['Descrizione']);
+                $articolo->um = $riga['UnitaMisura'];
+                $articolo->idconto_acquisto = $conto[$key] ?: null;
+                $articolo->abilita_serial = setting('Serial number abilitato di default');
 
-                    $articolo->save();
+                $articolo->save();
 
-                    // Se abbiamo identificato un barcode, lo salviamo nella tabella mg_articoli_barcode
-                    if (!empty($barcode)) {
-                        database()->insert('mg_articoli_barcode', [
-                            'idarticolo' => $articolo->id,
-                            'barcode' => $barcode,
-                        ]);
-                    }
+                // Se abbiamo identificato un barcode, lo salviamo nella tabella mg_articoli_barcode
+                if (!empty($barcode)) {
+                    database()->insert('mg_articoli_barcode', [
+                        'idarticolo' => $articolo->id,
+                        'barcode' => $barcode,
+                    ]);
                 }
             }
 

plugins/listino_clienti/src/DettaglioPrezzo.php

@@ -92,7 +92,8 @@ public static function dettaglioPredefinito($id_articolo, $id_anagrafica, $direz
             ->where('id_anagrafica', $id_anagrafica)
             ->where('dir', $direzione)
             ->whereNull('minimo')
-            ->whereNull('massimo');
+            ->whereNull('massimo')
+            ->whereNull('deleted_at');
     }
 
     public static function dettagli($id_articolo, $id_anagrafica, $direzione, $qta = null)