fix: avviso banca di addebito non selezionata per fatture con metodo di pagamento riba

Author: Pek5892

modules/fatture/edit.php

@@ -426,6 +426,11 @@
 				</div>
 
 				<div class="col-md-3">
+					<?php
+					$pagamento = $dbo->fetchOne('SELECT `codice_modalita_pagamento_fe` FROM `co_pagamenti` WHERE `id` = '.prepare($record['idpagamento']));
+					$show_riba_warning = $dir == 'entrata' && !empty($pagamento['codice_modalita_pagamento_fe']) && $pagamento['codice_modalita_pagamento_fe'] == 'MP12' && empty($record['id_banca_controparte']);
+					?>
+					<span id="riba-warning" class="badge badge-warning pull-right"><?php echo tr('Nessuna banca di addebito selezionata'); ?></span>
 					{[ "type": "select", "label": "<?php echo tr('Pagamento'); ?>", "name": "idpagamento", "required": 1, "ajax-source": "pagamenti", "value": "$idpagamento$" ]}
 				</div>
 
@@ -1029,69 +1034,11 @@
 echo '
 <script type="text/javascript">
 
-    // Imposta la direzione del documento nella sessione
-    session_set("superselect,dir", "'.$dir.'", 0);
-
-    // Imposta la banca controparte nella sessione
-    session_set("superselect,id_banca_controparte", $("#id_banca_controparte").val(), 0);
-
-    // Funzione per aggiornare il testo del pagamento in base alla banca
-    function aggiornaTestoPagamento() {
-        var bancaVal = $("#id_banca_controparte").val();
-        var pagamentoData = $("#idpagamento").selectData();
-        var bancaMancante = !bancaVal || bancaVal == "" || bancaVal == "null";
-
-        if (pagamentoData && pagamentoData.codice_modalita_pagamento_fe == "MP12") {
-            var avviso = " ('.tr('Nessuna banca di addebito selezionata').')";
-            var testoOriginale = pagamentoData.text || "";
-            var descrizioneBase = (pagamentoData.descrizione || "").replace(avviso, "");
-
-            // Rimuovi avviso esistente dal testo
-            var testoSenzaAvviso = testoOriginale.replace(avviso, "");
-
-            if (bancaMancante) {
-                // Aggiungi avviso prima della chiusura del link </a> o alla fine
-                var nuovoTesto;
-                if (testoSenzaAvviso.indexOf("</a>") !== -1) {
-                    nuovoTesto = testoSenzaAvviso.replace("</a>", avviso + "</a>");
-                } else {
-                    nuovoTesto = testoSenzaAvviso + avviso;
-                }
-
-                var nuovoData = $.extend({}, pagamentoData);
-                nuovoData.text = nuovoTesto;
-                nuovoData.descrizione = descrizioneBase + avviso;
-
-                $("#idpagamento").selectSetNew(pagamentoData.id, nuovoTesto, nuovoData);
-            } else {
-                var nuovoData = $.extend({}, pagamentoData);
-                nuovoData.text = testoSenzaAvviso;
-                nuovoData.descrizione = descrizioneBase;
-
-                $("#idpagamento").selectSetNew(pagamentoData.id, testoSenzaAvviso, nuovoData);
-            }
-        }
-    }
-
-    // Controllo iniziale al caricamento
-    $(document).ready(function() {
-        setTimeout(function() {
-            aggiornaTestoPagamento();
-            content_was_modified = false;
-        }, 500);
-    });
-
     $("#idtipodocumento").change(function() {
          updateSelectOption("idtipodocumento", $(this).val());
          session_set("superselect,idtipodocumento",$(this).val(), 0);
     });
 
-    $("#id_banca_controparte").change(function() {
-        session_set("superselect,id_banca_controparte", $(this).val(), 0).then(function() {
-            aggiornaTestoPagamento();
-        });
-    });
-
 	$("#idanagrafica").change(function() {
         updateSelectOption("idanagrafica", $(this).val());
         session_set("superselect,idanagrafica", $(this).val(), 0);
@@ -1119,6 +1066,27 @@ function aggiornaTestoPagamento() {
 echo '
 	});
 
+    $("#idpagamento").change(function() {
+        checkRibaWarning();
+    });
+
+    $("#id_banca_controparte").change(function() {
+        checkRibaWarning();
+    });
+
+    function checkRibaWarning() {
+        if ("'.($dir == 'entrata' ? 'true' : 'false').'") {
+            let pagamentoData = $("#idpagamento").selectData();
+            let bancaControparte = $("#id_banca_controparte").val();
+
+            if (pagamentoData && pagamentoData.codice_modalita_pagamento_fe === "MP12" && !bancaControparte) {
+                $("#riba-warning").show();
+            } else {
+                $("#riba-warning").hide();
+            }
+        }
+    }
+
     $("#ricalcola_scadenze").click(function() {
         swal({
             title: "'.tr('Desideri ricalcolare le scadenze?').'",

modules/pagamenti/ajax/select.php

@@ -66,18 +66,6 @@
         );
         $rs = $data['results'];
 
-        // Controllo metodi di pagamento con Ri.Ba. per le fatture di vendita
-        if (!empty($superselect['dir']) && $superselect['dir'] == 'entrata') {
-            $id_banca_controparte = $superselect['id_banca_controparte'] ?? '';
-            $banca_mancante = empty($id_banca_controparte) || $id_banca_controparte == 'null';
-
-            foreach ($rs as $k => $r) {
-                if ($r['codice_modalita_pagamento_fe'] == 'MP12' && $banca_mancante) {
-                    $rs[$k]['text'] = $r['descrizione'].' '.tr('(Nessuna banca di addebito selezionata)');
-                }
-            }
-        }
-
         $results = [
             'results' => $rs,
             'recordsFiltered' => $data['recordsFiltered'],