refactor: gestione natura iva non piu valide

Author: Pek5892

modules/fatture/row-list.php

@@ -283,15 +283,25 @@
             </td>';
 
         // Iva
-        // Controllo aliquota esente senza codice natura
+        // Controllo aliquota esente senza codice natura o con codice natura obsoleto (N2, N3, N6 senza sottocodice)
+        $codici_natura_obsoleti = ['N2', 'N3', 'N6'];
         $iva_esente_senza_natura = $riga->aliquota && $riga->aliquota->esente && empty($riga->aliquota->codice_natura_fe);
-        $iva_class = ($riga->aliquota->deleted_at || $iva_esente_senza_natura) ? 'text-danger' : 'text-muted';
-        $iva_tooltip = $iva_esente_senza_natura ? ' title="'.tr('Attenzione: aliquota esente senza codice natura IVA. Correggere prima di emettere fattura elettronica.').'" style="cursor: help;"' : '';
+        $iva_natura_obsoleta = $riga->aliquota && $riga->aliquota->esente && in_array($riga->aliquota->codice_natura_fe, $codici_natura_obsoleti);
+        $iva_errore = $iva_esente_senza_natura || $iva_natura_obsoleta;
+        $iva_class = ($riga->aliquota->deleted_at || $iva_errore) ? 'text-danger' : 'text-muted';
+
+        if ($iva_esente_senza_natura) {
+            $iva_tooltip = ' title="'.tr('Attenzione: aliquota esente senza codice natura IVA. Correggere prima di emettere fattura elettronica.').'" style="cursor: help;"';
+        } elseif ($iva_natura_obsoleta) {
+            $iva_tooltip = ' title="'.tr('Attenzione: il codice natura "_NATURA_" non è più valido dal 1° gennaio 2021. Utilizzare un sottocodice specifico.', ['_NATURA_' => $riga->aliquota->codice_natura_fe]).'" style="cursor: help;"';
+        } else {
+            $iva_tooltip = '';
+        }
 
         echo '
             <td class="text-right">
                 '.moneyFormat($riga->iva_unitaria_scontata).'
-                <br><small class="'.$iva_class.'"'.$iva_tooltip.'>'.($iva_esente_senza_natura ? '<i class="fa fa-exclamation-triangle"></i> ' : '').($riga->aliquota ? $riga->aliquota->getTranslation('title') : '').' ('.$riga->aliquota->esigibilita.') '.(($riga->aliquota->esente) ? ' ('.$riga->aliquota->codice_natura_fe.')' : null).'</small>
+                <br><small class="'.$iva_class.'"'.$iva_tooltip.'>'.($iva_errore ? '<i class="fa fa-exclamation-triangle"></i> ' : '').($riga->aliquota ? $riga->aliquota->getTranslation('title') : '').' ('.$riga->aliquota->esigibilita.') '.(($riga->aliquota->esente) ? ' ('.$riga->aliquota->codice_natura_fe.')' : null).'</small>
             </td>';
 
         // Importo

modules/iva/actions.php

@@ -72,9 +72,9 @@
         $esigibilita = post('esigibilita');
         $indetraibile = post('indetraibile');
 
-        $aliquota = Aliquota::where('id', '=', (new Aliquota())->getByField('title', $descrizione))->where('codice', '=', $codice)->first();
+        $aliquota = Aliquota::where('name', '=', $descrizione)->where('codice', '=', $codice)->first();
         if (!$aliquota) {
-            $iva = Aliquota::build($descrizione, $esente, $percentuale, $indetraibile, $dicitura, $codice, $codice_natura_fe, $esigibilita);
+            $iva = Aliquota::build($descrizione, $esente, $percentuale, $indetraibile, $dicitura, $codice, $codice_natura, $esigibilita);
             $id_record = $dbo->lastInsertedID();
             $iva->save();
 

modules/iva/ajax/select.php

@@ -41,8 +41,8 @@
         if (empty($filter)) {
             $where[] = '`deleted_at` IS NULL';
 
-            // Escludo le aliquote esenti senza codice natura iva
-            $where[] = '(`esente` = 0 OR (`esente` = 1 AND `codice_natura_fe` IS NOT NULL))';
+            // Escludo le aliquote esenti senza codice natura iva o con codice natura obsoleto (N2, N3, N6 senza sottocodice)
+            $where[] = '(`esente` = 0 OR (`esente` = 1 AND `codice_natura_fe` IS NOT NULL AND `codice_natura_fe` NOT IN ("N2", "N3", "N6")))';
 
             // se sto valorizzando un documento con lo split payment impedisco la selezione delle aliquote iva con natura N6.X (reverse charge)
             if (isset($superselect['split_payment']) and !empty($superselect['split_payment'])) {

modules/iva/edit.php

@@ -46,12 +46,18 @@
     $is_readonly = '1';
 }
 
-// Alert per aliquota esente senza codice natura
+// Alert per aliquota esente senza codice natura o con codice natura obsoleto
+$codici_natura_obsoleti = ['N2', 'N3', 'N6'];
 if ($record['esente'] && empty($record['codice_natura_fe'])) {
     echo '
     <div class="alert alert-warning">
         <i class="fa fa-exclamation-triangle"></i> <strong>'.tr('Attenzione').':</strong> '.tr('Questa aliquota è marcata come esente ma non ha un codice natura IVA impostato. Per la fatturazione elettronica è obbligatorio specificare il codice natura per le aliquote esenti.').'
     </div>';
+} elseif ($record['esente'] && in_array($record['codice_natura_fe'], $codici_natura_obsoleti)) {
+    echo '
+    <div class="alert alert-warning">
+        <i class="fa fa-exclamation-triangle"></i> <strong>'.tr('Attenzione').':</strong> '.tr('Il codice natura "_NATURA_" non è più valido dal 1° gennaio 2021. È necessario utilizzare un sottocodice specifico (es. _NATURA_.1, _NATURA_.2, ecc.).', ['_NATURA_' => $record['codice_natura_fe']]).'
+    </div>';
 }
 
 ?><form action="" method="post" id="edit-form">
@@ -86,7 +92,7 @@
 				</div>
 
                 <div class="col-md-4">
-					{[ "type": "select", "label": "<?php echo tr('Codice Natura (Fatturazione Elettronica)'); ?>", "name": "codice_natura_fe", "value": "$codice_natura_fe$", "required": <?php echo intval($record['esente']); ?>, "disabled": <?php echo intval(!$record['esente']); ?>, "values": "query=SELECT `codice` as id, CONCAT(`codice`, ' - ', `title`) AS descrizione FROM fe_natura LEFT JOIN `fe_natura_lang` ON (`fe_natura`.`codice` = `fe_natura_lang`.`id_record` AND `fe_natura_lang`.`id_lang` = <?php echo prepare(Models\Locale::getDefault()->id); ?>)", "readonly": "<?php echo $is_readonly; ?>", "extra": "<?php echo $attr; ?>" ]}
+					{[ "type": "select", "label": "<?php echo tr('Codice Natura (Fatturazione Elettronica)'); ?>", "name": "codice_natura_fe", "value": "$codice_natura_fe$", "required": <?php echo intval($record['esente']); ?>, "disabled": <?php echo intval(!$record['esente']); ?>, "values": "query=SELECT `codice` as id, CONCAT(`codice`, ' - ', `title`) AS descrizione FROM fe_natura LEFT JOIN `fe_natura_lang` ON (`fe_natura`.`codice` = `fe_natura_lang`.`id_record` AND `fe_natura_lang`.`id_lang` = <?php echo prepare(Models\Locale::getDefault()->id); ?>) WHERE `codice` NOT IN ('N2', 'N3', 'N6')", "readonly": "<?php echo $is_readonly; ?>", "extra": "<?php echo $attr; ?>" ]}
 				</div>
 			</div>