fix: registrazione insoluto da tasto Registra insoluto

Pek5892 · Pek5892 · commit 1db15128406a · 2026-03-31T16:26:10.000+02:00
diff --git a/modules/primanota/add.php b/modules/primanota/add.php
@@ -32,7 +32,7 @@
  * Questo file prevede diverse operazioni per la generazione di un singolo array `$movimenti` contenente tutti i movimenti da presentare nella Prima Nota. In particolare:
  *  - Individua Scadenze e Fatture per ID da URL
  *  - Legge le informazioni relative alle Scadenze per presentare i movimenti in Dare e Avere
- *  - Legge le informazioni relative alla Scadenze le Fatture indicate (sola della prima Scadenza insoluta se `is_insoluto` impostato):
+ *  - Legge le informazioni relative alla Scadenze le Fatture indicate (sola della prima Scadenza disponibile se `is_insoluto` impostato):
  *      - Per Fatture di vendita, il totale è Avere (a meno di Note di credito oppure insoluto)
  *      - Per Fatture di acquisto, il totale è Dare (a meno di Note di credito oppure insoluto)
  *  - Inverte Dare e Avere se l'importo indicato è negativo [TODO: documentare la casistica]
@@ -124,7 +124,7 @@
 
     // Inclusione delle sole fatture in stato Emessa, Parzialmente pagato o Pagato
     if (!in_array($fattura->stato->name, ['Emessa', 'Parzialmente pagato', 'Pagato'])) {
-        ++$counter;
+        $counter++;
         continue;
     }
 
@@ -138,11 +138,11 @@
     $conto_field = 'idconto_'.($dir == 'entrata' ? 'vendite' : 'acquisti');
     $id_conto_aziendale = $banca->id_pianodeiconti3 ?: ($fattura->pagamento[$conto_field] ?: setting('Conto aziendale predefinito'));
 
-    // Se sto registrando un insoluto, leggo l'ultima scadenza pagata altrimenti leggo la scadenza della fattura
+    // Se sto registrando un insoluto, leggo la prima rata disponibile (non pagata) altrimenti leggo la scadenza della fattura
     if ($is_insoluto) {
-        $scadenze = $database->fetchArray('SELECT id, ABS(da_pagare) AS rata, iddocumento, tipo FROM co_scadenziario WHERE iddocumento='.prepare($id_documento).' AND ABS(da_pagare) = ABS(pagato) ORDER BY updated_at DESC LIMIT 0, 1');
+        $scadenze = $database->fetchArray('SELECT id, ABS(da_pagare - pagato) AS rata, iddocumento, tipo FROM co_scadenziario WHERE iddocumento='.prepare($id_documento).' AND ABS(da_pagare) > ABS(pagato)'.(! empty($id_scadenze) ? 'AND id IN('.implode(',', array_map(prepare(...), $id_scadenze)).')' : '').' ORDER BY YEAR(scadenza) ASC, MONTH(scadenza) ASC LIMIT 0, 1');
     } else {
-        $scadenze = $database->fetchArray('SELECT id, ABS(da_pagare - pagato) AS rata, iddocumento, tipo FROM co_scadenziario WHERE iddocumento='.prepare($id_documento).' AND ABS(da_pagare) > ABS(pagato)'.(!empty($id_scadenze) ? 'AND id IN('.implode(',', array_map(prepare(...), $id_scadenze)).')' : '').' ORDER BY YEAR(scadenza) ASC, MONTH(scadenza) ASC');
+        $scadenze = $database->fetchArray('SELECT id, ABS(da_pagare - pagato) AS rata, iddocumento, tipo FROM co_scadenziario WHERE iddocumento='.prepare($id_documento).' AND ABS(da_pagare) > ABS(pagato)'.(! empty($id_scadenze) ? 'AND id IN('.implode(',', array_map(prepare(...), $id_scadenze)).')' : '').' ORDER BY YEAR(scadenza) ASC, MONTH(scadenza) ASC');
     }
 
     // Selezione prima scadenza
@@ -169,44 +169,91 @@
 
     $righe_documento = [];
 
-    // Riga controparte
-    foreach ($scadenze as $scadenza) {
-        // Predisposizione conto
-        if ($scadenza['tipo'] == 'ritenutaacconto') {
-            $id_conto_controparte = setting("Conto per Erario c/ritenute d'acconto");
-        } else {
-            $conto_field = 'idconto_'.($dir == 'entrata' ? 'cliente' : 'fornitore');
-            $id_conto_controparte = $fattura->anagrafica[$conto_field];
-        }
+    if ($is_insoluto) {
+        $scadenza_disponibile = $scadenze[0];
+
+        $conto_field = 'idconto_'.($dir == 'entrata' ? 'cliente' : 'fornitore');
+        $id_conto_controparte = $fattura->anagrafica[$conto_field];
+
+        $conto_banca_effetti = $database->fetchOne('SELECT id FROM co_pianodeiconti3 WHERE descrizione = '.prepare('Banca effetti all\'incasso'));
+        $id_conto_banca_effetti = $conto_banca_effetti['id'] ?? null;
+
+        $conto_spese = $database->fetchOne('SELECT id FROM co_pianodeiconti3 WHERE descrizione = '.prepare('Effetti insoluti'));
+        $id_conto_spese_insoluti = $conto_spese['id'] ?? null;
 
         $righe_documento[] = [
-            'iddocumento' => $scadenza['iddocumento'],
-            'id_scadenza' => $scadenza['id'],
+            'iddocumento' => $scadenza_disponibile['iddocumento'],
+            'id_scadenza' => $scadenza_disponibile['id'],
             'id_conto' => $id_conto_controparte,
-            'dare' => $is_importo_avere ? 0 : $scadenza['rata'],
-            'avere' => $is_importo_avere ? $scadenza['rata'] : 0,
+            'dare' => $scadenza_disponibile['rata'],
+            'avere' => 0,
         ];
-    }
 
-    // Riga aziendale
-    $totale = sum(array_column($scadenze, 'rata'));
+        $righe_documento[] = [
+            'iddocumento' => $scadenza_disponibile['iddocumento'],
+            'id_scadenza' => $scadenza_disponibile['id'],
+            'id_conto' => $id_conto_banca_effetti,
+            'dare' => 0,
+            'avere' => $scadenza_disponibile['rata'],
+        ];
 
-    $righe_documento[] = [
-        'iddocumento' => $scadenze[0]['iddocumento'],
-        'id_scadenza' => $scadenze[0]['id'],
-        'id_conto' => $id_conto_aziendale,
-        'dare' => $is_importo_avere ? $totale : 0,
-        'avere' => $is_importo_avere ? 0 : $totale,
-    ];
+        $righe_documento[] = [
+            'iddocumento' => null,
+            'id_scadenza' => null,
+            'id_conto' => $id_conto_spese_insoluti,
+            'dare' => $scadenza_disponibile['rata'],
+            'avere' => 0,
+        ];
+
+        $righe_documento[] = [
+            'iddocumento' => null,
+            'id_scadenza' => null,
+            'id_conto' => $id_conto_aziendale,
+            'dare' => 0,
+            'avere' => $scadenza_disponibile['rata'],
+        ];
+    } else {
+        // Riga controparte
+        foreach ($scadenze as $scadenza) {
+            // Predisposizione conto
+            if ($scadenza['tipo'] == 'ritenutaacconto') {
+                $id_conto_controparte = setting("Conto per Erario c/ritenute d'acconto");
+            } else {
+                $conto_field = 'idconto_'.($dir == 'entrata' ? 'cliente' : 'fornitore');
+                $id_conto_controparte = $fattura->anagrafica[$conto_field];
+            }
+
+            $righe_documento[] = [
+                'iddocumento' => $scadenza['iddocumento'],
+                'id_scadenza' => $scadenza['id'],
+                'id_conto' => $id_conto_controparte,
+                'dare' => $is_importo_avere ? 0 : $scadenza['rata'],
+                'avere' => $is_importo_avere ? $scadenza['rata'] : 0,
+            ];
+        }
+
+        // Riga aziendale
+        $totale = sum(array_column($scadenze, 'rata'));
+
+        $righe_documento[] = [
+            'iddocumento' => $scadenze[0]['iddocumento'],
+            'id_scadenza' => $scadenze[0]['id'],
+            'id_conto' => $id_conto_aziendale,
+            'dare' => $is_importo_avere ? $totale : 0,
+            'avere' => $is_importo_avere ? 0 : $totale,
+        ];
+    }
 
     $movimenti = array_merge($movimenti, $righe_documento);
 }
 
 // Inverto dare e avere per importi negativi
 foreach ($movimenti as $key => $value) {
-    if ($movimenti[$key]['dare'] < 0 || $movimenti[$key]['avere'] < 0) {
-        $tmp = abs($movimenti[$key]['dare']);
-        $movimenti[$key]['dare'] = abs($movimenti[$key]['avere']);
+    $dare = is_numeric($movimenti[$key]['dare']) ? $movimenti[$key]['dare'] : 0;
+    $avere = is_numeric($movimenti[$key]['avere']) ? $movimenti[$key]['avere'] : 0;
+    if ($dare < 0 || $avere < 0) {
+        $tmp = abs($dare);
+        $movimenti[$key]['dare'] = abs($avere);
         $movimenti[$key]['avere'] = $tmp;
     }
 }
diff --git a/modules/primanota/src/Mastrino.php b/modules/primanota/src/Mastrino.php
@@ -38,7 +38,9 @@ class Mastrino extends Model
     use SimpleModelTrait;
 
     public $incrementing = false;
+
     protected $table = 'co_movimenti';
+
     protected $primaryKey = 'idmastrino';
 
     protected $hidden = [
@@ -50,7 +52,7 @@ class Mastrino extends Model
 
     public static function build($descrizione = null, $data = null, $is_insoluto = false, $contabile = false, $id_anagrafica = null)
     {
-        $model = new static();
+        $model = new static;
 
         $model->idmastrino = self::getNextMastrino();
         $model->data = $data;
@@ -119,17 +121,18 @@ public function aggiornaScadenzario($movimenti = null, $scadenza = null, $singol
             return;
         }
         $movimenti = $movimenti ?: $this->movimenti;
+        $is_insoluto = $this->is_insoluto;
 
         // Aggiornamento delle scadenze per i singoli documenti
         $documenti = $this->getUniqueDocumenti($movimenti);
         $scadenze = $this->getScadenzePerDocumenti($documenti);
 
-        if (!empty($scadenza) && $singola) {
+        if (! empty($scadenza) && $singola) {
             $scadenze = [$scadenza->iddocumento => [$scadenza->id]];
         }
 
         foreach ($movimenti as $movimento) {
-            $this->correggiScadenza($movimento, $scadenze[$movimento->iddocumento], $movimento->iddocumento);
+            $this->correggiScadenza($movimento, $scadenze[$movimento->iddocumento], $movimento->iddocumento, $is_insoluto);
         }
 
         // Fix dello stato della Fattura
@@ -188,13 +191,13 @@ public static function getNextMastrino()
     /**
      * Funzione dedicata alla distribuzione del totale pagato del movimento nelle relative scadenze associate.
      */
-    protected function correggiScadenza(Movimento $movimento, $scadenze = null, $id_documento = null)
+    protected function correggiScadenza(Movimento $movimento, $scadenze = null, $id_documento = null, $is_insoluto = false)
     {
         $is_nota = false;
         $documento = Fattura::find($id_documento);
         $totale_da_distribuire = 0;
 
-        if (!empty($scadenze)) {
+        if (! empty($scadenze)) {
             if (empty($documento)) {
                 $dir = $movimento->totale < 0 ? 'uscita' : 'entrata';
             } else {
@@ -204,12 +207,21 @@ protected function correggiScadenza(Movimento $movimento, $scadenze = null, $id_
             // Se il movimento ha una scadenza specifica, aggiorna solo quella
             if (count($scadenze) == 1) {
                 $scadenza = Scadenza::find($movimento->id_scadenza);
-                if (!empty($scadenza)) {
+                if (! empty($scadenza)) {
                     // Calcola il totale di TUTTI i movimenti per questa specifica scadenza
-                    $totale_movimenti_scadenza = Movimento::where('id_scadenza', '=', $scadenza->id)
+                    // Considerando il flag is_insoluto per sottrarre i movimenti di insoluto
+                    $totale_pagamenti = Movimento::where('id_scadenza', '=', $scadenza->id)
+                        ->where('totale', '>', 0)
+                        ->where('is_insoluto', '=', 0)
+                        ->sum('totale');
+
+                    $totale_insoluti = Movimento::where('id_scadenza', '=', $scadenza->id)
                         ->where('totale', '>', 0)
+                        ->where('is_insoluto', '=', 1)
                         ->sum('totale');
 
+                    $totale_movimenti_scadenza = $totale_pagamenti - $totale_insoluti;
+
                     $scadenza_da_pagare = abs($scadenza->da_pagare);
                     $pagato_assoluto = abs($totale_movimenti_scadenza);
 
@@ -229,14 +241,20 @@ protected function correggiScadenza(Movimento $movimento, $scadenze = null, $id_
                 // Ordina le scadenze per data scadenza
                 $scadenze = Scadenza::whereIn('id', $scadenze)->orderBy('scadenza', 'asc')->get()->pluck('id')->toArray();
 
-                $totale_movimenti = 0;
+                $totale_pagamenti = 0;
+                $totale_insoluti = 0;
                 foreach ($scadenze as $scadenza) {
-                    $totale_movimenti += Movimento::where('id_scadenza', '=', $scadenza)
-                    ->where('totale', '>', 0)
-                    ->sum('totale');
+                    $totale_pagamenti += Movimento::where('id_scadenza', '=', $scadenza)
+                        ->where('totale', '>', 0)
+                        ->where('is_insoluto', '=', 0)
+                        ->sum('totale');
+                    $totale_insoluti += Movimento::where('id_scadenza', '=', $scadenza)
+                        ->where('totale', '>', 0)
+                        ->where('is_insoluto', '=', 1)
+                        ->sum('totale');
                 }
 
-                $totale_da_distribuire = abs($totale_movimenti);
+                $totale_da_distribuire = abs($totale_pagamenti - $totale_insoluti);
 
                 // Ciclo tra le rate dei pagamenti per inserire su `pagato` l'importo effettivamente pagato
                 // Nel caso il pagamento superi la rata, devo distribuirlo sulle rate successive
@@ -282,7 +300,7 @@ private function getUniqueDocumenti($movimenti)
     {
         $documentIds = [];
         foreach ($movimenti as $movimento) {
-            if (!in_array($movimento->iddocumento, $documentIds)) {
+            if (! in_array($movimento->iddocumento, $documentIds)) {
                 $documentIds[] = $movimento->iddocumento;
             }
         }
