correction erreur modif mvt

_calcul_dispo.html

@@ -37,7 +37,13 @@
 {{:include file="./_get_config.html" keep="config"}}
 
 {{* parcourir les mouvements et cumuler les entrées/sorties *}}
-{{#load type="movement" where="$$.equipment IN %s"|args:$in_equipments assign="mvt"}}
+{{if $date == null}}
+	{{:assign condition="$$.equipment IN %s"|args:$in_equipments}}
+{{else}}
+	{{:assign condition="$$.equipment IN %s AND $$.date <= '%s'"|args:$in_equipments:$date}}
+{{/if}}
+
+{{#load type="movement" where="%s"|args:$condition assign="mvt" order="$$.date ASC"}}
 	{{* matériel propriété de l'asso en stock *}}
 	{{:assign
 		var="stock"

historique.html

@@ -10,6 +10,37 @@
 {{/load}}
 {{:admin_header title="Historique des mouvements de %s (%s)"|args:$equipment.designation:$category.name current="module_equipment"}}
 
+{{* récupérer la config des entrées/sorties *}}
+{{:include file="./_get_config.html" keep="config"}}
+
+{{* déterminer les type de mouvements selon l'affection du matériel *}}
+{{#foreach from=$config.input_nature key=mvt item=type_mvt}}
+	{{if $_GET.prop}}
+		{{* matériel propriété de l'asso *}}
+		{{if $type_mvt != 'temporaire'}}
+			{{:assign var="input_types." value=$mvt}}
+		{{/if}}
+	{{else}}
+		{{* matériel non propriété de l'asso *}}
+		{{if $type_mvt == 'temporaire'}}
+			{{:assign var="input_types." value=$mvt}}
+		{{/if}}
+	{{/if}}
+{{/foreach}}
+{{#foreach from=$config.output_nature key=mvt item=type_mvt}}
+	{{if $_GET.prop}}
+		{{* matériel propriété de l'asso *}}
+		{{if $type_mvt != 'retour'}}
+			{{:assign var="output_types." value=$mvt}}
+		{{/if}}
+	{{else}}
+		{{* matériel non propriété de l'asso *}}
+		{{if $type_mvt == 'retour'}}
+			{{:assign var="output_types." value=$mvt}}
+		{{/if}}
+	{{/if}}
+{{/foreach}}
+
 {{* lister tous les mouvements du matériel passé en paramètre *}}
 {{#list
 	type="movement"
@@ -21,6 +52,9 @@
 	equipment=$equipment_key
 	order=1
 }}
+	{{if $direction === "input"  && $input_nature|in:$input_types ||
+		 $direction === "output" && $output_nature|in:$output_types
+	}}
 		<tr>
 			<td>{{$date|date_short}}</td>
 			<td>{{$col2}}</td>
@@ -29,6 +63,7 @@
 			<td>{{$comment}}</td>
 			<td></td>
 		</tr>
+	{{/if}}
 {{/list}}
 
 {{:admin_footer}}

inventaire.html

@@ -1,15 +1,13 @@
+{{* -*- brindille -*- *}}
+
 {{* inventaire des entrées/sorties des matériels des catégories de la variable category_keys *}}
 
 {{* Extraire et compiler les infos de la base *}}
-{{:include file="./_calcul_dispo.html" keep="cumul_mvt,category_keys"}}
+{{:include file="./_calcul_dispo.html" keep="cumul_mvt,category_keys,equipments"}}
 
 {{* Afficher les résultats *}}
-{{if $cumul_mvt === null}}
+{{if $equipments === null}}
-	{{if $category_keys|count == 1}}
+		<p class="block alert">Aucun matériel.</p>
-		<p class="block alert">Il n'y a aucun matériel dans cette catégorie.</p>
-	{{else}}
-		<p class="block alert">Il n'y a aucun matériel dans ces catégories.</p>
-	{{/if}}
 {{else}}
 	<section class="inventaire">
 		<h2 class="ruler">Matériels dont l'association est propriétaire</h2>
@@ -17,10 +15,6 @@
 		{{* itérer sur les catégories *}}
 		{{#foreach from=$cumul_mvt key="cat_key" item="cat_elem"}}
 
-			{{if $print_cat_name}}
-				<h3>{{$cat_elem.name}}</h3>
-			{{/if}}
-
 			{{* vérifier s'il y a des matériels dans cette catégorie *}}
 			{{:assign present=false}}
 			{{if $cat_elem|keys|has:"eqpmt"}}
@@ -32,6 +26,10 @@
 				{{/foreach}}
 			{{/if}}
 			{{if $present}}
+				{{if $print_cat_name}}
+					<h3>{{$cat_elem.name}}</h3>
+				{{/if}}
+
 				<table class="list">
 					<thead>
 						<tr>
@@ -63,7 +61,7 @@
 									<td class="actions">
 										{{:linkbutton
 											label="Historique des mouvements"
-											href="%shistorique.html?key=%s"|args:$module.url:$eqpmt_key
+											href="%shistorique.html?key=%s&prop=1"|args:$module.url:$eqpmt_key
 											shape="table"
 											target="_dialog"}}
 										{{:linkbutton
@@ -79,8 +77,6 @@
 						{{/foreach}}
 					</tbody>
 				</table>
-			{{else}}
-				<p class="block alert">Il n'y a aucun matériel dans cette catégorie.</p>
 			{{/if}}
 		{{/foreach}}
 
@@ -89,10 +85,6 @@
 		{{* itérer sur les catégories *}}
 		{{#foreach from=$cumul_mvt key="cat_key" item="cat_elem"}}
 
-			{{if $print_cat_name}}
-				<h3>{{$cat_elem.name}}</h3>
-			{{/if}}
-
 			{{* vérifier s'il y a des matériels dans cette catégorie *}}
 			{{:assign present=false}}
 			{{if $cat_elem|keys|has:"eqpmt"}}
@@ -104,6 +96,11 @@
 				{{/foreach}}
 			{{/if}}
 			{{if $present}}
+
+				{{if $print_cat_name}}
+					<h3>{{$cat_elem.name}}</h3>
+				{{/if}}
+
 				<table class="list">
 					<thead>
 						<tr>
@@ -142,8 +139,6 @@
 						{{/foreach}}
 					</tbody>
 				</table>
-			{{else}}
-				<p class="block alert">Il n'y a aucun matériel dans cette catégorie.</p>
 			{{/if}}
 		{{/foreach}}
 	</section>

mouvements/modifier_mouvement.html

@@ -1,28 +1,43 @@
-{{:admin_header title="Modifier une %s"|args:$_GET.direction current="module_equipment"}}
+{{* -*- brindille -*- *}}
 
-{{:assign direction=$_GET.direction}}
+{{if $_GET.direction == "input"}}
+	{{:assign msg_direction="entrée"}}
+{{else}}
+	{{:assign msg_direction="sortie"}}
+{{/if}}
+{{:admin_header title="Modifier une %s"|args:$msg_direction current="module_equipment"}}
+
+{{:assign direction_mvt=$_GET.direction}}
 {{:assign key_mvt=$_GET.key}}
 
 {{* récupérer les infos du mouvement à modifier *}}
 {{#load key=$key_mvt assign="mvt_modif"}}
 {{else}}
-	{{:error message="Aucune %s avec la clé %s"|args:$_GET.direction:$key_mvt}}
+	{{:error message="Aucune %s avec la clé %s"|args:$msg_direction:$key_mvt}}
 {{/load}}
-{{if $direction == "input"}}
+
+{{if $direction_mvt == "input"}}
 	{{:assign input_init=$mvt_modif.input_nature}}
 	{{:assign url="entrees/index.html"}}
 {{else}}
 	{{:assign output_init=$mvt_modif.output_nature}}
 	{{:assign url="sorties/index.html"}}
 {{/if}}
+
 {{:assign amount_init=$mvt_modif.amount}}
 {{:assign date_init=$mvt_modif.date|date_short}}
 {{:assign key_eqpmt_init=$mvt_modif.equipment}}
 
+{{* récupérer les infos du matériel associé *}}
+{{#load key=$key_eqpmt_init assign="eqpmt_init"}}
+{{else}}
+	{{:error message="Aucun matériel avec la clé « %s »"|args:$key_eqpmt_init}}
+{{/load}}
+
 {{* récupérer la config des entrées/sorties *}}
 {{:include file="../_get_config.html" keep="config"}}
 
-{{if $direction == "input"}}
+{{if $direction_mvt == "input"}}
 	{{* types d'entrées *}}
 	{{#foreach from=$config.input_nature key="label" item="type"}}
 		{{:assign var="input_labels.%s"|args:$label value="%s"|args:$label}}
@@ -40,18 +55,12 @@
 	{{/foreach}}
 {{/if}}
 
-{{* récupérer les infos du matériel associé *}}
-{{#load key=$key_eqpmt_init assign="eqpmt_init"}}
-{{else}}
-	{{:error message="Aucun matériel avec la clé « %s »"|args:$key_eqpmt_init}}
-{{/load}}
-
 {{*
 	-------------------- Traiter la saisie --------------------
 *}}
 {{#form on="change"}}
 	{{* préparer le mouvement modifié *}}
-	{{if $direction == "input"}}
+	{{if $direction_mvt == "input"}}
 		{{:assign var="mvt_modif.input_nature" from="input_labels.%s"|args:$_POST.type_operation}}
 	{{else}}
 		{{:assign var="mvt_modif.output_nature" from="output_labels.%s"|args:$_POST.type_operation}}
@@ -66,7 +75,6 @@
 		en l'insérant à sa place par date croissante
 	*}}
 	{{:assign insere=false}}
-
 	{{#load
 		where="
 			$$.type = 'movement'
@@ -78,15 +86,18 @@
 		assign="movement"
 	}}
 		{{if $key != $key_mvt}}
-			{{if $date > $mvt_modif.date && ! $insere}}
+			{{if $direction_mvt == "input" && ! $insere && $date >= $mvt_modif.date}}
-				{{:assign var=movements_modif." from=mvt_modif}}
+				{{:assign var="movements_modif." from=mvt_modif}}
+				{{:assign insere=true}}
+			{{elseif $direction_mvt == "output" && ! $insere && $date > $mvt_modif.date}}
+				{{:assign var="movements_modif." from=mvt_modif}}
 				{{:assign insere=true}}
 			{{/if}}
-			{{:assign var=movements_modif." from=movement}}
+			{{:assign var="movements_modif." from=movement}}
 		{{/if}}
 	{{/load}}
 	{{if ! $insere}}
-		{{:assign var=movements_modif." from=mvt_modif}}
+		{{:assign var="movements_modif." from=mvt_modif}}
 	{{/if}}
 
 	{{* Vérifier la cohérence des opérations de l'ancien matériel *}}
@@ -111,7 +122,7 @@
 	{{if $erreur}}
 		<p class="block error">
 			Modification demandée impossible :
-		{{if $direction == "input"}}
+		{{if $direction_mvt == "input"}}
 			« {{$input_init}} de {{$eqpmt_init.designation}} (qté : {{$amount_init}}) en date du {{$date_init}} » vers
 			« {{$mvt_modif.input_nature}} de {{$eqpmt.designation}} (qté : {{$mvt_modif.amount}}) à la date du {{$mvt_modif.date|date_short}} »
 		{{else}}
@@ -125,7 +136,7 @@
 			key=$key_mvt
 			validate_schema="./movement.schema.json"
 			type="movement"
-			direction=$direction
+			direction=$direction_mvt
 			input_nature=$mvt_modif.input_nature
 			output_nature=$mvt_modif.output_nature
 			amount=$mvt_modif.amount|intval
@@ -154,7 +165,7 @@
 <form method="post" action="">
 
 	<fieldset class="entree">
-		{{if $direction == "input"}}
+		{{if $direction_mvt == "input"}}
 			<legend>Modifier l'entrée « {{$input_init}} {{$eqpmt_init.designation}} ({{$amount_init}}) en date du {{$date_init}} »</legend>
 			<dl>
 			{{:input type="select" name="type_operation" label="Type" required=true options=$input_labels default=$type_defaut}}

mouvements/sorties/emprunte.html

@@ -1,3 +1,5 @@
+{{* -*- brindille -*- *}}
+
 {{:admin_header title="Sortie de matériel" current="module_equipment"}}
 
 {{if ! $dialog}}
@@ -23,20 +25,66 @@
 		{{:assign var="category_keys." value=$equipment.category}}
 	{{/load}}
 
-	{{* Extraire et compiler les infos de la base *}}
+	{{:assign stock=0}}
-	{{:include
+	{{:assign exterieur=0}}
-		file="../../_calcul_dispo.html"
+	{{:assign nonproprio=0}}
-		keep="cumul_mvt"
+	{{:assign insere=false}}
-		date=$_POST.date|parse_date
-	}}
 
-	{{* déterminer la quantité présente de ce matériel à la date donnée *}}
+	{{* lister tous les mouvements du matériel *}}
-	{{:assign var=present from="cumul_mvt.%s.eqpmt.%s.nonproprio"|args:$equipment.category:$_POST.equipment}}
+	{{#load type="movement" where="$$.equipment = '%s'"|args:$_POST.equipment assign="mvt" order="$$.date ASC"}}
 
-	{{if $_POST.quantite|intval > $present}}
+		{{* traiter le nouveau mouvement *}}
-			{{:error message="Erreur : la quantité indiquée (%s) est supérieure à celle présente (%d) à la date du %s"|args:$_POST.quantite:$present:$_POST.date}}
+		{{if ! $insere && $mvt.date > $_POST.date|parse_date}}
+			{{:assign var="operation" from="output_labels.%s|args:$_POST.type_operation}}
+			{{:assign var="type_mvt" from="config.output_nature.%s"|args:$operation}}
+			{{:assign dispo_old=$nonproprio}}
+
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d-%d"|math:$stock:$_POST.quantite}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign exterieur="%d+%d"|math:$exterieur:$_POST.quantite}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign nonproprio="%d-%d"|math:$nonproprio:$_POST.quantite}}
 			{{/if}}
 
+			{{:assign insere=true}}
+			{{:assign dispo="%d-%d"|math:$stock:$exterieur}}
+
+			{{if $dispo < 0 || $stock < 0 || $exterieur < 0 || $nonproprio < 0}}
+				{{:assign date_err=$_POST.date|date:"d/m/Y"}}
+				{{:error message="Erreur : la quantité indiquée (%s) est supérieure à celle disponible (%d) à la date du %s"|args:$_POST.quantite:$dispo_old:$date_err}}
+			{{/if}}
+		{{/if}}
+
+		{{* traiter le mouvement courant *}}
+		{{:assign var="mvt_nature" from="mvt.%s_nature"|args:$mvt.direction}}
+		{{:assign var="type_mvt" from="config.%s_nature.%s"|args:$mvt.direction:$mvt_nature}}
+
+		{{if $mvt.direction === 'input'}}
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d+%d"|math:$stock:$mvt.amount}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign exterieur="%d-%d"|math:$exterieur:$mvt.amount}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign nonproprio="%d+%d"|math:$nonproprio:$mvt.amount}}
+			{{/if}}
+		{{elseif $mvt.direction === 'output'}}
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d-%d"|math:$stock:$mvt.amount}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign exterieur="%d+%d"|math:$exterieur:$mvt.amount}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign nonproprio="%d-%d"|math:$nonproprio:$mvt.amount}}
+			{{/if}}
+		{{/if}}
+
+		{{:assign dispo="%d-%d"|math:$stock:$exterieur}}
+		{{if $dispo < 0 || $stock < 0 || $exterieur < 0 || $nonproprio < 0}}
+			{{:assign date_err=$mvt.date|date:"d/m/Y"}}
+			{{:error message="Erreur : la quantité indiquée (%s) est incompatible avec la sortie « %s : %s » à la date du %s"|args:$_POST.quantite:$designation:$mvt.amount:$date_err}}
+		{{/if}}
+	{{/load}}
+
 	{{* Enregistrer le mouvement *}}
 
 	{{:assign mvt_key=""|uuid}}
@@ -68,7 +116,7 @@
 		{{* formulaire de sortie de matériel *}}
 		<form method="post" action="">
 			<fieldset class="sortie">
-				<legend>Ajouter une sortie d'un matériel disponible en stock</legend>
+				<legend>Ajouter une sortie d'un matériel présent temporairement</legend>
 				<dl>
 				{{:input type="select" name="type_operation" label="Type" required=true options=$output_labels}}
 				{{:input type="date" name="date" label="Date de sortie" required=true default=$now|date_short}}
@@ -77,7 +125,7 @@
 			</fieldset>
 			<fieldset>
 				<legend>Matériel</legend>
-				<p><span class="alert">La quantité disponible est celle à la date du jour</span></p>
+				<p><span class="alert">La quantité indiquée est celle à la date du jour</span></p>
 				<dl>
 				{{:input type="select_groups" name="equipment" label="Matériel" required=true options=$temporaire onchange="fixerValeurMax('f_equipment', 'f_quantite')"}}
 				{{:input type="textarea" name="remarques" label="Remarques" cols="40" rows="3" required=false}}
@@ -88,7 +136,8 @@
 			</p>
 		</form>
 	{{else}}
-		<p class="block error">Il n'y a aucun matériel présent temporairement</p>
+		{{:assign date_err=$now|date:'d/m/Y'}}
+		<p class="block error">Il n'y a aucun matériel présent temporairement à la date du {{$date_err}}</p>
 	{{/if}}
 {{else}}
 	<p class="block error">Il n'y a aucun matériel en stock</p>

mouvements/sorties/stock_disponible.html

@@ -1,3 +1,5 @@
+{{* -*- brindille -*- *}}
+
 {{:admin_header title="Sortie de matériel" current="module_equipment"}}
 
 {{if ! $dialog}}
@@ -23,21 +25,65 @@
 		{{:assign var="category_keys." value=$equipment.category}}
 	{{/load}}
 
-	{{* Extraire et compiler les infos de la base *}}
+	{{:assign stock=0}}
-	{{:include
+	{{:assign exterieur=0}}
-		file="../../_calcul_dispo.html"
+	{{:assign nonproprio=0}}
-		keep="cumul_mvt"
+	{{:assign insere=false}}
-		date=$_POST.date|parse_date
-	}}
 
-	{{* déterminer la quantité disponible de ce matériel à la date donnée *}}
+	{{* lister tous les mouvements du matériel *}}
-	{{:assign var=stock from="cumul_mvt.%s.eqpmt.%s.stock"|args:$equipment.category:$_POST.equipment}}
+	{{#load type="movement" where="$$.equipment = '%s'"|args:$_POST.equipment assign="mvt" order="$$.date ASC"}}
-	{{:assign var=exterieur from="cumul_mvt.%s.eqpmt.%s.exterieur"|args:$equipment.category:$_POST.equipment}}
+
+		{{* traiter le nouveau mouvement *}}
+		{{if ! $insere && $mvt.date > $_POST.date|parse_date}}
+			{{:assign var="operation" from="output_labels.%s|args:$_POST.type_operation}}
+			{{:assign var="type_mvt" from="config.output_nature.%s"|args:$operation}}
+			{{:assign dispo_old="%d-%d"|math:$stock:$exterieur}}
+
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d-%d"|math:$stock:$_POST.quantite}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign exterieur="%d+%d"|math:$exterieur:$_POST.quantite}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign nonproprio="%d-%d"|math:$nonproprio:$_POST.quantite}}
+			{{/if}}
+
+			{{:assign insere=true}}
 			{{:assign dispo="%d-%d"|math:$stock:$exterieur}}
 
-	{{if $_POST.quantite|intval > $dispo}}
+			{{if $dispo < 0 || $stock < 0 || $exterieur < 0 || $nonproprio < 0}}
-			{{:error message="Erreur : la quantité indiquée (%s) est supérieure à celle disponible (%d) à la date du %s"|args:$_POST.quantite:$dispo:$_POST.date}}
+				{{:assign date_err=$_POST.date|date:"d/m/Y"}}
+				{{:error message="Erreur : la quantité indiquée (%s) est supérieure à celle disponible (%d) à la date du %s"|args:$_POST.quantite:$dispo_old:$date_err}}
 			{{/if}}
+		{{/if}}
+
+		{{* traiter le mouvement courant *}}
+		{{:assign var="mvt_nature" from="mvt.%s_nature"|args:$mvt.direction}}
+		{{:assign var="type_mvt" from="config.%s_nature.%s"|args:$mvt.direction:$mvt_nature}}
+
+		{{if $mvt.direction === 'input'}}
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d+%d"|math:$stock:$mvt.amount}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign exterieur="%d-%d"|math:$exterieur:$mvt.amount}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign nonproprio="%d+%d"|math:$nonproprio:$mvt.amount}}
+			{{/if}}
+		{{elseif $mvt.direction === 'output'}}
+			{{if $type_mvt == 'définitif'}}
+				{{:assign stock="%d-%d"|math:$stock:$mvt.amount}}
+			{{elseif $type_mvt == 'temporaire'}}
+				{{:assign exterieur="%d+%d"|math:$exterieur:$mvt.amount}}
+			{{elseif $type_mvt == 'retour'}}
+				{{:assign nonproprio="%d-%d"|math:$nonproprio:$mvt.amount}}
+			{{/if}}
+		{{/if}}
+
+		{{:assign dispo="%d-%d"|math:$stock:$exterieur}}
+		{{if $dispo < 0 || $stock < 0 || $exterieur < 0 || $nonproprio < 0}}
+			{{:assign date_err=$mvt.date|date:"d/m/Y"}}
+			{{:error message="Erreur : la quantité indiquée (%s) est incompatible avec la sortie « %s : %s » à la date du %s"|args:$_POST.quantite:$designation:$mvt.amount:$date_err}}
+		{{/if}}
+	{{/load}}
 
 	{{* Enregistrer le mouvement *}}
 
@@ -75,10 +121,8 @@
 			{{/if}}
 		{{/foreach}}
 	{{/foreach}}
-	{{if $disponibilites === null}}
-		{{:error message="Le matériel choisi (%s) n'est pas présent à la date du %s"|args:$designation:$_POST.date}}
-	{{/if}}
 
+	{{if $disponibilites != null}}
 		{{* formulaire de sortie de matériel *}}
 		<form method="post" action="">
 			<fieldset class="sortie">
@@ -91,7 +135,7 @@
 			</fieldset>
 			<fieldset>
 				<legend>Matériel</legend>
-			<p><span class="alert">La quantité disponible est celle à la date du jour</span></p>
+				<p><span class="alert">La quantité indiquée est celle à la date du jour</span></p>
 				<dl>
 					{{:input type="select_groups" name="equipment" label="Matériel" required=true options=$disponibilites onchange="fixerValeurMax('f_equipment', 'f_quantite')"}}
 					{{:input type="textarea" name="remarques" label="Remarques" cols="40" rows="3" required=false}}
@@ -101,6 +145,10 @@
 				{{:button type="submit" name="save" label="Enregistrer" shape="right" class="main"}}
 			</p>
 		</form>
+	{{else}}
+		{{:assign date_err=$now|date:'d/m/Y'}}
+		<p class="block error">Il n'y a aucun matériel disponible à la date du {{$date_err}}</p>
+	{{/if}}
 {{else}}
 	<p class="block error">Il n'y a aucun matériel en stock</p>
 {{/if}}

mouvements/valider_modification.html

@@ -1,4 +1,5 @@
-{{*
+{{* -*- brindille -*-
+
 	Vérifier la cohérence des E/S d'un matériel
 	- paramètres :
 		- movements : liste de mouvements
@@ -8,7 +9,7 @@
 {{* récupérer la config des entrées/sorties *}}
 {{:include file="../_get_config.html" keep="config"}}
 
-{{:assign erreur = false}}
+{{:assign erreur=false}}
 {{:assign stock=0}}
 {{:assign exterieur=0}}
 {{:assign nonprop=0}}