Correction erreurs, sécurisation code, personnalisation présentation (Jordan)

CHANGELOG

@@ -1,3 +1,13 @@
+0.25
+- Options d'affichage du rendu facture (Jason)
+0.19
+- Réparation des schémas partiellement migrés (Jason)
+0.18
+- Personnalisation du rendu des factures (Jason)
+0.17
+- Ajout archivage factures
+0.15
+- Ajout champ contact à la fiche client
 0.12
 - Ajout Mollie à la table moyens de paiement
 0.11

admin/_facture_common.php

@@ -4,30 +4,8 @@ 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?
+	throw new \LogicException('Appel interne invalide du formulaire de document.');
 } else {
 	$target = ($target === 'new' ? true:false);
 }
@@ -35,7 +13,6 @@ if (!isset($target) || !in_array( $target, ['new', 'edit'])) {
 $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_WRITE);
 
 use Paheko\DB;
-use stdClass;
 use const \Paheko\Plugin\Facturation\PATTERNS_LIST;
 
 $db = DB::getInstance();
@@ -73,17 +50,14 @@ if (! $target && $f->archivee)
 {
 	throw new UserException("Ce document est archivé, vous n'avez pas le droit de le modifier");
 }
-// Traitement
+// Traitement
 $data=[];
 $form->runIf(f('save') && !$form->hasErrors(),
-             function () use ($client, &$data, $form)
+             function () use (&$data, $form)
     {
         try
         {
-            if ( count(f('designation')) !== count(f('prix')) )
+			$type = (int) f('type');
-            {
-                throw new UserException('Nombre de désignations et de prix reçus différent.');
-            }
 
             $data = [
                 'numero'			=> f('numero_facture'),
@@ -93,37 +67,51 @@ $form->runIf(f('save') && !$form->hasErrors(),
                 'archivee'			=> f('archivee') == 1?1:0,
                 'moyen_paiement'	=> f('moyen_paiement'),
 				'nom_contact'		=> f('nom_contact'),
-                'toto'				=> 0
+				'numero_commande'	=> f('numero_commande') ?: null,
+				'reference_acheteur' => f('reference_acheteur') ?: null,
             ];
-            $data['type_facture'] = f('type');
+
-            if (in_array(f('type'), [DEVIS, FACT]))
+            $data['type_facture'] = $type;
+
+            if (in_array($type, [DEVIS, FACT], true))
             {
-				foreach(f('designation') as $k=>$value)
+				$designations = (array) f('designation');
+				$prix = (array) f('prix');
+
+				if (count($designations) !== count($prix))
 				{
-					if (empty($value) && f('prix')[$k] != null) {
+					throw new UserException('Nombre de désignations et de prix reçus différent.');
-						throw new UserException("Il manque la désignation de la ligne " . $k+1 . " !!");
 				}
-					elseif ($value != '' && f('prix')[$k] == null) {
+
-						throw new UserException('Il manque le prix sur la ligne '. $k+1 . ' !!');
+				$total = 0;
-					} elseif (empty($value) && f('prix')[$k] == null) {
+
+				foreach($designations as $k=>$value)
+                {
+					$prix_ligne = $prix[$k] ?? null;
+
+					if (empty($value) && $prix_ligne !== null && $prix_ligne !== '') {
+						throw new UserException("Il manque la désignation de la ligne " . ($k + 1) . " !!");
+					}
+					elseif ($value != '' && ($prix_ligne === null || $prix_ligne === '')) {
+						throw new UserException('Il manque le prix sur la ligne '. ($k + 1) . ' !!');
+					}
+					elseif (empty($value) && ($prix_ligne === null || $prix_ligne === '')) {
 						continue;
 					}
 
+					$prix_ligne = Utils::moneyToInteger($prix_ligne);
                     $data['contenu'][$k]['designation'] = $value;
-                    $data['contenu'][$k]['prix'] = Utils::moneyToInteger(f('prix')[$k]);
+                    $data['contenu'][$k]['prix'] = $prix_ligne;
-                    $data['toto'] += Utils::moneyToInteger(f('prix')[$k]);
+                    $total += $prix_ligne;
                 }
-                $data['total'] = $data['toto'];
+
-                unset($data['toto']);
+                $data['total'] = $total;
+
 				if (! isset($data['contenu'])) {
 					throw new UserException("Aucune désignation ni aucun prix saisi !!");
 				}
-				if (f('type') == FACT) {
-					$data['numero_commande'] = f('numero_commande');
-					$data['reference_acheteur'] = f('reference_acheteur');
 			}
-			}
+            elseif ($type === CERFA)
-            elseif ( f('type') == CERFA )
             {
                 $data['moyen_paiement'] = f('moyen_paiement_cerfa');
                 $data['contenu'] = [
@@ -131,8 +119,12 @@ $form->runIf(f('save') && !$form->hasErrors(),
                     'nature' => f('nature_don'),
                     'texte' => f('texte_don')];
                 $data['total'] = Utils::moneyToInteger(f('total'));
-                unset($data['toto']);
             }
+            else
+            {
+                throw new UserException('Ce type de document doit être créé avec son formulaire dédié.');
+            }
+
             if (f('base_receveur') == 'client')
             {
                 $data['receveur_membre'] = 0;
@@ -157,18 +149,20 @@ $form->runIf(f('save') && !$form->hasErrors(),
     }, $csrf_key);
 
 $form->runIf(f('select_cotis') && !$form->hasErrors(),
-             function () use ($step)
+             function () use (&$step)
 	{
 		$step = true;
 	}, 'add_cotis_1');
 
 $form->runIf(f('add_cotis') && !$form->hasErrors(),
-             function () use ($radio, $fields, $facture, $form)
+             function () use (&$data, &$radio, $fields, $facture, $form)
 	{
 		$radio['type'] = f('cotisation');
 		try
 		{
 			$num = (int) str_replace('cotis_', '', $radio['type']);
+			$cotis = [];
+
 			foreach($fields as $field)
 			{
 				$cotis[$field] = f($field.'_'.$num);
@@ -183,8 +177,12 @@ $form->runIf(f('add_cotis') && !$form->hasErrors(),
 				'receveur_membre' => 1,
 				'receveur_id' => f('membre_cotis'),
 				'date_emission' => f('date_emission'),
+				'date_echeance' => f('date_echeance') ?: f('date_emission'),
+				'reglee' => 1,
+				'archivee' => 0,
 				'moyen_paiement'	=> 'AU',
 				'total' => $r->paid_amount ?? $r->amount,
+				'nom_contact' => null,
 				'contenu' => ['id' => $cotis['id'],
 							  'intitule' => $cotis['label'],
 							  'souscription' => $cotis['date'],
@@ -264,7 +262,7 @@ if ($target)
 	} // ... s'il est défini dans le document copié
 	elseif (isset($doc['type'])) {
 		$radio['type'] = $doc['type'];
-	} // ... ou par défaut
+	} // ... ou par défaut
 	else
 	{
 		$radio['type'] = FACT;
@@ -284,9 +282,9 @@ else
 
 	$doc['date_emission'] = f('date_emission') ?: $f->date_emission;
 	$doc['date_echeance'] = f('date_echeance')?: $f->date_echeance; // Smarty m'a saoulé pour utiliser form_field|date_fr:---
-	$doc['nom_contact']		= $f->nom_contact;
+	$doc['nom_contact']		= $f->nom_contact ?? null;
-	$doc['numero_commande']	= $f->numero_commande;
+	$doc['numero_commande']	= $f->numero_commande ?? null;
-	$doc['reference_acheteur'] = $f->reference_acheteur;
+	$doc['reference_acheteur'] = $f->reference_acheteur ?? null;
 	/* modif DD -- CERFA -------------------------------------- */
 	if ( $f->type_facture == CERFA ) {
 		$doc['total'] = $f->total;
@@ -346,11 +344,11 @@ $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', 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('users', iterator_to_array(Users\Users::iterateAssocByCategory()));
 $tpl->assign('clients', $db->getAssoc('SELECT id, nom FROM plugin_facturation_clients;'));
 $tpl->assign('contacts', $db->getAssoc('SELECT id, nom_contact FROM plugin_facturation_clients;'));
 $tpl->assign('require_number', $require_number);
-$tpl->assign('number_pattern', PATTERNS_LIST[$plugin->getConfig('pattern')]);
+$tpl->assign('number_pattern', PATTERNS_LIST[$plugin->getConfig('pattern')] ?? PATTERNS_LIST['']);
 
 if ($target) {
 	$tpl->display(PLUGIN_ROOT . '/templates/facture_ajouter.tpl');

admin/_inc.php

@@ -22,6 +22,42 @@ const PATTERNS_LIST = [
     '%06{id}'                      => 'Numéro unique du document sur 6 chiffres ("000042")',
 ];
 
+function ensureSchemaColumns(): void
+{
+	$db = \Paheko\DB::getInstance();
+
+	$tables = [
+		'plugin_facturation_clients' => [
+			'siret'       => 'siret TEXT',
+			'telephone'   => 'telephone TEXT',
+			'email'       => 'email TEXT',
+			'nom_contact' => 'nom_contact TEXT',
+			'note'        => 'note TEXT',
+		],
+		'plugin_facturation_factures' => [
+			'nom_contact'       => 'nom_contact TEXT',
+			'numero_commande'   => 'numero_commande TEXT',
+			'reference_acheteur' => 'reference_acheteur TEXT',
+		],
+	];
+
+	foreach ($tables as $table => $columns) {
+		$schema = $db->getTableSchema($table);
+
+		if (!$schema) {
+			continue;
+		}
+
+		foreach ($columns as $column => $definition) {
+			if (!isset($schema['columns'][$column])) {
+				$db->exec(sprintf('ALTER TABLE %s ADD COLUMN %s;', $db->quoteIdentifier($table), $definition));
+			}
+		}
+	}
+}
+
+ensureSchemaColumns();
+
 $client = new Client;
 $facture = new Facture;
 
@@ -63,7 +99,7 @@ $tpl->register_function('money_fac', function (array $params)
 			$current_value = htmlspecialchars($current_value, ENT_QUOTES, 'UTF-8');
 		}
 
-        $currency = Config::getInstance()->get('monnaie');
+        $currency = Config::getInstance()->get('currency');
         return sprintf('<td><nobr><input type="text" pattern="[0-9]*([.,][0-9]{1,2})?" inputmode="decimal" size="8" class="money" style="width: 60%%" onchange="updateSum();" name="%s" value="%s" /><b>%s</b></nobr></td>', $name, $current_value, $currency);
     }
 );

admin/client_supprimer.php

@@ -12,7 +12,7 @@ $id = (int) qg('id');
 
 $c = $client->get($id);
 
-if (!$client)
+if (!$c)
 {
 	throw new UserException("Ce client n'existe pas.");
 }

admin/config.php

@@ -11,6 +11,17 @@ $champsPaheko = DynamicFields::getInstance()->listAssocNames();
 $champsPaheko = array('' => '- Choisir un champ -') + $champsPaheko;
 
 $form->runIf('save', function () use ($plugin) {
+	$invoice_theme = f('invoice_theme') ?: 'modern';
+	$invoice_accent_color = trim(f('invoice_accent_color') ?: '#4656A5');
+
+	if (!in_array($invoice_theme, ['modern', 'classic', 'minimal'], true)) {
+		throw new UserException('Le style de facture sélectionné est invalide.');
+	}
+
+	if (!preg_match('/^#[0-9A-Fa-f]{6}$/', $invoice_accent_color)) {
+		throw new UserException('La couleur d\'accent doit être au format hexadécimal, par exemple #4656A5.');
+	}
+
 	$plugin->setConfigProperty('rna_asso', trim(f('rna_asso')));
 	$plugin->setConfigProperty('siret_asso', trim(f('siret_asso')));
 	$plugin->setConfigProperty('ttc', (bool) f('ttc'));
@@ -22,6 +33,14 @@ $form->runIf('save', function () use ($plugin) {
 
 	$plugin->setConfigProperty('logo', (bool)f('logo'));
 	$plugin->setConfigProperty('footer', f('footer'));
+	$plugin->setConfigProperty('invoice_theme', $invoice_theme);
+	$plugin->setConfigProperty('invoice_accent_color', strtoupper($invoice_accent_color));
+	$plugin->setConfigProperty('invoice_note', trim(f('invoice_note') ?: ''));
+	$plugin->setConfigProperty('payment_terms', trim(f('payment_terms') ?: ''));
+	$plugin->setConfigProperty('invoice_show_status', (bool) f('invoice_show_status'));
+	$plugin->setConfigProperty('invoice_show_address_borders', (bool) f('invoice_show_address_borders'));
+	$plugin->setConfigProperty('invoice_show_email', (bool) f('invoice_show_email'));
+	$plugin->setConfigProperty('invoice_show_web', (bool) f('invoice_show_web'));
 
 	$plugin->setConfigProperty('adresse_client', f('adresse_client'));
 	$plugin->setConfigProperty('code_postal_client', f('code_postal_client'));
@@ -36,8 +55,21 @@ $form->runIf('save', function () use ($plugin) {
 
 
 $tpl->assign('ok', qg('ok') !== null);
-$tpl->assign('conf', $plugin->getConfig());
+$conf = $plugin->getConfig();
+
+foreach (['invoice_show_status', 'invoice_show_address_borders', 'invoice_show_email', 'invoice_show_web'] as $key) {
+	if (!isset($conf->$key)) {
+		$conf->$key = true;
+	}
+}
+
+$tpl->assign('conf', $conf);
 $tpl->assign('patterns', \Paheko\Plugin\Facturation\PATTERNS_LIST);
 $tpl->assign('champsPaheko', $champsPaheko);
+$tpl->assign('invoice_themes', [
+	'modern' => 'Moderne',
+	'classic' => 'Classique',
+	'minimal' => 'Minimal',
+]);
 
 $tpl->display(PLUGIN_ROOT . '/templates/config.tpl');

admin/facture_supprimer.php

@@ -12,11 +12,13 @@ $id = (int) qg('id');
 
 $f = $facture->get($id);
 
-if (!$client)
+if (!$f)
 {
 	throw new UserException("Ce document n'existe pas.");
 }
 
+$csrf_key = 'delete_doc_' . $f->id;
+
 $form->runIf(f('delete') && !$form->hasErrors(),
              function () use ($facture, $f, $form)
              {
@@ -28,7 +30,8 @@ $form->runIf(f('delete') && !$form->hasErrors(),
                  {
                      $form->addError($e->getMessage());
                  }
-             });
+             }, $csrf_key);
 
 $tpl->assign('doc', $f);
+$tpl->assign(compact('csrf_key'));
 $tpl->display(PLUGIN_ROOT . '/templates/facture_supprimer.tpl');

admin/pdf.php

@@ -4,11 +4,27 @@ namespace Paheko;
 
 require_once __DIR__ . '/_inc.php';
 $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_READ);
-$users = new Users\Users;
 
 f(['id' => 'required|numeric']);
 $id = (int) qg('id');
 $sign_tag = UserTemplate\Functions::signature();
+$h = static function ($value): string {
+	return htmlspecialchars((string) $value, ENT_QUOTES, 'UTF-8');
+};
+$br = static function ($value) use ($h): string {
+	return nl2br($h($value));
+};
+$format_identifier = static function ($value) use ($h): string {
+	$value = preg_replace('/\s+/', '', (string) $value);
+	return $value !== '' ? $h(implode(' ', str_split($value, 3))) : '';
+};
+$detail_line = static function (string $label, $value) use ($h, $br): string {
+	if ($value === null || trim((string) $value) === '') {
+		return '';
+	}
+
+	return sprintf('<div><span>%s</span>%s</div>', $h($label), $br($value));
+};
 
 // Vérification que le document existe
 if (!$f = $facture->get($id))
@@ -18,47 +34,42 @@ if (!$f = $facture->get($id))
 
 $moyen_paiement = $facture->getMoyenPaiement($f->moyen_paiement);
 
-// Récupération infos membre
+// Récupération infos membre/client dans un objet normalisé pour le rendu
-try
-{
 if ($f->receveur_membre)
 {
-		$c = $users->get($f->receveur_id);
+	$user = Users\Users::get($f->receveur_id);
-		// l'identité du membre peut être redéfinie dans la configuration des membres
+
-		$name_fields = \Paheko\Users\DynamicFields::getNameFields();
+	if (!$user) {
-		array_walk($name_fields, function(&$elem) use ($c) {
+		throw new UserException("Ce membre n'existe plus.");
-			$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
+	$nom_client = Users\DynamicFields::getNameFromArray($user) ?: '** ABSENT **';
-		$adresse_client = $plugin->getConfig('adresse_client');
+	$user_fields = Users\DynamicFields::getInstance()->listAssocNames();
-		if ($adresse_client != null && $c->$adresse_client != null) { $c->adresse = $c->$adresse_client; }
+	$get_user_field = static function (?string $configured, string $fallback) use ($user, $user_fields) {
-		$code_postal_client = $plugin->getConfig('code_postal_client');
+		$field = $configured ?: $fallback;
-		if ($code_postal_client != null && $c->$code_postal_client != null) { $c->code_postal = $c->$code_postal_client; }
+		return array_key_exists($field, $user_fields) ? ($user->$field ?? null) : null;
-		$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)
+	$c = (object) [
-		{
+		'nom' => $nom_client,
-			if($c->$v == '')
+		'adresse' => $get_user_field($plugin->getConfig('adresse_client'), 'adresse') ?: '[À RENSEIGNER DANS LA FICHE MEMBRE]',
-			{
+		'code_postal' => $get_user_field($plugin->getConfig('code_postal_client'), 'code_postal') ?: '[À RENSEIGNER]',
-				$c->$v = '[À RENSEIGNER DANS LA FICHE MEMBRE]';
+		'ville' => $get_user_field($plugin->getConfig('ville_client'), 'ville') ?: '[À RENSEIGNER]',
-			}
+		'siret' => null,
-		}
+		'email' => Users\DynamicFields::getFirstEmailField() ? ($user->{Users\DynamicFields::getFirstEmailField()} ?? null) : null,
+		'telephone' => null,
+	];
 }
 else
 {
-		$c = $client->get($f->receveur_id);
+	$client_data = $client->get($f->receveur_id);
-		$nom_client = $c->nom;
+
+	if (!$client_data) {
+		throw new UserException("Ce client n'existe plus.");
 	}
-}
+
-catch(UserException $e)
+	$nom_client = $client_data->nom;
-{
+	$c = $client_data;
-	$form->addError($e);
 }
 
 // Formatage dates
@@ -74,307 +85,417 @@ if (isset($f->date_echeance))
 // Génération factures, devis et cotisation
 if ($f->type_facture != CERFA)
 {
-	switch ($f->type_facture)
+	$labels = [
-	{
+		FACT => ['Facture', 'Émise le', 'Adressée à', 'Échéance de paiement'],
-		case FACT:
+		DEVIS => ['Devis', 'Émis le', 'Adressé à', 'Échéance du devis'],
-		$doc = 'Facture n° '. $f->numero;
+		COTIS => ['Reçu de cotisation', 'Émis le', 'Adressé à', 'Date du reçu'],
-        $txtemis = $doc . " - Émise le " . $emission;
+	];
-        $txtdest = "Adressée à :";
+	[$document_label, $emission_label, $recipient_label, $due_label] = $labels[$f->type_facture] ?? $labels[FACT];
-		break;
+	$doc = sprintf('%s n° %s', $document_label, $f->numero);
-		case DEVIS:
+	$theme = $plugin->getConfig('invoice_theme') ?: 'modern';
-		$doc = 'Devis n° '. $f->numero;
+	$theme = in_array($theme, ['modern', 'classic', 'minimal'], true) ? $theme : 'modern';
-        $txtemis = $doc . " - Émis le " . $emission;
+	$accent = $plugin->getConfig('invoice_accent_color') ?: '#4656A5';
-        $txtdest = "Adressé à :";
+	$accent = preg_match('/^#[0-9A-Fa-f]{6}$/', $accent) ? $accent : '#4656A5';
-		break;
+	$show_address_borders = $plugin->getConfig('invoice_show_address_borders') !== false;
-		case COTIS:
+	$show_email = $plugin->getConfig('invoice_show_email') !== false;
-		$doc = 'Reçu de cotisation n° '. $f->numero;
+	$show_web = $plugin->getConfig('invoice_show_web') !== false;
-        $txtemis = $doc . " - Émis le " . $emission;
+	$address_border_class = $show_address_borders ? 'invoice-address-borders' : 'invoice-no-address-borders';
-        $txtdest = "Adressé à :";
+	$currency = $config->get('currency') ?: '€';
-		break;
+	$total = Utils::money_format($f->total, ',', ' ');
-	}
+	$ttc = $plugin->getConfig('ttc') ? 'TTC' : 'HT';
 
-    // utiliser l'adresse configurée dans le plugin sinon celle de l'asso sinon rien !
+	if ($plugin->getConfig('rue_asso') && $plugin->getConfig('cp_asso') && $plugin->getConfig('ville_asso')) {
-    if ($plugin->getConfig('rue_asso') != null &&
+		$adresse = trim(($plugin->getConfig('numero_rue_asso') ? $plugin->getConfig('numero_rue_asso') . ' ' : '') . $plugin->getConfig('rue_asso'))
-        $plugin->getConfig('cp_asso') != null &&
+			. "\n" . $plugin->getConfig('cp_asso') . ' ' . $plugin->getConfig('ville_asso');
-        $plugin->getConfig('ville_asso') != null)
-    {
-        $adresse =
-            (($plugin->getConfig('numero_rue_asso') != null) ? $plugin->getConfig('numero_rue_asso') . " " : "") .
-            $plugin->getConfig('rue_asso') . "<br>" .
-            $plugin->getConfig('cp_asso') . " " .
-            $plugin->getConfig('ville_asso');
-    }
-    else if ($config->get('org_address') != null)
-    {
-        $adresse = str_replace("\n", '<br>', $config->get('org_address'));
 	}
 	else {
-        $adresse = "";
+		$adresse = $config->get('org_address_public') ?: $config->get('org_address') ?: '';
 	}
 
 	$logo = '';
-	if ($plugin->getConfig('logo')) {
+	if ($plugin->getConfig('logo') && ($logo_url = $config->fileURL('logo'))) {
-		$logo = '<img id="logo" src="' . $config->fileURL('logo') . '" />';
+		$logo = sprintf('<img class="logo" src="%s" alt="" />', $h($logo_url));
 	}
-	$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 : " . implode(' ', str_split($t, 3)) . "<br>":'').
-		(($t = $config->get('email_asso'))?"Email : $t<br>":'').
-		(($t = $config->get('site_asso'))?"Site web : $t<br>":'');
 
-	$receveur =
+	$asso_details = '<strong>' . $h($config->get('org_name')) . '</strong>';
-		$txtdest.'<br>'.
+	$asso_details .= $adresse ? '<p>' . $br($adresse) . '</p>' : '';
-		'<b>'.$nom_client.'</b><br>'.
+	$asso_details .= ($t = $plugin->getConfig('rna_asso')) ? $detail_line('RNA', $t) : '';
-		(($t = $f->nom_contact)?"Contact : $t<br>":'').
+	$asso_details .= ($t = $plugin->getConfig('siret_asso')) ? '<div><span>SIRET</span>' . $format_identifier($t) . '</div>' : '';
-		$c->adresse."<br>".
+	$asso_details .= $show_email && ($t = $config->get('org_email')) ? $detail_line('Email', $t) : '';
-		$c->code_postal.' '.$c->ville."<br>".
+	$asso_details .= ($t = $config->get('org_phone')) ? $detail_line('Tel.', $t) : '';
-        (($t = $c->siret)?"SIREN/SIRET : " . implode(' ', str_split($t, 3)) . "<br>":'').
+	$asso_details .= $show_web && ($t = $config->get('org_web')) ? $detail_line('Web', $t) : '';
-		(($t = $c->email)?"Email : $t<br>":'').
+	$brand_summary = '<div class="brand-summary">' . $logo . '<strong>' . $h($config->get('org_name')) . '</strong></div>';
-		(($t = $c->telephone)?"Tel : $t<br>":'');
+
-	if ($f->type_facture == FACT) {
+	$receveur = '<strong>' . $h($nom_client) . '</strong>';
-		$receveur .=
+	$receveur .= ($t = ($f->nom_contact ?? null)) ? $detail_line('Contact', $t) : '';
-			(($t = $f->numero_commande)?"Commande N° : $t<br>":'').
+	$receveur .= '<p>' . $br($c->adresse ?? '') . '<br>' . $h(trim(($c->code_postal ?? '') . ' ' . ($c->ville ?? ''))) . '</p>';
-			(($t = $f->reference_acheteur)?"Référence  : $t<br>":'');
+	$receveur .= ($t = $c->siret ?? null) ? '<div><span>SIREN/SIRET</span>' . $format_identifier($t) . '</div>' : '';
+	$receveur .= $show_email && ($t = $c->email ?? null) ? $detail_line('Email', $t) : '';
+	$receveur .= ($t = $c->telephone ?? null) ? $detail_line('Tel.', $t) : '';
+	$receveur .= $f->type_facture == FACT && ($t = ($f->numero_commande ?? null)) ? $detail_line('Commande', $t) : '';
+	$receveur .= $f->type_facture == FACT && ($t = ($f->reference_acheteur ?? null)) ? $detail_line('Référence', $t) : '';
+
+	if ($f->type_facture != COTIS) {
+		$rows = '';
+
+		foreach($f->contenu as $v) {
+			$rows .= sprintf(
+				'<tr><td class="designation">%s</td><td class="amount">%s %s</td></tr>',
+				$br($v['designation']),
+				$h(Utils::money_format($v['prix'], ',', ' ')),
+				$h($currency)
+			);
 		}
-	$total = Utils::money_format($f->total, ',', ' ');
 
-	// Devis et facture
+		$payment_terms = trim((string) ($plugin->getConfig('payment_terms') ?: ''));
-	if ($f->type_facture != COTIS)
+		$invoice_note = trim((string) ($plugin->getConfig('invoice_note') ?: ''));
-	{
+		$status = '';
-		$echeance = ($f->type_facture?'Échéance de paiement':'Échéance du devis')." : ".$echeance;
+
-		if ($f->type_facture == FACT) {
+		if ($plugin->getConfig('invoice_show_status') && $f->type_facture == FACT) {
-			$reglee = !$f->reglee?'Cette facture est en attente de règlement.':'Cette facture a été réglée.';
+			$status_class = $f->reglee ? 'paid' : 'pending';
-		} else {
+			$status_label = $f->reglee ? 'Facture réglée' : 'En attente de règlement';
-			$reglee = "";
+			$status = sprintf('<p class="status %s">%s</p>', $status_class, $h($status_label));
 		}
-		$footer = str_replace("\n", '<br>', $plugin->getConfig('footer') ?? '[Pied de page à configurer]');
-		$ttc = $plugin->getConfig('ttc') ? 'TTC':'HT';
 
-		// Génération du contenu de la facture
+		$approval = $f->type_facture == DEVIS ? '<section class="approval"><strong>Bon pour accord</strong><span>Date, nom et signature</span></section>' : '';
-		ob_start();
+		$note = $invoice_note !== '' ? '<p class="invoice-note">' . $br($invoice_note) . '</p>' : '';
-		echo <<<EOF
+		$terms = $payment_terms !== '' ? $detail_line('Conditions', $payment_terms) : '';
-<div class="h2">
+		$footer = $br($plugin->getConfig('footer') ?? '[Pied de page à configurer]');
-    <span>Contenu</span> - $doc
+		$echeance_html = $detail_line($due_label, $echeance ?? '');
-</div>
+		$moyen_html = $detail_line('Moyen de paiement', $moyen_paiement);
-<hr>
 
-<table class="contenu">
+		$content = <<<HTML
+<section class="items">
+	<table>
 		<thead>
 			<tr>
-	    <th>
+				<th>Désignation</th>
-		Désignations
+				<th class="amount">Montant</th>
-	    </th>
-	    <th>
-		Prix
-	    </th>
 			</tr>
 		</thead>
 		<tbody>
-EOF;
+			$rows
-
-		$i = 1;
-		foreach($f->contenu as $k=>$v)
-		{
-			echo '<tr><td>';
-			echo str_replace("\n", '<br>', $v['designation']);
-			echo '</td><td>';
-			echo Utils::money_format($v['prix'], ',', ' ') .' €';
-			echo '</td></tr>';
-			$i++;
-		}
-
-		echo <<<EOF
 		</tbody>
-    <tfoot>
-	<tr>
-	<td><b>Total</b><br>Net à payer</td>
-	<td><b>$total €</b><br>({$ttc})</td>
-	</tr>
-    </tfoot>
 	</table>
-<footer>
+</section>
-    <div class="h2"><span>Détails</span></div>
+<section class="summary">
-    <hr>
+	<div class="details">
-
+		$echeance_html
-    $echeance <br>
+		$terms
-    $reglee
+		$moyen_html
-    Moyen de paiement : $moyen_paiement
+		$status
-    <p>
+	</div>
-	$footer
+	<div class="total">
-    </p>
+		<span>Total $ttc</span>
-</footer>
+		<strong>$total {$h($currency)}</strong>
-
+		<em>Net à payer</em>
-EOF;
+	</div>
-		if ($f->type_facture == DEVIS) {
+</section>
-			echo <<<EOF
+$note
-		<p><b>Bon pour accord, date et signature<b></p>
+$approval
-EOF;
+<footer>$footer</footer>
-		}
+HTML;
-
-		$content = ob_get_clean();
-
-	}
-	else // Reçu de cotisation
-	{
-		$lieu = $plugin->getConfig('ville_asso');
-		$intitule = $f->contenu['intitule'];
-
-		$souscription = date('d/m/Y', strtotime($f->contenu['souscription']));
-
-		if($f->contenu['expiration'] == '1970-01-01')
-		{
-			$expiration = "jour même, s'agissant d'une cotisation ponctuelle.";
 	}
 	else {
-			$expiration = date('d/m/Y', strtotime($f->contenu['expiration']));
+		$lieu = $plugin->getConfig('ville_asso') ?: '';
-		}
+		$intitule = $f->contenu['intitule'];
+		$souscription = date('d/m/Y', strtotime($f->contenu['souscription']));
+		$expiration = $f->contenu['expiration'] == '1970-01-01'
+			? "le jour même, s'agissant d'une cotisation ponctuelle"
+			: 'le ' . date('d/m/Y', strtotime($f->contenu['expiration']));
+		$footer = $br($plugin->getConfig('footer') ?? '');
 
-		// Génération du contenu du reçu de cotisation
+		$content = <<<HTML
-		$content = <<<EOF
+<section class="letter">
-<div class="h2">
+	<p>À {$h($lieu)}, le {$h($emission)},</p>
-	<span>Reçu de votre cotisation</span> - $doc
-</div>
-<hr>
-<div class="contenuTexte">
-	<p>À $lieu, le $emission,</p>
 	<p>Bonjour,</p>
-
+	<p>Nous accusons réception de votre cotisation « {$h($intitule)} » reçue le {$h($emission)} et nous vous en remercions.</p>
-	<p>Nous accusons réception de votre cotisation « $intitule » reçue le $emission et nous vous en remercions.</p>
+	<p>Nous reconnaissons que vous avez acquitté la somme de <strong>{$h($total)} {$h($currency)}</strong>. Votre adhésion sera donc effective à compter du {$h($souscription)} jusqu'à {$h($expiration)}.</p>
-	<p>Nous reconnaissons que vous avez acquitté la somme de {$total} €.<br>111
+	<p>Nous vous prions de recevoir nos meilleures salutations.</p>
-	Votre adhésion sera donc effective à compter du $souscription jusqu’au $expiration.</p>
+	<p><strong>Pour {$h($config->get('org_name'))}</strong></p>
-	<br>
+	<p class="legal">Nous vous rappelons que la cotisation n'est pas soumise à la TVA et qu'elle ne donne pas lieu à la délivrance d'une facture. Elle n'ouvre pas droit au bénéfice des dispositions des articles 200, 238 bis et 885-0 V bis A du code général des impôts.</p>
-
+</section>
-	<p>Nous vous prions de recevoir, chère adhérente, cher adhérent, nos meilleures salutations,</p>
+<footer>$footer</footer>
-	<br>
+HTML;
-	<p>-représentant·e de l'asso-</p>
-	<br>
-	<p><i>Nous vous rappelons que la cotisation n’est pas soumise à la TVA et qu’elle ne donne pas lieu à la délivrance d’une facture. Elle n’ouvre pas droit au bénéfice des dispositions des articles 200, 238 bis et 885-0 V bis A du code général des impôts.</i></p>
-</div>
-EOF;
 	}
 
-	//-- Layout du document
+	$html_title = $h($doc . '_' . $emission);
+	$doc_html = $h($doc);
+	$txtemis = $h($emission_label . ' ' . $emission);
+	$txtdest = $h($recipient_label);
 
-	ob_start();
+	$html = <<<HTML
-	echo <<<EOF
 <!DOCTYPE html>
-<html>
+<html class="invoice-theme-$theme $address_border_class">
 	<head>
 		<meta charset="UTF-8">
-		<title>{$doc}_{$emission}</title>
+		<title>$html_title</title>
 		<style>
-			@page {
+			@page { size: A4 portrait; margin: 14mm; }
-				size: A4 portrait;
+			* { box-sizing: border-box; }
-				margin: 0;
-			}
-
 			body {
-				padding: 4mm;
+				margin: 0;
-				font-family: Helvetica, Arial, sans;
+				font-family: Helvetica, Arial, sans-serif;
 				font-size: 10pt;
-				background: white;
+				line-height: 1.45;
+				color: #20242A;
+				background: #fff;
 			}
-
+			.invoice {
-			.titre {
+				display: block;
-				text-align: center;
+				padding-bottom: 28mm;
-				font-size: 8pt;
+				page-break-after: avoid;
-				margin-bottom: 6mm;
+				break-after: avoid-page;
-				margin-top: 0mm;
 			}
-			.h2 {
+			.hero {
-				margin: 20px 20px 0px 20px;
+				display: table;
-			}
-			.h2 span {
-				font-weight: bold;
-				font-size: 14pt;
-			}
-			hr {
-				margin: 5px 0px 15px 0px;
-				border: none;
-				border-top: 1px solid;
-			}
-			.adressage {
-				font-size: 11pt;
-				margin: auto;
-			}
-			.adressage td {
-				width: 40%;
-				vertical-align: top;
-			}
-			.adressage td#logo {
-				width: 20%;
-				vertical-align: top;
-			}
-			.adressage img#logo {
-				height : 3cm;
-			}
-
-			.contenuTexte {
-				padding: 0 6mm;
-			}
-
-			.contenu {
 				width: 100%;
+				padding-bottom: 10mm;
+				border-bottom: 2px solid $accent;
 			}
-
+			.brand, .document-title {
-			.contenu td, .contenu th {
+				display: table-cell;
 				vertical-align: top;
-				padding: 8px 10px;
+			}
-				margin : 0px;
+			.brand { width: 46%; }
+			.document-title { width: 54%; }
+			.brand-summary {
+				display: inline-block;
+				max-width: 72mm;
+				text-align: center;
+			}
+			.logo {
+				display: block;
+				max-width: 34mm;
+				max-height: 24mm;
+				margin: 0 auto 4mm;
+			}
+			.brand-summary strong {
+				display: block;
+				font-size: 10pt;
+				line-height: 1.3;
+			}
+			.document-title {
+				text-align: right;
+			}
+			.document-title h1 {
+				margin: 0 0 2mm;
+				color: $accent;
+				font-size: 19pt;
+				font-weight: 700;
+				line-height: 1.15;
+				letter-spacing: 0;
+				white-space: nowrap;
+			}
+			.document-title p {
+				margin: 0;
+				color: #566070;
+			}
+			.addresses {
+				display: table;
+				width: 100%;
+				margin: 10mm 0 8mm;
+			}
+			.card {
+				display: table-cell;
+				width: 50%;
+				padding: 6mm;
+				vertical-align: top;
+				border: 1px solid #D9DEE8;
+			}
+			.card + .card { border-left: 0; }
+			.card h2 {
+				margin: 0 0 4mm;
+				color: $accent;
+				font-size: 9pt;
+				text-transform: uppercase;
+				letter-spacing: 0;
+			}
+			.card strong {
+				display: block;
+				margin-bottom: 2mm;
+				font-size: 12pt;
+			}
+			.card p { margin: 2mm 0; }
+			.card div {
+				margin-top: 1.5mm;
+			}
+			.card span {
+				display: inline-block;
+				min-width: 27mm;
+				margin-right: 3mm;
+				color: #687386;
+			}
+			.invoice-no-address-borders .card {
+				padding: 0;
+				border: 0;
+			}
+			.invoice-no-address-borders .card + .card {
+				padding-left: 10mm;
+			}
+			.items table {
+				width: 100%;
 				border-collapse: collapse;
 			}
-
+			.items th {
-			.contenu tr th, .contenu tr td {
+				padding: 4mm 3mm;
-				text-align: right;
+				color: #fff;
-				width: 16%;
+				background: $accent;
-			}
-			.contenu tr td:nth-child(1), .contenu tr th:nth-child(1) {
 				text-align: left;
-				width: 84%;
+				font-weight: 700;
 			}
-
+			.items td {
-			.contenu thead tr, .contenu tfoot tr {
+				padding: 4mm 3mm;
-				background-color: #CCE;
+				border-bottom: 1px solid #E5E8EF;
-				border: 10px solid black;
+				vertical-align: top;
 			}
-			.contenu tbody tr:nth-child(even) {
+			.items tr:nth-child(even) td {
-				background-color: #DDF !important;
+				background: #F7F8FB;
 			}
-			.contenu tfoot {
+			.amount {
-				display: table-row-group;
+				width: 34mm;
+				text-align: right;
+				white-space: nowrap;
 			}
-
+			.summary {
-			footer {
+				display: table;
-				bottom: 0;
+				width: 100%;
-				margin: 14mm 0;
+				margin-top: 8mm;
-				width: inherit;
+			}
+			.details, .total {
+				display: table-cell;
+				vertical-align: top;
+			}
+			.details {
+				width: 64%;
+				color: #3B4350;
+			}
+			.details div {
+				margin-bottom: 2mm;
+			}
+			.details span {
+				display: inline-block;
+				min-width: 42mm;
+				margin-right: 4mm;
+				color: #687386;
+			}
+			.total {
+				width: 36%;
+				padding: 4mm 5mm;
+				color: #fff;
+				background: $accent;
+				text-align: right;
+			}
+			.total span, .total em {
+				display: block;
+				font-style: normal;
+				font-size: 9.5pt;
+			}
+			.total strong {
+				display: block;
+				margin: 1mm 0;
+				font-size: 17pt;
+				line-height: 1.15;
+			}
+			.status {
+				display: inline-block;
+				margin: 2mm 0 0;
+				padding: 1.5mm 3mm;
+				border: 1px solid #D9DEE8;
+				font-weight: 700;
+			}
+			.status.paid { color: #1F6B3A; }
+			.status.pending { color: #9A4D00; }
+			.invoice-note, .approval, .letter {
+				margin-top: 8mm;
+			}
+			.approval {
+				min-height: 25mm;
+				padding-top: 4mm;
+				border-top: 1px solid #D9DEE8;
+			}
+			.approval strong, .approval span {
+				display: block;
+			}
+			.legal {
+				margin-top: 8mm;
+				color: #566070;
 				font-size: 9pt;
 			}
+			footer {
+				position: fixed;
+				right: 0;
+				bottom: 0;
+				left: 0;
+				margin: 0;
+				padding-top: 6mm;
+				color: #566070;
+				font-size: 8.5pt;
+				text-align: center;
+				border-top: 1px solid #D9DEE8;
+			}
+			.invoice-theme-classic .hero {
+				border-bottom-width: 1px;
+			}
+			.invoice-theme-classic .items th,
+			.invoice-theme-classic .total {
+				background: #2F343D;
+			}
+			.invoice-theme-minimal .hero,
+			.invoice-theme-minimal .card,
+			.invoice-theme-minimal footer {
+				border-color: #E6E8ED;
+			}
+			.invoice-theme-minimal .items th,
+			.invoice-theme-minimal .total {
+				color: #20242A;
+				background: #F0F2F6;
+			}
+			@media screen {
+				body {
+					padding: 14mm;
+				}
+				footer {
+					right: 14mm;
+					bottom: 14mm;
+					left: 14mm;
+				}
+			}
+			@media print {
+				footer {
+					right: 0;
+					bottom: 0;
+					left: 0;
+				}
+			}
 		</style>
 	</head>
 	<body>
-		<p class="titre">
+		<main class="invoice">
-		$txtemis
+			<header class="hero">
-		</p>
+				<section class="brand">
-
+					$brand_summary
-		<table class="adressage">
+				</section>
-			<tr>
+				<section class="document-title">
-				<td id="logo">$logo</td>
+					<h1>$doc_html</h1>
-				<td>$asso</td>
+					<p>$txtemis</p>
-				<td>$receveur</td>
+				</section>
-			</tr>
+			</header>
-		</table>
+			<section class="addresses">
-		<br>
+				<div class="card">
+					<h2>Émetteur</h2>
+					$asso_details
+				</div>
+				<div class="card">
+					<h2>$txtdest</h2>
+					$receveur
+				</div>
+			</section>
 			$content
+		</main>
 	</body>
 </html>
-EOF;
+HTML;
-
-	$html = ob_get_clean();
-
 } // Génération du CERFA
 elseif ($f->type_facture == CERFA)
 {
@@ -394,9 +515,9 @@ elseif ($f->type_facture == CERFA)
 	$t['objet2'] = $plugin->getConfig('objet_2');
 
 	$t['nom'] = $nom_client;
-	$t['adresse'] = $c->adresse;
+	$t['adresse'] = $c->adresse ?? '';
-	$t['cp'] = $c->code_postal;
+	$t['cp'] = $c->code_postal ?? '';
-	$t['ville'] = $c->ville;
+	$t['ville'] = $c->ville ?? '';
 	$t['total'] = '***'.Utils::money_format($f->total).'***';
 	$t['total_lettre'] = numfmt_create('fr_FR', \NumberFormatter::SPELLOUT)->format($f->total/100). ' euros';
 

config.json

@@ -1,6 +1,14 @@
 {
 	"footer": "[EXEMPLE]\nAssociation exonérée des impôts commerciaux\nEn cas de retard de paiement, indemnité forfaitaire légale pour frais de recouvrement : 40,00 €\n[Coordonnées bancaires]\nAssociation enregistrée en préfecture de XXX au numéro YYY",
 	"pattern": "%{type}-%{year}-%{ynumber}",
+	"invoice_theme": "modern",
+	"invoice_accent_color": "#4656A5",
+	"invoice_note": "",
+	"payment_terms": "Paiement à réception",
+	"invoice_show_status": true,
+	"invoice_show_address_borders": true,
+	"invoice_show_email": true,
+	"invoice_show_web": true,
 	"ttc": false,
 	"validate_cp": true
 }

data/schema.sql

@@ -3,13 +3,13 @@ CREATE TABLE IF NOT EXISTS plugin_facturation_factures (
 	type_facture		INTEGER NOT NULL DEFAULT 0,
 	numero				TEXT NOT NULL UNIQUE,
 	receveur_membre 	INTEGER NOT NULL, -- bool
-	receveur_id			INTEGER NOT NULL,
+	receveur_id			INTEGER NOT NULL,
-	date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
+	date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
-	date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
+	date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
 	reglee				INTEGER DEFAULT 0, -- bool
 	archivee			INTEGER DEFAULT 0, -- bool
 	moyen_paiement		TEXT NOT NULL,
-	contenu 			TEXT NOT NULL,
+	contenu 			TEXT NOT NULL,
 	total   			INTEGER DEFAULT 0,
 	nom_contact			TEXT,
 	numero_commande		TEXT,
@@ -20,10 +20,10 @@ CREATE TABLE IF NOT EXISTS plugin_facturation_factures (
 
 CREATE TABLE IF NOT EXISTS plugin_facturation_clients (
 	id					INTEGER PRIMARY KEY,
-	nom					TEXT NOT NULL,
+	nom					TEXT NOT NULL,
-	adresse				TEXT NOT NULL,
+	adresse				TEXT NOT NULL,
-	code_postal			TEXT NOT NULL,
+	code_postal			TEXT NOT NULL,
-	ville				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), -- Date d\'inscription
 	telephone			TEXT,

install.php

@@ -1,8 +1,13 @@
 <?php
 namespace Paheko;
-use Paheko\Entities\Files\File;
-use Paheko\Plugin\Facturation\Facture;
 
 $db = DB::getInstance();
 
 $db->import(dirname(__FILE__) . "/data/schema.sql");
+
+$default_config = json_decode(file_get_contents(__DIR__ . '/config.json'));
+
+if ($default_config instanceof \stdClass) {
+	$plugin->setConfig($default_config);
+	$plugin->save();
+}

lib/Client.php

@@ -4,7 +4,7 @@ namespace Paheko\Plugin\Facturation;
 
 use Paheko\DB;
 use Paheko\DynamicList;
-use Paheko\Entities\Plugin;
+use Paheko\Plugins;
 use Paheko\UserException;
 use Paheko\Utils;
 
@@ -29,9 +29,12 @@ class Client
 
 	public function __construct()
 	{
-		$plugin = new Plugin('facturation');
+		$plugin = Plugins::get('facturation');
-		$this->config['unique_client_name'] = $plugin->getConfig('unique_client_name') ?: false;
+
-		$this->config['validate_cp'] = $plugin->getConfig('validate_cp') ?: false;
+		if ($plugin) {
+			$this->config['unique_client_name'] = (bool) $plugin->getConfig('unique_client_name');
+			$this->config['validate_cp'] = (bool) ($plugin->getConfig('validate_cp') ?? true);
+		}
 	}
 
 	public function _checkFields(&$data)
@@ -102,8 +105,20 @@ class Client
 	{
 		$db = DB::getInstance();
 
-		return $db->first('SELECT *, strftime(\'%s\', date_creation) AS date_creation
+		$client = $db->first('SELECT *, strftime(\'%s\', date_creation) AS date_creation
 			FROM plugin_facturation_clients WHERE id = ? LIMIT 1;', (int)$id);
+
+		if (!$client) {
+			return null;
+		}
+
+		foreach (['siret', 'telephone', 'email', 'nom_contact', 'note'] as $key) {
+			if (!property_exists($client, $key)) {
+				$client->$key = null;
+			}
+		}
+
+		return $client;
 	}
 
 	public function listAll()

lib/Facture.php

@@ -6,9 +6,10 @@ use DateTime;
 use Paheko\Config;
 use Paheko\DB;
 use Paheko\DynamicList;
+use Paheko\Plugins;
 use Paheko\UserException;
+use Paheko\Users\DynamicFields;
 use Paheko\Utils;
-use Paheko\Services\Services_User;
 
 class Facture
 {
@@ -47,17 +48,52 @@ class Facture
 			'accounts' => [],
 			'label' => 'Facture',
 			'help' => ''],
+		CERFA => [
+			'id' => CERFA,
+			'accounts' => [],
+			'label' => 'Reçu fiscal',
+			'help' => 'Fonction historique conservée pour les anciens reçus. Le module Paheko "Reçus fiscaux" est recommandé pour les nouveaux usages.'],
+		COTIS => [
+			'id' => COTIS,
+			'accounts' => [],
+			'label' => 'Reçu de cotisation',
+			'help' => 'Fonction historique conservée pour les anciens reçus. Le module Paheko "Reçu de paiement" est recommandé pour les nouveaux usages.'],
 	];
 
 	public function __construct()
 	{
 
 	}
+	private function normalizeDate($value, string $field): string
+	{
+		if ($value instanceof \DateTimeInterface) {
+			return $value->format('Y-m-d');
+		}
+
+		$value = trim((string) $value);
+
+		foreach (['!Y-m-d', '!d/m/Y'] as $format) {
+			$date = DateTime::createFromFormat($format, $value);
+			$errors = DateTime::getLastErrors();
+
+			if ($date && (!$errors || (!$errors['warning_count'] && !$errors['error_count']))) {
+				return $date->format('Y-m-d');
+			}
+		}
+
+		throw new UserException(sprintf('La date "%s" est invalide pour le champ %s.', $value, $field));
+	}
 
 	// Fix : est dépendant de l'ordre des données dans l'array
 	// et implique que toutes les données soient présentes (pas possible de faire un update partiel)
 	public function _checkFields(&$datas)
 	{
+		$type_facture = isset($datas['type_facture']) ? (int) $datas['type_facture'] : null;
+		$fac = in_array($type_facture, [DEVIS, FACT], true);
+		$cerfa = $type_facture === CERFA;
+		$recu = $type_facture === COTIS;
+		$total = 0;
+
 		foreach($datas as $k=>$data)
 		{
 			if (!in_array($k, $this->keys))
@@ -76,24 +112,11 @@ class Facture
 			switch($k)
 			{
 				case 'type_facture':
-				if (!array_key_exists($datas[$k], $this->types)) {
+				$datas[$k] = (int) $datas[$k];
+
+				if (!array_key_exists($datas[$k], self::TYPES_NAMES)) {
 					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':
@@ -108,11 +131,11 @@ class Facture
 				}
 				break;
 				case 'date_emission':
-				$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
+				$datas[$k] = $this->normalizeDate($data, $k);
 				break;
 				case 'date_echeance':
-				$datas[$k] = \DateTime::createFromFormat('!d/m/Y', $data)->format('Y-m-d');
+				$datas[$k] = $this->normalizeDate($data, $k);
-				if (DateTime::createFromFormat('!Y-m-d', $datas[$k])->format('U') < DateTime::createFromFormat('!Y-m-d', $datas['date_emission'])->format('U'))
+				if (isset($datas['date_emission']) && 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).");
 				}
@@ -134,12 +157,12 @@ class Facture
 						if (empty($r['designation']) && empty($r['prix']))
 						{
 							unset($datas[$k][$g]);
-							unset($datas[$k]['prix']);
 							continue;
 						}
-						elseif (! is_numeric($r['prix']) && empty($r['prix']))
+						elseif (!isset($r['prix']) || $r['prix'] === '')
 						{
-							$datas[$k]['prix'] = 0;
+							$datas[$k][$g]['prix'] = 0;
+							$r['prix'] = 0;
 						}
 						elseif (empty($r['designation'])) {
 							throw new UserException("Une au moins des désignations est absente.");
@@ -251,7 +274,7 @@ class Facture
             $type = ctype_digit($v) ? 'd' : 's';
             return sprintf('%' . $match[1] . $type, $v);
         }, $selpattern);
-        $modele = '/^' . $prefixe . '\d+$/';
+        $modele = '/^' . preg_quote($prefixe, '/') . '\d+$/';
         $numeros_filtres = array_filter($numeros, function($elem) use ($modele) {
             return preg_match($modele, $elem);
         }, 0);
@@ -292,6 +315,12 @@ class Facture
 			$r->contenu = json_decode($r->contenu, true);
 		}
 
+		foreach (['nom_contact', 'numero_commande', 'reference_acheteur'] as $key) {
+			if (!property_exists($r, $key)) {
+				$r->$key = null;
+			}
+		}
+
 		$r->date_emission = \DateTime::createFromFormat('!Y-m-d', $r->date_emission);
 		if ($r->date_echeance)
 		{
@@ -320,15 +349,19 @@ class Facture
 
 	public function list($all = true): DynamicList
 	{
-		$id_field = \Paheko\Users\DynamicFields::getNameFieldsSQL('u');
+		$db = DB::getInstance();
-		$plugin_name = preg_replace('/^.*\/(\w+)\/$/', '${1}', \Paheko\PLUGIN_ADMIN_URL);
+		$id_field = DynamicFields::getNameFieldsSQL('u');
-		$plugin = \Paheko\Plugins::get($plugin_name);
+		$plugin = Plugins::get('facturation');
+		$user_fields = DynamicFields::getInstance()->listAssocNames();
 
 		// adresse et ville peuvent être redéfinies dans la configuration du plugin
-		$adresse_client = $plugin->getConfig('adresse_client');
+		$adresse_client = $plugin ? $plugin->getConfig('adresse_client') : null;
-		if ($adresse_client == null) { $adresse = 'u.adresse'; } else { $adresse = 'u.' . $adresse_client; }
+		$adresse_client = $adresse_client ?: 'adresse';
-		$ville_client = $plugin->getConfig('ville_client');
+		$adresse = array_key_exists($adresse_client, $user_fields) ? 'u.' . $db->quoteIdentifier($adresse_client) : 'NULL';
-		if ($ville_client == null) { $ville = 'u.ville'; } else { $ville = 'u.' . $ville_client; }
+
+		$ville_client = $plugin ? $plugin->getConfig('ville_client') : null;
+		$ville_client = $ville_client ?: 'ville';
+		$ville = array_key_exists($ville_client, $user_fields) ? 'u.' . $db->quoteIdentifier($ville_client) : 'NULL';
 
 		$columns = [
 			// Sélectionner cette colonne, mais ne pas la mettre dans la liste des colonnes
@@ -357,7 +390,7 @@ class Facture
 			'receveur' => [
 				'label' => 'Receveur',
 				// 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),
+				'select' => sprintf('CASE WHEN receveur_membre THEN COALESCE(NULLIF(%s, \'\'), \'** ABSENT **\') ELSE c.nom END', $id_field),
 			],
 			'receveur_adresse' => [
 				// l'adresse peut être redéfinie dans la configuration du plugin
@@ -393,7 +426,7 @@ class Facture
 
 		$tables = 'plugin_facturation_factures AS f
 			INNER JOIN plugin_facturation_paiement AS mp ON mp.code = f.moyen_paiement
-			LEFT JOIN users AS u ON f.receveur_membre = 1 AND u.id = f.receveur_id
+			LEFT JOIN users_view AS u ON f.receveur_membre = 1 AND u.id = f.receveur_id
 			LEFT JOIN plugin_facturation_clients AS c ON f.receveur_membre = 0 AND c.id = f.receveur_id';
 
 		if ($all) {
@@ -404,7 +437,7 @@ class Facture
 		$list = new DynamicList($columns, $tables, $where);
 		$list->orderBy('date_emission', true);
 
-		$currency = Config::getInstance()->monnaie;
+		$currency = Config::getInstance()->currency;
 
 		$list->setModifier(function ($row) use ($currency) {
 			// Remplir la colonne virtuelle
@@ -460,7 +493,10 @@ class Facture
 		}
 		else // Si c'est un membre de l'asso
 		{
-			throw new UserException("Woopsie, g pô encore implémenté l'usage des users de l'asso comme clients");
+			if (!\Paheko\Users\Users::exists((int) $id))
+			{
+				throw new UserException("Ce membre n'existe pas.");
+			}
 		}
 
 		$r = (array)DB::getInstance()->get('SELECT *, strftime(\'%s\', date_emission) AS date_emission,
@@ -492,7 +528,10 @@ class Facture
 		}
 		else // Si c'est un membre de l'asso
 		{
-			throw new UserException("Woopsie, g pô encore implémenté l'usage des users de l'asso comme clients");
+			if (!\Paheko\Users\Users::exists((int) $id))
+			{
+				throw new UserException("Ce membre n'existe pas.");
+			}
 		}
 
 		return DB::getInstance()->test('plugin_facturation_factures', 'receveur_membre = ? AND receveur_id = ?', $base, $id);

plugin.ini

@@ -1,8 +1,8 @@
 name="Facturation"
 description="Permet d'éditer des factures et devis à ses membres ainsi qu'à une base de clients supplémentaire."
-author="zou ; adapté par Jean-Christophe Engel"
+author="Sur la base de Jean-Christophe Engel, amélioré par Jordan"
 url="https://gitea.zaclys.com/lesanges/facturation"
-version="0.17.2"
+version="0.27"
 menu=true
 restrict_section="accounting"
 restrict_level="read"

templates/_form.tpl

@@ -63,7 +63,7 @@
 			<dt><label>Document adressé à :</label></dt>
 			{if !empty($clients)}
 			<dd>
-				{input type="radio" name="base_receveur" value="membre" label="Un·e membre" default=1 source=$doc}
+				{input type="radio" name="base_receveur" value="membre" label="Un·e membre" default="membre" source=$doc}
 				{input type="radio" name="base_receveur" value="client" label="Un·e client·e" source=$doc}
 			</dd>
 			{/if}

templates/_js.tpl

@@ -45,9 +45,15 @@
         function changeTypeSaisie(type)
         {
             g.toggle(['.type_client', '.type_membre'], false);
+			document.querySelectorAll('.type_client input, .type_client select, .type_client textarea, .type_membre input, .type_membre select, .type_membre textarea').forEach((e) => {
+				e.disabled = true;
+			});
 
             if (type) {
             	g.toggle('.type_' + type, true);
+				document.querySelectorAll('.type_' + type + ' input, .type_' + type + ' select, .type_' + type + ' textarea').forEach((e) => {
+					e.disabled = false;
+				});
             }
         }
 
@@ -76,9 +82,11 @@
         }
 
 		const selclient = document.querySelector('#f_client');
+		if (selclient) {
 			selclient.addEventListener("change", () => {
 				modifierContact(selclient, '#f_contact_list', '#f_nom_contact');
 			});
+		}
 
     } ());
 
@@ -86,12 +94,18 @@
 	// Hide type specific parts of the form
 	function hideAllTypes() {
 		g.toggle('[data-types]', false);
+		document.querySelectorAll('[data-types] input, [data-types] select, [data-types] textarea, [data-types] button').forEach((e) => {
+			e.disabled = true;
+		});
 	}
 
 	// Toggle parts of the form when a type is selected
 	function selectType(v) {
 		hideAllTypes();
 		g.toggle('[data-types~=t' + v + ']', true);
+		document.querySelectorAll('[data-types~=t' + v + '] input, [data-types~=t' + v + '] select, [data-types~=t' + v + '] textarea, [data-types~=t' + v + '] button').forEach((e) => {
+			e.disabled = false;
+		});
 		g.toggle('[data-types=all-but-advanced]', v != 0);
 		// Disable required form elements, or the form won't be able to be submitted
 		$('[data-types=all-but-advanced] input[required]').forEach((e) => {

templates/clients.tpl

@@ -25,7 +25,7 @@
                     {if $session->canAccess($session::SECTION_ACCOUNTING, $session::ACCESS_ADMIN)}
                         {linkbutton shape="delete" href="client_supprimer.php?id=%d"|args:$row.id label="Supprimer"}
                     {/if}
-                    {if $session->canAccess($session::SECTION_USERS, $session::ACCESS_ADMIN)}
+                    {if $session->canAccess($session::SECTION_ACCOUNTING, $session::ACCESS_WRITE)}
                         {linkbutton shape="edit" href="client_modifier.php?id=%d"|args:$row.id label="Modifier"}
                     {/if}
                     {linkbutton shape="user" href="client.php?id=%d"|args:$row.id label="Fiche client"}

templates/config.tpl

@@ -38,14 +38,27 @@
 		    {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>Présentation des factures</legend>
+			<dl>
+				{input type="select" name="invoice_theme" label="Style visuel" options=$invoice_themes source=$conf}
+				{input type="color" name="invoice_accent_color" label="Couleur d'accent" source=$conf default="#4656A5"}
+				{input type="text" name="payment_terms" label="Conditions de paiement" source=$conf}
+				{input type="textarea" class="full-width" rows="3" name="invoice_note" source=$conf label="Note affichée sous le tableau"}
+				{input type="checkbox" name="invoice_show_status" value="1" source=$conf label="Afficher le statut de règlement sur la facture"}
+				{input type="checkbox" name="invoice_show_address_borders" value="1" source=$conf label="Afficher les bordures des blocs émetteur et destinataire"}
+				{input type="checkbox" name="invoice_show_email" value="1" source=$conf label="Afficher les adresses email"}
+				{input type="checkbox" name="invoice_show_web" value="1" source=$conf label="Afficher le site web de l'association"}
+			</dl>
+		</fieldset>
 		<fieldset>
 			<legend>
 				Choisir les champs à faire figurer sur la facture ou le devis pour l'adresse d'un membre
 			</legend>
 			<dl>
-				{input type="select" name="adresse_client" label="Adresse" required=true options=$champsPaheko source=$conf}
+				{input type="select" name="adresse_client" label="Adresse" required=false options=$champsPaheko source=$conf}
-				{input type="select" name="code_postal_client" label="Code postal" required=true options=$champsPaheko source=$conf}
+				{input type="select" name="code_postal_client" label="Code postal" required=false options=$champsPaheko source=$conf}
-				{input type="select" name="ville_client" label="Ville" required=true options=$champsPaheko source=$conf}
+				{input type="select" name="ville_client" label="Ville" required=false options=$champsPaheko source=$conf}
 			</dl>
 		</fieldset>
     </fieldset>
@@ -60,7 +73,6 @@
                 F = Facture, D = Devis
             </dd>
         </dl>
-        <i>Pour personnaliser l'apparence de la facture, il faut pour l'instant se retrousser les manches et éditer soi-même le fichier admin/pdf.php du plugin ! </i>
     </fieldset>
 
     <p class="submit">

templates/facture_supprimer.tpl

@@ -16,7 +16,7 @@
     </fieldset>
 
     <p class="submit">
-        {csrf_field key="delete_doc_"|cat:$doc.id}
+        {csrf_field key=$csrf_key}
         <input type="submit" name="delete" value="Supprimer &rarr;" />
     </p>
 

upgrade.php

@@ -2,7 +2,6 @@
 
 namespace Paheko;
 use Paheko\Plugin\Facturation\Facture;
-use Paheko\Entities\Files\File;
 
 $db = DB::getInstance();
 $old_version = $plugin->oldVersion();
@@ -25,11 +24,9 @@ if (version_compare($old_version, '0.3.0', '<'))
 	$conf = $db->getAssoc('SELECT cle, valeur FROM plugin_facturation_config ORDER BY cle;');
 	foreach($conf as $k=>$v)
 	{
-		if(!$plugin->setConfig($k, $v))
+		$plugin->setConfigProperty($k, $v);
-		{
-			throw new UserException('Erreur dans la conversion de la configuration pour la clé : '.$k);
-		}
 	}
+	$plugin->save();
 	$db->exec('DROP TABLE `plugin_facturation_config`;');
 }
 
@@ -59,13 +56,13 @@ if (version_compare($old_version, '0.4.0', '<'))
 			type_facture		INTEGER NOT NULL DEFAULT 0,
 			numero				TEXT NOT NULL UNIQUE,
 			receveur_membre 	INTEGER NOT NULL, -- bool
-			receveur_id			INTEGER NOT NULL,
+			receveur_id			INTEGER NOT NULL,
-			date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
+			date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
-			date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
+			date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
 			reglee				INTEGER DEFAULT 0, -- bool
 			archivee			INTEGER DEFAULT 0, -- bool
 			moyen_paiement		TEXT NOT NULL,
-			contenu 			TEXT NOT NULL,
+			contenu 			TEXT NOT NULL,
 			total   			REAL DEFAULT 0
 		);
 
@@ -81,13 +78,13 @@ EOT
 // 0.6.0 -
 if (version_compare($old_version, '0.6.0', '<'))
 {
-    define('DEVIS', 0);
+    defined('DEVIS') || define('DEVIS', 0);
-    define('FACT', 1);
+    defined('FACT') || define('FACT', 1);
-    define('CERFA', 2);
+    defined('CERFA') || define('CERFA', 2);
-    define('COTIS', 3);
+    defined('COTIS') || define('COTIS', 3);
     $facture = new Facture;
 	$r = $db->first('SELECT id, total FROM plugin_facturation_factures;');
-	if (strpos($r->total,'.'))
+	if ($r && strpos((string) $r->total, '.') !== false)
 	{
 		// SQL -> total integer
 		$db->exec(<<<EOT
@@ -97,13 +94,13 @@ if (version_compare($old_version, '0.6.0', '<'))
 			type_facture		INTEGER NOT NULL DEFAULT 0,
 			numero				TEXT NOT NULL UNIQUE,
 			receveur_membre 	INTEGER NOT NULL, -- bool
-			receveur_id			INTEGER NOT NULL,
+			receveur_id			INTEGER NOT NULL,
-			date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
+			date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
-			date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
+			date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
 			reglee				INTEGER DEFAULT 0, -- bool
 			archivee			INTEGER DEFAULT 0, -- bool
 			moyen_paiement		TEXT NOT NULL,
-			contenu 			TEXT NOT NULL,
+			contenu 			TEXT NOT NULL,
 			total   			INTEGER DEFAULT 0
 		);
 
@@ -116,6 +113,8 @@ EOT
 		$factures = $facture->listAll();
 		foreach($factures as $k=>$f)
 		{
+			$contenu = [];
+
 			foreach($f->contenu as $line => $content)
 			{
 				// Petit bug qui peut arriver avec des contenus mal enregistrés en db
@@ -137,19 +136,16 @@ EOT
 			$facture->edit($f->id, $data);
 			unset($contenu);
 		}
-
-		$path = __DIR__.'/data/default_sign.png';
-		$png = (new File)->createAndStore('skel/plugin/facturation','sign.png', $path, null);
 	}
 }
 
 // 0.6.2 -
 if (version_compare($old_version, '0.6.2', '<'))
 {
-    define('DEVIS', 0);
+    defined('DEVIS') || define('DEVIS', 0);
-    define('FACT', 1);
+    defined('FACT') || define('FACT', 1);
-    define('CERFA', 2);
+    defined('CERFA') || define('CERFA', 2);
-    define('COTIS', 3);
+    defined('COTIS') || define('COTIS', 3);
     $facture = new Facture;
 	$db->exec(<<<EOT
 	    INSERT OR IGNORE INTO plugin_facturation_paiement
@@ -201,7 +197,8 @@ EOT
 // 0.7.1 - Ajout clé config TTC/HT
 if (version_compare($old_version, '0.7.1', '<'))
 {
-	$plugin->setConfig('ttc', false);
+	$plugin->setConfigProperty('ttc', false);
+	$plugin->save();
 }
 
 // 0.8.1 - Signal menu item
@@ -217,10 +214,10 @@ if (version_compare($old_version, '0.8.5', '<'))
               CREATE TABLE IF NOT EXISTS plugin_facturation_clients_tmp
               (
                   id				INTEGER PRIMARY KEY,
-                  nom				TEXT NOT NULL,
+                  nom				TEXT NOT NULL,
-                  adresse			TEXT NOT NULL,
+                  adresse			TEXT NOT NULL,
-                  code_postal		TEXT NOT NULL,
+                  code_postal		TEXT NOT NULL,
-                  ville				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,
@@ -261,10 +258,10 @@ if (version_compare($old_version, '0.15', '<'))
               CREATE TABLE IF NOT EXISTS plugin_facturation_clients_tmp
               (
                   id				INTEGER PRIMARY KEY,
-                  nom				TEXT NOT NULL,
+                  nom				TEXT NOT NULL,
-                  adresse			TEXT NOT NULL,
+                  adresse			TEXT NOT NULL,
-                  code_postal		TEXT NOT NULL,
+                  code_postal		TEXT NOT NULL,
-                  ville				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,
@@ -293,13 +290,13 @@ EOT
 				type_facture		INTEGER NOT NULL DEFAULT 0,
 				numero				TEXT NOT NULL UNIQUE,
 				receveur_membre 	INTEGER NOT NULL, -- bool
-				receveur_id			INTEGER NOT NULL,
+				receveur_id			INTEGER NOT NULL,
-				date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
+				date_emission		TEXT NOT NULL, -- CHECK (date(date_emission) IS NOT NULL AND date(date_emission) = date_emission),
-				date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
+				date_echeance		TEXT NOT NULL, -- CHECK (date(date_echeance) IS NOT NULL AND date(date_echeance) = date_echeance),
 				reglee				INTEGER DEFAULT 0, -- bool
 				archivee			INTEGER DEFAULT 0, -- bool
 				moyen_paiement		TEXT NOT NULL,
-				contenu 			TEXT NOT NULL,
+				contenu 			TEXT NOT NULL,
 				total   			INTEGER DEFAULT 0,
 				nom_contact			TEXT,
 				numero_commande		TEXT,
@@ -322,3 +319,64 @@ EOT
 	);
 
 }
+
+// 0.18 - Personnalisation du rendu des factures
+if (version_compare($old_version, '0.18', '<'))
+{
+	$defaults = json_decode(file_get_contents(__DIR__ . '/config.json'), true);
+
+	foreach (['invoice_theme', 'invoice_accent_color', 'invoice_note', 'payment_terms', 'invoice_show_status'] as $key) {
+		if ($plugin->getConfig($key) === null && array_key_exists($key, $defaults)) {
+			$plugin->setConfigProperty($key, $defaults[$key]);
+		}
+	}
+
+	$plugin->save();
+}
+
+// 0.19 - Réparation des schémas partiellement migrés
+if (version_compare($old_version, '0.19', '<'))
+{
+	$tables = [
+		'plugin_facturation_clients' => [
+			'siret'       => 'siret TEXT',
+			'telephone'   => 'telephone TEXT',
+			'email'       => 'email TEXT',
+			'nom_contact' => 'nom_contact TEXT',
+			'note'        => 'note TEXT',
+		],
+		'plugin_facturation_factures' => [
+			'nom_contact'       => 'nom_contact TEXT',
+			'numero_commande'   => 'numero_commande TEXT',
+			'reference_acheteur' => 'reference_acheteur TEXT',
+		],
+	];
+
+	foreach ($tables as $table => $columns) {
+		$schema = $db->getTableSchema($table);
+
+		if (!$schema) {
+			continue;
+		}
+
+		foreach ($columns as $column => $definition) {
+			if (!isset($schema['columns'][$column])) {
+				$db->exec(sprintf('ALTER TABLE %s ADD COLUMN %s;', $db->quoteIdentifier($table), $definition));
+			}
+		}
+	}
+}
+
+// 0.25 - Options d'affichage du rendu facture
+if (version_compare($old_version, '0.25', '<'))
+{
+	$defaults = json_decode(file_get_contents(__DIR__ . '/config.json'), true);
+
+	foreach (['invoice_show_address_borders', 'invoice_show_email', 'invoice_show_web'] as $key) {
+		if ($plugin->getConfig($key) === null && array_key_exists($key, $defaults)) {
+			$plugin->setConfigProperty($key, $defaults[$key]);
+		}
+	}
+
+	$plugin->save();
+}