From d1b8ba9c2b5f016096471f8f146f971075c18e16 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sun, 23 Feb 2020 14:42:25 +0100
Subject: [PATCH] Correzione #764

Rimozione dei controlli sui totali per le scadenze generiche.
---
 modules/scadenzario/edit.php | 82 +++++++++++++++---------------------
 1 file changed, 34 insertions(+), 48 deletions(-)

diff --git a/modules/scadenzario/edit.php b/modules/scadenzario/edit.php
index 4eeb85b0a..86e42ff7f 100755
--- a/modules/scadenzario/edit.php
+++ b/modules/scadenzario/edit.php
@@ -13,7 +13,7 @@ echo '
 	<input type="hidden" name="op" value="update">
 	<input type="hidden" name="backto" value="record-edit">
 	<input type="hidden" name="id_record" value="'.$id_record.'">
-	
+
 	<input type="hidden" name="tipo" value="'.$record['tipo'].'">
 	<input type="hidden" name="descrizione" value="'.$record['descrizione'].'">
 	<input type="hidden" name="iddocumento" value="'.$record['iddocumento'].'">
@@ -103,14 +103,13 @@ echo '
 
                         <tbody id="scadenze">';
 
-$totale_da_pagare = 0;
-$totale_pagato = 0;
-
-// Scelgo la query in base al segmento
+// Scelgo la query in base alla scadenza
 if (!empty($documento)) {
     $rs = $dbo->fetchArray('SELECT * FROM co_scadenziario WHERE iddocumento = '.prepare($documento->id).' ORDER BY scadenza ASC');
+    $totale_da_pagare = $documento->netto;
 } else {
     $rs = $dbo->fetchArray('SELECT * FROM co_scadenziario WHERE id = '.prepare($id_record).' ORDER BY scadenza ASC');
+    $totale_da_pagare = sum(array_column($rs, 'da_pagare'));
 }
 
 foreach ($rs as $i => $scadenza) {
@@ -146,18 +145,6 @@ foreach ($rs as $i => $scadenza) {
                         </tr>';
 }
 
-$totale_da_pagare = $documento->netto;
-$totale_pagato = sum(array_column($rs, 'pagato'));
-
-if ($totale_da_pagare == $totale_pagato) {
-    echo '
-<script>
-$(document).ready(function(){
-    $("#pulsanti").children().find("a:nth-child(2)").attr("disabled", true).addClass("disabled");
-})
-</script>';
-}
-
 echo '
                         </tbody>
                         <tfoot>
@@ -178,7 +165,7 @@ echo '
 
 					<div class="clearfix"></div>
                     <br>
-                    
+
 					<div class="alert alert-error hide" id="totale"><?php echo tr('Il totale da pagare deve essere pari a _MONEY_', [
                         '_MONEY_' => '<b>'.moneyFormat($totale_da_pagare).'</b>',
                     ]); ?>.<br><?php echo tr('Differenza di _TOT_ _CURRENCY_', [
@@ -197,32 +184,31 @@ echo '
 {( "name": "log_email", "id_module": "$id_module$", "id_record": "$id_record$" )}
 
 <?php
-if ($records[0]['iddocumento'] == 0) {
-                            ?>
+if (empty($documento)) {
+echo '
 <a class="btn btn-danger ask" data-backto="record-list">
-    <i class="fa fa-trash"></i> <?php echo tr('Elimina'); ?>
-</a>
-<?php
-                        }
+    <i class="fa fa-trash"></i> '.tr('Elimina').'
+</a>';
+}
 
 echo '
 <table class="hide">
     <tbody id="scadenza-template">
         <tr class="danger">
             <input type="hidden" name="id_scadenza[-id-]" value="">
-            
+
             <td align="center">
                 {[ "type": "date", "name": "scadenza[-id-]" ]}
             </td>
-    
+
             <td align="right">
                 {[ "type": "number", "name": "da_pagare[-id-]", "decimals": 2, "onchange": "controlloTotale()" ]}
             </td>
-    
+
             <td align="right">
                 {[ "type": "number", "name": "pagato[-id-]", "decimals": 2 ]}
             </td>
-            
+
             <td align="center">
                 {[ "type": "date", "name": "data_concordata[-id-]" ]}
             </td>
@@ -234,36 +220,36 @@ echo '
     var i = '.$i.';
 	$(document).on("click", "#add-scadenza", function(){
         cleanup_inputs();
-        
+
         i++;
 		var text = replaceAll($("#scadenza-template").html(), "-id-", "" + i);
-		
+
 		$("#scadenze").append(text);
 		restart_inputs();
 	});
 </script>';
 
-?>
-
+// Abilitazione dei controlli solo per Scadenze collegate a documenti
+if (!empty($documento)) {
+    echo '
 <script>
     globals.cifre_decimali = 2;
 
 	$(document).ready(function(){
-        controlloTotale();
+        controlloTotale();';
 
-        <?php
-        if ($dir == 'uscita') {
-            echo '
+    if ($dir == 'uscita') {
+        echo '
         $("#email-button").remove();';
-        }
-        ?>
+    }
+    echo '
 	});
 
     function controlloTotale() {
-        totale_da_pagare = $('#totale_da_pagare').val().toEnglish();
+        totale_da_pagare = $("#totale_da_pagare").val().toEnglish();
         totale_utente = 0;
 
-        $('input[name*=da_pagare]').each(function() {
+        $("input[name*=da_pagare]").each(function() {
 
             totale_utente += $(this).val().toEnglish();
 
@@ -276,18 +262,18 @@ echo '
         totale_utente = Math.round(totale_utente * 100) / 100;
         totale_da_pagare = Math.round(totale_da_pagare * 100) / 100;
 
-
         diff = Math.abs(totale_da_pagare) - Math.abs(totale_utente);
 
         if (diff == 0) {
-            $('#btn-saves').removeClass('hide');
-            $('#totale').addClass('hide');
+            $("#save").removeClass("hide");
+            $("#totale").addClass("hide");
         } else {
-            $('#btn-saves').addClass('hide');
-            $('#totale').removeClass('hide');
+            $("#save").addClass("hide");
+            $("#totale").removeClass("hide");
         }
 
-        $('#diff').html(diff.toLocale());
-        $('#totale_utente').html(totale_utente.toLocale());
+        $("#diff").html(diff.toLocale());
+        $("#totale_utente").html(totale_utente.toLocale());
     }
-</script>
+</script>';
+}