From ff8f0cfca2c5bcb44cb575239b4668e06ae89673 Mon Sep 17 00:00:00 2001
From: Dasc3er <Dasc3er@users.noreply.github.com>
Date: Mon, 20 Sep 2021 17:49:30 +0200
Subject: [PATCH] Correzioni minori

Limite ricorsione a 50 per elenco contratti precedenti (per evitare loop infinito).
---
 modules/contratti/actions.php      | 2 +-
 modules/contratti/buttons.php      | 2 +-
 modules/preventivi/actions.php     | 2 +-
 plugins/rinnovi_contratti/edit.php | 5 +++--
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/modules/contratti/actions.php b/modules/contratti/actions.php
index df4ecc30d..481c45738 100644
--- a/modules/contratti/actions.php
+++ b/modules/contratti/actions.php
@@ -419,7 +419,7 @@ $riga = $contratto->getRiga($type, $id_riga);
 
         break;
 
-        case 'import':
+    case 'import':
         $rs = $dbo->fetchArray('SELECT * FROM co_contratti_tipiintervento WHERE idcontratto = '.prepare(post('idcontratto')).' AND idtipointervento='.prepare(post('idtipointervento')));
 
         // Se la riga in_tipiintervento esiste, la aggiorno...
diff --git a/modules/contratti/buttons.php b/modules/contratti/buttons.php
index 533971c97..fc1609c7e 100755
--- a/modules/contratti/buttons.php
+++ b/modules/contratti/buttons.php
@@ -42,7 +42,7 @@ $stati_pianificabili = $dbo->fetchOne('SELECT GROUP_CONCAT(`descrizione` SEPARAT
 echo '
 <div class="tip" data-toggle="tooltip" title="'.tr('Il contratto è rinnovabile se sono definite le date di accettazione e conclusione e si trova in uno stato di questi stati: _STATE_LIST_', [
         '_STATE_LIST_' => $stati_pianificabili,
-    ]).'">
+    ]).'" id="rinnova">
     <button type="button" class="btn btn-warning ask '.($rinnova ? '' : 'disabled').'" data-backto="record-edit" data-op="renew" data-msg="'.tr('Rinnovare questo contratto?').'" data-button="'.tr('Rinnova').'" data-class="btn btn-lg btn-warning">
         <i class="fa fa-refresh"></i> '.tr('Rinnova').'...
     </button>
diff --git a/modules/preventivi/actions.php b/modules/preventivi/actions.php
index c8c761c63..d180988b4 100755
--- a/modules/preventivi/actions.php
+++ b/modules/preventivi/actions.php
@@ -117,7 +117,7 @@ switch (post('op')) {
         }
 
         flash()->info(tr('Preventivo duplicato correttamente!'));
-    break;
+        break;
 
     case 'addintervento':
         if (post('idintervento') !== null) {
diff --git a/plugins/rinnovi_contratti/edit.php b/plugins/rinnovi_contratti/edit.php
index 3e812a3d4..50f13ffd6 100755
--- a/plugins/rinnovi_contratti/edit.php
+++ b/plugins/rinnovi_contratti/edit.php
@@ -43,7 +43,6 @@ echo '
                     {[ "type": "checkbox", "label": "'.tr('Rinnovo automatico').'", "name": "rinnovo_automatico", "help": "'.tr('Il contratto è da rinnovare automaticamente alla scadenza').'", "value": "$rinnovo_automatico$", "disabled": '.($record['rinnovabile'] ? 0 : 1).' ]}
                 </div>
 
-
                 <div class="col-md-3">
                     {[ "type": "number", "label": "'.tr('Preavviso per rinnovo').'", "name": "giorni_preavviso_rinnovo", "decimals": "2", "value": "$giorni_preavviso_rinnovo$", "icon-after": "giorni", "disabled": '.($record['rinnovabile'] ? 0 : 1).' ]}
                 </div>
@@ -73,7 +72,9 @@ echo '
 
     <tbody>';
 
-while (!empty($id_contratto_precedente)) {
+$counter = 0;
+while (!empty($id_contratto_precedente) && $counter < 50) {
+    $counter++;
     $rs = $dbo->fetchArray('SELECT nome, numero, data_accettazione, data_conclusione, budget, idcontratto_prev FROM co_contratti WHERE id='.prepare($id_contratto_precedente));
 
     echo '