From 435edd153a649a4800a0ae8d1598108f44c45fd6 Mon Sep 17 00:00:00 2001
From: Pek5892 <pek5892@gmail.com>
Date: Fri, 16 Feb 2024 12:46:17 +0100
Subject: [PATCH] Fix modifica stati dei contratti

---
 modules/stati_contratto/actions.php | 29 ++++++++++++++++++-----------
 modules/stati_contratto/edit.php    |  2 +-
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/modules/stati_contratto/actions.php b/modules/stati_contratto/actions.php
index 7b3bb3cc4..35d1506a0 100755
--- a/modules/stati_contratto/actions.php
+++ b/modules/stati_contratto/actions.php
@@ -23,20 +23,27 @@ use Modules\Contratti\Stato;
 
 switch (post('op')) {
     case 'update':
+        $id_stato_old = (new Stato())->getByName($record['name'])->id_record;
         $id_stato = (new Stato())->getByName(post('descrizione'))->id_record;
-        $dbo->update('co_staticontratti', [
-            'icona' => post('icona'),
-            'colore' => post('colore'),
-            'is_completato' => post('is_completato') ?: null,
-            'is_fatturabile' => post('is_fatturabile') ?: null,
-            'is_pianificabile' => post('is_pianificabile') ?: null,
-        ], ['id' => $id_record]);
 
-        $dbo->update('co_staticontratti_lang', [
-            'name' => post('descrizione'),
-        ], ['id_record' => $id_stato]);
+        if (($id_stato)) {
+            flash()->error(tr('Questo nome è già stato utilizzato per un altro stato dei contratti.'));
+        } else {
+            $dbo->update('co_staticontratti', [
+                'icona' => post('icona'),
+                'colore' => post('colore'),
+                'is_completato' => post('is_completato') ?: null,
+                'is_fatturabile' => post('is_fatturabile') ?: null,
+                'is_pianificabile' => post('is_pianificabile') ?: null,
+            ], ['id' => $id_stato_old]);
+    
+            $dbo->update('co_staticontratti_lang', [
+                'name' => post('descrizione'),
+            ], ['id_record' => $id_stato_old]);
+    
+            flash()->info(tr('Informazioni salvate correttamente.'));
+        }
 
-        flash()->info(tr('Informazioni salvate correttamente.'));
 
         break;
 
diff --git a/modules/stati_contratto/edit.php b/modules/stati_contratto/edit.php
index 136aec586..6e20dde78 100755
--- a/modules/stati_contratto/edit.php
+++ b/modules/stati_contratto/edit.php
@@ -34,7 +34,7 @@ if ($record['can_delete']) {
 
 	<div class="row">
 		<div class="col-md-4">
-			{[ "type": "text", "label": "<?php echo tr('Descrizione'); ?>", "name": "descrizione", "required": 1, "value": "$descrizione$", "extra": "<?php echo $attr; ?>" ]}
+			{[ "type": "text", "label": "<?php echo tr('Descrizione'); ?>", "name": "descrizione", "required": 1, "value": "$name$", "extra": "<?php echo $attr; ?>" ]}
 		</div>
 
 		<div class="col-md-3">