màj CHANGELOG, README et plugin.ini

This reverts commit 16daac2e24, reversing
changes made to 46d0c838ae.

CHANGELOG

@@ -1,3 +1,33 @@
+0.11
+- Changement mention finale pour devis
+0.9
+- Ajout possibilité choisir champs identité et adresse membre
+0.8.8
+- correction typo
+0.8.7
+- correction typo
+0.8.6
+- Correction erreur si pas de prix saisi
+0.8.5
+- Ajout numéro SIREN/SIRET pour les clients
+- Ajout pagination liste factures
+
+0.8.4
+- Correction bug si identité définie par plusieurs champs
+- Correction bug oubli choix receveur
+- Mise-à-jour Changelog
+
+0.8.3 :
+- Mise-à-jour README
+
+0.8.2 :
+- Tri des membres en ordre alphabétique
+- Élimination de plusieurs bugs
+
+0.8.1 :
+- reprise du code par jce (lesanges@zaclys.net ; https://git.roflcopter.fr/lesanges/paheko-plugin-facturation)
+- Achèvement migration vers Paheko 1.3
+
 0.7.4 :
 - Corrections de dépréciations PHP, le plugin être compatible PHP8.2
 - Quelques problèmes réglés de formulaires invalidables

README.md

@@ -1,53 +1,49 @@
-# Plugin Facturation pour Garradin
+# Plugin Facturation pour Paheko (ex Garradin)
 
-Plugin de facturation pour le logiciel de gestion d'association Garradin ( https://garradin.eu/ - https://fossil.kd2.org/garradin ).
+Plugin de facturation pour le logiciel de gestion d'association Paheko
-Source : https://gitlab.com/ramoloss/garradin-plugin-facturation
+( https://paheko.eu/ - https://fossil.kd2.org/paheko ).
 
-## Installation:
+Source :
-Vous pouvez télécharger l'archive .tar.gz depuis la page des [releases](https://gitlab.com/ramoloss/garradin-plugin-facturation/-/releases), et la placer directement dans le dossier plugins de Garradin.
+- version compatible paheko 1.3.x : https://git.roflcopter.fr/lesanges/facturation
+- version historique non compatible paheko 1.3.x : https://gitlab.com/noizette/garradin-plugin-facturation
 
-### Anciennes versions (<0.6)
+## Installation :
-
+Vous pouvez télécharger l'archive .tar.gz depuis la page des
-Normalement, les plugins de Garradin doivent seulement être laissé sous forme d'archive .tar.gz dans le dossier plugins, or pour la génération des PDF, la librairie mPDF a besoin d'écrire des fichiers temporaires.
+[releases](https://git.roflcopter.fr/lesanges/paheko-plugin-facturation/releases),
-
+supprimer le numéro de version du nom de l'archive puis la placer dans
-Il faut pour cela faire :
+le dossier plugins de Paheko.
-
-    tar xvf garradin-plugin-facturation-v0.5.0.tar.gz
-    mv garradin-plugin-facturation-v0.5.0 facturation
-    rm garradin-plugin-facturation-v0.5.0.tar.gz
-    chown -R www-data:www-data facturation/
-    chmod -R g+w facturation/ 
-
-
-Supprimer l'archive permet à Garradin de ne pas la lire à la place du dossier.
-
-*Note : www-data correspond dans la plupart des cas à l'utlisateur d'Apache, si vous utilisez un autre serveur web, il faudra probablement adapter.*
-
-## Migration vers Garradin 1.0
-
-Lorsque vous tentez de mettre à jour une installation de Garradin avec le plugin facturation vers Garradin 1.0, l'upgrade est bloquée par le plugin.
-
-Pour remédier à cela et parvenir à faire la mise à jour, il faut dans un premier temps installer la version 0.4 du plugin sur votre installation Garradin 0.9.8, se rendre sur la page principale du plugin (menu > Facturation), vous pouvez ensuite mettre à jour Garradin vers la 1.0.
 
 ## Fonctionnalités :
 - Créer et gérer une base de client·es
 - Créer et modifier des factures et devis adressés aux membres de l'association ou des client·es ajouté·es
-- Créer des reçus fiscaux pour des dons et génération du cerfa correspondant
+- (obsolète) Créer des reçus fiscaux pour des dons et génération du CERFA correspondant
-- Créer des reçus sur des cotisations
+- (obsolète) Créer des reçus sur des cotisations
-- Génération des documents (facture et devis) en PDF grâce à la librairie mPDF
+- Génération des documents (facture et devis) en PDF
 - Liste les documents associés sur la fiche d'un·e client·e
-- Permet de définir le statut du document sur reglée
+- Permet de définir le statut du document à « réglé »
 - **Configuration** :
     - Possibilité d'ajouter un numéro RNA et SIRET de l'association si elle en possède (apparait alors sur les documents)
-    - Modification du pied de page des documents (notament pour y inscrire des mentions légales)
+	- Possibilité de choisir certains champs à faire figurer sur la facture (adresse, code postal, ville)
+    - Modification du pied de page des documents (notamment pour y inscrire des mentions légales)
     - Vérifier le code postal : si coché, lors d'ajout ou de modification de client, le plugin vérifiera que le code postal entré est bien formaté (par rapport aux codes postaux français seulement)
-    - Noms de client·es uniques : si coché, lors d'ajout ou de modicifation de client·e, le nom du/de la client·e ne pourra pas être le même que celui d'un·e client·e déjà existant
+    - Noms de client·es uniques : si coché, lors d'ajout ou de modification de client·e, le nom du/de la client·e ne pourra pas être le même que celui d'un·e client·e déjà existant
-    - Informations relatives au cerfa pour les reçus fiscaux
+    - (obsolète) Informations relatives au CERFA pour les reçus fiscaux
-    - Image qui set de signature sur le cerfa
+    - (obsolète) Image qui sert de signature sur le CERFA
 
-Note : pour le moment, les actions sur la liste des clients à cocher ne fonctionnent pas. Pour supprimer un client, le faire depuis sa fiche.
+Note : pour le moment, les actions sur la liste des clients à cocher
+ne fonctionnent pas. Pour supprimer un client, le faire depuis sa
+fiche.
 
-## Futur :
+## Futur de ce plugin :
+Un nouveau plugin est en cours de développement par BohwaZ, donc il
+n'est pas pertinent d'ajouter de nouvelles fonctionnalités à celui-ci.
+
+Par contre, si des bugs sont signalés sur la liste
+hebergement@paheko.cloud ou aide@paheko.cloud, je
+(lesanges@zaclys.net) peux tenter de les corriger, à condition que ça
+n'impacte pas trop la structure du plugin.
+
+## Futur improbable (obsolète) :
 - Ajout des champs Référence, Prix unitaire, Quantité sur les documents
 - Actions sur liste de client·es (exporter, supprimer)
 - Afficher/filtrer les documents par statuts réglé/archivé
@@ -57,15 +53,12 @@ Note : pour le moment, les actions sur la liste des clients à cocher ne fonctio
 - Gestion TVA ?
 - Un devis ne devrait pas pouvoir être réglé
 - Quid si un·e membre de l'asso est supprimé·e alors que des documents lui sont adressés ?
-
-## Futur improbable :
 - Opérations de paiements dans la compta liés à une facture
 - Gestion de produits
 
-
 Le plugin nécessite l'extension PHP mbstring.
 
-## Inclus les bibliothèques suivantes :
+## (???) Inclus les bibliothèques suivantes :
 
 - Composer :
     https://getcomposer.org/

admin/_facture_common.php

@@ -4,6 +4,28 @@ namespace Paheko;
 
 require_once __DIR__ . '/_inc.php';
 
+function toArray($array, $cle, $sep=",")
+{
+	$result = array();
+	foreach ($array as $elem)
+	{
+		$ro = new \ReflectionObject($elem);
+		$proprietes = $ro->getProperties();
+		$ligne = "";
+		foreach ($proprietes as $p)
+		{
+			if ($p->getName() == $cle) {
+				$key = $p->getValue($elem);
+			}
+			else {
+				$ligne .= $sep . $p->getValue($elem);
+			}
+		}
+		$result[$key] = substr($ligne, strlen($sep));
+	}
+	return $result;
+}
+
 if (!isset($target) || !in_array( $target, ['new', 'edit'])) {
 	throw new Exception('blabla illegal call'); // Fix: exception type?
 } else {
@@ -34,8 +56,8 @@ $tpl->assign('formes_don', array('1' => 'Acte authentique',
                                  '3' => 'Don manuel',
                                  '4' => 'Autres'));
 $tpl->assign('natures_don', array('1' => 'Numéraire',
-                                 '2' => 'Chèque',
+                                  '2' => 'Chèque',
-                                 '3' => 'Virement, CB; ...'));
+                                  '3' => 'Virement, CB; ...'));
 $tpl->assign('textes_don', $facture->listTextesCerfa());
 
 if ( !$target ) {
@@ -51,134 +73,145 @@ if ( !$target ) {
 // Traitement
 $data=[];
 $form->runIf(f('save') && !$form->hasErrors(),
-             function () use ($client, &$data)
+             function () use ($client, &$data, $form)
-             {
+    {
-                 try
+        try
-                 {
+        {
-                     if ( count(f('designation')) !== count(f('prix')) )
+            if ( count(f('designation')) !== count(f('prix')) )
-                     {
+            {
-                         throw new UserException('Nombre de désignations et de prix reçus différent.');
+                throw new UserException('Nombre de désignations et de prix reçus différent.');
-                     }
+            }
 
-                     $data = [
+            $data = [
-                         'numero'			=> f('numero_facture'),
+                'numero'			=> f('numero_facture'),
-                         'date_emission'	=> f('date_emission'),
+                'date_emission'	=> f('date_emission'),
-                         'date_echeance'	=> f('date_echeance'),
+                'date_echeance'	=> f('date_echeance'),
-                         'reglee'			=> f('reglee') == 1?1:0,
+                'reglee'			=> f('reglee') == 1?1:0,
-                         'archivee'			=> f('archivee') == 1?1:0,
+                'archivee'			=> f('archivee') == 1?1:0,
-                         'moyen_paiement'	=> f('moyen_paiement'),
+                'moyen_paiement'	=> f('moyen_paiement'),
-                         'toto'				=> 0
+                'toto'				=> 0
-                     ];
+            ];
-                     $data['type_facture'] = f('type');
+            $data['type_facture'] = f('type');
-                     if (in_array(f('type'), [DEVIS, FACT]))
+            if (in_array(f('type'), [DEVIS, FACT]))
-                     {
+            {
-                         foreach(f('designation') as $k=>$value)
+                foreach(f('designation') as $k=>$value)
-                         {
+                {
-                             $data['contenu'][$k]['designation'] = $value;
+					if ($value != '' && f('prix')[$k] == null) {
-                             $data['contenu'][$k]['prix'] = Utils::moneyToInteger(f('prix')[$k]);
+						throw new UserException('Il manque le prix sur la ligne '. $k+1 . ' !!');
-                             $data['toto'] += Utils::moneyToInteger(f('prix')[$k]);
+					}
-                         }
-                         $data['total'] = $data['toto'];
-                         unset($data['toto']);
-                     }
-                     elseif ( f('type') == CERFA )
-                     {
-                         $data['moyen_paiement'] = f('moyen_paiement_cerfa');
-                         $data['contenu'] = [
-                             'forme' => f('forme_don'),
-                             'nature' => f('nature_don'),
-                             'texte' => f('texte_don')];
-                         $data['total'] = Utils::moneyToInteger(f('total'));
-                         unset($data['toto']);
-                     }
-                     if (f('base_receveur') == 'client')
-                     {
-                         $data['receveur_membre'] = 0;
-                         $data['receveur_id'] = f('client');
-                     }
-                     elseif (f('base_receveur') == 'membre')
-                     {
-                         $data['receveur_membre'] = 1;
-                         $data['receveur_id'] = f('membre');
-                     }
 
-                 }
+                    $data['contenu'][$k]['designation'] = $value;
-                 catch(UserException $e)
+                    $data['contenu'][$k]['prix'] = Utils::moneyToInteger(f('prix')[$k]);
-                 {
+                    $data['toto'] += Utils::moneyToInteger(f('prix')[$k]);
-                     $form->addError($e->getMessage());
+                }
-                 }
+                $data['total'] = $data['toto'];
+                unset($data['toto']);
+            }
+            elseif ( f('type') == CERFA )
+            {
+                $data['moyen_paiement'] = f('moyen_paiement_cerfa');
+                $data['contenu'] = [
+                    'forme' => f('forme_don'),
+                    'nature' => f('nature_don'),
+                    'texte' => f('texte_don')];
+                $data['total'] = Utils::moneyToInteger(f('total'));
+                unset($data['toto']);
+            }
+            if (f('base_receveur') == 'client')
+            {
+                $data['receveur_membre'] = 0;
+                $data['receveur_id'] = f('client');
+            }
+            elseif (f('base_receveur') == 'membre')
+            {
+                $data['receveur_membre'] = 1;
+                $data['receveur_id'] = f('membre');
+            }
+            else
+            {
+                throw new UserException('Vous devez indiquer si le receveur est un client ou un membre');
+            }
 
-             }, $csrf_key);
+        }
+        catch(UserException $e)
+        {
+            $form->addError($e->getMessage());
+        }
+
+    }, $csrf_key);
 
 $form->runIf(f('select_cotis') && !$form->hasErrors(),
              function () use ($step)
-             {
+		     {
-                 $step = true;
+				 $step = true;
-             }, 'add_cotis_1');
+			 }, 'add_cotis_1');
 
 $form->runIf(f('add_cotis') && !$form->hasErrors(),
-             function () use ($radio, $fields, $facture)
+             function () use ($radio, $fields, $facture, $form)
-             {
+			 {
-                 $radio['type'] = f('cotisation');
+				 $radio['type'] = f('cotisation');
-                 try
+				 try
-                 {
+				 {
-                     $num = (int) str_replace('cotis_', '', $radio['type']);
+					 $num = (int) str_replace('cotis_', '', $radio['type']);
-                     foreach($fields as $field)
+					 foreach($fields as $field)
-                     {
+					 {
-                         $cotis[$field] = f($field.'_'.$num);
+						 $cotis[$field] = f($field.'_'.$num);
-                     }
+					 }
 
-                     $r = $facture->getCotis(f('membre_cotis'), $cotis['id']);
+					 $r = $facture->getCotis(f('membre_cotis'), $cotis['id']);
-                     $r = $r[0];
+					 $r = $r[0];
 
-                     $data = [
+					 $data = [
-                         'type_facture' => COTIS,
+						 'type_facture' => COTIS,
-                         'numero' => f('numero_facture'),
+						 'numero' => f('numero_facture'),
-                         'receveur_membre' => 1,
+						 'receveur_membre' => 1,
-                         'receveur_id' => f('membre_cotis'),
+						 'receveur_id' => f('membre_cotis'),
-                         'date_emission' => f('date_emission'),
+						 'date_emission' => f('date_emission'),
-                         'moyen_paiement'	=> 'AU',
+						 'moyen_paiement'	=> 'AU',
-                         'total' => $r->paid_amount ?? $r->amount,
+						 'total' => $r->paid_amount ?? $r->amount,
-                         'contenu' => ['id' => $cotis['id'],
+						 'contenu' => ['id' => $cotis['id'],
-                                       'intitule' => $cotis['label'],
+									   'intitule' => $cotis['label'],
-                                       'souscription' => $cotis['date'],
+									   'souscription' => $cotis['date'],
-                                       'expiration' => $cotis['expiry'] ]
+									   'expiration' => $cotis['expiry'] ]
-                     ];
+					 ];
 
-                 }
+				 }
-                 catch (UserException $e)
+				 catch (UserException $e)
-                 {
+				 {
-                     $form->addError($e->getMessage());
+					 $form->addError($e->getMessage());
-                 }
+				 }
-             }, 'add_cotis_2');
+			 }, 'add_cotis_2');
 
-if ($step)
+if (! $form->hasErrors())
 {
-    try
+    if ($step)
     {
-        $liste = $facture->getCotis((int)f('membre_cotis'));
+        try
+        {
+            $liste = $facture->getCotis((int)f('membre_cotis'));
+        }
+        catch (UserException $e)
+        {
+            $form->addError($e->getMessage());
+        }
     }
-    catch (UserException $e)
+    elseif (count($data) > 0)
     {
-        $form->addError($e->getMessage());
+        if ($target)
+        {
+            $id = $facture->add($data, $plugin->getConfig('pattern'));
+            Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
+        }
+        else
+        {
+            if ($facture->edit($id, $data))
+            {
+                Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
+            }
+            throw new UserException('Erreur d\'édition du reçu');
+        }
     }
 }
-elseif (count($data) > 0)
-{
-	if ($target)
-	{
-		$id = $facture->add($data, $plugin->getConfig('pattern'));
-		Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
-	}
-	else
-	{
-		if ($facture->edit($id, $data))
-		{
-			Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
-		}
-		throw new UserException('Erreur d\'édition du reçu');
-	}
-}
 
 // Affichage
 
@@ -290,9 +323,8 @@ $date = new \DateTime;
 $date->setTimestamp(time());
 $tpl->assign('date', $date->format('d/m/Y'));
 
-
 $tpl->assign(compact('liste', 'radio', 'step', 'designations', 'prix', 'from_user', 'identite', 'csrf_key', 'doc'));
-$tpl->assign('users', $db->getAssoc('SELECT id, '.$identite.' FROM users WHERE id_category != -2 NOT IN (SELECT id FROM users_categories WHERE hidden = 1) ORDER BY ' .$identite. ';'));
+$tpl->assign('users', toArray($db->get('SELECT id, '.$identite.' FROM users WHERE id_category != -2 NOT IN (SELECT id FROM users_categories WHERE hidden = 1) ORDER BY ' .$identite. ';'), 'id', " "));
 $tpl->assign('clients', $db->getAssoc('SELECT id, nom FROM plugin_facturation_clients;'));
 $tpl->assign('require_number', $require_number);
 $tpl->assign('number_pattern', PATTERNS_LIST[$plugin->getConfig('pattern')]);

admin/client_modifier.php

@@ -17,7 +17,7 @@ if (!$c)
 }
 
 $form->runIf(f('save') && !$form->hasErrors(),
-             function () use ($client, $id)
+             function () use ($client, $id, $form)
              {
                  try
                  {
@@ -26,6 +26,7 @@ $form->runIf(f('save') && !$form->hasErrors(),
                          'adresse'		=>  f('adresse'),
                          'code_postal'	=>  f('code_postal'),
                          'ville'		=>  f('ville'),
+                         'siret'        =>  f('siret'),
                          'telephone'	=>  f('telephone'),
                          'email'		=>  f('email')
                      ]);

admin/clients.php

@@ -7,7 +7,7 @@ require_once __DIR__ . '/_inc.php';
 $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_READ);
 
 $form->runIf(f('add') && !$form->hasErrors(),
-             function () use ($client)
+             function () use ($client, $form)
              {
                  try
                  {
@@ -16,6 +16,7 @@ $form->runIf(f('add') && !$form->hasErrors(),
                          'adresse'		=>  f('adresse'),
                          'code_postal'	=>  f('code_postal'),
                          'ville'		=>  f('ville'),
+                         'siret'        =>  f('siret'),
                          'telephone'	=>  f('telephone'),
                          'email'		=>  f('email')
                      ]);

admin/config.php

@@ -1,11 +1,15 @@
 <?php
 
 namespace Paheko;
+use Paheko\Users\DynamicFields;
 
 require_once __DIR__ . '/_inc.php';
 
 $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_ADMIN);
 
+$champsPaheko = DynamicFields::getInstance()->listAssocNames();
+$champsPaheko = array('' => '- Choisir un champ -') + $champsPaheko;
+
 $form->runIf('save', function () use ($plugin) {
 	$plugin->setConfigProperty('rna_asso', trim(f('rna_asso')));
 	$plugin->setConfigProperty('siret_asso', trim(f('siret_asso')));
@@ -16,18 +20,11 @@ $form->runIf('save', function () use ($plugin) {
 	$plugin->setConfigProperty('cp_asso', trim(f('cp_asso')));
 	$plugin->setConfigProperty('ville_asso', trim(f('ville_asso')));
 
-	$plugin->setConfigProperty('droit_art200', (bool)f('droit_art200'));
+	$plugin->setConfigProperty('logo', (bool)f('logo'));
-	$plugin->setConfigProperty('droit_art238bis', (bool)f('droit_art238bis'));
-	$plugin->setConfigProperty('droit_art885_0VbisA', (bool)f('droit_art885_0VbisA'));
-	$plugin->setConfigProperty('objet_0', trim(f('objet_0')));
-	$plugin->setConfigProperty('objet_1', trim(f('objet_1')));
-	$plugin->setConfigProperty('objet_2', trim(f('objet_2')));;
-
 	$plugin->setConfigProperty('footer', f('footer'));
 
 	$plugin->setConfigProperty('validate_cp', (bool)f('validate_cp'));
 	$plugin->setConfigProperty('unique_client_name', (bool)f('unique_client_name'));
-
 	$plugin->setConfigProperty('pattern', f('pattern'));
 
 	$plugin->save();
@@ -37,5 +34,6 @@ $form->runIf('save', function () use ($plugin) {
 $tpl->assign('ok', qg('ok') !== null);
 $tpl->assign('conf', $plugin->getConfig());
 $tpl->assign('patterns', \Paheko\Plugin\Facturation\PATTERNS_LIST);
+$tpl->assign('champsPaheko', $champsPaheko);
 
 $tpl->display(PLUGIN_ROOT . '/templates/config.tpl');

admin/facture_supprimer.php

@@ -18,7 +18,7 @@ if (!$client)
 }
 
 $form->runIf(f('delete') && !$form->hasErrors(),
-             function () use ($facture, $f)
+             function () use ($facture, $f, $form)
              {
                  try {
                      $facture->delete($f->id);

admin/index.php

@@ -12,6 +12,7 @@ $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_READ);
 
 $list = $facture->list();
 $list->loadFromQueryString();
+$list->setPageSize(50);
 
 $tpl->assign(compact('list'));
 

admin/pdf.php

@@ -8,7 +8,6 @@ $users = new Users\Users;
 
 f(['id' => 'required|numeric']);
 $id = (int) qg('id');
-$id_field = Users\DynamicFields::getNameFieldsSQL();
 $sign_tag = UserTemplate\Functions::signature();
 
 // Vérification que le document existe
@@ -25,19 +24,36 @@ try
 	if ($f->receveur_membre)
 	{
 		$c = $users->get($f->receveur_id);
-		$c->$id_field = $c->$identite;
+		// l'identité du membre peut être redéfinie dans la configuration des membres
+		$name_fields = \Paheko\Users\DynamicFields::getNameFields();
+		array_walk($name_fields, function(&$elem) use ($c) {
+			$elem = $c->$elem;
+		});
+		$nom_client = implode(" ", $name_fields);
+		if (preg_match('/^ +$/', $nom_client)) {
+			$nom_client = "** ABSENT **";
+		}
+
+		// adresse, code postal et ville peuvent être redéfini(e)s dans la configuration du plugin
+		$adresse_client = $plugin->getConfig('adresse_client');
+		if ($adresse_client != null && $c->$adresse_client != null) { $c->adresse = $c->$adresse_client; }
+		$code_postal_client = $plugin->getConfig('code_postal_client');
+		if ($code_postal_client != null && $c->$code_postal_client != null) { $c->code_postal = $c->$code_postal_client; }
+		$ville_client = $plugin->getConfig('ville_client');
+		if ($ville_client != null && $c->$ville_client != null) { $c->ville = $c->$ville_client; }
+
 		foreach(['ville','code_postal','adresse'] as $v)
 		{
 			if($c->$v == '')
 			{
-				$c->$v = '[A RENSEIGNER DANS LA FICHE MEMBRE]';
+				$c->$v = '[À RENSEIGNER DANS LA FICHE MEMBRE]';
 			}
 		}
 	}
 	else
 	{
 		$c = $client->get($f->receveur_id);
-		$c->$id_field = $c->nom;
+		$nom_client = $c->nom;
 	}
 }
 catch(UserException $e)
@@ -61,14 +77,20 @@ if ($f->type_facture != CERFA)
 	switch ($f->type_facture)
 	{
 		case FACT:
-			$doc = 'Facture n°'. $f->numero;
+		$doc = 'Facture n° '. $f->numero;
-			break;		
+        $txtemis = $doc . " - Émise le " . $emission;
+        $txtdest = "Adressée à :";
+		break;
 		case DEVIS:
-			$doc = 'Devis n°'. $f->numero;
+		$doc = 'Devis n° '. $f->numero;
-			break;		
+        $txtemis = $doc . " - Émis le " . $emission;
+        $txtdest = "Adressé à :";
+		break;
 		case COTIS:
-			$doc = 'Reçu de cotisation n°'. $f->numero;
+		$doc = 'Reçu de cotisation n° '. $f->numero;
-			break;		
+        $txtemis = $doc . " - Émis le " . $emission;
+        $txtdest = "Adressé à :";
+		break;
 	}
 
     // utiliser l'adresse configurée dans le plugin sinon celle de l'asso sinon rien !
@@ -90,20 +112,25 @@ if ($f->type_facture != CERFA)
         $adresse = "";
     }
 
+	$logo='';
+	if ($plugin->getConfig('logo')) {
+		$logo = '<img id="logo" src="' . $config->fileURL('logo') . '" />';
+	}
 	$asso =
 		// 'Émis par :<br><br>'.
 		'<b>'.$config->get('org_name')."</b><br>".
 		$adresse ."<br>".
 		(($t = $plugin->getConfig('rna_asso'))?"RNA : $t<br>":'').
-		(($t = $plugin->getConfig('siret_asso'))?"SIRET : $t<br>":'').
+        (($t = $plugin->getConfig('siret_asso'))?"SIRET : " . implode(' ', str_split($t, 3)) . "<br>":'').
 		(($t = $config->get('email_asso'))?"Email : $t<br>":'').
 		(($t = $config->get('site_asso'))?"Site web : $t<br>":'');
 
 	$receveur =
-		'Adressé à :<br><br>'.
+		$txtdest.'<br>'.
-		'<b>'.$c->nom.'</b><br>'.
+		'<b>'.$nom_client.'</b><br>'.
 		$c->adresse."<br>".
 		$c->code_postal.' '.$c->ville."<br>".
+        (($t = $c->siret)?"SIREN/SIRET : " . implode(' ', str_split($t, 3)) . "<br>":'').
 		(($t = $c->email)?"Email : $t<br>":'').
 		(($t = $c->telephone)?"Tel : $t<br>":'');
 
@@ -113,7 +140,11 @@ if ($f->type_facture != CERFA)
 	if ($f->type_facture != COTIS)
 	{
 		$echeance = ($f->type_facture?'Échéance de paiement':'Échéance du devis')." : ".$echeance;
-		$reglee = !$f->reglee?'Cette facture est en attente de règlement.':'Cette facture a été reglée.';
+		if ($f->type_facture == FACT) {
+			$reglee = !$f->reglee?'Cette facture est en attente de règlement.':'Cette facture a été réglée.';
+		} else {
+			$reglee = "";
+		}
 		$footer = str_replace("\n", '<br>', $plugin->getConfig('footer') ?? '[Pied de page à configurer]');
 		$ttc = $plugin->getConfig('ttc') ? 'TTC':'HT';
 
@@ -166,12 +197,17 @@ EOF;
     $echeance <br>
     $reglee
     Moyen de paiement : $moyen_paiement
-
     <p>
 	$footer
     </p>
 </footer>
+
 EOF;
+	if ($f->type_facture == DEVIS) {
+		echo <<<EOF
+		<p><b>Bon pour accord, date et signature<b></p>
+EOF;
+	}
 
 		$content = ob_get_clean();
 
@@ -263,6 +299,13 @@ EOF;
 				width: 40%;
 				vertical-align: top;
 			}
+			.adressage td#logo {
+				width: 20%;
+				vertical-align: top;
+			}
+			.adressage img#logo {
+				height : 3cm;
+			}
 
 			.contenuTexte {
 				padding: 0 6mm;
@@ -309,11 +352,12 @@ EOF;
 	</head>
 	<body>
 		<p class="titre">
-		$doc - Émis le $emission
+		$txtemis
 		</p>
 
 		<table class="adressage">
 			<tr>
+				<td id="logo">$logo</td>
 				<td>$asso</td>
 				<td>$receveur</td>
 			</tr>
@@ -344,7 +388,7 @@ elseif ($f->type_facture == CERFA)
 	$t['objet1'] = $plugin->getConfig('objet_1');
 	$t['objet2'] = $plugin->getConfig('objet_2');
 
-	$t['nom'] = $c->nom;
+	$t['nom'] = $nom_client;
 	$t['adresse'] = $c->adresse;
 	$t['cp'] = $c->code_postal;
 	$t['ville'] = $c->ville;
@@ -515,7 +559,7 @@ if(qg('d') !== null)
 {
 	$filename = 'Print';
 	if (preg_match('!<title>(.*)</title>!U', $html, $match)) {
-		$filename = trim($match[1]);
+		$filename = str_replace(" ", "_", trim($match[1]));
 	}
 
 	header('Content-type: application/pdf');

data/schema.sql

@@ -21,7 +21,7 @@ CREATE TABLE IF NOT EXISTS plugin_facturation_clients (
 	adresse				TEXT NOT NULL,
 	code_postal			TEXT NOT NULL,
 	ville				TEXT NOT NULL,
-	-- date_creation		INTEGER NOT NULL,
+    siret               TEXT,
 	date_creation		TEXT NOT NULL DEFAULT CURRENT_DATE CHECK (date(date_creation) IS NOT NULL AND date(date_creation) = date_creation), -- Date d\'inscription
 	telephone			TEXT,
 	email				TEXT

lib/Client.php

@@ -15,6 +15,7 @@ class Client
 		'adresse',
 		'code_postal',
 		'ville',
+        'siret',
 		'telephone',
 		'email'
 	];
@@ -39,7 +40,7 @@ class Client
 			{
 				$data[$key] = trim($data[$key]);
 
-				if($data[$key] == '' && ($key != 'telephone' && $key != 'email'))
+				if($data[$key] == '' && ($key != 'siret' && $key != 'telephone' && $key != 'email'))
 				{
 					throw new UserException('Le champs '.$key.' doit être renseigné.');
 				}
@@ -55,6 +56,10 @@ class Client
 						throw new UserException('Le code postal est erroné.');
 					}
 				}
+                elseif ($key == "siret")
+                {
+                    $data[$key] = str_replace(' ', '', $data[$key]);
+                }
 				elseif ($key == "telephone")
 				{
 					$data[$key] = Utils::normalizePhoneNumber($data[$key]);
@@ -84,7 +89,7 @@ class Client
 
 		if($this->config['unique_client_name'] && isset($data['nom']) && $db->test('plugin_facturation_clients', 'nom = ? COLLATE NOCASE', $data['nom']))
 		{
-			throw new UserException('La valeur du champ nom est déjà utilisée, hors ce champ doit être unique à chaque client.');
+			throw new UserException('La valeur du champ nom est déjà utilisée, or ce champ doit être unique à chaque client.');
 		}
 
 		$db->insert('plugin_facturation_clients', $data);
@@ -122,6 +127,9 @@ class Client
 			'ville' => [
 				'label' => 'Ville',
 			],
+            'siret' => [
+                'label' => 'SIRET',
+            ],
 			'telephone' => [
 				'label' => 'Téléphone',
 			],
@@ -150,7 +158,7 @@ class Client
 
 		if($this->config['unique_client_name'] && isset($data['nom']) && $db->test('plugin_facturation_clients', 'nom = ? COLLATE NOCASE AND id != ?', $data['nom'], (int)$id))
 		{
-			throw new UserException('La valeur du champ nom est déjà utilisée, hors ce champ doit être unique à chaque client.');
+			throw new UserException('La valeur du champ nom est déjà utilisée, or ce champ doit être unique à chaque client.');
 		}
 
 		return $db->update('plugin_facturation_clients', $data, $db->where('id', (int)$id));

lib/Facture.php

@@ -34,27 +34,27 @@ class Facture
 	];
 
 	public $types = [
-			DEVIS => [
+		DEVIS => [
-				'id' => DEVIS,
+			'id' => DEVIS,
-				'accounts' => [],
+			'accounts' => [],
-				'label' => 'Devis',
+			'label' => 'Devis',
-				'help' => ''],
+			'help' => ''],
-			FACT => [
+		FACT => [
-				'id' => FACT,
+			'id' => FACT,
-				'accounts' => [],
+			'accounts' => [],
-				'label' => 'Facture',
+			'label' => 'Facture',
-				'help' => ''],
+			'help' => ''],
-			CERFA => [
+		CERFA => [
-				'id' => CERFA,
+			'id' => CERFA,
-				'accounts' => [],
+			'accounts' => [],
-				'label' => 'Reçu fiscal',
+			'label' => 'Reçu fiscal',
-				'help' => 'Reçu fiscal pour un don (membre ou client)'],
+			'help' => 'Reçu fiscal pour un don (membre ou client)'],
-			COTIS => [
+		COTIS => [
-				'id' => COTIS,
+			'id' => COTIS,
-				'accounts' => [],
+			'accounts' => [],
-				'label' => 'Reçu de cotisation',
+			'label' => 'Reçu de cotisation',
-				'help' => 'Reçu pour une cotisation payée par un·e membre'],
+			'help' => 'Reçu pour une cotisation payée par un·e membre'],
-		];
+	];
 
 	public function __construct()
 	{
@@ -83,108 +83,108 @@ class Facture
 			switch($k)
 			{
 				case 'type_facture':
-					if (!array_key_exists($datas[$k], $this->types)) {
+				if (!array_key_exists($datas[$k], $this->types)) {
-						throw new UserException("$k est de type non-attendue ($data).");
+					throw new UserException("$k est de type non-attendue ($data).");
-					}
+				}
-					if ($datas[$k] < 2) {
+				if ($datas[$k] < 2) {
-						$fac = true;
+					$fac = true;
-						$cerfa = false;
+					$cerfa = false;
-						$recu = false;
+					$recu = false;
-					}
+				}
-					elseif ($datas[$k] == 2) {
+				elseif ($datas[$k] == 2) {
-						$fac = false;
+					$fac = false;
-						$cerfa = true;
+					$cerfa = true;
-						$recu = false;
+					$recu = false;
-					}
+				}
-					elseif ($datas[$k] == 3) {
+				elseif ($datas[$k] == 3) {
-						$fac = false;
+					$fac = false;
-						$cerfa = false;
+					$cerfa = false;
-						$recu = true;
+					$recu = true;
-					}
+				}
-					break;
+				break;
 				case 'receveur_membre':
 				case 'reglee':
 				case 'archivee':
-					if ($datas[$k] != 1 && $datas[$k] != 0) {
+				if ($datas[$k] != 1 && $datas[$k] != 0) {
-						throw new UserException("$k est de valeur non-attendue ($data).");
+					throw new UserException("$k est de valeur non-attendue ($data).");
-					}
+				}
-					break;
+				break;
 				case 'receveur_id':
-					if (!is_numeric($datas[$k]) || $datas[$k] < 0) {
+				if (!is_numeric($datas[$k]) || $datas[$k] < 0) {
-						throw new UserException("L'id du receveur est non-attendu ($data).");
+					throw new UserException("L'id du receveur est non-attendu ($data).");
-					}
+				}
-					break;
+				break;
 				case 'date_emission':
-					$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
+				$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
-					break;
+				break;
 				case 'date_echeance':
-					$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
+				$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
-					if (DateTime::createFromFormat('!Y-m-d', $datas[$k])->format('U') < DateTime::createFromFormat('!Y-m-d', $datas['date_emission'])->format('U'))
+				if (DateTime::createFromFormat('!Y-m-d', $datas[$k])->format('U') < DateTime::createFromFormat('!Y-m-d', $datas['date_emission'])->format('U'))
-					{
+				{
-						throw new UserException("La date d'échéance est antérieure à la date d'émission ($data).");
+					throw new UserException("La date d'échéance est antérieure à la date d'émission ($data).");
-					}
+				}
-					break;
+				break;
 				case 'moyen_paiement':
-					if (!array_key_exists($datas[$k], $this->listMoyensPaiement())) {
+				if (!array_key_exists($datas[$k], $this->listMoyensPaiement())) {
-						throw new UserException("Le moyen de paiement ne correspond pas à la liste interne ($data).");
+					throw new UserException("Le moyen de paiement ne correspond pas à la liste interne ($data).");
-					}
+				}
-					break;
+				break;
 				case 'contenu':
-					if ($fac)
+				if ($fac)
+				{
+					if (!is_array($datas[$k]) || empty($datas[$k])) {
+						throw new UserException("Le contenu du document est vide ($data).");
+					}
+					$total = 0;
+					foreach($datas[$k] as $g => $r)
 					{
-						if (!is_array($datas[$k]) || empty($datas[$k])) {
+						if (empty($r['designation']) && empty($r['prix']))
-							throw new UserException("Le contenu du document est vide ($data).");
-						}
-						$total = 0;
-						foreach($datas[$k] as $g => $r)
 						{
-							if (empty($r['designation']) && empty($r['prix']))
+							unset($datas[$k][$g]);
-							{
+							unset($datas[$k]['prix']);
-								unset($datas[$k][$g]);
+							continue;
-								unset($datas[$k]['prix']);
-								continue;
-							}
-							elseif (empty($r['prix']))
-							{
-								$datas[$k]['prix'] = 0;
-							}
-							
-							if (!is_int($r['prix']))
-							{
-								throw new UserException('Un (ou plus) des prix n\'est pas un entier.');
-							}
-
-							$total += $r['prix']; 
 						}
-
+						elseif (empty($r['prix']))
-						if($fac && !$total)
 						{
-							throw new UserException("Toutes les désignations/prix sont vides.");
+							$datas[$k]['prix'] = 0;
 						}
-					}
-					elseif ($cerfa)
-					{
 
+						if (!is_int($r['prix']))
+						{
+							throw new UserException('Un (ou plus) des prix n\'est pas un entier.');
+						}
+
+						$total += $r['prix'];
 					}
-					elseif ($recu)
+
+					if($fac && !$total)
 					{
-						// $fields = ['id', 'intitule', 'date', 'expiration'];
+						throw new UserException("Toutes les désignations/prix sont vides.");
-						// foreach ($datas[$k]as $)
 					}
-					$datas[$k] = json_encode($datas[$k]);
+				}
-					break;
+				elseif ($cerfa)
+				{
+
+				}
+				elseif ($recu)
+				{
+					// $fields = ['id', 'intitule', 'date', 'expiration'];
+					// foreach ($datas[$k]as $)
+				}
+				$datas[$k] = json_encode($datas[$k]);
+				break;
 				case 'total':
-					if ($cerfa && $datas[$k] < 1) {
+				if ($cerfa && $datas[$k] < 1) {
-						throw new UserException('Le total ne peut être inférieur à 1€ pour les reçus (bug encore non résolu).');
+					throw new UserException('Le total ne peut être inférieur à 1€ pour les reçus (bug encore non résolu).');
-					}
+				}
-					if ($fac && !isset($datas['contenu'])) {
+				if ($fac && !isset($datas['contenu'])) {
-						throw new UserException("Pas de contenu fourni pour vérifier le total.");
+					throw new UserException("Pas de contenu fourni pour vérifier le total.");
-					}
+				}
-					if ($fac && $total != $datas[$k])
+				if ($fac && $total != $datas[$k])
-					{
+				{
-						throw new UserException("Les totaux sont différents ($total != $datas[$k].");
+					throw new UserException("Les totaux sont différents ($total != $datas[$k].");
-					}
+				}
-					break;
+				break;
 			}
 		}
 	}
@@ -325,6 +325,14 @@ class Facture
 	public function list(): DynamicList
 	{
 		$id_field = \Paheko\Users\DynamicFields::getNameFieldsSQL('u');
+		$plugin_name = preg_replace('/^.*\/(\w+)\/$/', '${1}', \Paheko\PLUGIN_ADMIN_URL);
+		$plugin = \Paheko\Plugins::get($plugin_name);
+
+		// adresse et ville peuvent être redéfinies dans la configuration du plugin
+		$adresse_client = $plugin->getConfig('adresse_client');
+		if ($adresse_client == null) { $adresse = 'u.adresse'; } else { $adresse = 'u.' . $adresse_client; }
+		$ville_client = $plugin->getConfig('ville_client');
+		if ($ville_client == null) { $ville = 'u.ville'; } else { $ville = 'u.' . $ville_client; }
 
 		$columns = [
 			// Sélectionner cette colonne, mais ne pas la mettre dans la liste des colonnes
@@ -349,15 +357,18 @@ class Facture
 			],
 			'receveur' => [
 				'label' => 'Receveur',
-				'select' => sprintf('CASE WHEN receveur_membre THEN %s ELSE c.nom END', $id_field),
+				// l'identité du membre peut être redéfinie dans la configuration des membres
+				'select' => sprintf('CASE WHEN receveur_membre THEN CASE %s WHEN "" THEN "** ABSENT **" ELSE %s END ELSE c.nom END', $id_field, $id_field),
 			],
 			'receveur_adresse' => [
-				'label' => 'Son adresse',
+				// l'adresse peut être redéfinie dans la configuration du plugin
-				'select' => 'CASE WHEN receveur_membre THEN u.adresse ELSE c.adresse END',
+				'label' => 'Adresse',
+				'select' => sprintf('CASE WHEN receveur_membre THEN %s ELSE c.adresse END', $adresse),
 			],
 			'receveur_ville' => [
-				'label' => 'Sa ville',
+				// la ville peut être redéfinie dans la configuration du plugin
-				'select' => 'CASE WHEN receveur_membre THEN u.ville ELSE c.ville END',
+				'label' => 'Ville',
+				'select' => sprintf('CASE WHEN receveur_membre THEN %s ELSE c.ville END', $ville),
 			],
 			'date_emission' => [
 				'label' => 'Émission',
@@ -391,6 +402,7 @@ class Facture
 
 		$list = new DynamicList($columns, $tables);
 		$list->orderBy('date_emission', true);
+        $list->setCount('COUNT(f.id)');
 
 		$currency = Config::getInstance()->monnaie;
 
@@ -405,8 +417,8 @@ class Facture
 
 			if ($row->type_facture == COTIS && isset($content->intitule, $content->souscription)) {
 				$row->contenu = sprintf("Cotisation %s\nSouscrite le %s",
-					$content->intitule,
+										$content->intitule,
-					Utils::date_fr($content->souscription, 'd/m/Y')
+										Utils::date_fr($content->souscription, 'd/m/Y')
 				);
 			}
 			elseif ($row->type_facture != CERFA) {
@@ -420,7 +432,6 @@ class Facture
 			}
 		});
 
-		$list->setPageSize(1000);
 		return $list;
 	}
 
@@ -432,7 +443,7 @@ class Facture
 
 		if(isset($data['numero']) && $db->test('plugin_facturation_factures', 'numero = ? COLLATE NOCASE AND id != ?', $data['numero'], (int)$id))
 		{
-			throw new UserException('Un document avec ce numéro existe déjà, hors le numéro doit être unique.');
+			throw new UserException('Un document avec ce numéro existe déjà, or le numéro doit être unique.');
 		}
 		return $db->update('plugin_facturation_factures', $data, $db->where('id', (int)$id));
 	}
@@ -506,7 +517,7 @@ class Facture
 			LEFT JOIN services_fees sf ON sf.id = su.id_fee
 			LEFT JOIN acc_transactions_users tu ON tu.id_service_user = su.id
 			LEFT JOIN acc_transactions_lines tl ON tl.id_transaction = tu.id_transaction
-			'.$where.'
+		'.$where.'
 			GROUP BY su.id
 			ORDER BY su.date;';
 

plugin.ini

@@ -1,8 +1,8 @@
 name="Facturation"
 description="Permet d'éditer des factures, devis et reçus à ses membres ainsi qu'à une base de clients supplémentaire."
-author="zou"
+author="zou ; adapté par jce"
 url="https://git.roflcopter.fr/lesanges/paheko-plugin-facturation"
-version="0.8.2"
+version="0.11"
 menu=true
 restrict_section="accounting"
 restrict_level="read"

templates/_form.tpl

@@ -28,7 +28,7 @@
 		<legend data-types="t3">Créer un reçu de cotisation</legend>
 		<dl>
 
-			{input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=$require_number source=$doc}
+			{input type="text" name="numero_facture" maxlength=18 label="Numéro du document" required=$require_number source=$doc}
 
 			<dd class="help">
 				{if $require_number}

templates/_style.css

@@ -7,4 +7,15 @@
 	display: none;
 }
 
+div.aide {
+	margin: 1em;
+	padding: 1em;
+}
+
+div.aide ul {
+	list-style: square;
+	margin: 1em;
+	padding: 1em;
+}
+
 {/literal}

templates/aide.tpl

@@ -1,21 +1,30 @@
 {include file="_head.tpl" title="Aide — %s"|args:$plugin.name current="plugin_%s"|args:$plugin.id}
 {include file="%s/templates/_menu.tpl"|args:$plugin_root current="aide"}
 
-<fieldset>
+<h2>Quelques remarques et conseils sur l'utilisation du plugin Facturation</h2>
-<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>
+<div class="aide">
-    <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 Paheko qui fait foi plutôt que celle dans le plugin (qui sert en revanche pour les reçus).</p>
+	<p>Une des premières choses à faire est d'aller dans l'onglet {link href="config.php" label="Configuration du plugin"} pour renseigner les valeurs nécessaires à la génération des documents.</p>
-    <p>Pensez à mettre une image en signature (cela sert pour les reçus fiscaux), cela se passe dans la configuration de Paheko, onglet personnalisation. Il est préférable d'avoir un fond transparent.
+	<fieldset>
-</p>
+		<legend>Informations à configurer</legend>
-    <br>
+		<dl>
-    <p>- Pour créer un reçu sur une cotisation, il faut pour le moment que cette cotisation soit attachée à la compta.</p>
+			<dt><label>Adresse</label></dt>
-    <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>
+			<dd>Si elle n'est pas remplie, c'est l'adresse qui figure dans la {link href="!config" label="configuration de l'association"} qui sera utilisée</dd>
-    <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>
+			<dt><label>Nom et prénom des membres</label></dt>
-    <br>
+			<dd>Les champs qui définissent l'identité d'un membre peuvent être choisis dans la {link href="!config/users" label="configuration de la fiche membre"} </dd>
-    <p>Hésitez pas à faire des retours, proposer meilleures explications, ou quoi, vous pouvez venir en causer soit <a href="https://gitlab.com/ramoloss/Paheko-plugin-facturation">sur mon gitlab</a>, soit sur l'adresse d'entraide de Paheko. 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>
+			<dt><label>Champs à faire figurer sur la facture</label></dt>
+			<dd>Permet de choisir les champs pour l'adresse, le code postal et la ville parmi les {link href="!config/fields" label="champs de la fiche membre"}</dd>
+		</dl>
+	</fieldset>
 
-</fieldset>
+	<p>Pensez à mettre une image en signature (cela sert pour les reçus fiscaux), cela se passe dans la {link href="!config/custom.php" label="configuration de Paheko, onglet personnalisation"}. Il est préférable d'avoir un fond transparent.
+	</p>
+	<ul>
+		<li>Pour créer un reçu sur une cotisation, il vaut mieux utiliser le module {link href="!config/ext" label="Reçu de paiement"} intégré à Paheko.</li>
+		<li>Pour créer un reçu fiscal, il vaut mieux utiliser le module {link href="!config/ext" label="Reçus fiscaux"} intégré à Paheko.</li>
+	</ul>
+	<p>N'hésitez pas à faire des retours, proposer de meilleures explications dans le forum d'entraide de Paheko. Si vous êtes un peu dev, le code est un peu cracra mais j'espère que ça ne vous repoussera pas trop à le bidouiller :)</p>
 
+</div>
 
 {include file="_foot.tpl"}

templates/client.tpl

@@ -18,6 +18,18 @@
     <dt>Code postal</dt>
     <dd>{$client.code_postal|escape|rtrim|nl2br}</dd>
 
+	<dt>SIREN/SIRET</dt>
+	<dd>
+	{if empty($client->siret)}
+        <em>(Non renseigné)</em>
+    {else}
+	<?php
+		$siret = implode(' ', str_split($client->siret, 3));
+	?>
+	{$siret|escape|trim}
+	{/if}
+	</dd>
+
     <dt>Adresse électronique</dt>
     <dd>
     {if empty($client.email)}
@@ -32,7 +44,7 @@
     {if empty($client.telephone)}
         <em>(Non renseigné)</em>
     {else}
-        <a href="tel:{$client.telephone}">{$client.telephone}</a>
+        <a href="tel:{$client.telephone}">{$client.telephone|format_phone_number}</a>
     {/if}
     </dd>
 

templates/client_modifier.tpl

@@ -11,6 +11,7 @@
             {input type="text" name="adresse" label="Adresse" required=true source=$client}
             {input type="text" name="code_postal" label="Code postal" required=true source=$client}
             {input type="text" name="ville" label="Ville" required=true source=$client}
+            {input type="text" name="siret" label="SIREN/SIRET" source=$client}
             {input type="tel" name="telephone" label="Téléphone" source=$client}
             {input type="email" name="email" label="Adresse e-mail" source=$client}
         </dl>

templates/clients.tpl

@@ -13,6 +13,12 @@
                     {if $key == 'id' || $key == 'nom'}
                         <?php continue; ?>
                     {/if}
+					{if $key == 'siret'}
+						<?php
+							if (null === $value) { $value = ""; }
+							$value = implode(' ', str_split($value, 3));
+						?>
+					{/if}
                     <td>{$value}</td>
                 {/foreach}
                 <td class="actions">
@@ -52,6 +58,7 @@
             {input type="text" name="adresse" label="Adresse" required=true}
             {input type="text" name="code_postal" label="Code postal" required=true}
             {input type="text" name="ville" label="Ville" required=true}
+            {input type="text" name="siret" label="SIREN/SIRET"}
             {input type="tel" name="telephone" label="Téléphone"}
             {input type="email" name="email" label="Adresse e-mail"}
         </dl>

templates/config.tpl

@@ -27,33 +27,44 @@
                 {input type="text" name="ville_asso" source=$conf label="Ville"}
             </dl>
         </fieldset>
-            <fieldset>
+		<fieldset>
-                <legend>Objet</legend>
+			<legend>Objet</legend>
-                <dl>
+			<dl>
-                    <dt><label>L'objet (but) de l'association doit tenir sur 3 lignes, chaque ligne pouvant accueillir un maximum de 100 caractères.</label><b title="(Champ obligatoire)">obligatoire pour reçus fiscaux</b></dt>
+				<dt><label>L'objet (but) de l'association doit tenir sur 3 lignes, chaque ligne pouvant accueillir un maximum de 100 caractères.</label><b title="(Champ obligatoire)">obligatoire pour reçus fiscaux</b></dt>
-                    {input type="text" name="objet_0" source=$conf label="Ligne 1" maxlength=95}
+				{input type="text" name="objet_0" source=$conf label="Ligne 1" maxlength=95}
-                    {input type="text" name="objet_1" source=$conf label="Ligne 2" maxlength=95}
+				{input type="text" name="objet_1" source=$conf label="Ligne 2" maxlength=95}
-                    {input type="text" name="objet_2" source=$conf label="Ligne 3" maxlength=95}
+				{input type="text" name="objet_2" source=$conf label="Ligne 3" maxlength=95}
-                </dl>
+			</dl>
-            </fieldset>
+		</fieldset>
 
-            <fieldset>
+		<fieldset>
-                <legend>Droit à la réduction d'impôt</legend>
+			<legend>Droit à la réduction d'impôt</legend>
-                <dl>
+			<dl>
-                    <dt><label>Articles concernés par l'association :</label> <b title="(Champ obligatoire)">obligatoire pour reçus fiscaux</b></dt>
+				<dt><label>Articles concernés par l'association :</label> <b title="(Champ obligatoire)">obligatoire pour reçus fiscaux</b></dt>
-                    {input type="checkbox" name="droit_art200" value="1" source=$conf label="Article 200"}
+				{input type="checkbox" name="droit_art200" value="1" source=$conf label="Article 200"}
-                    {input type="checkbox" name="droit_art238bis" value="1" source=$conf label="Article 238 bis"}
+				{input type="checkbox" name="droit_art238bis" value="1" source=$conf label="Article 238 bis"}
-                    {input type="checkbox" name="droit_art885_0VbisA" value="1" source=$conf label="Article 885-0V bis A"}
+				{input type="checkbox" name="droit_art885_0VbisA" value="1" source=$conf label="Article 885-0V bis A"}
-                </dl>
+			</dl>
-            </fieldset>
+		</fieldset>
 
     </fieldset>
 
     <fieldset>
         <legend>Factures</legend>
         <dl>
-            {input type="textarea" class="full-width" rows="10" name="footer" source=$conf label="Pied de document — informations légales" required=true}
+		    {input type="checkbox" name="logo" value="1" source=$conf label="Imprimer le logo de l'association"}
+            {input type="textarea" class="full-width" rows="5" name="footer" source=$conf label="Pied de document — informations légales" required=true}
         </dl>
+		<fieldset>
+			<legend>
+				Choisir les champs à faire figurer sur la facture
+			</legend>
+			<dl>
+				{input type="select" name="adresse_client" label="Adresse" required=true options=$champsPaheko source=$conf}
+				{input type="select" name="code_postal_client" label="Code postal" required=true options=$champsPaheko source=$conf}
+				{input type="select" name="ville_client" label="Ville" required=true options=$champsPaheko source=$conf}
+			</dl>
+		</fieldset>
     </fieldset>
 
     <fieldset>

templates/index.tpl

@@ -33,6 +33,7 @@
 
     </tbody>
 </table>
+{$list->getHTMLPagination()|raw}
 
 <p class="help">
     Export de la liste&nbsp;:

upgrade.php

@@ -209,3 +209,37 @@ if (version_compare($old_version, '0.8.1', '<'))
 {
 	$plugin->unregisterSignal('menu.item');
 }
+
+// 0.8.5 Ajout champs SIREN/SIRET à la table clients
+
+if (version_compare($old_version, '0.8.5', '<'))
+{
+    $db->exec(<<<EOT
+              CREATE TABLE IF NOT EXISTS plugin_facturation_clients_tmp
+              (
+                  id				INTEGER PRIMARY KEY,
+                  nom				TEXT NOT NULL,
+                  adresse			TEXT NOT NULL,
+                  code_postal		TEXT NOT NULL,
+                  ville				TEXT NOT NULL,
+                  siret             TEXT,
+                  date_creation		TEXT NOT NULL DEFAULT CURRENT_DATE CHECK (date(date_creation) IS NOT NULL AND date(date_creation) = date_creation),
+                  telephone			TEXT,
+                  email				TEXT
+              );
+              EOT
+    );
+    // copier les clients dans la table temporaire en ajoutant un siret fictif
+    $sql = 'SELECT * FROM plugin_facturation_clients';
+    foreach ($db->iterate($sql) as $client)
+    {
+        $db->insert('plugin_facturation_clients_tmp', $client);
+    }
+    // remplacer l'ancienne table par la nouvelle
+    $db->exec(<<<EOT
+              DROP TABLE plugin_facturation_clients;
+              ALTER TABLE plugin_facturation_clients_tmp RENAME TO plugin_facturation_clients;
+              EOT
+    );
+}
+