From 39dd23151fd6aeec4d415464f9019fe92cda50e4 Mon Sep 17 00:00:00 2001
From: Noizette <ac@b>
Date: Sat, 24 Oct 2020 22:51:01 +0200
Subject: [PATCH] Ajout page d'aide

---
 templates/_menu.tpl          |  1 +
 templates/aide.tpl           | 20 +++++++++++++
 templates/cotis_ajouter.tpl  |  2 +-
 templates/cotis_modifier.tpl |  2 +-
 templates/recu.tpl           | 55 ------------------------------------
 templates/supprimer.tpl      | 26 -----------------
 www/admin/aide.php           |  9 ++++++
 www/admin/cotis_ajouter.php  |  3 ++
 www/admin/cotis_modifier.php |  3 --
 www/admin/recu.php           | 19 -------------
 www/admin/supprimer.php      | 44 -----------------------------
 11 files changed, 35 insertions(+), 149 deletions(-)
 create mode 100644 templates/aide.tpl
 delete mode 100644 templates/recu.tpl
 delete mode 100644 templates/supprimer.tpl
 create mode 100644 www/admin/aide.php
 delete mode 100644 www/admin/recu.php
 delete mode 100644 www/admin/supprimer.php

diff --git a/templates/_menu.tpl b/templates/_menu.tpl
index f4d22b2..f947524 100644
--- a/templates/_menu.tpl
+++ b/templates/_menu.tpl
@@ -7,4 +7,5 @@
     {if $session->canAccess('compta', Membres::DROIT_ADMIN)}
     <li{if $current == 'config'} class="current"{/if}><a href="{plugin_url file="config.php"}">Configuration</a></li>
     {/if}
+    <li{if $current == 'aide'} class="current"{/if}><a href="{plugin_url file="aide.php"}">Aide</a></li>
 </ul>
diff --git a/templates/aide.tpl b/templates/aide.tpl
new file mode 100644
index 0000000..3f574ac
--- /dev/null
+++ b/templates/aide.tpl
@@ -0,0 +1,20 @@
+{include file="admin/_head.tpl" title="Aide — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id}
+{include file="%s/templates/_menu.tpl"|args:$plugin_root current="aide"}
+
+<fieldset>
+<legend><h2>Quelques remarques et conseils sur l'utilisation du plugin Facturation</h2></legend>
+
+    <p>Une des premières choses à faire est d'aller dans l'onglet Configuration pour renseigner les valeurs nécessaires à la génération des documents.</p>
+    <p>Pour l'instant, il y a des choses encore brouillonnes. Notamment, dans les factures et devis, c'est l'adresse postale renseignée dans la configuration de Garradin qui fait foi plutôt que celle dans le plugin (qui sert en revanche pour les reçus).</p>
+    <br>
+    <p>Pour créer un reçu sur une cotisation, il faut pour le moment que cette cotisation soit attachée à la compta.</p>
+    <p>Pour créer un reçu fiscal, l'interface est pour l'instant la même que pour créer une facture/devis. Les champs correspondent mais les noms/labels autour ne sont pas adaptés. Vous pouvez de toutes façons tester, et si le résultat est pas celui attendu, remodifiez derrière :)</p>
+    <p>La partie « Droit à la réduction d'impôt » peut faire peur, elle correspond simplement à des cases du cerfa pour les reçus fiscaux. Je n'y connais pas grand chose pour le moment, je ne peux vous éclairer davantage, il va falloir se retourner vers légifrance :(</p>
+    <br>
+    <p>Hésitez pas à faire des retours, proposer meilleures explications, ou quoi, vous pouvez venir en causer soit <a href="https://gitlab.com/ramoloss/garradin-plugin-facturation">sur mon gitlab</a>, soit sur l'adresse d'entraide de garradin. Si vous êtes un peu dev, le code est un peu cracra mais j'espère que ça vous repoussera pas trop à le bidouiller :)</p>
+
+
+</fieldset>
+
+
+{include file="admin/_foot.tpl"}
\ No newline at end of file
diff --git a/templates/cotis_ajouter.tpl b/templates/cotis_ajouter.tpl
index 10c8ab2..c3c6d83 100644
--- a/templates/cotis_ajouter.tpl
+++ b/templates/cotis_ajouter.tpl
@@ -1,4 +1,4 @@
-{include file="admin/_head.tpl" title="Créer un document — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1}
+{include file="admin/_head.tpl" title="Créer un reçu — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1}
 {include file="%s/templates/_menu.tpl"|args:$plugin_root current="facture"}
 
 <style>
diff --git a/templates/cotis_modifier.tpl b/templates/cotis_modifier.tpl
index bfbe860..0a8dd7a 100644
--- a/templates/cotis_modifier.tpl
+++ b/templates/cotis_modifier.tpl
@@ -1,4 +1,4 @@
-{include file="admin/_head.tpl" title="Créer un document — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1}
+{include file="admin/_head.tpl" title="Modifier un reçu — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1}
 {include file="%s/templates/_menu.tpl"|args:$plugin_root current="index"}
 
 <style>
diff --git a/templates/recu.tpl b/templates/recu.tpl
deleted file mode 100644
index 84c15a4..0000000
--- a/templates/recu.tpl
+++ /dev/null
@@ -1,55 +0,0 @@
-{include file="admin/_head.tpl" title="Extension — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id}
-{include file="%s/templates/_menu.tpl"|args:$plugin_root current="recu"}
-
-{if $error}
-    {if $error == 'OK'}
-    <p class="confirm">
-        La configuration a bien été enregistrée.
-    </p>
-    {else}
-    <p class="error">
-        {$error|escape}
-    </p>
-    {/if}
-{/if}
-
-
-{if !empty($trecus)}
-    <p>Retrouvez l'ensemble des reçus fiscaux générés:</p>
-    <table class="list">
-        <thead>
-            <tr>
-                <td title="Numéro unique"></td>
-                <td>Nom</td>
-                <td>Prénom</td>
-                <td>Ville</td>
-                <td>Montant</td>
-                <td>Date</td>
-                <td>Télécharger</td>
-                <td></td>
-            </tr>
-        </thead>
-        <tbody>
-        {foreach from=$trecus item="gendon"}
-        <tr>
-            <td>{$gendon.id|escape}</td>
-            <td>{$gendon.nom|escape}</td>
-            <td>{$gendon.prenom|escape}</td>
-            <td>{$gendon.ville|escape}</td>
-            <td>{$gendon.montant|escape}</td>
-            <td>{$gendon.date|escape}</td>
-            <td><a href="{plugin_url file="generation.php"}?id={$gendon.id}">{$gendon.gen_ordre|escape}.pdf</a></td>
-            <td class="action">
-                <a class="icn" href="{plugin_url file="supprimer.php"}?id={$gendon.id|escape}" title="Supprimer">✘</a>
-            </td>
-        </tr>
-        {/foreach}
-        </tbody>
-    </table>
-{else}
-    <p class="alert">
-        Aucun reçu fiscal trouvé.
-    </p>
-{/if}
-
-{include file="admin/_foot.tpl"}
\ No newline at end of file
diff --git a/templates/supprimer.tpl b/templates/supprimer.tpl
deleted file mode 100644
index 8d54df3..0000000
--- a/templates/supprimer.tpl
+++ /dev/null
@@ -1,26 +0,0 @@
-{include file="admin/_head.tpl" title="Extension — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id}
-{include file="%s/templates/_menu.tpl"|args:$plugin_root current="recu"}
-
-{if $error}
-    <p class="error">
-        {$error|escape}
-    </p>
-{/if}
-
-<form method="post" action="{$self_url|escape}">
-
-    <fieldset>
-        <legend>Supprimer ce reçu fiscal ?</legend>
-        <h3 class="warning">
-            Êtes-vous sûr de vouloir supprimer le reçu fiscal n°{$recu.gen_ordre|escape} concernant un don de {$recu.montant} € effectué par {$recu.prenom} {$recu.nom} le {$recu.date} ?
-        </h3>
-    </fieldset>
-
-    <p class="submit">
-        {csrf_field key="recu_supprimer_`$recu.id`"}
-        <input type="submit" name="remove" value="Supprimer &rarr;" />
-    </p>
-
-</form>
-
-{include file="admin/_foot.tpl"}
\ No newline at end of file
diff --git a/www/admin/aide.php b/www/admin/aide.php
new file mode 100644
index 0000000..7932ef1
--- /dev/null
+++ b/www/admin/aide.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Garradin;
+
+require_once __DIR__ . '/_inc.php';
+
+$session->requireAccess('compta', Membres::DROIT_ACCES);
+
+$tpl->display(PLUGIN_ROOT . '/templates/aide.tpl');
\ No newline at end of file
diff --git a/www/admin/cotis_ajouter.php b/www/admin/cotis_ajouter.php
index afb282f..a70ba50 100644
--- a/www/admin/cotis_ajouter.php
+++ b/www/admin/cotis_ajouter.php
@@ -46,6 +46,9 @@ elseif (f('add'))
 			{
 				$cotis[$field] = f($field.'_'.$num);
 			}
+			
+			$cotis['date'] = date('Y-m-d', $cotis['date']);
+			$cotis['expiration'] = date('Y-m-d', $cotis['expiration']);
 	
 			$r = $db->get('SELECT moyen_paiement, montant FROM membres_operations AS mo INNER JOIN compta_journal AS cj ON cj.id = mo.id_operation
 						WHERE mo.id_cotisation = ?;', (int)$cotis['id']);
diff --git a/www/admin/cotis_modifier.php b/www/admin/cotis_modifier.php
index 0440562..4ce990f 100644
--- a/www/admin/cotis_modifier.php
+++ b/www/admin/cotis_modifier.php
@@ -63,9 +63,6 @@ elseif (f('add'))
 			$cotis['date'] = date('Y-m-d', $cotis['date']);
 			$cotis['expiration'] = date('Y-m-d', $cotis['expiration']);
 
-			var_export($cotis);
-			// die();
-
 			$r = $db->get('SELECT moyen_paiement, montant FROM membres_operations AS mo INNER JOIN compta_journal AS cj ON cj.id = mo.id_operation
 						WHERE mo.id_cotisation = ?;', (int)$cotis['id']);
 			$r = $r[0];
diff --git a/www/admin/recu.php b/www/admin/recu.php
deleted file mode 100644
index 7192c1a..0000000
--- a/www/admin/recu.php
+++ /dev/null
@@ -1,19 +0,0 @@
-<?php
-namespace Garradin;
-
-$session->requireAccess('compta', Membres::DROIT_ADMIN);
-
-$error = false;
-
-if (isset($_GET['ok']))
-{
-    $error = 'OK';
-}
-
-$recus = new Plugin\RecuDon\GenDon;
-
-$trecus = $recus->listSimple();
-
-$tpl->assign('trecus', $trecus);
-$tpl->assign('error', $error);
-$tpl->display(PLUGIN_ROOT . '/templates/recu.tpl');
diff --git a/www/admin/supprimer.php b/www/admin/supprimer.php
deleted file mode 100644
index f660560..0000000
--- a/www/admin/supprimer.php
+++ /dev/null
@@ -1,44 +0,0 @@
-<?php
-namespace Garradin;
-
-if ($user['droits']['compta'] < Membres::DROIT_ADMIN)
-{
-    throw new UserException("Vous n'avez pas le droit d'accéder à cette page.");
-}
-
-$gendon = new Plugin\RecuDon\GenDon;
-
-$recu = $gendon->get(Utils::get('id'));
-
-if (!$recu)
-{
-    throw new UserException("Le reçu demandé n'existe pas.");
-}
-
-$error = false;
-
-if (!empty($_POST['remove']))
-{
-    if (!Utils::CSRF_check('recu_supprimer_'.$recu['id']))
-    {
-        $error = 'Une erreur est survenue, merci de renvoyer le formulaire.';
-    }
-    else
-    {
-        try
-        {
-            $gendon->remove($recu['id']);
-            Utils::redirect(PLUGIN_URL . 'recu.php');
-        }
-        catch (UserException $e)
-        {
-            $error = $e->getMessage();
-        }
-    }
-}
-
-$tpl->assign('error', $error);
-
-$tpl->assign('recu', $recu);
-
-$tpl->display(PLUGIN_ROOT . '/templates/supprimer.tpl');
\ No newline at end of file