fatture

app/Http/Controllers/ContrattoServizioController.php

@@ -351,16 +351,17 @@ class ContrattoServizioController extends Controller implements HasMiddleware
       if ($dataFine === '') {
         $dataFine = null;
       }
+      $unita = $row['unita_misura'] ?? ($prodotto->unita_misura ?? null);
       $riga->fill([
         'prodotto_id' => $prodottoId,
         'quantita' => $row['quantita'] ?? 1,
-        'unita_misura' => $row['unita_misura'] ?? ($prodotto->unita_misura ?? null),
+        'unita_misura' => $unita,
         'prezzo_unitario' => ($row['prezzo_unitario'] === '' || ! array_key_exists('prezzo_unitario', $row))
           ? null
           : $row['prezzo_unitario'],
         'sconto_tipo' => $scontoTipo,
         'sconto_valore' => $scontoValore,
-        'periodo' => $row['periodo'] ?? ($prodotto->periodo ?? null),
+        'periodo' => ContrattoServizioRiga::periodoSalvato($unita, $row['periodo'] ?? ($prodotto->periodo ?? null)),
         'data_inizio' => $dataInizio,
         'data_fine' => $dataFine,
         'note' => $row['note'] ?? null,

app/Http/Controllers/ContrattoTemplateController.php

@@ -3,6 +3,7 @@
 namespace App\Http\Controllers;
 
 use App\DataTables\ContrattoTemplateDataTable;
+use App\Models\ContrattoServizioRiga;
 use App\Models\ContrattoTemplate;
 use App\Models\ContrattoTemplateRiga;
 use App\Models\ContrattoTemplateTodo;
@@ -213,15 +214,16 @@ class ContrattoTemplateController extends Controller implements HasMiddleware
         continue;
       }
       $prodotto = Prodotto::find($prodottoId);
+      $unita = $row['unita_misura'] ?? ($prodotto->unita_misura ?? null);
       ContrattoTemplateRiga::create([
         'contratto_template_id' => $template->id,
         'prodotto_id' => $prodottoId,
         'quantita' => $row['quantita'] ?? 1,
-        'unita_misura' => $row['unita_misura'] ?? ($prodotto->unita_misura ?? null),
+        'unita_misura' => $unita,
         'prezzo_unitario' => ($row['prezzo_unitario'] === '' || ! array_key_exists('prezzo_unitario', $row))
           ? null
           : $row['prezzo_unitario'],
-        'periodo' => $row['periodo'] ?? ($prodotto->periodo ?? null),
+        'periodo' => ContrattoServizioRiga::periodoSalvato($unita, $row['periodo'] ?? ($prodotto->periodo ?? null)),
         'note' => $row['note'] ?? null,
         'ordine' => $ordine++,
       ]);

app/Models/ContrattoServizio.php

@@ -394,7 +394,7 @@ class ContrattoServizio extends \App\Models\AbstractModels\AbstractContrattoServ
                 'quantita' => $riga->quantita,
                 'unita_misura' => $riga->unita_misura,
                 'prezzo_unitario' => $riga->prezzo_unitario,
-                'periodo' => $riga->periodo,
+                'periodo' => ContrattoServizioRiga::periodoSalvato($riga->unita_misura, $riga->periodo),
                 'note' => $riga->note,
                 'ordine' => $riga->ordine,
             ]);

app/Models/ContrattoServizioRiga.php

@@ -315,6 +315,44 @@ class ContrattoServizioRiga extends \App\Models\AbstractModels\AbstractContratto
         return filled($um) && ! in_array($um, ['mese', 'anno', 'forfait'], true);
     }
 
+    /**
+     * Ciclo di fatturazione. Mese, anno e forfait sono già il ciclo:
+     * un eventuale periodo salvato sulla riga (es. mensile su «1 anno») non conta.
+     * Il forfait con periodo esplicito è il canone di testata, che segue il contratto.
+     */
+    public function periodoCiclo(?string $periodoContratto = null): string
+    {
+        $um = $this->unita_misura ?: optional($this->prodotto)->unita_misura;
+
+        if ($um === 'anno') {
+            return 'annuale';
+        }
+        if ($um === 'mese') {
+            return 'mensile';
+        }
+        if ($um === 'forfait' && ! filled($this->periodo)) {
+            return 'una_tantum';
+        }
+
+        $periodo = (string) ($this->periodo ?: $periodoContratto ?: 'una_tantum');
+
+        return $periodo !== '' ? $periodo : 'una_tantum';
+    }
+
+    /**
+     * Mese, anno e forfait non hanno un periodo separato: non va persistito.
+     */
+    public static function periodoSalvato(?string $unita, ?string $periodo): ?string
+    {
+        if (in_array($unita, ['mese', 'anno', 'forfait'], true)) {
+            return null;
+        }
+
+        $periodo = $periodo !== null ? trim($periodo) : null;
+
+        return $periodo === '' ? null : $periodo;
+    }
+
     public function getPeriodoLabelAttribute(): string
     {
         if (! $this->needsPeriodo()) {

app/Services/FatturazioneGeneraService.php

@@ -651,7 +651,7 @@ class FatturazioneGeneraService
             }
 
             foreach ($contratto->righe as $riga) {
-                $periodoKey = (string) ($riga->periodo ?: $contratto->periodo ?: 'una_tantum');
+                $periodoKey = $riga->periodoCiclo($contratto->periodo);
                 foreach ($this->dateOccorrenzeRigaNelPeriodo($contratto, $riga, $from, $to) as $occurrence) {
                     [$pDa, $pA] = $this->finestraCiclo($periodoKey, $occurrence, $this->decorrenzaRiga($contratto, $riga));
                     if ($this->cicloPrecedenteAllaSoglia($occurrence, $pDa, $rinnoviDa)) {
@@ -740,7 +740,7 @@ class FatturazioneGeneraService
             return [];
         }
 
-        $periodo = (string) ($riga->periodo ?: $contratto->periodo ?: 'una_tantum');
+        $periodo = $riga->periodoCiclo($contratto->periodo);
         if ($periodo === 'a_consumo') {
             return [];
         }