màj CHANGELOG, README et plugin.ini

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

CHANGELOG

@@ -1,3 +1,9 @@
+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

README.md

@@ -1,55 +1,49 @@
 # Plugin Facturation pour Paheko (ex Garradin)
 
-Plugin de facturation pour le logiciel de gestion d'association Paheko ( https://paheko.eu/ - https://fossil.kd2.org/paheko ).
+Plugin de facturation pour le logiciel de gestion d'association Paheko
+( https://paheko.eu/ - https://fossil.kd2.org/paheko ).
+
 Source :
-- version historique : https://gitlab.com/noizette/garradin-plugin-facturation
 - 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
 
-## Installation:
-Vous pouvez télécharger l'archive .tar.gz depuis la page des [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 le dossier plugins de Paheko.
-
-### Anciennes versions (<0.6)
-
-Normalement, les plugins de Paheko 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.
-
-Il faut pour cela faire :
-
-    tar xvf paheko-plugin-facturation-v0.5.0.tar.gz
-    mv paheko-plugin-facturation-v0.5.0 facturation
-    rm paheko-plugin-facturation-v0.5.0.tar.gz
-    chown -R www-data:www-data facturation/
-    chmod -R g+w facturation/ 
-
-
-Supprimer l'archive permet à Paheko 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 (obsolète)
-
-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.
+## Installation :
+Vous pouvez télécharger l'archive .tar.gz depuis la page des
+[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
+le dossier plugins de Paheko.
 
 ## 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
-- Créer des reçus sur des cotisations
-- Génération des documents (facture et devis) en PDF grâce à la librairie mPDF
+- (obsolète) Créer des reçus fiscaux pour des dons et génération du CERFA correspondant
+- (obsolète) Créer des reçus sur des cotisations
+- 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
-    - Informations relatives au cerfa pour les reçus fiscaux
-    - Image qui set de signature sur le cerfa
+    - 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
+    - (obsolète) Informations relatives au CERFA pour les reçus fiscaux
+    - (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é
@@ -59,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 {
@@ -301,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/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('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('logo', (bool)f('logo'));
 	$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/pdf.php

@@ -24,17 +24,36 @@ try
 	if ($f->receveur_membre)
 	{
 		$c = $users->get($f->receveur_id);
+		// 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);
+		$nom_client = $c->nom;
 	}
 }
 catch(UserException $e)
@@ -58,20 +77,20 @@ if ($f->type_facture != CERFA)
 	switch ($f->type_facture)
 	{
 		case FACT:
-			$doc = 'Facture n° '. $f->numero;
-            $txtemis = $doc . " - Émise le " . $emission;
-            $txtdest = "Adressée à :";
-			break;		
+		$doc = 'Facture n° '. $f->numero;
+        $txtemis = $doc . " - Émise le " . $emission;
+        $txtdest = "Adressée à :";
+		break;
 		case DEVIS:
-			$doc = 'Devis n° '. $f->numero;
-            $txtemis = $doc . " - Émis le " . $emission;
-            $txtdest = "Adressé à :";
-			break;		
+		$doc = 'Devis n° '. $f->numero;
+        $txtemis = $doc . " - Émis le " . $emission;
+        $txtdest = "Adressé à :";
+		break;
 		case COTIS:
-			$doc = 'Reçu de cotisation n° '. $f->numero;
-            $txtemis = $doc . " - Émis le " . $emission;
-            $txtdest = "Adressé à :";
-			break;		
+		$doc = 'Reçu de cotisation n° '. $f->numero;
+        $txtemis = $doc . " - Émis le " . $emission;
+        $txtdest = "Adressé à :";
+		break;
 	}
 
     // utiliser l'adresse configurée dans le plugin sinon celle de l'asso sinon rien !
@@ -93,6 +112,10 @@ 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>".
@@ -104,7 +127,7 @@ if ($f->type_facture != CERFA)
 
 	$receveur =
 		$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>":'').
@@ -117,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é réglé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';
 
@@ -170,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();
 
@@ -267,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;
@@ -318,6 +357,7 @@ EOF;
 
 		<table class="adressage">
 			<tr>
+				<td id="logo">$logo</td>
 				<td>$asso</td>
 				<td>$receveur</td>
 			</tr>
@@ -348,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;
@@ -519,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');

lib/Facture.php

@@ -34,27 +34,27 @@ class Facture
 	];
 
 	public $types = [
-			DEVIS => [
-				'id' => DEVIS,
-				'accounts' => [],
-				'label' => 'Devis',
-				'help' => ''],
-			FACT => [
-				'id' => FACT,
-				'accounts' => [],
-				'label' => 'Facture',
-				'help' => ''],
-			CERFA => [
-				'id' => CERFA,
-				'accounts' => [],
-				'label' => 'Reçu fiscal',
-				'help' => 'Reçu fiscal pour un don (membre ou client)'],
-			COTIS => [
-				'id' => COTIS,
-				'accounts' => [],
-				'label' => 'Reçu de cotisation',
-				'help' => 'Reçu pour une cotisation payée par un·e membre'],
-		];
+		DEVIS => [
+			'id' => DEVIS,
+			'accounts' => [],
+			'label' => 'Devis',
+			'help' => ''],
+		FACT => [
+			'id' => FACT,
+			'accounts' => [],
+			'label' => 'Facture',
+			'help' => ''],
+		CERFA => [
+			'id' => CERFA,
+			'accounts' => [],
+			'label' => 'Reçu fiscal',
+			'help' => 'Reçu fiscal pour un don (membre ou client)'],
+		COTIS => [
+			'id' => COTIS,
+			'accounts' => [],
+			'label' => 'Reçu de cotisation',
+			'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)) {
-						throw new UserException("$k est de type non-attendue ($data).");
-					}
-					if ($datas[$k] < 2) {
-						$fac = true;
-						$cerfa = false;
-						$recu = false;
-					}
-					elseif ($datas[$k] == 2) {
-						$fac = false;
-						$cerfa = true;
-						$recu = false;
-					}
-					elseif ($datas[$k] == 3) {
-						$fac = false;
-						$cerfa = false;
-						$recu = true;
-					}
-					break;
+				if (!array_key_exists($datas[$k], $this->types)) {
+					throw new UserException("$k est de type non-attendue ($data).");
+				}
+				if ($datas[$k] < 2) {
+					$fac = true;
+					$cerfa = false;
+					$recu = false;
+				}
+				elseif ($datas[$k] == 2) {
+					$fac = false;
+					$cerfa = true;
+					$recu = false;
+				}
+				elseif ($datas[$k] == 3) {
+					$fac = false;
+					$cerfa = false;
+					$recu = true;
+				}
+				break;
 				case 'receveur_membre':
 				case 'reglee':
 				case 'archivee':
-					if ($datas[$k] != 1 && $datas[$k] != 0) {
-						throw new UserException("$k est de valeur non-attendue ($data).");
-					}
-					break;
+				if ($datas[$k] != 1 && $datas[$k] != 0) {
+					throw new UserException("$k est de valeur non-attendue ($data).");
+				}
+				break;
 				case 'receveur_id':
-					if (!is_numeric($datas[$k]) || $datas[$k] < 0) {
-						throw new UserException("L'id du receveur est non-attendu ($data).");
-					}
-					break;
+				if (!is_numeric($datas[$k]) || $datas[$k] < 0) {
+					throw new UserException("L'id du receveur est non-attendu ($data).");
+				}
+				break;
 				case 'date_emission':
-					$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
-					break;
+				$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
+				break;
 				case 'date_echeance':
-					$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'))
-					{
-						throw new UserException("La date d'échéance est antérieure à la date d'émission ($data).");
-					}
-					break;
+				$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'))
+				{
+					throw new UserException("La date d'échéance est antérieure à la date d'émission ($data).");
+				}
+				break;
 				case 'moyen_paiement':
-					if (!array_key_exists($datas[$k], $this->listMoyensPaiement())) {
-						throw new UserException("Le moyen de paiement ne correspond pas à la liste interne ($data).");
-					}
-					break;
+				if (!array_key_exists($datas[$k], $this->listMoyensPaiement())) {
+					throw new UserException("Le moyen de paiement ne correspond pas à la liste interne ($data).");
+				}
+				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])) {
-							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']))
 						{
-							if (empty($r['designation']) && empty($r['prix']))
-							{
-								unset($datas[$k][$g]);
-								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']; 
+							unset($datas[$k][$g]);
+							unset($datas[$k]['prix']);
+							continue;
 						}
-
-						if($fac && !$total)
+						elseif (empty($r['prix']))
 						{
-							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'];
-						// foreach ($datas[$k]as $)
+						throw new UserException("Toutes les désignations/prix sont vides.");
 					}
-					$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) {
-						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'])) {
-						throw new UserException("Pas de contenu fourni pour vérifier le total.");
-					}
-					if ($fac && $total != $datas[$k])
-					{
-						throw new UserException("Les totaux sont différents ($total != $datas[$k].");
-					}
-					break;
+				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).');
+				}
+				if ($fac && !isset($datas['contenu'])) {
+					throw new UserException("Pas de contenu fourni pour vérifier le total.");
+				}
+				if ($fac && $total != $datas[$k])
+				{
+					throw new UserException("Les totaux sont différents ($total != $datas[$k].");
+				}
+				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' => [
+				// l'adresse peut être redéfinie dans la configuration du plugin
 				'label' => 'Adresse',
-				'select' => 'CASE WHEN receveur_membre THEN u.adresse ELSE c.adresse END',
+				'select' => sprintf('CASE WHEN receveur_membre THEN %s ELSE c.adresse END', $adresse),
 			],
 			'receveur_ville' => [
+				// la ville peut être redéfinie dans la configuration du plugin
 				'label' => 'Ville',
-				'select' => 'CASE WHEN receveur_membre THEN u.ville ELSE c.ville END',
+				'select' => sprintf('CASE WHEN receveur_membre THEN %s ELSE c.ville END', $ville),
 			],
 			'date_emission' => [
 				'label' => 'Émission',
@@ -406,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,
-					Utils::date_fr($content->souscription, 'd/m/Y')
+										$content->intitule,
+										Utils::date_fr($content->souscription, 'd/m/Y')
 				);
 			}
 			elseif ($row->type_facture != CERFA) {
@@ -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

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

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>
-<legend><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>
 
-    <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 Paheko qui fait foi plutôt que celle dans le plugin (qui sert en revanche pour les reçus).</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.
-</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/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>
+<div class="aide">
+	<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>
+	<fieldset>
+		<legend>Informations à configurer</legend>
+		<dl>
+			<dt><label>Adresse</label></dt>
+			<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>
+			<dt><label>Nom et prénom des membres</label></dt>
+			<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>
+			<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/config.tpl

@@ -27,33 +27,44 @@
                 {input type="text" name="ville_asso" source=$conf label="Ville"}
             </dl>
         </fieldset>
-            <fieldset>
-                <legend>Objet</legend>
-                <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>
-                    {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_2" source=$conf label="Ligne 3" maxlength=95}
-                </dl>
-            </fieldset>
+		<fieldset>
+			<legend>Objet</legend>
+			<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>
+				{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_2" source=$conf label="Ligne 3" maxlength=95}
+			</dl>
+		</fieldset>
 
-            <fieldset>
-                <legend>Droit à la réduction d'impôt</legend>
-                <dl>
-                    <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_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"}
-                </dl>
-            </fieldset>
+		<fieldset>
+			<legend>Droit à la réduction d'impôt</legend>
+			<dl>
+				<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_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"}
+			</dl>
+		</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>