From 92408f663c87a52da2185bd466ab9f0690035a51 Mon Sep 17 00:00:00 2001 From: bohwaz Date: Fri, 17 Dec 2021 14:35:12 +0100 Subject: [PATCH 1/3] =?UTF-8?q?Mise=20en=20commun=20du=20code=20entre=20mo?= =?UTF-8?q?difier=20et=20ajouter,=20moins=20de=20duplication=20=C3=A0=20g?= =?UTF-8?q?=C3=A9rer?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- templates/_form.tpl | 221 ++++++++++++++++++++++++++++++++ templates/facture_ajouter.tpl | 227 +-------------------------------- templates/facture_modifier.tpl | 222 +------------------------------- www/admin/facture_ajouter.php | 13 +- www/admin/facture_modifier.php | 13 +- 5 files changed, 239 insertions(+), 457 deletions(-) create mode 100644 templates/_form.tpl diff --git a/templates/_form.tpl b/templates/_form.tpl new file mode 100644 index 0000000..e557e2f --- /dev/null +++ b/templates/_form.tpl @@ -0,0 +1,221 @@ +{form_errors} + +
+ +
+ Type d'écriture +
+ {foreach from=$types_details item="type"} +
+ {input type="radio" name="type" value=$type.id source=$radio label=null} + +
+ {/foreach} +
+
+ +
+ Créer un devis + Créer une facture + Créer un reçu fiscal + Créer un reçu de cotisation +
+ + {input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=1 source=$doc help="Chaque document doit comporter un numéro unique délivré chronologiquement et de façon continue. Il faut que le système adopté par l'association garantisse que deux factures émises la même année ne peuvent pas porter le même numéro."} + + {input type="date" name="date_emission" default=$date label="Date d'émission" required=1 source=$doc} +
+

Date du versemen du don

+
+
+ {input type="date" name="date_echeance" default=$date label="Date d'échéance" required=1 source=$doc} +
+

Date d'établissement du document

+
+
+ +
+ + {input type="checkbox" name="reglee" value="1" label="Réglée" source=$doc data-types="t1"} +
+ {input type="checkbox" name="archivee" value="1" label="Archivée" source=$doc disabled="disabled"} +
+ +
+
+ +
+ Client + +
+
+ {if !empty($clients)} +
+ {input type="radio" name="base_receveur" value="membre" label="Un·e membre" default=1 source=$doc} + {input type="radio" name="base_receveur" value="client" label="Un·e client·e" source=$doc} +
+ {/if} +
+ +
+ {input type="select" name="membre" label="Membre" options=$membres required=1 source=$doc} +
+ + {if !empty($clients)} +
+ {input type="select" name="client" label="Client" options=$clients required=1 class="type_client" source=$doc} +
+ {else} + + {/if} +
+ +
+ Contenu + +
+ {input type="select" name="moyen_paiement" required=1 label="Moyen de paiement" source=$doc options=$moyens_paiement} + +
+
+ + + + + + + + + + + + + + + {if count($designations) > 0} + + + {money_fac name="prix_tpl[]"} + + + + {foreach from=$designations item=designation key=key} + + + {money_fac value=$prix[$key] user=$from_user} + + + {/foreach} + {else} + + + {money_fac name="prix_tpl[]"} + + + {/if} + + + + + + + + +
DésignationPrix
{button label="Enlever" title="Enlever la ligne" shape="minus" min="2" name="remove_line"}
Total :0.00{button label="Ajouter" title="Ajouter une ligne" id="ajouter_ligne" shape="plus"}
+
+
+
+ +

+ {csrf_field key=$csrf_key} + {button type="submit" name="save" label="Enregistrer" shape="right" class="main"} +

+ +
+ Membre +
+
+
+ {input type="select" name="membre_cotis" label="Membre" options=$membres required=1 source=$doc} +
+
+
+ +

+ {csrf_field key="add_cotis_1"} + {button type="submit" name="select_cotis" label="Sélectionner" shape="right" class="main"} +

+ + +{if $step} +
+ Cotisation + {if count($liste)} +
+
Sélectionnez la cotisation concernée :
+ + + + + + + + + + + + + + {foreach from=$liste item=cotis key=i} + {if !$cotis.paid} + {continue} + {/if} + + + {foreach from=$cotis item=element key=key} + {if $key == 'paid'} + {continue} + {/if} + + {/foreach} + + {/foreach} +
IdIntituléDate d'inscriptionExpiration d'expirationTarifMontantSomme payée
+ {input type="radio" name="cotisation" value="%s"|args:$i} + + +
+ +
+
+ +

+ {csrf_field key="add_cotis_2"} + {button type="submit" name="add_cotis" label="Enregistrer" shape="right" class="main"} +

+ {else} +

Ce membre n'a aucune cotisation payée.

+ + {/if} +{/if} + +
+ +{include file="%s/templates/_js.tpl"|args:$plugin_root} diff --git a/templates/facture_ajouter.tpl b/templates/facture_ajouter.tpl index 9aa9471..6e6ae58 100644 --- a/templates/facture_ajouter.tpl +++ b/templates/facture_ajouter.tpl @@ -1,232 +1,7 @@ {include file="admin/_head.tpl" title="Créer un document — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1} {include file="%s/templates/_menu.tpl"|args:$plugin_root current="facture"} -{form_errors} - -
- -
- Type d'écriture -
- {foreach from=$types_details item="type"} -
- {input type="radio" name="type" value=$type.id source=$radio label=null} - -
- {/foreach} -
-
- -
- Créer un devis - Créer une facture - Créer un reçu fiscal - Créer un reçu de cotisation -
- - {input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=1 help="Chaque document doit comporter un numéro unique délivré chronologiquement et de façon continue. Il faut que le système adopté par l'association garantisse que deux factures émises la même année ne peuvent pas porter le même numéro."} - - {input type="date" name="date_emission" default=$date label="Date d'émission" required=1 } -
-

Date du versemen du don

-
-
- {input type="date" name="date_echeance" default=$date label="Date d'échéance" required=1 } -
-

Date d'établissement du document

-
-
- -
- - {input type="checkbox" name="reglee" value="" label="Réglée" data-types="t1"} -
- {input type="checkbox" name="archivee" value="" label="Archivée" disabled="disabled"} -
- -
-
- -
- Client - -
-
- {if !empty($clients)} -
- {input type="radio" name="base_receveur" value="membre" label="Un·e membre" default=1} - {input type="radio" name="base_receveur" value="client" label="Un·e client·e"} -
- {/if} -
- -
- {input type="select" name="membre" label="Membre" options=$membres required=1} -
- - {if !empty($clients)} -
- {input type="select" name="client" label="Client" options=$clients required=1 class="type_client"} -
- {else} - - {/if} -
- -
- Contenu - -
-
obligatoire
-
- -
- -
-
- - - - - - - - - - - - - - - {if count($designations) > 0} - - - {money_fac name="prix_tpl[]"} - - - - {foreach from=$designations item=designation key=key} - - - {money_fac value=$prix[$key] user=$from_user} - - - {/foreach} - {else} - - - {money_fac name="prix_tpl[]"} - - - {/if} - - - - - - - - -
DésignationPrix
{button label="Enlever" title="Enlever la ligne" shape="minus" min="2" name="remove_line"}
Total :0.00{button label="Ajouter" title="Ajouter une ligne" id="ajouter_ligne" shape="plus"}
-
-
-
- -

- {csrf_field key="ajout_facture"} - {button type="submit" name="add" label="Enregistrer" shape="right" class="main"} -

- -
- Membre -
-
-
- {input type="select" name="membre_cotis" label="Membre" options=$membres required=1} -
-
-
- -

- {csrf_field key="add_cotis_1"} - {button type="submit" name="select_cotis" label="Sélectionner" shape="right" class="main"} -

- - -{if $step} -
- Cotisation - {if count($liste)} -
-
Sélectionnez la cotisation concernée :
- - - - - - - - - - - - - - {foreach from=$liste item=cotis key=i} - {if !$cotis.paid} - {continue} - {/if} - - - {foreach from=$cotis item=element key=key} - {if $key == 'paid'} - {continue} - {/if} - - {/foreach} - - {/foreach} -
IdIntituléDate d'inscriptionExpiration d'expirationTarifMontantSomme payée
- {input type="radio" name="cotisation" value="%s"|args:$i} - - -
- -
-
- -

- {csrf_field key="add_cotis_2"} - {button type="submit" name="add_cotis" label="Enregistrer" shape="right" class="main"} -

- {else} -

Ce membre n'a aucune cotisation payée.

- - {/if} -{/if} - -
+{include file="%s/templates/_form.tpl"|args:$plugin_root} {include file="%s/templates/_js.tpl"|args:$plugin_root} diff --git a/templates/facture_modifier.tpl b/templates/facture_modifier.tpl index 195753d..a738782 100644 --- a/templates/facture_modifier.tpl +++ b/templates/facture_modifier.tpl @@ -1,226 +1,6 @@ {include file="admin/_head.tpl" title="Modifier un document — %s"|args:$plugin.nom current="plugin_%s"|args:$plugin.id js=1} {include file="%s/templates/_menu.tpl"|args:$plugin_root current="index"} -{form_errors} - -
- -
- Type d'écriture -
- {foreach from=$types_details item="type"} -
- {input type="radio" name="type" value=$type.id source=$radio label=null} - -
- {/foreach} -
-
- -
- Créer un devis - Créer une facture - Créer un reçu fiscal - Créer un reçu de cotisation -
- - {input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=1 source=$doc help="Chaque document doit comporter un numéro unique délivré chronologiquement et de façon continue. Il faut que le système adopté par l'association garantisse que deux factures émises la même année ne peuvent pas porter le même numéro."} - - {input type="date" name="date_emission" default=$date label="Date d'émission" required=1 source=$doc} -
-

Date du versemen du don

-
-
- {input type="date" name="date_echeance" default=$date label="Date d'échéance" required=1 source=$doc} -
-

Date d'établissement du document

-
-
- -
- - {input type="checkbox" name="reglee" value="1" label="Réglée" source=$doc data-types="t1"} -
- {input type="checkbox" name="archivee" value="1" label="Archivée" source=$doc disabled="disabled"} -
- -
-
- -
- Client - -
-
- {if !empty($clients)} -
- {input type="radio" name="base_receveur" value="membre" label="Un·e membre" default=1 source=$doc} - {input type="radio" name="base_receveur" value="client" label="Un·e client·e" source=$doc} -
- {/if} -
- -
- {input type="select" name="membre" label="Membre" options=$membres required=1 source=$doc} -
- - {if !empty($clients)} -
- {input type="select" name="client" label="Client" options=$clients required=1 class="type_client" source=$doc} -
- {else} - - {/if} -
- -
- Contenu - -
- {input type="select" name="moyen_paiement" required=1 label="Moyen de paiement" source=$doc options=$doc.moyens_paiement} - -
-
- - - - - - - - - - - - - - - {if count($designations) > 0} - - - {money_fac name="prix_tpl[]"} - - - - {foreach from=$designations item=designation key=key} - - - {money_fac value=$prix[$key] user=$from_user} - - - {/foreach} - {else} - - - {money_fac name="prix_tpl[]"} - - - {/if} - - - - - - - - -
DésignationPrix
{button label="Enlever" title="Enlever la ligne" shape="minus" min="2" name="remove_line"}
Total :0.00{button label="Ajouter" title="Ajouter une ligne" id="ajouter_ligne" shape="plus"}
-
-
-
- -

- {csrf_field key="modifier_facture"} - {button type="submit" name="save" label="Enregistrer" shape="right" class="main"} -

- -
- Membre -
-
-
- {input type="select" name="membre_cotis" label="Membre" options=$membres required=1 source=$doc} -
-
-
- -

- {csrf_field key="add_cotis_1"} - {button type="submit" name="select_cotis" label="Sélectionner" shape="right" class="main"} -

- - -{if $step} -
- Cotisation - {if count($liste)} -
-
Sélectionnez la cotisation concernée :
- - - - - - - - - - - - - - {foreach from=$liste item=cotis key=i} - {if !$cotis.paid} - {continue} - {/if} - - - {foreach from=$cotis item=element key=key} - {if $key == 'paid'} - {continue} - {/if} - - {/foreach} - - {/foreach} -
IdIntituléDate d'inscriptionExpiration d'expirationTarifMontantSomme payée
- {input type="radio" name="cotisation" value="%s"|args:$i} - - -
- -
-
- -

- {csrf_field key="add_cotis_2"} - {button type="submit" name="add_cotis" label="Enregistrer" shape="right" class="main"} -

- {else} -

Ce membre n'a aucune cotisation payée.

- - {/if} -{/if} - -
- -{include file="%s/templates/_js.tpl"|args:$plugin_root} +{include file="%s/templates/_form.tpl"|args:$plugin_root} {include file="admin/_foot.tpl"} diff --git a/www/admin/facture_ajouter.php b/www/admin/facture_ajouter.php index 903d6f3..880d2a0 100644 --- a/www/admin/facture_ajouter.php +++ b/www/admin/facture_ajouter.php @@ -14,14 +14,19 @@ $db = DB::getInstance(); $step = $radio = false; $liste = []; +$csrf_key = 'ajout_facture'; $fields = $facture->recu_fields; -$tpl->assign('moyens_paiement', $facture->listMoyensPaiement()); +$moyens_paiement = $facture->listMoyensPaiement(true); + +$doc = null; + +$tpl->assign('moyens_paiement', $moyens_paiement); $tpl->assign('moyen_paiement', f('moyen_paiement') ?: 'ES'); if (f('add')) { - $form->check('ajout_facture', [ + $form->check($csrf_key, [ 'type' => 'required|in:'.implode(',', [DEVIS, FACT, CERFA]), 'numero_facture' => 'required|string', 'date_emission' => 'required|date_format:d/m/Y', @@ -31,7 +36,7 @@ if (f('add')) 'base_receveur' => 'required|in:membre,client', // 'client' => '', // 'membre' => '', - 'moyen_paiement' => 'required|in:' . implode(',', array_keys($facture->listMoyensPaiement())), + 'moyen_paiement' => 'required|in:' . implode(',', array_keys($moyens_paiement)), 'designation' => 'array|required', 'prix' => 'array|required' ]); @@ -214,7 +219,7 @@ $date = new \DateTime; $date->setTimestamp(time()); $tpl->assign('date', $date->format('d/m/Y')); -$tpl->assign(compact('designations', 'prix', 'from_user', 'identite')); +$tpl->assign(compact('designations', 'prix', 'from_user', 'identite', 'csrf_key', 'doc')); $tpl->assign('membres', $db->getAssoc('SELECT id, '.$identite.' FROM membres WHERE id_category != -2 NOT IN (SELECT id FROM users_categories WHERE hidden = 1);')); $tpl->assign('clients', $db->getAssoc('SELECT id, nom FROM plugin_facturation_clients;')); diff --git a/www/admin/facture_modifier.php b/www/admin/facture_modifier.php index 013be9e..b0d34f6 100644 --- a/www/admin/facture_modifier.php +++ b/www/admin/facture_modifier.php @@ -13,9 +13,9 @@ $db = DB::getInstance(); $step = false; $liste = []; -$fields = $facture->recu_fields; +$moyens_paiement = $facture->listMoyensPaiement(true); -$tpl->assign('moyens_paiement', $facture->listMoyensPaiement()); +$tpl->assign('moyens_paiement', $moyens_paiement); $tpl->assign('moyen_paiement', f('moyen_paiement') ?: 'ES'); qv(['id' => 'required|numeric']); @@ -26,11 +26,13 @@ if (!$f = $facture->get($id)) throw new UserException("Ce document n'existe pas."); } +$csrf_key = 'modifier_facture'; + // Traitement if(f('save')) { - $form->check('modifier_facture', [ + $form->check($csrf_key, [ 'type' => 'required|in:'.implode(',', [DEVIS, FACT, CERFA]), 'numero_facture' => 'required|string', 'date_emission' => 'required|date_format:d/m/Y', @@ -40,7 +42,7 @@ if(f('save')) 'base_receveur' => 'required|in:membre,client', // 'client' => '', // 'membre' => '', - 'moyen_paiement' => 'required|in:' . implode(',', array_keys($facture->listMoyensPaiement())), + 'moyen_paiement' => 'required|in:' . implode(',', array_keys($moyens_paiement)), 'designation' => 'array|required', 'prix' => 'array|required' ]); @@ -193,7 +195,6 @@ if ($step) // Affichage -$doc['moyens_paiement'] = $facture->listMoyensPaiement(true); // $doc['moyen_paiement'] = $doc['moyens_paiement'][$f->moyen_paiement]; $doc['moyen_paiement'] = $f->moyen_paiement; $doc['type'] = $f->type_facture; @@ -253,7 +254,7 @@ if ($f->type_facture != COTIS) } -$tpl->assign(compact('designations', 'prix', 'from_user', 'identite')); +$tpl->assign(compact('designations', 'prix', 'from_user', 'identite', 'csrf_key')); $tpl->assign('membres', $db->getAssoc('SELECT id, '.$identite.' FROM membres WHERE id_category != -2 NOT IN (SELECT id FROM users_categories WHERE hidden = 1);')); $tpl->assign('clients', $db->getAssoc('SELECT id, nom FROM plugin_facturation_clients;')); From 8e2ddf19c6a839a3d3720b027fb22923ea874684 Mon Sep 17 00:00:00 2001 From: bohwaz Date: Fri, 17 Dec 2021 14:49:03 +0100 Subject: [PATCH 2/3] Duplication de document pour transformer rapidement un devis en facture --- templates/_form.tpl | 4 ++-- templates/facture.tpl | 1 + www/admin/facture_ajouter.php | 31 ++++++++++++++++++++++++++++--- 3 files changed, 31 insertions(+), 5 deletions(-) diff --git a/templates/_form.tpl b/templates/_form.tpl index e557e2f..1b7b875 100644 --- a/templates/_form.tpl +++ b/templates/_form.tpl @@ -81,7 +81,7 @@ Contenu
- {input type="select" name="moyen_paiement" required=1 label="Moyen de paiement" source=$doc options=$moyens_paiement} + {input type="select" name="moyen_paiement" required=1 label="Moyen de paiement" source=$doc options=$moyens_paiement default="ES"}
@@ -143,7 +143,7 @@
- {input type="select" name="membre_cotis" label="Membre" options=$membres required=1 source=$doc} + {input type="select" name="membre_cotis" label="Membre" options=$membres required=1 default=$doc.membre}
diff --git a/templates/facture.tpl b/templates/facture.tpl index 5c26840..8edecd5 100644 --- a/templates/facture.tpl +++ b/templates/facture.tpl @@ -5,6 +5,7 @@ {if $session->canAccess($session::SECTION_ACCOUNTING, $session::ACCESS_WRITE)} {linkbutton shape="edit" href="%sfacture_modifier.php?id=%d"|args:$plugin_url,$facture.id label="Modifier ce document"} +{linkbutton shape="plus" href="%sfacture_ajouter.php?copy=%d"|args:$plugin_url,$facture.id label="Dupliquer ce document"} {/if} {linkbutton shape="download" href="%spdf.php?d&id=%d"|args:$plugin_url,$facture.id label="Télécharger ce document"} diff --git a/www/admin/facture_ajouter.php b/www/admin/facture_ajouter.php index 880d2a0..3f0d63d 100644 --- a/www/admin/facture_ajouter.php +++ b/www/admin/facture_ajouter.php @@ -13,6 +13,8 @@ $db = DB::getInstance(); $step = $radio = false; $liste = []; +$designations = []; +$prix = []; $csrf_key = 'ajout_facture'; $fields = $facture->recu_fields; @@ -21,10 +23,21 @@ $moyens_paiement = $facture->listMoyensPaiement(true); $doc = null; +if (qg('copy') !== null && $f = $facture->get((int)qg('copy'))) { + $doc = (array) $f; + + // Copié depuis facture_modifier.php + $doc['type'] = $f->type_facture; + $doc['numero_facture'] = $f->numero; + $doc['base_receveur'] = $f->receveur_membre ? 'membre' : 'client'; + $doc['client'] = $f->receveur_id; + $doc['membre'] = $f->receveur_id; +} + $tpl->assign('moyens_paiement', $moyens_paiement); $tpl->assign('moyen_paiement', f('moyen_paiement') ?: 'ES'); -if (f('add')) +if (f('save')) { $form->check($csrf_key, [ 'type' => 'required|in:'.implode(',', [DEVIS, FACT, CERFA]), @@ -181,6 +194,9 @@ elseif (null !== f('type')) { $radio['type'] = f('type'); } +elseif (isset($doc['type'])) { + $radio['type'] = $doc['type']; +} else { $radio['type'] = FACT; @@ -192,8 +208,6 @@ $tpl->assign('types_details', $facture->types); $tpl->assign('client_id', f('client') ?: -1); $tpl->assign('membre_id', f('membre') ?: -1); -$designations = []; -$prix = []; $from_user = false; if (($d = f('designation')) && ($p = f('prix')) && implode($d)) { @@ -208,6 +222,17 @@ if (($d = f('designation')) && ($p = f('prix')) && implode($d)) } $from_user = true; } +else if (!empty($doc['contenu'])) { + foreach($doc['contenu'] as $k=>$v) + { + if (empty($v['designation']) && empty($v['prix'])) + { + continue; + } + $designations[] = $v['designation']; + $prix[] = $v['prix']; + } +} else { $designations = ['Exemple']; $prix = [250]; From 2e7710f29c13fccfd4444c9b00c98aac288fedcc Mon Sep 17 00:00:00 2001 From: bohwaz Date: Fri, 17 Dec 2021 21:10:54 +0100 Subject: [PATCH 3/3] =?UTF-8?q?Ajout=20fonction=20:=20num=C3=A9rotation=20?= =?UTF-8?q?automatique=20des=20documents?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- config.json | 2 +- lib/Facture.php | 63 +++++++++++++++++++++++++++++++--- templates/_form.tpl | 9 ++++- templates/config.tpl | 12 ++++--- www/admin/_inc.php | 20 ++++++++--- www/admin/config.php | 4 +++ www/admin/facture_ajouter.php | 20 +++++++---- www/admin/facture_modifier.php | 2 ++ 8 files changed, 110 insertions(+), 22 deletions(-) diff --git a/config.json b/config.json index 0e0dcd2..5b28698 100644 --- a/config.json +++ b/config.json @@ -1,3 +1,3 @@ { - + 'pattern': '%{type}-%{year}-%{ynumber}' } \ No newline at end of file diff --git a/lib/Facture.php b/lib/Facture.php index b414394..6dc7b61 100644 --- a/lib/Facture.php +++ b/lib/Facture.php @@ -75,7 +75,7 @@ class Facture if(!is_array($data)){ $datas[$k] = trim($data); } - if ($datas[$k] === '') + if ($datas[$k] === '' && $k != 'numero') { throw new UserException("La valeur de $k est vide"); } @@ -189,18 +189,70 @@ class Facture } } - public function add($data) + public function add($data, ?string $number_pattern = null) { $db = DB::getInstance(); $this->_checkFields($data); + $generate_number = false; - if(isset($data['numero']) && $db->test('plugin_facturation_factures', 'numero = ? COLLATE NOCASE', $data['numero'])) + if (empty($data['numero']) && $number_pattern) { + $generate_number = true; + $data['numero'] = sha1(random_bytes(10)); + } + + if ($db->test('plugin_facturation_factures', 'numero = ? COLLATE NOCASE', $data['numero'])) { throw new UserException('Un document avec ce numéro existe déjà, hors le numéro doit être unique.'); } + $db->insert('plugin_facturation_factures', $data); - return $db->lastInsertRowId(); + $id = $db->lastInsertRowId(); + + if ($generate_number) { + $numero = $this->getNewNumber($number_pattern, $data['type_facture'], \DateTime::createFromFormat('!Y-m-d', $data['date_emission']), $id); + $db->update('plugin_facturation_factures', compact('numero'), 'id = ' . (int) $id); + } + + return $id; + } + + /** + * Renvoie un nouveau numéro de facture selon un motif défini et le type de document + */ + public function getNewNumber(string $pattern, int $type, \DateTimeInterface $date, int $id) + { + if ($type == DEVIS) { + $type = 'DEVIS'; + $t = 'D'; + } + elseif ($type == FACT) { + $type = 'FACT'; + $t = 'F'; + } + elseif ($type == CERFA) { + $type = 'CERFA'; + $t = 'RF'; + } + else { + $type = 'COTIS'; + $t = 'RC'; + } + + $year = $date->format('Y'); + $y = $date->format('y'); + + // On récupère le nombre de documents pour cette année + // vu qu'on vient d'ajouter un document, celui-ci est bien le dernier numéro + $ynumber = DB::getInstance()->count('plugin_facturation_factures', 'strftime(\'%Y\', date_emission) = ?', (string) $year); + + $data = compact('type', 't', 'year', 'y', 'ynumber', 'id'); + + return preg_replace_callback('/%(\d+)?\{([a-z]+)\}/', function ($match) use ($data) { + $v = $data[$match[2]]; + $type = ctype_digit($data[$match[2]]) ? 'd' : 's'; + return sprintf('%' . $match[1] . $type, $v); + }, $pattern); } public function get($id) @@ -284,6 +336,7 @@ class Facture ], 'date_emission' => [ 'label' => 'Émission', + 'order' => 'date_emission %s, id %1$s', ], 'date_echeance' => [ 'label' => 'Échéance', @@ -312,7 +365,7 @@ class Facture LEFT JOIN plugin_facturation_clients AS c ON f.receveur_membre = 0 AND c.id = f.receveur_id'; $list = new DynamicList($columns, $tables); - $list->orderBy('numero', true); + $list->orderBy('date_emission', true); $currency = Config::getInstance()->monnaie; diff --git a/templates/_form.tpl b/templates/_form.tpl index 1b7b875..8a55b0c 100644 --- a/templates/_form.tpl +++ b/templates/_form.tpl @@ -28,7 +28,14 @@ Créer un reçu de cotisation
- {input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=1 source=$doc help="Chaque document doit comporter un numéro unique délivré chronologiquement et de façon continue. Il faut que le système adopté par l'association garantisse que deux factures émises la même année ne peuvent pas porter le même numéro."} + {input type="text" name="numero_facture" maxlength=12 label="Numéro du document" required=$require_number source=$doc} + +
+ {if $require_number} + Chaque document doit comporter un numéro unique délivré chronologiquement et de façon continue. Il faut que le système adopté par l'association garantisse que deux factures émises la même année ne peuvent pas porter le même numéro. + {else} + Laisser vide pour qu'un numéro soit automatiquement affecté au format {$number_pattern}. + {/if} {input type="date" name="date_emission" default=$date label="Date d'émission" required=1 source=$doc}
diff --git a/templates/config.tpl b/templates/config.tpl index 286582c..6402890 100644 --- a/templates/config.tpl +++ b/templates/config.tpl @@ -20,10 +20,10 @@
Adresse
- {input type="text" name="numero_rue_asso" source=$plugin.config label="Numéro de rue" required=1 maxlength=5} - {input type="text" name="rue_asso" source=$plugin.config label="Nom de rue" required=1} - {input type="text" name="cp_asso" source=$plugin.config label="Code postal" required=1} - {input type="text" name="ville_asso" source=$plugin.config label="Ville" required=1} + {input type="text" name="numero_rue_asso" source=$plugin.config label="Numéro de rue" maxlength=5} + {input type="text" name="rue_asso" source=$plugin.config label="Nom de rue"} + {input type="text" name="cp_asso" source=$plugin.config label="Code postal"} + {input type="text" name="ville_asso" source=$plugin.config label="Ville"}
@@ -60,6 +60,10 @@
{input type="checkbox" name="validate_cp" value="1" source=$plugin.config label="Vérifier le code postal lors de saisie/modification de client (seulement FR)"} {input type="checkbox" name="unique_client_name" value="1" source=$plugin.config label="Noms des clients uniques"} + {input type="select" name="pattern" label="Format de numéro de document" required=false options=$patterns source=$plugin.config} +
+ F = Facture, D = Devis, RF = Reçu fiscal, RC = Reçu cotisation +
Pour personnaliser l'apparence de la facture, il faut pour l'instant se retrousser les manches et éditer soi-même le fichier www/admin/pdf.php du plugin !
diff --git a/www/admin/_inc.php b/www/admin/_inc.php index 463f4bb..b052a36 100644 --- a/www/admin/_inc.php +++ b/www/admin/_inc.php @@ -1,19 +1,31 @@ 'Aucun, le numéro sera à spécifier manuellement pour chaque document', + '%{type}-%{year}-%{ynumber}' => 'Type-Année-Numéro du document par année ("FACT-2021-42")', + '%{year}-%{type}-%04{ynumber}' => 'Année-Type-Numéro du document par année ("2021-DEVIS-0042")', + '%{t}-%{year}-%{ynumber}' => 'Type court-Année-Numéro du document par année ("F-2021-42")', + '%{y}%{t}%{ynumber}' => 'Année courte-Type court-Numéro du document par année ("21D42")', + '%{type}-%{id}' => 'Type - Numéro unique du document ("FACT-42")', + '%{t}%{id}' => 'Type court et numéro unique du document ("F42")', + '%{id}' => 'Numéro unique du document ("42"))', + '%06{id}' => 'Numéro unique du document sur 6 chiffres ("000042")', +]; $client = new Client; $facture = new Facture; -$tpl->assign('www_url', WWW_URL); +$tpl->assign('www_url', \Garradin\WWW_URL); $tpl->assign('f_obj', $facture); $tpl->assign('plugin_url', Utils::plugin_url()); diff --git a/www/admin/config.php b/www/admin/config.php index 9b9c8ce..dd449fe 100644 --- a/www/admin/config.php +++ b/www/admin/config.php @@ -29,6 +29,8 @@ if (f('save') && $form->check('facturation_config')) $plugin->setConfig('validate_cp', (bool)f('validate_cp')); $plugin->setConfig('unique_client_name', (bool)f('unique_client_name')); + $plugin->setConfig('pattern', f('pattern')); + Utils::redirect(PLUGIN_URL . 'config.php?ok'); } catch (UserException $e) @@ -122,6 +124,8 @@ else $tpl->assign('ok', qg('ok') !== null); +$tpl->assign('patterns', \Garradin\Plugin\Facturation\PATTERNS_LIST); + // $tpl->assign('max_size', Utils::getMaxUploadSize()); $tpl->display(PLUGIN_ROOT . '/templates/config.tpl'); diff --git a/www/admin/facture_ajouter.php b/www/admin/facture_ajouter.php index 3f0d63d..7d56042 100644 --- a/www/admin/facture_ajouter.php +++ b/www/admin/facture_ajouter.php @@ -2,6 +2,8 @@ namespace Garradin; +use const \Garradin\Plugin\Facturation\PATTERNS_LIST; + require_once __DIR__ . '/_inc.php'; $session->requireAccess($session::SECTION_ACCOUNTING, $session::ACCESS_WRITE); @@ -22,18 +24,22 @@ $fields = $facture->recu_fields; $moyens_paiement = $facture->listMoyensPaiement(true); $doc = null; +$require_number = $plugin->getConfig('pattern') ? false : true; if (qg('copy') !== null && $f = $facture->get((int)qg('copy'))) { $doc = (array) $f; // Copié depuis facture_modifier.php $doc['type'] = $f->type_facture; - $doc['numero_facture'] = $f->numero; + $doc['numero_facture'] = ''; $doc['base_receveur'] = $f->receveur_membre ? 'membre' : 'client'; $doc['client'] = $f->receveur_id; $doc['membre'] = $f->receveur_id; } +$tpl->assign('require_number', $require_number); +$tpl->assign('number_pattern', PATTERNS_LIST[$plugin->getConfig('pattern')]); + $tpl->assign('moyens_paiement', $moyens_paiement); $tpl->assign('moyen_paiement', f('moyen_paiement') ?: 'ES'); @@ -41,7 +47,7 @@ if (f('save')) { $form->check($csrf_key, [ 'type' => 'required|in:'.implode(',', [DEVIS, FACT, CERFA]), - 'numero_facture' => 'required|string', + 'numero_facture' => $require_number ? 'required|string' : 'string', 'date_emission' => 'required|date_format:d/m/Y', 'date_echeance' => 'required|date_format:d/m/Y', // 'reglee' => '', @@ -61,10 +67,10 @@ if (f('save')) if ( count(f('designation')) !== count(f('prix')) ) { throw new UserException('Nombre de désignations et de prix reçus différent.'); - } - + } + $truc = [ - 'numero' =>f('numero_facture'), + 'numero' => f('numero_facture'), 'date_emission' => f('date_emission'), 'date_echeance' => f('date_echeance'), 'reglee' => f('reglee') == 1?1:0, @@ -98,7 +104,7 @@ if (f('save')) $truc['receveur_id'] = f('membre'); } - $id = $facture->add($truc); + $id = $facture->add($truc, $plugin->getConfig('pattern')); Utils::redirect(PLUGIN_URL . 'facture.php?id='.(int)$id); @@ -158,7 +164,7 @@ elseif (f('add_cotis')) 'expiration' => $cotis['expiry'] ] ]; - $id = $facture->add($data); + $id = $facture->add($data, $plugin->getConfig('pattern')); Utils::redirect(PLUGIN_URL . 'facture.php?id='.(int)$id); } diff --git a/www/admin/facture_modifier.php b/www/admin/facture_modifier.php index b0d34f6..6b950fc 100644 --- a/www/admin/facture_modifier.php +++ b/www/admin/facture_modifier.php @@ -262,4 +262,6 @@ $date = new \DateTime; $date->setTimestamp(time()); $tpl->assign('date', $date->format('d/m/Y')); +$tpl->assign('require_number', true); + $tpl->display(PLUGIN_ROOT . '/templates/facture_modifier.tpl'); \ No newline at end of file