Corrections contrôles validité date et affichage état immo ignorées

add_asset.html

@@ -108,6 +108,11 @@
 		{{/if}}
 		{{:assign duration=$_POST.duree|intval}}
 		{{:assign status="managed"}}
+		{{if $_POST.date_mes == null ||  $_POST.date_mes == $_POST.date_achat}}
+			{{:assign date_mes=null}}
+		{{else}}
+			{{:assign date_mes=$_POST.date_mes|date_short}}
+		{{/if}}
 	{{/if}}
 
 	{{* enregistrer l'écriture *}}
@@ -141,7 +146,7 @@
 			type="immo"
 			line=$immo_line_id
 			duration=$duration
-			date_mes=$date_debut
+			date_mes=$date_mes
 			status=$status
 		}}
 	{{/if}}

add_infos.html

@@ -60,6 +60,11 @@
 		{{:assign status="ignored"}}
 	{{else}}
 
+		{{if $_POST.date_achat|parse_date > $ligne_immo.date_achat}}
+			{{:assign da=$ligne_immo.date_achat|date_short}}
+			{{:error message="Erreur : la date d'achat (%s) ne peut être postérieure à la date de l'écriture d'immobilisation (%s)"|args:$_POST.date_achat:$da}}
+		{{/if}}
+
 		{{* vérifier que la date de mise en service est postérieure à la date d'acquisition *}}
 		{{:assign d1=$_POST.date_achat|or:$ligne_immo.date_achat|parse_date}}
 		{{:assign d2=$_POST.date_mes|or:$d1|parse_date}}
@@ -126,7 +131,6 @@
 		date_achat=$date_achat
 		date_mes=$date_mes
 		status=$status
-		assign_new_id="new_id"
 	}}
 
 	{{if $_POST.amortir}}
@@ -136,14 +140,13 @@
 	{{/if}}
 	{{:redirect force="index.html?ok=1&msg=infos&type_immo=%s"|args:$type_immo}}
 {{/form}}
-{{:form_errors}}
 
 {{:admin_header title="Classer l'immobilisation" custom_css="./style.css" current="module_amortization"}}
-
 {{* barre de navigation *}}
 {{if ! $dialog}}
 	{{:include file="_nav.html" current="index" subcurrent="%s"|args:$_GET.type_immo subsubcurrent="parameter"}}
 {{/if}}
+{{:form_errors}}
 
 {{:assign choix_defaut=$_GET.type_immo}}
 
@@ -174,7 +177,12 @@
 	<fieldset>
 		<legend>Amortissement</legend>
 		<dl>
-			{{:input id="amortir" type="checkbox" value=1 name="amortir" label="Amortir cette immobilisation sur une ou plusieurs années" checked="checked" help="Dé-cocher pour ne pas amortir"}}
+			{{if $info_immo.status == 'ignored'}}
+				{{:assign checked=null}}
+			{{else}}
+				{{:assign checked="checked"}}
+			{{/if}}
+			{{:input id="amortir" type="checkbox" value=1 name="amortir" label="Amortir cette immobilisation sur une ou plusieurs années" checked="%s"|args:$checked help="Dé-cocher pour ne pas amortir"}}
 			<div class="masquable">
 				{{:input type="number" name="duree" label="Durée d'amortissement" required=true min=1}}
 			</div>
@@ -201,13 +209,20 @@
 <script type="text/javascript" src="scripts.js"></script>
 <script type="text/javascript">
 
-function changeVisibility(evt, idcheck = 'f_amortir_1', hiddenclass = 'masquable')
+function changeVisibility(evt, idcheck = 'f_amortir_1', hiddenclass = 'masquable', id_help = 'help_f_amortir_1')
 {
 	toggleVisibility(idcheck, document.querySelectorAll('.' + hiddenclass));
+	const help_elem = document.getElementById(id_help);
+	if (document.getElementById(idcheck).checked) {
+		help_elem.innerText = "Dé-cocher pour ne pas amortir";
+	} else {
+		help_elem.innerText = "Cocher pour amortir";
+	}
 }
 
 (function () {
 	document.getElementById('f_amortir_1').onclick = changeVisibility;
+	changeVisibility(null);
 })();
 
 </script>

modify_infos.html

@@ -92,7 +92,6 @@
 		{{:assign date_mes=null}}
 		{{:assign status="ignored"}}
 	{{else}}
-
 		{{if $_POST.date_achat == null || $_POST.date_achat|parse_date == $ligne_immo.date_achat}}
 			{{:assign date_achat=null}}
 		{{elseif $_POST.date_achat|parse_date == $info_immo.date_achat}}
@@ -100,14 +99,23 @@
 		{{else}}
 			{{:assign date_achat=$_POST.date_achat|parse_date}}
 		{{/if}}
+		{{if $_POST.date_achat|parse_date > $ligne_immo.date_achat}}
+			{{:assign da=$ligne_immo.date_achat|date_short}}
+			{{:error message="Erreur : la date d'achat (%s) ne peut être postérieure à la date de l'écriture d'immobilisation (%s)"|args:$_POST.date_achat:$da}}
+		{{/if}}
+
 		{{if $_POST.date_mes == null || $_POST.date_mes|parse_date == $ligne_immo.date_achat}}
 			{{:assign date_mes=null}}
 		{{elseif $_POST.date_mes|parse_date == $info_immo.date_achat}}
 			{{:assign date_mes=null}}
 		{{else}}
 			{{:assign date_mes=$_POST.date_mes|parse_date}}
-			{{if $date_mes < $ligne_immo.date_achat || $date_mes < $info_immo.date_achat}}
-				{{:assign da=$date_achat|or:$info_immo.date_achat|or:$ligne_immo.date_achat|date_short}}
+			{{:assign date_achat_comp=$ligne_immo.date_achat}}
+			{{if $date_achat != null}}
+				{{:assign date_achat_comp=$date_achat}}
+			{{/if}}
+			{{if $date_mes < $date_achat_comp}}
+				{{:assign da=$date_achat_comp|date_short}}
 				{{:error message="Erreur : la date de mise en service (%s) ne peut être antérieure à la date d'acquisition (%s)"|args:$_POST.date_mes:$da}}
 			{{/if}}
 		{{/if}}

scripts.js

@@ -79,7 +79,7 @@ function setDateEnd(id_exercices, id_date, id_years) {
 
 // renvoyer la valeur numérique d'un montant formaté en €
 function getNumber(text) {
-	return Number(text.replace(/[^0-9,]/g, '').replace(/,/, '.')); 
+	return Number(text.replace(/[^0-9,]/g, '').replace(/,/, '.'));
 }
 
 // calculer le montant d'un amortissement

style.css

@@ -94,3 +94,7 @@ html.dialog.dark main {
 html.dark .block.alert {
 	background-color : rgba(87, 206, 235, 0.5);
 }
+
+span.input-list.avoirs label {
+	padding : 0.4rem 0.6rem;
+}