From fcac139b893cccf170f3b4d06bd740fb18bc420e Mon Sep 17 00:00:00 2001
From: engel <>
Date: Wed, 4 Oct 2023 12:27:56 +0000
Subject: [PATCH] =?UTF-8?q?adaptation=20=C3=A0=20la=20nouvelle=20gestion?=
 =?UTF-8?q?=20de=20la=20config?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

FossilOrigin-Name: 8021f28f6ba17ffafbb4a1e5ffb4cb8486c3b585a486afd6a24f61db0b64841c
---
 admin/config.php        | 188 ++++++++++++++++++++--------------------
 install.php             |  25 ++++--
 lib/Utils.php           |   7 +-
 templates/recu.skel     |   4 +-
 templates/recu_html.tpl |   4 +-
 uninstall.php           |   3 +-
 upgrade.php             |   4 +-
 7 files changed, 125 insertions(+), 110 deletions(-)

diff --git a/admin/config.php b/admin/config.php
index e5db4e6..60c4981 100644
--- a/admin/config.php
+++ b/admin/config.php
@@ -7,114 +7,116 @@ use Paheko\Entities\Files\File;
 use Paheko\Plugin\RecusFiscaux\Utils;
 
 $session->requireAccess($session::SECTION_CONFIG, $session::ACCESS_ADMIN);
-$art_sel  = f('articlesCGI') ?: [];
-$taux_sel = f('tauxReduction') ?: [];
-$noms_sel = f('champsNom') ?: [];
 
 // récupérer les champs des noms
 $champsNom = Utils::getChampsNom($config, $plugin);
 
-if (f('save') && $form->check('recusfiscaux_config')) {
-    try {
-        // objet de l'association
-        if ($plugin->getConfig('objet_asso') != trim(f('objet_asso'))) {
-            $plugin->setConfig('objet_asso', trim(f('objet_asso')));
-        }
+$csrf_key = 'recusfiscaux_config';
 
-        // articles du CGI
-        $confArticles = $plugin->getConfig('articlesCGI');
-        // effacer l'ancienne configuration
-        for ($i = 0; $i < count($confArticles); ++$i) {
-            $confArticles[$i]->valeur = false; // 0
-        }
-        // et copier la nouvelle
-        foreach ($art_sel as $article) {
-            $confArticles[$article]->valeur = true; // 1
-        }
-        $plugin->setConfig("articlesCGI", $confArticles);
+$form->runIf('save', function () use ($plugin, $champsNom) {
+    // Objet de l'asso
+    $plugin->setConfigProperty('objet_asso', trim(f('objet_asso')));
 
-        // taux de réduction
-        $confTaux = $plugin->getConfig('reduction');
-        // effacer l'ancienne configuration
-        for ($i = 0; $i < count($confTaux); ++$i) {
-            $confTaux[$i]->valeur = false; // 0
-        }
-        // et copier la nouvelle
-        foreach ($taux_sel as $taux) {
-            $confTaux[$taux]->valeur = true; // 1
-        }
-        $plugin->setConfig("reduction", $confTaux);
-
-        // Informations au sujet du responsable
-        if ($plugin->getConfig('nom_responsable') != trim(f('nom_responsable'))) {
-            $plugin->setConfig('nom_responsable', trim(f('nom_responsable')));
-        }
-        if ($plugin->getConfig('fonction_responsable') != trim(f('fonction_responsable'))) {
-            $plugin->setConfig('fonction_responsable', trim(f('fonction_responsable')));
-        }
-        // ville
-        if ($plugin->getConfig('ville_asso') != trim(f('ville_asso'))) {
-            $plugin->setConfig('ville_asso', trim(f('ville_asso')));
-        }
-        // signature
-
-        if (isset($_SESSION['sig_file']) && count($_SESSION['sig_file']) > 0) {
-            // supprimer la signature précédente, si besoin
-            if (
-                null !== $plugin->getConfig('signature') &&
-                $plugin->getConfig('signature') != $_SESSION['sig_file'][0]->path
-            ) {
-                $sig_file = \Paheko\Files\Files::get($plugin->getConfig('signature'));
-                if (null !== $sig_file) {
-                    $sig_file->delete();
-                }
-            }
-            // puis installer la nouvelle
-            $plugin->setConfig('signature', $_SESSION['sig_file'][0]->path);
-        }
-
-        // autres informations
-        // numérotation des reçus
-        $configNum = $plugin->getConfig('numerotation');
-        $formNum = clone $configNum;
-        if ($configNum->prefixe != trim(f('prefixe'))) {
-            $formNum->prefixe = trim(f('prefixe'));
-        }
-        $formNum->annee = f('annee');
-        $formNum->membre = f('membre');
-        $formNum->sequentiel = f('sequentiel');
-        $formNum->valeur_init = f('valeur_init');
-        $plugin->setConfig('numerotation', $formNum);
-
-        // impression des adresses de courriel
-        $plugin->setConfig('imprimerCourriel', f('imprimerCourriel'));
-
-        // champs pour le nom et prénom
-        foreach ($champsNom as $nom => $champ) {
-            $champ->position = 0;
-        }
-        $i = -count($noms_sel);
-        foreach ($noms_sel as $nom) {
-            $champsNom[$nom]->position = $i++;
-        }
-        $plugin->setConfig('champsNom', $champsNom);
-
-        \Paheko\Utils::redirect(PLUGIN_URL . 'config.php?ok');
-    } catch (UserException $e) {
-        $form->addError($e->getMessage());
+    // Articles du CGI
+    $confArticles = $plugin->getConfig('articlesCGI');
+    // effacer l'ancienne configuration
+    for ($i = 0; $i < count($confArticles); ++$i) {
+        $confArticles[$i]->valeur = false;
     }
-}
+    // et copier la nouvelle
+    $art_sel  = f('articlesCGI') ?: [];
+    foreach ($art_sel as $article) {
+        $confArticles[$article]->valeur = true;
+    }
+    $plugin->setConfigProperty('articlesCGI', $confArticles);
+
+    // Taux de réduction
+    $confTaux = $plugin->getConfig('reduction');
+    // effacer l'ancienne configuration
+    for ($i = 0; $i < count($confTaux); ++$i) {
+        $confTaux[$i]->valeur = false;
+    }
+    // et copier la nouvelle
+    $taux_sel = f('tauxReduction') ?: [];
+    foreach ($taux_sel as $taux) {
+        $confTaux[$taux]->valeur = true;
+    }
+    $plugin->setConfigProperty("reduction", $confTaux);
+
+    // Informations au sujet du responsable
+    $plugin->setConfigProperty('nom_responsable', trim(f('nom_responsable') ?: '') ?: null);
+    $plugin->setConfigProperty('fonction_responsable', trim(f('fonction_responsable') ?: '') ?: null);
+    $plugin->setConfigProperty('ville_asso', trim(f('ville_asso') ?: '') ?: null);
+
+    // signature
+    if (isset($_SESSION['sig_file']) && count($_SESSION['sig_file']) > 0) {
+        // supprimer la signature précédente, si besoin
+        if (
+            null !== $plugin->getConfig('signature')
+                &&
+            $plugin->getConfig('signature') != $_SESSION['sig_file'][0]->path
+        ) {
+            $sig_file = \Paheko\Files\Files::get($plugin->getConfig('signature'));
+            if (null !== $sig_file) {
+                $sig_file->delete();
+            }
+        }
+        // puis installer la nouvelle
+        $plugin->setConfigProperty('signature', $_SESSION['sig_file'][0]->path);
+    }
+
+    // Numérotation des reçus
+    $configNum = $plugin->getConfig('numerotation');
+    $formNum = clone $configNum;
+    if ($configNum->prefixe != trim(f('prefixe'))) {
+        $formNum->prefixe = trim(f('prefixe'));
+    }
+    $formNum->annee = f('annee');
+    $formNum->membre = f('membre');
+    $formNum->sequentiel = f('sequentiel');
+    $formNum->valeur_init = f('valeur_init');
+    $plugin->setConfigProperty('numerotation', $formNum);
+
+    // Impression des adresses de courriel
+    $plugin->setConfigProperty('imprimerCourriel', trim(f('imprimerCourriel') ?: '') ?: null);
+
+    // champs pour le nom et prénom
+    foreach ($champsNom as $nom => $champ) {
+        $champ->position = 0;
+    }
+    $noms_sel = f('champsNom') ?: [];
+    $i = -count($noms_sel);
+    foreach ($noms_sel as $nom) {
+        $champsNom[$nom]->position = $i++;
+    }
+    $plugin->setConfigProperty('champsNom', $champsNom);
+
+    // enregistrer la nouvelle config
+    $plugin->save();
+}, $csrf_key, PLUGIN_ADMIN_URL . 'config.php?ok');
+
+
+// test fonctions fichiers : voir files.sor
+// $fichiers = Files::list('config');
+// error_log("fichiers config = " . print_r($fichiers, true));
+// $fichiers = Files::list('ext/recusfiscaux');
+// error_log("fichiers ext/recusfiscaux = " . print_r($fichiers, true));
+$sig_file = Files::get('ext/recusfiscaux/default_signature.png');
+// error_log("sig_file = " . print_r($sig_file, true));
+
+//error_log("config.php::config=" . print_r($plugin->getConfig(), true));
+
 
 // trier les champs de nom pour l'affichage
 uasort($champsNom, function ($a, $b) {
     return $a->position - $b->position;
 });
 
-$tpl->assign('ok', qg('ok') !== null);
 $path = qg('path') ?: File::CONTEXT_CONFIG;
-$tpl->assign('default_signature', \Paheko\WWW_URL . "plugin/recusfiscaux/default_signature.png");
+$tpl->assign('default_signature',  '/' . 'ext/recusfiscaux/default_signature.png');
+// $tpl->assign('default_signature', \Paheko\WWW_URL . "plugin/recusfiscaux/default_signature.png");
 $tpl->assign('plugin_config', $plugin->getConfig());
 $tpl->assign('plugin_css', ['style.css']);
 $tpl->assign('numerotation', $plugin->getConfig('numerotation'));
-$tpl->assign(compact('path', 'champsNom'));
+$tpl->assign(compact('csrf_key', 'path', 'champsNom'));
 $tpl->display(PLUGIN_ROOT . '/templates/config.tpl');
diff --git a/install.php b/install.php
index 6fe5a5a..c9b26e3 100644
--- a/install.php
+++ b/install.php
@@ -1,10 +1,23 @@
 <?php
 namespace Paheko;
-use Paheko\Entities\Files\File;
+
+use Paheko\Files\Files;
+
+$nom_plugin = $plugin->get('name');
+const SIGNATURE_DEFAUT = 'default_signature.png';
+const CONFIG_INIT = 'config.json';
+
+// configuration initiale
+$config_init = json_decode(file_get_contents(Plugins::getPath($nom_plugin) . '/' . CONFIG_INIT),
+                           true);
+
+// enregistrer dans la config du plugin
+foreach ($config_init as $cle => $valeur) {
+    $plugin->setConfigProperty($cle, $valeur);
+}
+$plugin->save();
 
 // « signature » par défaut à remplacer (voir l'onglet de configuration)
-$path = __DIR__ . '/data/default_signature.png';
-$default_signature_file = (new File)->createAndStore('skel/plugin/recusfiscaux',
-                                                     'default_signature.png',
-                                                     $path,
-                                                     null);
+$path = __DIR__ . '/data/' . SIGNATURE_DEFAUT;
+$default_signature_file = Files::createFromPath('ext/' . $nom_plugin . '/' . SIGNATURE_DEFAUT,
+                                                $path);
diff --git a/lib/Utils.php b/lib/Utils.php
index 2109cdb..d7c5fcf 100644
--- a/lib/Utils.php
+++ b/lib/Utils.php
@@ -3,6 +3,7 @@
 namespace Paheko\Plugin\RecusFiscaux;
 
 use Paheko\DB;
+use Paheko\Users\DynamicFields;
 use KD2\ZipWriter;
 
 class Utils
@@ -428,8 +429,8 @@ class Utils
         $champsNom = (array) $plugin->getConfig('champsNom');
 
         // récupérer dans la config Paheko les champs des membres
-        // utilisés pour le nom et le préno
-        $champsPaheko = $config->get('champs_membres')->listAssocNames();
+        // utilisés pour le nom et le prénom
+        $champsPaheko = DynamicFields::getInstance()->listAssocNames();
 
         foreach ($champsPaheko as $name => $title)
         {
@@ -458,7 +459,7 @@ class Utils
             }
         }
         // mettre à jour la config du plugin
-        $plugin->setConfig('champsNom', $champsNom);
+        $plugin->setConfigProperty('champsNom', $champsNom);
         return $champsNom;
     }
 
diff --git a/templates/recu.skel b/templates/recu.skel
index e3dcf3b..8691222 100644
--- a/templates/recu.skel
+++ b/templates/recu.skel
@@ -79,8 +79,8 @@
 
         <div class="cartouche" id="beneficiaire">
             <h3 class="rubrique">Bénéficiaire des versements</h3>
-            <p class="important">Association « {{$config.nom_asso}} »<br />
-              {{$config.adresse_asso}}<br />
+            <p class="important">Association « {{$config.org_name}} »<br />
+              {{$config.org_address}}<br />
               <span class="titre">Objet : </span><span class="libelle">{{$objet_asso}}</span>
             </p>
         </div>
diff --git a/templates/recu_html.tpl b/templates/recu_html.tpl
index 8fb840a..2abd833 100644
--- a/templates/recu_html.tpl
+++ b/templates/recu_html.tpl
@@ -33,8 +33,8 @@
 
     <div class="cartouche" id="beneficiaire">
         <h3 class="rubrique">Bénéficiaire des versements</h3>
-        <p class="important">Association « {$nom_asso} »<br />
-          {$adresse_asso}<br />
+        <p class="important">Association « {$org_name} »<br />
+          {$org_address}<br />
           <span class="titre">Objet&nbsp;:&nbsp;</span><span class="libelle">{$objet_asso}</span>
         </p>
     </div>
diff --git a/uninstall.php b/uninstall.php
index 430e8f0..4246dbe 100644
--- a/uninstall.php
+++ b/uninstall.php
@@ -1,11 +1,10 @@
 <?php
 namespace Paheko;
-use Paheko\Entities\Files\File;
 
 // supprimer les fichiers créés
 
 // signature par défaut
-$default_signature_file = \Paheko\Files\Files::get('skel/plugin/recusfiscaux/default_signature.png');
+$default_signature_file = \Paheko\Files\Files::get('ext/recusfiscaux/default_signature.png');
 if (null !== $default_signature_file) {
     $default_signature_file->delete();
 }
diff --git a/upgrade.php b/upgrade.php
index 7b08b05..699d018 100644
--- a/upgrade.php
+++ b/upgrade.php
@@ -14,6 +14,6 @@ if (version_compare($old_version, '0.9', '<'))
     $configNum->membre = false;
     $configNum->sequentiel = false;
     $configNum->valeur_init = 1;
-    $plugin->setConfig('numerotation', $configNum);
-    $plugin->setConfig('imprimerCourriel', false);
+    $plugin->setConfigProperty('numerotation', $configNum);
+    $plugin->setConfigProperty('imprimerCourriel', false);
 }