From 530f0886b37b33e9d03976266ab33cc8cb0f17bb Mon Sep 17 00:00:00 2001
From: bohwaz <github.bohwaz@miam.kd2.org>
Date: Fri, 17 Dec 2021 12:38:58 +0100
Subject: [PATCH] =?UTF-8?q?Suppression=20de=20client=20:=20utilisation=20d?=
 =?UTF-8?q?u=20formulaire=20g=C3=A9n=C3=A9rique?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 templates/client.tpl           |  2 +-
 templates/client_supprimer.tpl | 29 +++++++----------------------
 www/admin/client_supprimer.php | 21 +++++----------------
 3 files changed, 13 insertions(+), 39 deletions(-)

diff --git a/templates/client.tpl b/templates/client.tpl
index e9bbaec..fbf2dd1 100644
--- a/templates/client.tpl
+++ b/templates/client.tpl
@@ -79,7 +79,7 @@
 </table>
 </div>
 {else}
-<h4>Cet utilisateur n'a pas de document associé.</h4>
+    <p class="alert block">Ce client n'a pas de document associé.</p>
 {/if}
 
 {include file="admin/_foot.tpl"}
\ No newline at end of file
diff --git a/templates/client_supprimer.tpl b/templates/client_supprimer.tpl
index 75dbfea..991ccf7 100644
--- a/templates/client_supprimer.tpl
+++ b/templates/client_supprimer.tpl
@@ -1,30 +1,15 @@
 {include file="admin/_head.tpl" title="Supprimer un client — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=0}
 {include file="%s/templates/_menu_client.tpl"|args:$plugin_root current="client_supprimer"}
 
-
-{form_errors}
-
 {if !$deletable}
-<form method="post" action="{$self_url}">
-
-    <fieldset>
-        <legend>Supprimer ce client ?</legend>
-        <h3 class="warning">
-            Êtes-vous sûr de vouloir supprimer le membre «&nbsp;{$client.nom}&nbsp;» ?
-        </h3>
-        <p class="alert">
-            <strong>Attention</strong> : cette action est irréversible.
-        </p>
-    </fieldset>
-
-    <p class="submit">
-        {csrf_field key="delete_client_"|cat:$client.id}
-        <input type="submit" name="delete" value="Supprimer &rarr;" />
-    </p>
-
-</form>
+    <p class="error block">Ce/cette client·e ne peut pas être supprimé·e car des documents lui y sont liés.</p>
 {else}
-<p>Ce/cette client·e ne peut pas être supprimé·e car des documents lui y sont liés.</p>
+
+    {include file="common/delete_form.tpl"
+        legend="Supprimer ce client ?"
+        warning="Êtes-vous sûr de vouloir supprimer le client « %s » ?"|args:$client.nom
+        alert="Attention, cette action est irréversible."}
+
 {/if}
 
 {include file="admin/_foot.tpl"}
\ No newline at end of file
diff --git a/www/admin/client_supprimer.php b/www/admin/client_supprimer.php
index a66f135..d810142 100644
--- a/www/admin/client_supprimer.php
+++ b/www/admin/client_supprimer.php
@@ -17,24 +17,13 @@ if (!$client)
 	throw new UserException("Ce client n'existe pas.");
 }
 
-if (f('delete'))
-{
-	$form->check('delete_client_'.$c->id);
-
-	if (!$form->hasErrors())
-	{
-		try {
-			$client->delete($c->id);
-			Utils::redirect(PLUGIN_URL . 'clients.php');
-	}
-	catch (UserException $e)
-	{
-		$form->addError($e->getMessage());
-	}
-    }
-}
+$csrf_key = 'delete_client_'.$c->id;
 
+$form->runIf('delete', function () use ($client, $c) {
+	$client->delete($c->id);
+}, $csrf_key, PLUGIN_URL . 'clients.php');
 
 $tpl->assign('deletable', $client->isDeletable($id));
 $tpl->assign('client', $c);
+$tpl->assign(compact('csrf_key'));
 $tpl->display(PLUGIN_ROOT . '/templates/client_supprimer.tpl');