refactor: spostamento rinnovi in edit contratti

Author: Pek5892

modules/contratti/actions.php

@@ -111,6 +111,12 @@
             $contratto->codice_cig = post('codice_cig');
             $contratto->codice_cup = post('codice_cup');
 
+            // Salvataggio informazioni sul rinnovo
+            $contratto->rinnovabile = post('rinnovabile');
+            $contratto->rinnovo_automatico = post('rinnovo_automatico');
+            $contratto->giorni_preavviso_rinnovo = post('giorni_preavviso_rinnovo');
+            $contratto->ore_preavviso_rinnovo = post('ore_preavviso_rinnovo');
+
             $contratto->setScontoFinale(post('sconto_finale'), post('tipo_sconto_finale'));
             $contratto->idtipointervento = post('idtipointervento');
 
@@ -155,16 +161,6 @@
 
         break;
 
-    case 'update_rinnovo':
-        $contratto->rinnovabile = post('rinnovabile');
-        $contratto->rinnovo_automatico = post('rinnovo_automatico');
-        $contratto->giorni_preavviso_rinnovo = post('giorni_preavviso_rinnovo');
-        $contratto->ore_preavviso_rinnovo = post('ore_preavviso_rinnovo');
-        $contratto->save();
-        flash()->info(tr('Contratto modificato correttamente!'));
-
-        break;
-
         // Duplica contratto
     case 'copy':
         $new = $contratto->replicate(['idcontratto_prev']);

modules/contratti/edit.php

@@ -168,6 +168,24 @@
                 </div>
             </div>
 
+            <div class="row">
+                <div class="col-md-3">
+                    {[ "type": "checkbox", "label": "<?php echo tr('Rinnovabile'); ?>", "name": "rinnovabile", "help": "<?php echo tr('Il contratto è rinnovabile?'); ?>", "value": "$rinnovabile$", "class": "unblockable" ]}
+                </div>
+
+                <div class="col-md-3 rinnovo-field" <?php echo $record['rinnovabile'] ? '' : 'style="display:none;"'; ?>>
+                    {[ "type": "checkbox", "label": "<?php echo tr('Tacito rinnovo'); ?>", "name": "rinnovo_automatico", "help": "<?php echo tr('Il contratto può essere rinnovato senza chiedere conferma al cliente alla scadenza, questa impostazione non implica nessun automatismo di rinnovo.'); ?>", "value": "$rinnovo_automatico$", "disabled": <?php echo $record['rinnovabile'] ? 0 : 1; ?>, "class": "unblockable" ]}
+                </div>
+
+                <div class="col-md-3 rinnovo-field" <?php echo $record['rinnovabile'] ? '' : 'style="display:none;"'; ?>>
+                    {[ "type": "number", "label": "<?php echo tr('Preavviso per rinnovo'); ?>", "name": "giorni_preavviso_rinnovo", "decimals": "2", "value": "$giorni_preavviso_rinnovo$", "icon-after": "giorni", "disabled": <?php echo $record['rinnovabile'] ? 0 : 1; ?>, "class": "unblockable" ]}
+                </div>
+
+                <div class="col-md-3 rinnovo-field" <?php echo $record['rinnovabile'] ? '' : 'style="display:none;"'; ?>>
+                    {[ "type": "number", "label": "<?php echo tr('Ore rimanenti rinnovo'); ?>", "name": "ore_preavviso_rinnovo", "decimals": "2", "value": "$ore_preavviso_rinnovo$", "icon-after": "ore", "disabled": <?php echo $record['rinnovabile'] ? 0 : 1; ?>, "help": "<?php echo tr('Ore residue nel contratto prima di visualizzare una avviso per un eventuale rinnovo anticipato.'); ?>", "class": "unblockable" ]}
+                </div>
+            </div>
+
 			<div class="row">
 				<div class="col-md-4">
 					{[ "type": "textarea", "label": "<?php echo tr('Esclusioni'); ?>", "name": "esclusioni", "class": "autosize", "value": "$esclusioni$", "extra": "rows='5'" ]}
@@ -682,4 +700,20 @@ function caricaRighe(id_riga) {
 
     $("#id_sottocategoria").val(null).trigger("change");
 });
+
+input("rinnovabile").on("change", function() {
+    const disabled = parseInt($(this).val()) === 0;
+
+    // Disabilita i campi
+    input("giorni_preavviso_rinnovo").setDisabled(disabled);
+    input("ore_preavviso_rinnovo").setDisabled(disabled);
+    input("rinnovo_automatico").setDisabled(disabled);
+
+    // Mostra/nascondi i campi
+    if (disabled) {
+        $(".rinnovo-field").hide();
+    } else {
+        $(".rinnovo-field").show();
+    }
+});
 </script>';

plugins/rinnovi_contratti/edit.php

@@ -25,45 +25,9 @@
 $id_contratto_precedente = $record['idcontratto_prev'];
 $module = Module::find($id_module);
 
-echo '
-<form action="" method="post" id="rinnovo-form">
-	<input type="hidden" name="backto" value="record-edit">
-	<input type="hidden" name="op" value="update_rinnovo">
-    <input type="hidden" name="id_record" value="'.$id_record.'">
-
-    <div class="card card-primary">
-		<div class="card-header">
-			<h3 class="card-title">'.tr('Informazioni per rinnovo').'</h3>
-		</div>
-
-		<div class="card-body">
-            <div class="row">
-                <div class="col-md-3">
-                    {[ "type": "checkbox", "label": "'.tr('Rinnovabile').'", "name": "rinnovabile", "help": "'.tr('Il contratto è rinnovabile?').'", "value": "$rinnovabile$" ]}
-                </div>
-
-                <div class="col-md-3">
-                    {[ "type": "checkbox", "label": "'.tr('Tacito rinnovo').'", "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>
-
-                <div class="col-md-3">
-                    {[ "type": "number", "label": "'.tr('Ore rimanenti rinnovo').'", "name": "ore_preavviso_rinnovo", "decimals": "2", "value": "$ore_preavviso_rinnovo$", "icon-after": "ore", "disabled": '.($record['rinnovabile'] ? 0 : 1).', "help": "'.tr('Ore residue nel contratto prima di visualizzare una avviso per un eventuale rinnovo anticipato.').'" ]}
-                </div>
-            </div>
-
-            <div class="col-md-12 text-right">
-                <button type="submit" class="btn btn-success"><i class="fa fa-check"></i> '.tr('Salva').'</button>
-            </div>
-        </div>
-    </div>
-</form>';
-
 if (!empty($id_contratto_precedente)) {
     echo '
+<h4>'.tr('Storico rinnovi').'</h4>
 <table class="table table-hover table-sm table-bordered table-striped">
     <thead>
         <tr>
@@ -75,6 +39,11 @@
     </thead>
 
     <tbody>';
+} else {
+    echo '
+<div class="alert alert-info">
+    <i class="fa fa-info-circle"></i> '.tr('Non sono presenti voci da visualizzare nello storico dei rinnovi.').'
+</div>';
 }
 
 $counter = 0;
@@ -97,10 +66,14 @@
     $id_contratto_precedente = $rs[0]['idcontratto_prev'];
 }
 
-echo '
+// Chiudo la tabella solo se ci sono record da visualizzare
+if (!empty($record['idcontratto_prev'])) {
+    echo '
     </tbody>
-</table>
+</table>';
+}
 
+echo '
 <script type="text/javascript">
     input("rinnovabile").on("change", function() {
         const disabled = parseInt($(this).val()) === 0;