Correction erreur validation retour

_nav.html

@@ -15,7 +15,6 @@
 		<li {{if $current == 'inventaire'}} class="current"{{/if}}><a href="{{$module.url}}index.html?prop=1">Inventaire</a></li>
 		<li {{if $current == 'historique'}} class="current"{{/if}}><a href="{{$module.url}}global_history.html">Historique</a></li>
 		<li {{if $current == 'archives'}} class="current"{{/if}}><a href="{{$module.url}}archives.html">Archives</a></li>
-		<li {{if $current == 'storage'}} class="current"{{/if}}><a href="{{$module.url}}storage/index.html">Stockage</a></li>
 		<li {{if $current == 'config'}} class="current"{{/if}}><a href="{{$module.url}}categories/index.html">Configuration</a></li>
 	</ul>
 
@@ -36,10 +35,6 @@
 				<li class="title"><strong>Historique — {{$eqpmt}} ({{$category}})</strong></li>
 			</ul>
 		{{/if}}
-	{{elseif $current == 'storage'}}
-		<aside>
-			{{:linkbutton label="Ajouter un lieu de stockage" shape="plus" href="add_storage.html" target="_dialog"}}
-		</aside>
 	{{elseif $current == 'config'}}
 		{{if $subcurrent == 'categories'}}
 			<aside>

global_history.html

@@ -46,19 +46,11 @@
 			FROM @TABLE AS mat
 			WHERE mat.key = @TABLE.$$.equipment) AS 'Matériel' ;
 		$$.amount AS 'Quantité';
-		CASE WHEN $$.user NOT NULL
-			THEN (SELECT %s AS nom FROM users WHERE id = $$.user)
-			ELSE ''
-		END as 'Dépositaire';
-		CASE WHEN $$.storage NOT NULL
-				THEN (SELECT $$.name FROM @TABLE as storage WHERE storage.key = @TABLE.$$.storage)
-				ELSE ''
-			END as 'Stockage';
-		$$.comment AS 'Commentaire'
-	"|args:$config.user_fields.name_sql
+		$$.comment AS 'Commentaire'"
 	order=1
 	desc=true
-}}
+	}}
+
 	{{* filtrer selon la catégorie *}}
 	{{#load key=$equipment}}
 		{{:assign cat_key=$category}}
@@ -74,8 +66,6 @@
 		<td class="nosort">{{$op_label}}</td>
 		<td>{{$col4}}</td>
 		<td class="num">{{$amount}}</td>
-		<td>{{:link href="/admin/users/details.php?id=%s"|args:$user label="%s"|args:$col6}}</td>
-		<td>{{$col7}}</td>
 		<td>{{$comment}}</td>
 		<td></td>
 	</tr>

historique.html

@@ -126,16 +126,7 @@
 			"" as 'Stock';
 			"" as 'Sorti';
 			"" as 'Disponible';
-			CASE WHEN $$.user NOT NULL
-				THEN (SELECT %s AS nom FROM users WHERE id = $$.user)
-				ELSE ''
-			END as 'Dépositaire';
-			CASE WHEN $$.storage NOT NULL
-				THEN (SELECT $$.name FROM @TABLE as storage WHERE storage.key = @TABLE.$$.storage)
-				ELSE ''
-			END as 'Stockage';
-			$$.comment AS 'Commentaire'
-		"|args:$config.user_fields.name_sql
+			$$.comment AS 'Commentaire'"
 		equipment=$equipment_key
 		order=1
 	}}
@@ -156,8 +147,6 @@
 				<td class="num nosort">{{$stock}}</td>
 				<td class="num nosort">{{$exterieur}}</td>
 				<td class="num nosort">{{$dispo}}</td>
-				<td>{{:link href="/admin/users/details.php?id=%s"|args:$user label="%s"|args:$col8}}</td>
-				<td>{{$col9}}</td>
 				<td>{{$comment}}</td>
 				<td class="actions">
 					{{if $current != "archives"}}
@@ -190,10 +179,6 @@
 			$$.operation AS 'Opération';
 			$$.amount AS 'Quantité';
 			"" as 'Présent';
-			CASE WHEN $$.storage NOT NULL
-				THEN (SELECT $$.name FROM @TABLE as storage WHERE storage.key = @TABLE.$$.storage)
-				ELSE ''
-			END as 'Stockage';
 			$$.comment AS 'Commentaire'"
 		equipment=$equipment_key
 		order=1
@@ -211,7 +196,6 @@
 				<td>{{$op_label}}</td>
 				<td class="num">{{$amount}}</td>
 				<td class="num nosort">{{$stock}}</td>
-				<td>{{$col6}}</td>
 				<td>{{$comment}}</td>
 				<td class="actions">
 					{{:linkbutton

mouvements/ajouter_entree.html

@@ -24,7 +24,7 @@
 {{* Traiter l'envoi du formulaire *}}
 {{#form on="save"}}
 
-	{{* vérifier la quantité saisie *}}
+	{{* vérifier la quantité saisie (UTILE ? : fait par la navigateur) *}}
 	{{if $_POST.quantite <= 0}}
 		{{:error message="La quantité (%s) doit être strictement positive !!"|args:$_POST.quantite}}
 	{{/if}}
@@ -35,7 +35,7 @@
 	{{/if}}
 
 	{{* vérifier l'existence d'un matériel de même nom *}}
-	{{#load type="equipment" where="$$.name = :name COLLATE U_NOCASE" :name=$_POST.name|trim limit=1}}
+	{{#load type="equipment" where="$$.name = :name COLLATE U_NOCASE" :name=$_POST.name|trim limit=1}}}}
 		{{:assign equipment_key=$key}}
 		{{:assign var=nom_cat from="categories.%s"|args:$category}}
 		{{:assign present=true}}
@@ -77,7 +77,6 @@
 		equipment=$equipment_key
 		date=$_POST.date|parse_date
 		comment=$_POST.remarques|trim
-		storage=$_POST.storage
 	}}
 
 	{{:assign var=entree from="config.input_nature.%s"|args:$_POST.operation}}
@@ -86,29 +85,30 @@
 	{{else}}
 		{{:assign prop=1}}
 	{{/if}}
-	{{:redirect force="../index.html?prop=%s&ok=1&msg=ajout"|args:$prop}}
+	{{:redirect force="../index.html?prop=%s&ok=1&msg=ajout"|args:$prop:$msg}}
 {{/form}}
 
 {{:admin_header title="Entrée de matériel" current="module_equipment"}}
 {{:form_errors}}
 
 {{if $categories != null}}
-	{{#load type="storage" order="$$.name"}}
-		{{:assign var="storage.%s"|args:$key value=$name}}
-	{{/load}}
-
 	{{* formulaire ajout matériel *}}
 	<form method="post" action="">
 
 		<fieldset class="entree">
-			<legend>Enregistrer une entrée de matériel</legend>
+			<legend>Ajouter une entrée de matériel</legend>
+			<dl>
+			{{:input type="select" name="operation" label="Type" required=true default_empty="— Aucun —" options=$input_labels|sort}}
+			{{:input type="date" name="date" label="Date" required=true default=$now|date_short}}
+			{{:input type="number" name="quantite" label="Quantité" min=1 required=true default=1}}
+			</dl>
+		</fieldset>
+
+		<fieldset>
+			<legend>Sélectionner une catégorie et indiquer le nom du nouveau matériel</legend>
 			<dl>
-				{{:input type="select" name="operation" label="Type" required=true default_empty="— Aucun —" options=$input_labels|sort}}
-				{{:input type="date" name="date" label="Date" required=true default=$now|date_short}}
-				{{:input type="number" name="quantite" label="Quantité" min=1 required=true default=1}}
-				{{:input type="text" name="name" label="Désignation" required=true}}
 				{{:input type="select" name="category" label="Catégorie" default_empty="— Aucune —" options=$categories required=true}}
-				{{:input type="select" name="storage" label="Lieu de stockage" default_empty="— Aucun —" options=$storage required=false}}
+				{{:input type="text" name="name" label="Désignation" required=true}}
 				{{:input type="textarea" name="remarques" label="Remarques" cols="40", rows="3" required=false}}
 			</dl>
 		</fieldset>

mouvements/dupliquer_mouvement.html

@@ -13,23 +13,6 @@
 	{{:error message="Aucun mouvement avec la clé %s"|args:$_GET.key}}
 {{/load}}
 
-{{* récupérer la config des entrées/sorties *}}
-{{:include file="../_get_config.html" keep="config, directions"}}
-{{:assign var="type_mvt" from="config.%s_nature.%s.type"|args:$mvt_new.direction:$mvt_new.operation}}
-{{if $mvt_new.direction == "input"}}
-	{{if $type_mvt == "temporaire"}}
-		{{:assign prop=0}}
-	{{else}}
-		{{:assign prop=1}}
-	{{/if}}
-{{else}}
-	{{if $type_mvt == "retour"}}
-		{{:assign prop=0}}
-	{{else}}
-		{{:assign prop=1}}
-	{{/if}}
-{{/if}}
-
 {{* infos pour affichage *}}
 {{:assign amount_init=$mvt_new.amount}}
 {{:assign date_init=$mvt_new.date|date_short}}
@@ -77,19 +60,24 @@
 		{{:error message="Impossible de saisir une date dans le futur (%s)"|args:$_POST.date}}
 	{{/if}}
 
-	{{if $mvt_new.direction != "output" || $type_mvt != "temporaire" && $_POST.user != null}}
-		{{:error message="Un membre ne peut être associé qu'à une sortie temporaire"}}
-	{{/if}}
-	{{if $_POST.user|count > 1}}
-		{{:error message="Un membre au plus peut être associé à une sortie"}}
-	{{/if}}
-	{{#foreach from=$_POST.user key="id" item="name"}}
-		{{:assign var="user.id" value=$id}}
-		{{:assign var="user.name" value=$name}}
-	{{/foreach}}
-
 	{{* préparer le mouvement copié *}}
 	{{:assign var="mvt_new.operation" value=$_POST.operation}}
+	{{:assign var="type_mvt" from="config.%s_nature.%s.type"|args:$mvt_new.direction:$mvt_new.operation}}
+
+	{{if $mvt_new.direction == "input"}}
+		{{if $type_mvt == "temporaire"}}
+			{{:assign prop=0}}
+		{{else}}
+			{{:assign prop=1}}
+		{{/if}}
+	{{else}}
+		{{if $type_mvt == "retour"}}
+			{{:assign prop=0}}
+		{{else}}
+			{{:assign prop=1}}
+		{{/if}}
+	{{/if}}
+
 	{{:assign var="mvt_new.amount" value=$_POST.amount}}
 	{{:assign var="mvt_new.date" value=$_POST.date|parse_date}}
 	{{:assign var="mvt_new.comment" value=$_POST.comment}}
@@ -148,7 +136,6 @@
 			equipment=$eqpmt_key
 			date=$mvt_new.date
 			comment=$mvt_new.comment
-			user=$user.id
 		}}
 		{{:redirect force="../historique.html?ok=1&key=%s&prop=%s&msg=copie"|args:$eqpmt_key:$prop}}
 	{{/if}}
@@ -165,45 +152,21 @@
 {{#load key=$eqpmt.category assign="category"}}{{/load}}
 {{:assign var="op_label" from="config.%s_nature.%s.label"|args:$mvt_new.direction:$mvt_new.operation}}
 
-{{if $mvt_new.user != null}}
-	{{#select id, !name as nom FROM users WHERE id=:id; !name=$config.user_fields.name_sql :id=$mvt_new.user}}
-		{{:assign var="user.%s"|args:$id value=$nom}}
-	{{/select}}
-{{/if}}
-
-{{#load type="storage" order="$$.name"}}
-	{{:assign var="storage.%s"|args:$key value=$name}}
-{{/load}}
-
 {{* formulaire de copie du mouvement *}}
 <form method="post" action="">
 	<fieldset>
 		<legend>Créer une copie de « {{$op_label}} {{$eqpmt.name}} (Catégorie : {{$category.name}}) »</legend>
-		<dl>
-			{{if $mvt_new.direction == "input"}}
+		{{if $mvt_new.direction == "input"}}
+			<dl>
 				{{:input type="select" name="operation" label="Type" required=true options=$input_labels|sort default=$mvt_new.operation}}
-			{{else}}
+		{{else}}
+			<dl>
 				{{:input type="select" name="operation" label="Type" required=true options=$output_labels|sort default=$mvt_new.operation}}
-			{{/if}}
-			{{:input type="date" name="date" label="Date" required=true default=$mvt_new.date}}
-			{{:input type="number" name="amount" label="Quantité" min=1 required=true default=$mvt_new.amount}}
-			{{if $prop == 1 && $mvt_new.direction == "output"}}
-				{{:input
-					type="list"
-					name="user"
-					label="Membre"
-					default=$user
-					target="!users/selector.php"
-					multiple=true
-					max=1
-					help="dans le cas d'une sortie temporaire, il est possible d'associer un membre bénéficiaire"
-				}}
-			{{/if}}
-			{{if $mvt_new.direction == "input"}}
-				{{:input type="select" name="storage" label="Lieu de stockage" default_empty="— Aucun —" options=$storage required=false}}
-			{{/if}}
-			{{:input type="textarea" name="comment" label="Remarques" cols="40", rows="3" required=false default=$mvt_new.comment}}
-		</dl>
+		{{/if}}
+		{{:input type="date" name="date" label="Date" required=true default=$mvt_new.date}}
+		{{:input type="number" name="amount" label="Quantité" min=1 required=true default=$mvt_new.amount}}
+		{{:input type="textarea" name="comment" label="Remarques" cols="40", rows="3" required=false default=$mvt_new.comment}}
+			</dl>
 	</fieldset>
 
 	<p class="submit">

mouvements/entrees/repertorie.html

@@ -1,7 +1,7 @@
 {{* -*- brindille -*- *}}
 
 {{*
-	Enregistrer une entrée de matériel
+	Ajouter une entrée de matériel
 	paramètres :
 	- key : clé du matériel à ajouter
 *}}
@@ -52,7 +52,6 @@
 		equipment=$equipment.key
 		date=$_POST.date|parse_date
 		comment=$_POST.remarques|trim
-		storage=$_POST.storage
 	}}
 	{{:assign var=type_entree from="config.input_nature.%s.type"|args:$_POST.operation}}
 	{{if $type_entree == "temporaire"}}
@@ -67,9 +66,6 @@
 {{:form_errors}}
 
 {{* formulaire de saisie d'une entrée de matériel *}}
-{{#load type="storage" order="$$.name"}}
-	{{:assign var="storage.%s"|args:$key value=$name}}
-{{/load}}
 <form method="post" action="">
 	<fieldset class="informations">
 		<legend>Informations matériel</legend>
@@ -81,12 +77,11 @@
 		</dl>
 	</fieldset>
 	<fieldset class="entree">
-		<legend>Enregistrer une entrée de matériel</legend>
+		<legend>Ajouter une entrée de matériel</legend>
 		<dl>
 			{{:input type="select" name="operation" label="Type" required=true default_empty="— Aucun —" options=$input_labels|sort}}
 			{{:input type="date" name="date" label="Date" required=true default=$now|date_short}}
 			{{:input type="number" name="quantite" label="Quantité" required=true min=1 default=1}}
-			{{:input type="select" name="storage" label="Lieu de stockage" default_empty="— Aucun —" options=$storage required=false}}
 			{{:input type="textarea" name="remarques" label="Remarques" cols="40" rows="3" required=false}}
 		</dl>
 	</fieldset>

mouvements/entrees/retour.html

@@ -13,55 +13,89 @@
 {{* récupérer la config des entrées/sorties *}}
 {{:include file="../../_get_config.html" keep="config"}}
 
-{{* types d'entrées *}}
-{{#foreach from=$config.input_nature key=key}}
-	{{if $type == 'retour'}}
-		{{:assign var="input_labels.%s"|args:$key value=$label}}
-	{{/if}}
-{{/foreach}}
+{{* déterminer le matériel concerné par le mouvement *}}
+{{#load key=$_GET.key assign="equipment"}}{{/load}}
 
-{{* Traiter l'envoi du formulaire *}}
+{{*
+	-------------------- Traiter l'envoi du formulaire --------------------
+*}}
 {{#form on="save"}}
-	{{* interdire date dans le futur *}}
+	{{* vérifier les données saisies *}}
+	{{if $_POST.amount <= 0}}
+		{{:error message="La quantité (%s) doit être strictement positive !!"|args:$_POST.amount}}
+	{{/if}}
+
 	{{if $_POST.date|parse_date|strtotime > $now}}
 		{{:error message="Impossible de saisir une date dans le futur (%s)"|args:$_POST.date}}
 	{{/if}}
 
-	{{* récupérer les infos du matériel *}}
-	{{#load type="equipment" key=$_GET.key assign="equipment"}}
-		{{:assign name=$equipment.name}}
-	{{/load}}
+	{{* préparer le nouveau mouvement *}}
+	{{:assign var="mvt_new.key" value=""|uuid}}
+	{{:assign var="mvt_new.direction" value="input"}}
+	{{:assign var="mvt_new.date" value=$_POST.date|parse_date}}
+	{{:assign var="mvt_new.operation" value=$_POST.operation}}
+	{{:assign var="mvt_new.amount" value=$_POST.amount}}
 
-	{{* Extraire et compiler les infos de la base *}}
-	{{:include
-		file="../../_calcul_dispo.html"
-		keep="equipments"
-		eqpmt_key=$_GET.key
-		date=$_POST.date|parse_date
+	{{*
+		lister les mouvements
+		- insérer le nouveau mvt à sa place par date croissante
+	*}}
+	{{:assign insere=false}}
+	{{#load
+		where="
+			$$.type = 'movement'
+		AND
+			$$.equipment = :eqpmt_key"
+		:eqpmt_key=$equipment.key
+		order="$$.date"
+		assign="movement"
 	}}
-
-	{{* déterminer la quantité sortie temporairement de ce matériel à la date donnée *}}
-	{{:assign var=sorti from="equipments.%s.exterieur"|args:$_GET.key}}
-
-	{{if $_POST.quantite|intval > $sorti}}
-		{{:error message="Erreur : la quantité indiquée (%s) est supérieure à celle sortie (%d) à la date du %s"|args:$_POST.quantite:$sorti:$_POST.date}}
+		{{if! $insere}}
+			{{if
+				$mvt_new.direction == "input" && $date >= $mvt_new.date ||
+				$mvt_new.direction == "output" && $date > $mvt_new.date
+			}}
+				{{:assign var="movements_new." from=mvt_new}}
+				{{:assign insere=true}}
+			{{/if}}
+		{{/if}}
+		{{:assign var="movements_new." from=movement}}
+	{{/load}}
+	{{if ! $insere}}
+		{{:assign var="movements_new." from=mvt_new}}
 	{{/if}}
 
-	{{* Enregistrer le mouvement *}}
-	{{:assign mvt_key=""|uuid}}
-	{{:save
-		key=$mvt_key
-		validate_schema="../movement.schema.json"
-		type="movement"
-		direction="input"
-		operation=$_POST.operation
-		amount=$_POST.quantite|intval
-		equipment=$equipment.key
-		date=$_POST.date|parse_date
-		comment=$_POST.remarques|trim
+	{{* Vérifier la cohérence des mouvements du matériel *}}
+	{{:include
+		file="../valider_modification.html"
+		keep="erreur, pb"
+		movements=$movements_new
 	}}
 
-	{{:redirect force="../../historique.html?ok=1&key=%s&prop=1&msg=retour"|args:$_GET.key}}
+	{{if $erreur}}
+		<p class="block error">Impossible d'enregistrer ce mouvement.
+			{{if $pb.mvt.key != $mvt_new.key}}
+				{{:assign var="err_mvt_label" from="config.input_nature.%s.label"|args:$pb.mvt.operation}}
+				<br />Mouvement incompatible avec
+				« {{$err_mvt_label}} de {{$equipment.name}} (qté : {{$pb.mvt.amount}})
+					en date du {{$pb.mvt.date|date_short}} ».
+				{{/if}}
+		</p>
+	{{else}}
+		{{* vérification réussie : enregistrer le mouvement modifié *}}
+		{{:save
+			key=""|uuid
+			validate_schema="../movement.schema.json"
+			type="movement"
+			direction="input"
+			operation=$_POST.operation
+			amount=$_POST.amount|intval
+			equipment=$equipment.key
+			date=$_POST.date|parse_date
+			comment=$_POST.comment|trim
+		}}
+		{{:redirect force="../../historique.html?ok=1&key=%s&prop=1&msg=retour"|args:$_GET.key}}
+{{/if}}
 {{/form}}
 
 {{:admin_header title="Retour de matériel" custom_css="./../../style.css" current="module_equipment"}}
@@ -71,13 +105,15 @@
 {{:include file="../../_calcul_dispo.html" eqpmt_key=$_GET.key keep="categories, equipments"}}
 {{:assign var=sorti from="equipments.%s.exterieur"|args:$_GET.key}}
 
+{{* types d'entrées *}}
+{{#foreach from=$config.input_nature key=key}}
+	{{if $type == 'retour'}}
+		{{:assign var="input_labels.%s"|args:$key value=$label}}
+	{{/if}}
+{{/foreach}}
+
 {{if $sorti > 0}}
-
-	{{* déterminer le matériel concerné par le mouvement *}}
-	{{#load key=$_GET.key assign="equipment"}}
-	{{/load}}
 	{{:assign var=cat_name from=categories.%s|args:$equipment.category}}
-
 	{{* formulaire d'entrée de matériel *}}
 	<form method="post" action="">
 		<fieldset class="informations">
@@ -92,13 +128,13 @@
 			</dl>
 		</fieldset>
 		<fieldset class="entree">
-			<legend>Enregistrer un retour de matériel</legend>
+			<legend>Ajouter un retour de matériel</legend>
 			<dl>
 				{{:input type="select" name="operation" label="Type" required=true options=$input_labels|sort}}
 				{{:input type="date" name="date" label="Date" required=true default=$now|date_short}}
-				{{:input type="number" name="quantite" label="Quantité" min=1 required=true default=$sorti max=$sorti}}
-				{{:input type="textarea" name="remarques" label="Remarques" cols="40", rows="3" required=false}}
-			</dl>
+				{{:input type="number" name="amount" label="Quantité" min=1 required=true default=$sorti max=$sorti}}
+				{{:input type="textarea" name="comment" label="Remarques" cols="40", rows="3" required=false}}
+				</dl>
 		</fieldset>
 		<p class="submit">
 			{{:button type="submit" name="save" label="Enregistrer" shape="right" class="main"}}

mouvements/modifier_mouvement.html

@@ -14,20 +14,6 @@
 
 {{* récupérer la config des entrées/sorties *}}
 {{:include file="../_get_config.html" keep="config, directions"}}
-{{:assign var="type_mvt" from="config.%s_nature.%s.type"|args:$mvt_new.direction:$mvt_new.operation}}
-{{if $mvt_new.direction == "input"}}
-	{{if $type_mvt == "temporaire"}}
-		{{:assign prop=0}}
-	{{else}}
-		{{:assign prop=1}}
-	{{/if}}
-{{else}}
-	{{if $type_mvt == "retour"}}
-		{{:assign prop=0}}
-	{{else}}
-		{{:assign prop=1}}
-	{{/if}}
-{{/if}}
 
 {{* infos pour affichage *}}
 {{:assign var="op_label" from="config.%s_nature.%s.label"|args:$mvt_new.direction:$mvt_new.operation}}
@@ -56,30 +42,29 @@
 		{{:error message="La quantité (%s) doit être strictement positive !!"|args:$_POST.amount}}
 	{{/if}}
 
-	{{* vérifier validité des données *}}
+	{{* interdire date dans le futur *}}
 	{{if $_POST.date|parse_date|strtotime > $now}}
 		{{:error message="Impossible de saisir une date dans le futur (%s)"|args:$_POST.date}}
 	{{/if}}
 
-	{{if $_POST.user|count > 1}}
-		{{:error message="Un membre au plus peut être associé à une sortie"}}
-	{{/if}}
+	{{* préparer le mouvement modifié *}}
+	{{:assign var="mvt_new.operation" value=$_POST.operation}}
+	{{:assign var="type_mvt" from="config.%s_nature.%s.type"|args:$mvt_new.direction:$mvt_new.operation}}
 
-	{{if $_POST.user != null}}
-		{{if $mvt_new.direction == "input"}}
-			{{:error message="Un membre ne peut être associé qu'à une sortie temporaire"}}
-		{{elseif $type_mvt != "temporaire"}}
-			{{:error message="Un membre ne peut être associé qu'à une sortie temporaire"}}
+	{{if $mvt_new.direction == "input"}}
+		{{if $type_mvt == "temporaire"}}
+			{{:assign prop=0}}
+		{{else}}
+			{{:assign prop=1}}
+		{{/if}}
+	{{else}}
+		{{if $type_mvt == "retour"}}
+			{{:assign prop=0}}
+		{{else}}
+			{{:assign prop=1}}
 		{{/if}}
 	{{/if}}
 
-	{{#foreach from=$_POST.user key="id" item="name"}}
-		{{:assign var="user.id" value=$id}}
-		{{:assign var="user.name" value=$name}}
-	{{/foreach}}
-
-	{{* préparer le mouvement modifié *}}
-	{{:assign var="mvt_new.operation" value=$_POST.operation}}
 	{{:assign var="mvt_new.amount" value=$_POST.amount}}
 	{{:assign var="mvt_new.date" value=$_POST.date|parse_date}}
 	{{:assign var="mvt_new.comment" value=$_POST.comment}}
@@ -143,8 +128,6 @@
 			equipment=$eqpmt_key
 			date=$mvt_new.date
 			comment=$mvt_new.comment
-			user=$user.id
-			storage=$_POST.storage
 		}}
 		{{:redirect force="../historique.html?ok=1&key=%s&prop=%s&msg=modification"|args:$eqpmt_key:$prop}}
 	{{/if}}
@@ -160,45 +143,21 @@
 {{* récupérer les infos de la catégorie *}}
 {{#load key=$eqpmt.category assign="category"}}{{/load}}
 
-{{if $mvt_new.user != null}}
-	{{#select id, !name as nom FROM users WHERE id=:id; !name=$config.user_fields.name_sql :id=$mvt_new.user}}
-		{{:assign var="user.%s"|args:$id value=$nom}}
-	{{/select}}
-{{/if}}
-
 {{* formulaire de modification du mouvement *}}
-{{#load type="storage" order="$$.name"}}
-	{{:assign var="storage.%s"|args:$key value=$name}}
-{{/load}}
-
 <form method="post" action="">
 	<fieldset>
 		<legend>Modifier « {{$op_label}} {{$eqpmt.name}} (Catégorie : {{$category.name}}) »</legend>
+		{{if $mvt_new.direction == "input"}}
 			<dl>
-			{{if $mvt_new.direction == "input"}}
 				{{:input type="select" name="operation" label="Type" required=true options=$input_labels|sort default=$mvt_new.operation}}
-			{{else}}
+		{{else}}
+			<dl>
 				{{:input type="select" name="operation" label="Type" required=true options=$output_labels|sort default=$mvt_new.operation}}
-			{{/if}}
-			{{:input type="date" name="date" label="Date" required=true default=$mvt_new.date}}
-			{{:input type="number" name="amount" label="Quantité" min=1 required=true default=$mvt_new.amount}}
-			{{if $prop == 1 && $mvt_new.direction == "output"}}
-				{{:input
-					type="list"
-					name="user"
-					label="Membre"
-					default=$user
-					target="!users/selector.php"
-					multiple=true
-					max=1
-					help="dans le cas d'une sortie temporaire, il est possible d'associer un membre bénéficiaire"
-				}}
-			{{/if}}
-			{{if $mvt_new.direction == "input"}}
-				{{:input type="select" name="storage" label="Lieu de stockage" default=$mvt_new.storage default_empty="— Aucun —" options=$storage required=false}}
-			{{/if}}
-			{{:input type="textarea" name="comment" label="Remarques" cols="40", rows="3" required=false default=$mvt_new.comment}}
-		</dl>
+		{{/if}}
+		{{:input type="date" name="date" label="Date" required=true default=$mvt_new.date}}
+		{{:input type="number" name="amount" label="Quantité" min=1 required=true default=$mvt_new.amount}}
+		{{:input type="textarea" name="comment" label="Remarques" cols="40", rows="3" required=false default=$mvt_new.comment}}
+			</dl>
 	</fieldset>
 
 	<p class="submit">

mouvements/movement.schema.json

@@ -32,14 +32,6 @@
 		"comment": {
 			"description": "Commentaire additionnel",
 			"type": ["null", "string"]
-		},
-		"user": {
-			"description": "identifiant membre dépositaire",
-			"type": ["integer", "null"]
-		},
-		"storage": {
-			"description": "clé du lieu de stockage",
-			"type": ["string", "null"]
 		}
 	},
 	"required": ["type", "direction", "operation", "amount", "equipment", "date", "comment"]

mouvements/sorties/emprunte.html

@@ -130,7 +130,7 @@
 				</dl>
 			</fieldset>
 			<fieldset class="sortie">
-				<legend>Enregistrer la sortie d'un matériel présent temporairement</legend>
+				<legend>Ajouter une sortie d'un matériel présent temporairement</legend>
 				<dl>
 				{{:input type="select" name="operation" label="Type" required=true options=$output_labels|sort}}
 				{{:input type="date" name="date" label="Date de sortie" required=true default=$now|date_short}}

mouvements/sorties/stock_disponible.html

@@ -5,6 +5,11 @@
 	- key		: clé du matériel à sortir
 *}}
 
+{{if ! $dialog}}
+	{{* barre de navigation *}}
+	{{:include file="../../_nav.html" current="sorties"}}
+{{/if}}
+
 {{* récupérer la config des entrées/sorties *}}
 {{:include file="../../_get_config.html" keep="config"}}
 
@@ -22,25 +27,9 @@
 		{{:error message="Impossible de saisir une date dans le futur (%s)"|args:$_POST.date}}
 	{{/if}}
 
-	{{* vérifier les infos saisies *}}
-	{{if $_POST.operation == ""}}
-		{{:error message="Vous devez choisir un type de sortie"}}
-	{{/if}}
-
-	{{:assign var="type_mvt" from="config.output_nature.%s.type"|args:$_POST.operation}}
-	{{if $type_mvt != "temporaire" && $_POST.user != null}}
-		{{:error message="Un membre ne peut être associé qu'à une sortie temporaire"}}
-	{{/if}}
-	{{if $_POST.user|count > 1}}
-		{{:error message="Un membre au plus peut être associé à une sortie"}}
-	{{/if}}
-	{{#foreach from=$_POST.user key="id" item="name"}}
-		{{:assign var="user.id" value=$id}}
-		{{:assign var="user.name" value=$name}}
-	{{/foreach}}
-
 	{{* récupérer les infos du matériel *}}
-	{{#load type="equipment" key=$_GET.key assign="equipment"}}{{/load}}
+	{{#load type="equipment" key=$_GET.key assign="equipment"}}
+	{{/load}}
 
 	{{:assign stock=0}}
 	{{:assign exterieur=0}}
@@ -53,6 +42,7 @@
 		{{if ! $insere && $mvt.date > $_POST.date|parse_date}}
 			{{:assign insere=true}}
 			{{:assign dispo_old="%d-%d"|math:$stock:$exterieur}}
+			{{:assign var="type_mvt" from="config.output_nature.%s.type"|args:$_POST.operation}}
 
 			{{if $type_mvt == 'définitif'}}
 				{{:assign stock="%d-%d"|math:$stock:$_POST.quantite}}
@@ -124,16 +114,11 @@
 		equipment=$equipment.key
 		date=$_POST.date|parse_date
 		comment=$_POST.remarques|trim
-		user=$user.id
 	}}
 	{{:redirect force="../../historique.html?ok=1&key=%s&prop=1&msg=sortie"|args:$_GET.key}}
 {{/form}}
 
 {{:admin_header title="Sortie de matériel" custom_css="./../../style.css" current="module_equipment"}}
-{{if ! $dialog}}
-	{{* barre de navigation *}}
-	{{:include file="../../_nav.html" current="sorties"}}
-{{/if}}
 {{:form_errors}}
 
 {{* Extraire et compiler les infos de la base *}}
@@ -162,20 +147,11 @@
 			</dl>
 		</fieldset>
 		<fieldset class="sortie">
-			<legend>Enregistrer une sortie</legend>
+			<legend>Ajouter une sortie</legend>
 			<dl>
 				{{:input type="select" name="operation" label="Type" required=true default_empty="— Aucun —" options=$output_labels|sort}}
 				{{:input type="date" name="date" label="Date de sortie" required=true default=$now|date_short}}
 				{{:input type="number" name="quantite" label="Quantité" required=true default=$dispo min=1 max=$dispo}}
-				{{:input
-					type="list"
-					name="user"
-					label="Membre"
-					target="!users/selector.php"
-					multiple=true
-					max=1
-					help="dans le cas d'une sortie temporaire, il est possible d'associer un membre bénéficiaire"
-				}}
 				{{:input type="textarea" name="remarques" label="Remarques" cols="40" rows="3" required=false}}
 			</dl>
 		</fieldset>
@@ -184,15 +160,7 @@
 		</p>
 	</form>
 {{else}}
-	<p class="block error">Il n'y a aucune unité de ce matériel disponible à la date du {{$now|date_short}}</p>
+	<p class="block error">Il y a 0 unité de ce matériel disponible à la date du {{$now|date_short}}</p>
 {{/if}}
 
 {{:admin_footer}}
-
-<script type="text/javascript" src="scripts.js"></script>
-<script type="text/javascript">
-	let op = $('#f_operation');
-	op.addEventListener("change", (evt) => {
-		toggleUserChoice(document.querySelector("#user"));
-	}
-</script>

mouvements/supprimer_mouvement.html

@@ -17,9 +17,10 @@
 	{{:error message="Aucun matériel avec la clé « %s »"|args:$mvt_suppr.equipment}}
 {{/load}}
 
+{{* récupérer la config des entrées/sorties *}}
+{{:include file="../_get_config.html" keep="config"}}
+
 {{#form on="delete"}}
-	{{* récupérer la config des entrées/sorties *}}
-	{{:include file="../_get_config.html" keep="config"}}
 
 	{{* vérifier s'il est possible de supprimer le mouvement *}}
 	{{if $mvt_suppr.direction == 'input'}}
@@ -115,10 +116,11 @@
 {{:form_errors}}
 
 {{:assign var="date_suppr" value="%s"|args:$mvt_suppr.date|date_short}}
+{{:assign var="mvt_label" from="config.%s_nature.%s.label"|args:$mvt_suppr.direction:$mvt_suppr.operation}}
 
 {{:delete_form
 	legend="Supprimer ?"
-	warning="Supprimer « %s (%d) en date du %s » ?"|args:$equipment.name:$mvt_suppr.amount:$date_suppr
+	warning="Supprimer le mouvement « %s de %s (quantité : %d) en date du %s » ?"|args:$mvt_label:$equipment.name:$mvt_suppr.amount:$date_suppr
 }}
 
 

mouvements/valider_modification.html

@@ -38,6 +38,7 @@
 	{{:assign dispo="%d-%d"|math:$stock:$exterieur}}
 	{{if $dispo < 0 || $stock < 0 || $exterieur < 0 || $nonprop < 0}}
 		{{:assign erreur=true}}
+		{{:assign var="pb.mvt" value=$mvt}}
 		{{:break}}
 	{{/if}}
 {{/foreach}}

snippets/user_details.html

@@ -1,35 +0,0 @@
-{{#restrict section="users" level="read"}}
-	<div>
-		<h3>Matériel en prêt - {{$user._name}}</h3>
-		<table class="list">
-			<thead>
-				<tr>
-					<td>Matériel</td>
-					<td>Date</td>
-					<td>Quantité</td>
-				</tr>
-			</thead>
-			<tbody>
-		{{#select
-			json_extract(eqpmt.document, '$.name') as mat,
-			json_extract(mvt.document, '$.date') as date,
-			json_extract(mvt.document, '$.amount') as qte
-			FROM module_data_equipment AS mvt
-			INNER JOIN users ON json_extract(mvt.document, '$.user') = users.id
-			INNER JOIN module_data_equipment AS eqpmt
-			ON json_extract(mvt.document, '$.equipment') = eqpmt.key
-			WHERE users.id = :user
-			ORDER BY mat
-			;
-			:user = $user.id
-			}}
-			<tr>
-				<td>{{$mat}}</td>
-				<td>{{$date|date_short}}</td>
-				<td>{{$qte}}</td>
-			</tr>
-		{{/select}}
-			</tbody>
-		</table>
-	</div>
-{{/restrict}}

storage/add_storage.html

@@ -1,43 +0,0 @@
-{{* -*- brindille -*- *}}
-
-
-{{* Traiter l'envoi du formulaire *}}
-{{#form on="save"}}
-
-	{{* vérification *}}
-	{{if $_POST.name == null}}
-		{{:error message="Le nom du lieu de stockage est obligatoire"}}
-	{{/if}}
-	{{#load type="storage" where="$$.name = :name COLLATE U_NOCASE" :name=$_POST.name|trim limit=1}}
-		{{:error message="Ce nom est déjà utilisé"}}
-	{{/load}}
-
-	{{:save
-		key=""|uuid
-	    validate_schema="storage.schema.json"
-		type="storage"
-		name=$_POST.name|tolower|ucwords
-		location=$_POST.location
-	}}
-	{{:redirect force="index.html?ok=1&msg=ajout"}}
-{{/form}}
-
-{{:admin_header title="Lieu de stockage" current="module_equipment"}}
-{{* barre de navigation *}}
-{{if ! $dialog}}
-	{{:include file="../_nav.html" current="storage"}}
-{{/if}}
-{{:form_errors}}
-
-<form method="post" action="" data-focus="1">
-	<fieldset class="storage">
-		<legend>Ajouter un lieu de stockage</legend>
-		<dl>
-			{{:input type="text" name="name" label="Nom" required=true}}
-			{{:input type="textarea" name="location" label="Emplacement" cols="40", rows="3" required=false}}
-		</dl>
-	</fieldset>
-	<p class="submit">
-		{{:button type="submit" name="save" label="Enregistrer" shape="right" class="main"}}
-	</p>
-</form>

storage/delete_storage.html

@@ -1,21 +0,0 @@
-{{* -*- brindille -*- *}}
-
-{{*
-	paramètres :
-	- key : clé du stockage à supprimer
-*}}
-
-{{#form on="delete"}}
-	{{:delete where="key = :key" :key=$_GET.key}}
-	{{:redirect force="index.html?ok=1&msg=suppression"}}
-{{/form}}
-
-
-{{:admin_header title="Lieux de stockage" custom_css="./style.css" current="module_equipment"}}
-{{:form_errors}}
-
-{{#load key=$_GET.key limit="1" assign="store"}}{{/load}}
-{{:delete_form
-	legend="Supprimer %s ?"|args:$store.name
-	warning="Confirmer la suppression de « %s »"|args:$store.name
-}}

storage/index.html

@@ -1,37 +0,0 @@
-{{* -*- brindille -*- *}}
-
-{{*
-	Afficher les lieux de stockage
-*}}
-
-{{:admin_header title="Lieux de stockage" custom_css="./style.css" current="module_equipment"}}
-
-{{:include file="../_nav.html" current="storage"}}
-
-{{if $_GET.ok}}
-	{{if $_GET.msg == "ajout"}}
-		<p class="block confirm">Ajout effectué</p>
-	{{elseif $_GET.msg == "suppression"}}
-		<p class="block confirm">Suppression effectuée</p>
-	{{/if}}
-{{elseif $_GET.err}}
-	<p class="block error">Opération refusée</p>
-{{/if}}
-
-{{#list
-	type="storage"
-	select="
-		$$.name AS 'Nom';
-		$$.location AS 'Emplacement'
-	"
-	}}
-	<tr>
-		<td>{{$name}}</td>
-		<td>{{$location}}</td>
-		<td class="actions">
-			{{:linkbutton label="Supprimer" shape="delete" href="delete_storage.html?key=%s"|args:$key target="_dialog"}}
-		</td>
-	</tr>
-{{/list}}
-
-{{:admin_footer}}

storage/storage.schema.json

@@ -1,19 +0,0 @@
-{
-	"$schema": "https://json-schema.org/draft/2020-12/schema",
-	"type": "object",
-	"properties": {
-		"type": {
-			"type": "string",
-			"enum": ["storage"]
-		},
-		"name" : {
-			"description": "Nom du lieu de stockage",
-			"type": "string"
-		},
-		"location": {
-			"description": "Emplacement du lieu de stockage",
-			"type": "string"
-		}
-	},
-	"required": ["type", "name"]
-}