refactor: miglioria importazione preventivi

Pek5892 · Pek5892 · commit bf7d26e23a1c · 2025-10-21T15:16:01.000+02:00
diff --git a/modules/preventivi/src/Import/CSV.php b/modules/preventivi/src/Import/CSV.php
@@ -122,41 +122,33 @@ public function import($record, $update_record = true, $add_record = true)
         try {
             $database = database();
 
-            // Validazione dei campi obbligatori
             if (empty($record['numero']) || empty($record['nome']) || empty($record['ragione_sociale'])
                 || empty($record['data_bozza']) || empty($record['prezzo_unitario'])) {
                 return false;
             }
 
-            // Validazione per righe: deve essere presente codice articolo O descrizione riga
             if (empty($record['codice']) && empty($record['descrizione_riga'])) {
                 return false;
             }
 
-            // Ricerca del preventivo esistente
             $preventivo = $this->trovaPreventivo($record, $database);
 
-            // Controllo se creare o aggiornare il record
             if (($preventivo && !$update_record) || (!$preventivo && !$add_record)) {
                 return null;
             }
 
-            // Creazione o aggiornamento del preventivo
             if (empty($preventivo)) {
                 $preventivo = $this->creaPreventivo($record);
                 if (empty($preventivo)) {
                     return false;
                 }
             }
 
-            // Aggiunta della riga al preventivo (articolo o riga generica)
             $this->aggiungiRigaAlPreventivo($preventivo, $record);
 
             return true;
         } catch (\Exception $e) {
-            // Registra l'errore in un log
             error_log('Errore durante l\'importazione del preventivo: '.$e->getMessage());
-
             return false;
         }
     }
@@ -170,9 +162,9 @@ public static function getExample()
     {
         return [
             ['Numero', 'Nome Preventivo', 'Descrizione Preventivo', 'Cliente', 'Partita IVA Cliente', 'Tipo Attività', 'Data', 'Codice Articolo', 'Descrizione riga generica', 'Aliquota IVA riga (%)', 'Quantità riga', 'Data prevista evasione riga', 'Prezzo unitario riga'],
-            ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Rossi', '12345678901', 'Generico', '27/04/2024', '001', '', '', '2', '30/04/2024', '50'],
-            ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Rossi', '12345678901', 'Generico', '27/04/2024', '043', '', '', '1', '10/05/2024', '100'],
-            ['16', 'Preventivo Servizi', 'Preventivo servizi', 'Bianchi', '98765432109', 'Generico', '28/04/2024', '', 'Consulenza tecnica', '22', '1', '05/05/2024', '150'],
+            ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Mario Rossi', '123456789', 'Generico', '27/04/2025', 'OSM-BUDGET', '', '22', '1', '30/04/2025', '100'],
+            ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Mario Rossi', '123456789', 'Generico', '27/04/2025', '', 'Manodopera', '22', '1', '10/05/2025', '150'],
+            ['16', 'Preventivo Servizi', 'Preventivo servizi', 'Mario Rossi', '123456789', 'Generico', '28/04/2025', '', 'Consulenza tecnica', '22', '1', '05/05/2025', '150'],
         ];
     }
 
@@ -205,16 +197,13 @@ protected function trovaPreventivo($record, $database)
     protected function creaPreventivo($record)
     {
         try {
-            // Ricerca o creazione dell'anagrafica cliente
             $anagrafica = $this->trovaOCreaAnagrafica($record);
             if (empty($anagrafica)) {
                 return null;
             }
 
-            // Ricerca del tipo di intervento
             $tipo = $this->trovaTipoIntervento($record);
 
-            // Creazione del preventivo
             $preventivo = Preventivo::build($anagrafica, $tipo, $record['nome'], $this->parseData($record['data_bozza']), 0);
             $preventivo->numero = $record['numero'];
             $preventivo->idstato = Stato::where('name', 'Bozza')->first()->id;
@@ -224,7 +213,6 @@ protected function creaPreventivo($record)
             return $preventivo;
         } catch (\Exception $e) {
             error_log('Errore durante la creazione del preventivo: '.$e->getMessage());
-
             return null;
         }
     }
@@ -244,21 +232,17 @@ protected function trovaOCreaAnagrafica($record)
 
         $anagrafica = null;
 
-        // Prima ricerca per partita IVA se presente
         if (!empty($record['partita_iva'])) {
             $anagrafica = Anagrafica::where('piva', $record['partita_iva'])->first();
         }
 
-        // Se non trovata per partita IVA, ricerca per ragione sociale
         if (empty($anagrafica)) {
             $anagrafica = Anagrafica::where('ragione_sociale', $record['ragione_sociale'])->first();
         }
 
-        // Se non trovata, crea nuova anagrafica
         if (empty($anagrafica)) {
             $anagrafica = Anagrafica::build($record['ragione_sociale']);
 
-            // Imposta la partita IVA se fornita
             if (!empty($record['partita_iva'])) {
                 $anagrafica->partita_iva = $record['partita_iva'];
             }
@@ -294,23 +278,20 @@ protected function trovaTipoIntervento($record)
     protected function aggiungiRigaAlPreventivo($preventivo, $record)
     {
         try {
-            // Se è presente il codice articolo, prova a creare una riga articolo
             if (!empty($record['codice'])) {
                 $articolo_orig = ArticoloOriginale::where('codice', $record['codice'])->first();
                 if (!empty($articolo_orig)) {
                     return $this->aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_orig);
                 }
             }
 
-            // Se non è un articolo o l'articolo non è stato trovato, crea una riga generica
             if (!empty($record['descrizione_riga'])) {
                 return $this->aggiungiRigaGenericaAlPreventivo($preventivo, $record);
             }
 
             return false;
         } catch (\Exception $e) {
             error_log('Errore durante l\'aggiunta della riga al preventivo: '.$e->getMessage());
-
             return false;
         }
     }
@@ -330,12 +311,10 @@ protected function aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_
             $riga_articolo = Articolo::build($preventivo, $articolo_orig);
             $riga_articolo->um = $articolo_orig->um ?: null;
 
-            // Gestione della data di evasione
             if (!empty($record['data_evasione'])) {
                 $riga_articolo->data_evasione = $this->parseData($record['data_evasione']);
             }
 
-            // Gestione dell'IVA
             $anagrafica = $preventivo->anagrafica;
             $idiva = $articolo_orig->idiva_vendita ?: ($anagrafica->idiva_vendite ?: setting('Iva predefinita'));
 
@@ -348,7 +327,6 @@ protected function aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_
             return true;
         } catch (\Exception $e) {
             error_log('Errore durante l\'aggiunta dell\'articolo al preventivo: '.$e->getMessage());
-
             return false;
         }
     }
@@ -366,12 +344,10 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)
         try {
             $riga = Riga::build($preventivo);
 
-            // Gestione della data di evasione
             if (!empty($record['data_evasione'])) {
                 $riga->data_evasione = $this->parseData($record['data_evasione']);
             }
 
-            // Gestione dell'IVA
             $idiva = $this->trovaAliquotaIva($record, $preventivo);
 
             $riga->descrizione = $record['descrizione_riga'];
@@ -383,7 +359,6 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)
             return true;
         } catch (\Exception $e) {
             error_log('Errore durante l\'aggiunta della riga generica al preventivo: '.$e->getMessage());
-
             return false;
         }
     }
@@ -398,15 +373,13 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)
      */
     protected function trovaAliquotaIva($record, $preventivo)
     {
-        // Se è specificata un'aliquota IVA nel record, cerca per percentuale
         if (!empty($record['aliquota_iva'])) {
             $aliquota = Aliquota::where('percentuale', $record['aliquota_iva'])->first();
             if (!empty($aliquota)) {
                 return $aliquota->id;
             }
         }
 
-        // Fallback: usa l'IVA dell'anagrafica o quella predefinita
         $anagrafica = $preventivo->anagrafica;
         return $anagrafica->idiva_vendite ?: setting('Iva predefinita');
     }
@@ -423,14 +396,11 @@ protected function parseData($data_string)
         try {
             return new Carbon($data_string);
         } catch (\Exception $e) {
-            // Prova a interpretare formati di data italiani (dd/mm/yyyy)
             if (preg_match('/^(\d{1,2})\/(\d{1,2})\/(\d{4})$/', $data_string, $matches)) {
                 return Carbon::createFromDate($matches[3], $matches[2], $matches[1]);
             }
 
-            // Fallback alla data corrente
             error_log('Errore nel parsing della data: '.$e->getMessage());
-
             return Carbon::now();
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -122,41 +122,33 @@ public function import($record, $update_record = true, $add_record = true)`
`122`	`122`	`try {`
`123`	`123`	`$database = database();`
`124`	`124`
`125`		`- // Validazione dei campi obbligatori`
`126`	`125`	`if (empty($record['numero']) \|\| empty($record['nome']) \|\| empty($record['ragione_sociale'])`
`127`	`126`	`\|\| empty($record['data_bozza']) \|\| empty($record['prezzo_unitario'])) {`
`128`	`127`	`return false;`
`129`	`128`	`}`
`130`	`129`
`131`		`- // Validazione per righe: deve essere presente codice articolo O descrizione riga`
`132`	`130`	`if (empty($record['codice']) && empty($record['descrizione_riga'])) {`
`133`	`131`	`return false;`
`134`	`132`	`}`
`135`	`133`
`136`		`- // Ricerca del preventivo esistente`
`137`	`134`	`$preventivo = $this->trovaPreventivo($record, $database);`
`138`	`135`
`139`		`- // Controllo se creare o aggiornare il record`
`140`	`136`	`if (($preventivo && !$update_record) \|\| (!$preventivo && !$add_record)) {`
`141`	`137`	`return null;`
`142`	`138`	`}`
`143`	`139`
`144`		`- // Creazione o aggiornamento del preventivo`
`145`	`140`	`if (empty($preventivo)) {`
`146`	`141`	`$preventivo = $this->creaPreventivo($record);`
`147`	`142`	`if (empty($preventivo)) {`
`148`	`143`	`return false;`
`149`	`144`	`}`
`150`	`145`	`}`
`151`	`146`
`152`		`- // Aggiunta della riga al preventivo (articolo o riga generica)`
`153`	`147`	`$this->aggiungiRigaAlPreventivo($preventivo, $record);`
`154`	`148`
`155`	`149`	`return true;`
`156`	`150`	`} catch (\Exception $e) {`
`157`		`- // Registra l'errore in un log`
`158`	`151`	`error_log('Errore durante l\'importazione del preventivo: '.$e->getMessage());`
`159`		`-`
`160`	`152`	`return false;`
`161`	`153`	`}`
`162`	`154`	`}`
`@@ -170,9 +162,9 @@ public static function getExample()`
`170`	`162`	`{`
`171`	`163`	`return [`
`172`	`164`	`['Numero', 'Nome Preventivo', 'Descrizione Preventivo', 'Cliente', 'Partita IVA Cliente', 'Tipo Attività', 'Data', 'Codice Articolo', 'Descrizione riga generica', 'Aliquota IVA riga (%)', 'Quantità riga', 'Data prevista evasione riga', 'Prezzo unitario riga'],`
`173`		`- ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Rossi', '12345678901', 'Generico', '27/04/2024', '001', '', '', '2', '30/04/2024', '50'],`
`174`		`- ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Rossi', '12345678901', 'Generico', '27/04/2024', '043', '', '', '1', '10/05/2024', '100'],`
`175`		`- ['16', 'Preventivo Servizi', 'Preventivo servizi', 'Bianchi', '98765432109', 'Generico', '28/04/2024', '', 'Consulenza tecnica', '22', '1', '05/05/2024', '150'],`
	`165`	`+ ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Mario Rossi', '123456789', 'Generico', '27/04/2025', 'OSM-BUDGET', '', '22', '1', '30/04/2025', '100'],`
	`166`	`+ ['15', 'Preventivo Materiali', 'Preventivo iniziale', 'Mario Rossi', '123456789', 'Generico', '27/04/2025', '', 'Manodopera', '22', '1', '10/05/2025', '150'],`
	`167`	`+ ['16', 'Preventivo Servizi', 'Preventivo servizi', 'Mario Rossi', '123456789', 'Generico', '28/04/2025', '', 'Consulenza tecnica', '22', '1', '05/05/2025', '150'],`
`176`	`168`	`];`
`177`	`169`	`}`
`178`	`170`
`@@ -205,16 +197,13 @@ protected function trovaPreventivo($record, $database)`
`205`	`197`	`protected function creaPreventivo($record)`
`206`	`198`	`{`
`207`	`199`	`try {`
`208`		`- // Ricerca o creazione dell'anagrafica cliente`
`209`	`200`	`$anagrafica = $this->trovaOCreaAnagrafica($record);`
`210`	`201`	`if (empty($anagrafica)) {`
`211`	`202`	`return null;`
`212`	`203`	`}`
`213`	`204`
`214`		`- // Ricerca del tipo di intervento`
`215`	`205`	`$tipo = $this->trovaTipoIntervento($record);`
`216`	`206`
`217`		`- // Creazione del preventivo`
`218`	`207`	`$preventivo = Preventivo::build($anagrafica, $tipo, $record['nome'], $this->parseData($record['data_bozza']), 0);`
`219`	`208`	`$preventivo->numero = $record['numero'];`
`220`	`209`	`$preventivo->idstato = Stato::where('name', 'Bozza')->first()->id;`
`@@ -224,7 +213,6 @@ protected function creaPreventivo($record)`
`224`	`213`	`return $preventivo;`
`225`	`214`	`} catch (\Exception $e) {`
`226`	`215`	`error_log('Errore durante la creazione del preventivo: '.$e->getMessage());`
`227`		`-`
`228`	`216`	`return null;`
`229`	`217`	`}`
`230`	`218`	`}`
`@@ -244,21 +232,17 @@ protected function trovaOCreaAnagrafica($record)`
`244`	`232`
`245`	`233`	`$anagrafica = null;`
`246`	`234`
`247`		`- // Prima ricerca per partita IVA se presente`
`248`	`235`	`if (!empty($record['partita_iva'])) {`
`249`	`236`	`$anagrafica = Anagrafica::where('piva', $record['partita_iva'])->first();`
`250`	`237`	`}`
`251`	`238`
`252`		`- // Se non trovata per partita IVA, ricerca per ragione sociale`
`253`	`239`	`if (empty($anagrafica)) {`
`254`	`240`	`$anagrafica = Anagrafica::where('ragione_sociale', $record['ragione_sociale'])->first();`
`255`	`241`	`}`
`256`	`242`
`257`		`- // Se non trovata, crea nuova anagrafica`
`258`	`243`	`if (empty($anagrafica)) {`
`259`	`244`	`$anagrafica = Anagrafica::build($record['ragione_sociale']);`
`260`	`245`
`261`		`- // Imposta la partita IVA se fornita`
`262`	`246`	`if (!empty($record['partita_iva'])) {`
`263`	`247`	`$anagrafica->partita_iva = $record['partita_iva'];`
`264`	`248`	`}`
`@@ -294,23 +278,20 @@ protected function trovaTipoIntervento($record)`
`294`	`278`	`protected function aggiungiRigaAlPreventivo($preventivo, $record)`
`295`	`279`	`{`
`296`	`280`	`try {`
`297`		`- // Se è presente il codice articolo, prova a creare una riga articolo`
`298`	`281`	`if (!empty($record['codice'])) {`
`299`	`282`	`$articolo_orig = ArticoloOriginale::where('codice', $record['codice'])->first();`
`300`	`283`	`if (!empty($articolo_orig)) {`
`301`	`284`	`return $this->aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_orig);`
`302`	`285`	`}`
`303`	`286`	`}`
`304`	`287`
`305`		`- // Se non è un articolo o l'articolo non è stato trovato, crea una riga generica`
`306`	`288`	`if (!empty($record['descrizione_riga'])) {`
`307`	`289`	`return $this->aggiungiRigaGenericaAlPreventivo($preventivo, $record);`
`308`	`290`	`}`
`309`	`291`
`310`	`292`	`return false;`
`311`	`293`	`} catch (\Exception $e) {`
`312`	`294`	`error_log('Errore durante l\'aggiunta della riga al preventivo: '.$e->getMessage());`
`313`		`-`
`314`	`295`	`return false;`
`315`	`296`	`}`
`316`	`297`	`}`
`@@ -330,12 +311,10 @@ protected function aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_`
`330`	`311`	`$riga_articolo = Articolo::build($preventivo, $articolo_orig);`
`331`	`312`	`$riga_articolo->um = $articolo_orig->um ?: null;`
`332`	`313`
`333`		`- // Gestione della data di evasione`
`334`	`314`	`if (!empty($record['data_evasione'])) {`
`335`	`315`	`$riga_articolo->data_evasione = $this->parseData($record['data_evasione']);`
`336`	`316`	`}`
`337`	`317`
`338`		`- // Gestione dell'IVA`
`339`	`318`	`$anagrafica = $preventivo->anagrafica;`
`340`	`319`	`$idiva = $articolo_orig->idiva_vendita ?: ($anagrafica->idiva_vendite ?: setting('Iva predefinita'));`
`341`	`320`
`@@ -348,7 +327,6 @@ protected function aggiungiArticoloAlPreventivo($preventivo, $record, $articolo_`
`348`	`327`	`return true;`
`349`	`328`	`} catch (\Exception $e) {`
`350`	`329`	`error_log('Errore durante l\'aggiunta dell\'articolo al preventivo: '.$e->getMessage());`
`351`		`-`
`352`	`330`	`return false;`
`353`	`331`	`}`
`354`	`332`	`}`
`@@ -366,12 +344,10 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)`
`366`	`344`	`try {`
`367`	`345`	`$riga = Riga::build($preventivo);`
`368`	`346`
`369`		`- // Gestione della data di evasione`
`370`	`347`	`if (!empty($record['data_evasione'])) {`
`371`	`348`	`$riga->data_evasione = $this->parseData($record['data_evasione']);`
`372`	`349`	`}`
`373`	`350`
`374`		`- // Gestione dell'IVA`
`375`	`351`	`$idiva = $this->trovaAliquotaIva($record, $preventivo);`
`376`	`352`
`377`	`353`	`$riga->descrizione = $record['descrizione_riga'];`
`@@ -383,7 +359,6 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)`
`383`	`359`	`return true;`
`384`	`360`	`} catch (\Exception $e) {`
`385`	`361`	`error_log('Errore durante l\'aggiunta della riga generica al preventivo: '.$e->getMessage());`
`386`		`-`
`387`	`362`	`return false;`
`388`	`363`	`}`
`389`	`364`	`}`
`@@ -398,15 +373,13 @@ protected function aggiungiRigaGenericaAlPreventivo($preventivo, $record)`
`398`	`373`	`*/`
`399`	`374`	`protected function trovaAliquotaIva($record, $preventivo)`
`400`	`375`	`{`
`401`		`- // Se è specificata un'aliquota IVA nel record, cerca per percentuale`
`402`	`376`	`if (!empty($record['aliquota_iva'])) {`
`403`	`377`	`$aliquota = Aliquota::where('percentuale', $record['aliquota_iva'])->first();`
`404`	`378`	`if (!empty($aliquota)) {`
`405`	`379`	`return $aliquota->id;`
`406`	`380`	`}`
`407`	`381`	`}`
`408`	`382`
`409`		`- // Fallback: usa l'IVA dell'anagrafica o quella predefinita`
`410`	`383`	`$anagrafica = $preventivo->anagrafica;`
`411`	`384`	`return $anagrafica->idiva_vendite ?: setting('Iva predefinita');`
`412`	`385`	`}`
`@@ -423,14 +396,11 @@ protected function parseData($data_string)`
`423`	`396`	`try {`
`424`	`397`	`return new Carbon($data_string);`
`425`	`398`	`} catch (\Exception $e) {`
`426`		`- // Prova a interpretare formati di data italiani (dd/mm/yyyy)`
`427`	`399`	`if (preg_match('/^(\d{1,2})\/(\d{1,2})\/(\d{4})$/', $data_string, $matches)) {`
`428`	`400`	`return Carbon::createFromDate($matches[3], $matches[2], $matches[1]);`
`429`	`401`	`}`
`430`	`402`
`431`		`- // Fallback alla data corrente`
`432`	`403`	`error_log('Errore nel parsing della data: '.$e->getMessage());`
`433`		`-`
`434`	`404`	`return Carbon::now();`
`435`	`405`	`}`
`436`	`406`	`}`