diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php index 8a222c3f6..998918612 100644 --- a/.php-cs-fixer.php +++ b/.php-cs-fixer.php @@ -3,8 +3,11 @@ $finder = (new PhpCsFixer\Finder()) ->name('*.php') ->in([ + __DIR__ . '/app', __DIR__ . '/db', - __DIR__ . '/sources/AppBundle', + __DIR__ . '/htdocs', + __DIR__ . '/sources', + __DIR__ . '/tests', ]) ; @@ -12,6 +15,7 @@ ->setRiskyAllowed(true) ->setRules([ '@PSR2' => true, + '@PHP74Migration:risky' => true, 'blank_line_after_opening_tag' => true, 'concat_space' => ['spacing' => 'one'], 'no_alias_functions' => true, diff --git a/app/AppCache.php b/app/AppCache.php index 639ec2cd7..185a6990b 100644 --- a/app/AppCache.php +++ b/app/AppCache.php @@ -1,5 +1,7 @@ getEnvironment(), ['dev', 'test'], true)) { - $bundles[] = new Symfony\Bundle\DebugBundle\DebugBundle(); - $bundles[] = new Symfony\Bundle\WebProfilerBundle\WebProfilerBundle(); + $bundles[] = new DebugBundle(); + $bundles[] = new WebProfilerBundle(); } return $bundles; @@ -37,16 +53,16 @@ public function getRootDir() public function getCacheDir() { - return dirname(__DIR__).'/var/cache/'.$this->getEnvironment(); + return dirname(__DIR__) . '/var/cache/' . $this->getEnvironment(); } public function getLogDir() { - return dirname(__DIR__).'/var/logs'; + return dirname(__DIR__) . '/var/logs'; } - public function registerContainerConfiguration(LoaderInterface $loader) + public function registerContainerConfiguration(LoaderInterface $loader): void { - $loader->load($this->getRootDir().'/config/config_'.$this->getEnvironment().'.yml'); + $loader->load($this->getRootDir() . '/config/config_' . $this->getEnvironment() . '.yml'); } } diff --git a/app/Resources/views/admin/association/membership/register.html.twig b/app/Resources/views/admin/association/membership/register.html.twig index 4f9703f1f..94375c81f 100644 --- a/app/Resources/views/admin/association/membership/register.html.twig +++ b/app/Resources/views/admin/association/membership/register.html.twig @@ -4,7 +4,7 @@ {% block submenu %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_ASSOCIATION') } )) }} {% endblock %} diff --git a/app/Resources/views/site/base.html.twig b/app/Resources/views/site/base.html.twig index 1d7facd2e..2bdadb03c 100644 --- a/app/Resources/views/site/base.html.twig +++ b/app/Resources/views/site/base.html.twig @@ -79,7 +79,7 @@
- {{ render(controller('AppBundle:Website\\Newsletter:subscribeForm')) }} + {{ render(controller('AppBundle\\Controller\\Website\\NewsletterController:subscribeForm')) }} Tous les trois mois, des nouvelles de L'AFUP
diff --git a/app/Resources/views/site/become_member.html.twig b/app/Resources/views/site/become_member.html.twig index 0785720a3..ffe632d24 100644 --- a/app/Resources/views/site/become_member.html.twig +++ b/app/Resources/views/site/become_member.html.twig @@ -10,7 +10,7 @@ {% block content %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_ASSOCIATION') } )) }}
diff --git a/app/Resources/views/site/cms_page/display.html.twig b/app/Resources/views/site/cms_page/display.html.twig index ead69a7f8..11180a5b9 100644 --- a/app/Resources/views/site/cms_page/display.html.twig +++ b/app/Resources/views/site/cms_page/display.html.twig @@ -9,7 +9,7 @@ {% block content %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': rubrique.feuille_associee } )) }}
diff --git a/app/Resources/views/site/company_membership/adhesion_entreprise.html.twig b/app/Resources/views/site/company_membership/adhesion_entreprise.html.twig index 4f517a491..b68743ca2 100644 --- a/app/Resources/views/site/company_membership/adhesion_entreprise.html.twig +++ b/app/Resources/views/site/company_membership/adhesion_entreprise.html.twig @@ -4,7 +4,7 @@ {% block submenu %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_ASSOCIATION') } )) }} {% endblock %} diff --git a/app/Resources/views/site/meetups/list.html.twig b/app/Resources/views/site/meetups/list.html.twig index 68da395b9..0695165aa 100644 --- a/app/Resources/views/site/meetups/list.html.twig +++ b/app/Resources/views/site/meetups/list.html.twig @@ -26,7 +26,7 @@ {% block content %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_ANTENNES') } )) }} diff --git a/app/Resources/views/site/news/list.html.twig b/app/Resources/views/site/news/list.html.twig index ed723ccb8..95954cd80 100644 --- a/app/Resources/views/site/news/list.html.twig +++ b/app/Resources/views/site/news/list.html.twig @@ -30,7 +30,7 @@ {% endif %} {{ render(controller( - 'AppBundle\\Controller\\Website\\PagerController::displayAction', + 'AppBundle\\Controller\\Website\\PagerController::display', { 'total_items': total_items, 'current_page': current_page, diff --git a/app/Resources/views/site/offices.html.twig b/app/Resources/views/site/offices.html.twig index 8b03769aa..14bfc06c5 100644 --- a/app/Resources/views/site/offices.html.twig +++ b/app/Resources/views/site/offices.html.twig @@ -289,7 +289,7 @@ {% block content %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_ANTENNES') } )) }}
diff --git a/app/Resources/views/site/techletter/index.html.twig b/app/Resources/views/site/techletter/index.html.twig index 33ae29666..6d5df30b7 100644 --- a/app/Resources/views/site/techletter/index.html.twig +++ b/app/Resources/views/site/techletter/index.html.twig @@ -10,7 +10,7 @@ {% block content %} {{ render(controller( - 'AppBundle\\Controller\\Website\\SecondaryMenuController::displayAction', + 'AppBundle\\Controller\\Website\\SecondaryMenuController::display', { 'feuille_id': constant('Afup\\Site\\Corporate\\Feuille::ID_FEUILLE_NOS_ACTIONS') } )) }} diff --git a/app/config/config.php b/app/config/config.php index fe2fbbf87..2a198b164 100644 --- a/app/config/config.php +++ b/app/config/config.php @@ -1,14 +1,16 @@ * @author Olivier Hoareau * @copyright 2010 Association Française des Utilisateurs de PHP - * + * * @category AFUP * @package AFUP * @group Pages diff --git a/htdocs/pages/administration/compta_balance.php b/htdocs/pages/administration/compta_balance.php index 70c24fdd6..9d82864ce 100644 --- a/htdocs/pages/administration/compta_balance.php +++ b/htdocs/pages/administration/compta_balance.php @@ -1,12 +1,13 @@ assign('action', $action); -if (isset($_GET['details']) && $_GET['details']) - $details=$_GET['details']; -else - $details =""; +$details = isset($_GET['details']) && $_GET['details'] ? $_GET['details'] : ""; $compta = new Comptabilite($bdd); -if (isset($_GET['id_periode']) && $_GET['id_periode']) - $id_periode=$_GET['id_periode']; -else - $id_periode=""; - +$id_periode = isset($_GET['id_periode']) && $_GET['id_periode'] ? $_GET['id_periode'] : ""; + $id_periode = $compta->obtenirPeriodeEnCours($id_periode); $smarty->assign('id_periode', $id_periode); $listPeriode = $compta->obtenirListPeriode(); -$smarty->assign('listPeriode', $listPeriode ); +$smarty->assign('listPeriode', $listPeriode); - $periode_debut=$listPeriode[$id_periode-1]['date_debut']; - $periode_fin=$listPeriode[$id_periode-1]['date_fin']; + $periode_debut=$listPeriode[$id_periode-1]['date_debut']; + $periode_fin=$listPeriode[$id_periode-1]['date_fin']; + + $smarty->assign('compteurLigne',1); - $smarty->assign('compteurLigne',1); - if ($action == 'lister') { - - $balance = $compta->obtenirBalance('',$periode_debut,$periode_fin); - $smarty->assign('balance', $balance); - - $totalDepense = $compta->obtenirTotalBalance(1,$periode_debut,$periode_fin); - $smarty->assign('totalDepense', $totalDepense); - - $totalRecette = $compta->obtenirTotalBalance(2,$periode_debut,$periode_fin); - $smarty->assign('totalRecette', $totalRecette); - - $difMontant = $totalRecette - $totalDepense ; - $smarty->assign('difMontant', $difMontant); - - if ($details!='') - { - $dataDetails = $compta->obtenirBalanceDetails($details,$periode_debut,$periode_fin); - $smarty->assign('dataDetails', $dataDetails); - - $sousTotal = $compta->obtenirSousTotalBalance($details,$periode_debut,$periode_fin); - $smarty->assign('sousTotal', $sousTotal); - } + $balance = $compta->obtenirBalance('',$periode_debut,$periode_fin); + $smarty->assign('balance', $balance); -} + $totalDepense = $compta->obtenirTotalBalance($periode_debut,$periode_fin, 1); + $smarty->assign('totalDepense', $totalDepense); -if ($action == 'view' && $details) { + $totalRecette = $compta->obtenirTotalBalance($periode_debut,$periode_fin, 2); + $smarty->assign('totalRecette', $totalRecette); - $dataDetails = $compta->obtenirBalanceDetails($details,$periode_debut,$periode_fin); - $smarty->assign('dataDetails', $dataDetails); + $difMontant = $totalRecette - $totalDepense ; + $smarty->assign('difMontant', $difMontant); - $sousTotal = $compta->obtenirSousTotalBalance($details,$periode_debut,$periode_fin); - $smarty->assign('sousTotal', $sousTotal); - + if ($details!='') { + $dataDetails = $compta->obtenirBalanceDetails($details,$periode_debut,$periode_fin); + $smarty->assign('dataDetails', $dataDetails); -} + $sousTotal = $compta->obtenirSousTotalBalance($details,$periode_debut,$periode_fin); + $smarty->assign('sousTotal', $sousTotal); + } +} +if ($action == 'view' && $details) { + $dataDetails = $compta->obtenirBalanceDetails($details,$periode_debut,$periode_fin); + $smarty->assign('dataDetails', $dataDetails); -?> + $sousTotal = $compta->obtenirSousTotalBalance($details,$periode_debut,$periode_fin); + $smarty->assign('sousTotal', $sousTotal); +} diff --git a/htdocs/pages/administration/compta_banque.php b/htdocs/pages/administration/compta_banque.php index 7acbf2be4..dd3633503 100644 --- a/htdocs/pages/administration/compta_banque.php +++ b/htdocs/pages/administration/compta_banque.php @@ -1,5 +1,7 @@ assign('action', $action); -if (isset($_GET['compte']) && $_GET['compte']) { - $compte = $_GET['compte']; -} else { - $compte = 1; -} +$compte = isset($_GET['compte']) && $_GET['compte'] ? $_GET['compte'] : 1; $compta = new Comptabilite($bdd); -if (isset($_GET['id_periode']) && $_GET['id_periode']) { - $id_periode = $_GET['id_periode']; -} else { - $id_periode = ""; -} +$id_periode = isset($_GET['id_periode']) && $_GET['id_periode'] ? $_GET['id_periode'] : ""; $id_periode = $compta->obtenirPeriodeEnCours($id_periode); $smarty->assign('id_periode', $id_periode); @@ -60,17 +53,17 @@ $journal = $compta->obtenirJournalBanque($compte, $periode_debut, $periode_fin); $smarty->assign('journal', $journal); - $sousTotal = $compta->obtenirSousTotalJournalBanque($compte, $periode_debut, $periode_fin); + $sousTotal = $compta->obtenirSousTotalJournalBanque($periode_debut, $periode_fin, $compte); $smarty->assign('sousTotal', $sousTotal); - $total = $compta->obtenirTotalJournalBanque($compte, $periode_debut, $periode_fin); + $total = $compta->obtenirTotalJournalBanque($periode_debut, $periode_fin, $compte); $smarty->assign('total', $total); } elseif ($action == 'exporter') { $periode_debut = $listPeriode[$id_periode - 1]['date_debut']; $periode_fin = $listPeriode[$id_periode - 1]['date_fin']; $journal = $compta->obtenirJournalBanque($compte, $periode_debut, $periode_fin); - $sousTotal = $compta->obtenirSousTotalJournalBanque($compte, $periode_debut, $periode_fin); + $sousTotal = $compta->obtenirSousTotalJournalBanque($periode_debut, $periode_fin, $compte); setlocale(LC_TIME, 'fr_FR.utf8', 'fra'); //require_once 'PEAR/PHPExcel.php'; @@ -79,9 +72,9 @@ for ($i = 1; $i < 13; $i++) { $compteurLigne[$i] = 4; $sheet = $workbook->createSheet($i); - $sheet->setTitle('Mois de ' . strftime('%B %Y', mktime(0, 0, 0, $i, 1, date('Y', strtotime($periode_debut))))); + $sheet->setTitle('Mois de ' . strftime('%B %Y', mktime(0, 0, 0, $i, 1, (int) date('Y', strtotime($periode_debut))))); $sheet->setCellValue('A1', - 'Mois de ' . strftime('%B %Y', mktime(0, 0, 0, $i, 1, date('Y', strtotime($periode_debut))))); + 'Mois de ' . strftime('%B %Y', mktime(0, 0, 0, $i, 1, (int) date('Y', strtotime($periode_debut))))); $sheet->setCellValue('A3', 'Date'); $sheet->setCellValue('B3', 'Opération'); $sheet->setCellValue('C3', 'Description'); @@ -185,7 +178,6 @@ $objDrawing->setHeight(35); $objDrawing->setWidth(70); $objDrawing->setWorksheet($sheet); - } //$workbook->removeSheetByIndex(0); @@ -231,7 +223,6 @@ unlink($zipFilename); exit; } - } catch (Exception $e) { header('HTTP/1.1 400 Bad Request'); header('X-Info: ' . $e->getMessage()); diff --git a/htdocs/pages/administration/compta_bilan.php b/htdocs/pages/administration/compta_bilan.php index 3ebf97294..8fb9f99b9 100644 --- a/htdocs/pages/administration/compta_bilan.php +++ b/htdocs/pages/administration/compta_bilan.php @@ -1,89 +1,78 @@ assign('action', $action); //$compte=$_GET['compte']; -if (isset($_GET['details']) && $_GET['details']) - $details=$_GET['details']; -else - $details =""; - +$details = isset($_GET['details']) && $_GET['details'] ? $_GET['details'] : ""; + $compta = new Comptabilite($bdd); -if (isset($_GET['id_periode']) && $_GET['id_periode']) - $id_periode=$_GET['id_periode']; -else - $id_periode=""; - +$id_periode = isset($_GET['id_periode']) && $_GET['id_periode'] ? $_GET['id_periode'] : ""; + $id_periode = $compta->obtenirPeriodeEnCours($id_periode); $smarty->assign('id_periode', $id_periode); $listPeriode = $compta->obtenirListPeriode(); -$smarty->assign('listPeriode', $listPeriode ); +$smarty->assign('listPeriode', $listPeriode); + + $periode_debut=$listPeriode[$id_periode-1]['date_debut']; + $periode_fin=$listPeriode[$id_periode-1]['date_fin']; - $periode_debut=$listPeriode[$id_periode-1]['date_debut']; - $periode_fin=$listPeriode[$id_periode-1]['date_fin']; - if ($action == 'lister') { + $debit = $compta->obtenirBilan(1,$periode_debut,$periode_fin); + $smarty->assign('debit', $debit); + + $credit = $compta->obtenirBilan(2,$periode_debut,$periode_fin); + $smarty->assign('credit', $credit); + + // $journal = $compta->obtenirBilan(); + // $smarty->assign('bilan', $journal); + + $totalDepense = $compta->obtenirTotalBilan($periode_debut,$periode_fin, 1); + $smarty->assign('totalDepense', $totalDepense); - $debit = $compta->obtenirBilan(1,$periode_debut,$periode_fin); - $smarty->assign('debit', $debit); - - $credit = $compta->obtenirBilan(2,$periode_debut,$periode_fin); - $smarty->assign('credit', $credit); - -// $journal = $compta->obtenirBilan(); -// $smarty->assign('bilan', $journal); - - $totalDepense = $compta->obtenirTotalBilan(1,$periode_debut,$periode_fin); - $smarty->assign('totalDepense', $totalDepense); - - $totalRecette = $compta->obtenirTotalBilan(2,$periode_debut,$periode_fin); - $smarty->assign('totalRecette', $totalRecette); - - $difMontant = $totalRecette - $totalDepense ; - $smarty->assign('difMontant', $difMontant); - - if ($details!='') - { - $dataDetailsDebit = $compta->obtenirBilanDetails(1,$periode_debut,$periode_fin,$details); - $smarty->assign('dataDetailsDebit', $dataDetailsDebit); - - $dataDetailsCredit = $compta->obtenirBilanDetails(2,$periode_debut,$periode_fin,$details); - $smarty->assign('dataDetailsCredit', $dataDetailsCredit); - - } - + $totalRecette = $compta->obtenirTotalBilan($periode_debut,$periode_fin, 2); + $smarty->assign('totalRecette', $totalRecette); + + $difMontant = $totalRecette - $totalDepense ; + $smarty->assign('difMontant', $difMontant); + + if ($details!='') { + $dataDetailsDebit = $compta->obtenirBilanDetails(1, $details,$periode_debut,$periode_fin); + $smarty->assign('dataDetailsDebit', $dataDetailsDebit); + + $dataDetailsCredit = $compta->obtenirBilanDetails(2, $details,$periode_debut,$periode_fin); + $smarty->assign('dataDetailsCredit', $dataDetailsCredit); + } } elseif ($action == 'view' && $details) { + $dataDetailsDebit = $compta->obtenirBilanDetails(1, $details,$periode_debut,$periode_fin); + $smarty->assign('dataDetailsDebit', $dataDetailsDebit); + + $dataDetailsCredit = $compta->obtenirBilanDetails(2, $details,$periode_debut,$periode_fin); + $smarty->assign('dataDetailsCredit', $dataDetailsCredit); + + $sousTotalDebit = $compta->obtenirSousTotalBilan($periode_debut,$periode_fin,$details, 1); + $smarty->assign('sousTotalDebit', $sousTotalDebit); + + $sousTotalCredit = $compta->obtenirSousTotalBilan($periode_debut,$periode_fin,$details, 2); + $smarty->assign('sousTotalCredit', $sousTotalCredit); - $dataDetailsDebit = $compta->obtenirBilanDetails(1,$periode_debut,$periode_fin,$details); - $smarty->assign('dataDetailsDebit', $dataDetailsDebit); - - $dataDetailsCredit = $compta->obtenirBilanDetails(2,$periode_debut,$periode_fin,$details); - $smarty->assign('dataDetailsCredit', $dataDetailsCredit); - - $sousTotalDebit = $compta->obtenirSousTotalBilan(1,$periode_debut,$periode_fin,$details); - $smarty->assign('sousTotalDebit', $sousTotalDebit); - - $sousTotalCredit = $compta->obtenirSousTotalBilan(2,$periode_debut,$periode_fin,$details); - $smarty->assign('sousTotalCredit', $sousTotalCredit); - - $difMontant = $sousTotalCredit - $sousTotalDebit ; - $smarty->assign('difMontant', $difMontant); - + $difMontant = $sousTotalCredit - $sousTotalDebit ; + $smarty->assign('difMontant', $difMontant); } elseif ($action == 'supprimer') { - if ($compta->supprimerEcriture($_GET['id']) ) { + if ($compta->supprimerEcriture($_GET['id'])) { Logs::log('Suppression de l\'écriture ' . $_GET['id']); afficherMessage('L\'écriture a été supprimée', 'index.php?page=compta_journal&action=lister'); } else { diff --git a/htdocs/pages/administration/compta_conf_categorie.php b/htdocs/pages/administration/compta_conf_categorie.php index 2685c7628..ac7759b78 100644 --- a/htdocs/pages/administration/compta_conf_categorie.php +++ b/htdocs/pages/administration/compta_conf_categorie.php @@ -1,10 +1,11 @@ obtenirListCategories(true); - $smarty->assign('data', $data); - - - + $data = $compta->obtenirListCategories(true); + $smarty->assign('data', $data); } elseif ($action == 'ajouter' || $action == 'modifier') { + $formulaire = instancierFormulaire(); - $formulaire = instancierFormulaire(); - - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenirListCategories('',$_GET['id']); $champs['categorie'] = $champsRecup['categorie']; - $formulaire->setDefaults($champs); + $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } + $formulaire->addElement('hidden', 'id', $_GET['id']); + } -// partie saisie - $formulaire->addElement('header' , '' , ''); - $formulaire->addElement('text', 'categorie', 'Categorie' , ['size' => 30, 'maxlength' => 40]); + // partie saisie + $formulaire->addElement('header' , '' , ''); + $formulaire->addElement('text', 'categorie', 'Categorie' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); + $valeur = $formulaire->exportValues(); - if ($action == 'ajouter') { - $ok = $compta->ajouterConfig( - 'compta_categorie', - 'categorie', - $valeur['categorie'] - ); + if ($action === 'ajouter') { + $ok = $compta->ajouterConfig( + 'compta_categorie', + 'categorie', + $valeur['categorie'] + ); } else { - $ok = $compta->modifierConfig( - 'compta_categorie', - $valeur['id'], - 'categorie', - $valeur['categorie'] - ); + $ok = $compta->modifierConfig( + 'compta_categorie', + $valeur['id'], + 'categorie', + $valeur['categorie'] + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_categorie&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_categorie&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); - } - -?> diff --git a/htdocs/pages/administration/compta_conf_compte.php b/htdocs/pages/administration/compta_conf_compte.php index e45ab350f..67d43ca26 100644 --- a/htdocs/pages/administration/compta_conf_compte.php +++ b/htdocs/pages/administration/compta_conf_compte.php @@ -1,10 +1,11 @@ obtenirListComptes(true); - $smarty->assign('data', $data); - - - + $data = $compta->obtenirListComptes(true); + $smarty->assign('data', $data); } elseif ($action == 'ajouter' || $action == 'modifier') { + $formulaire = instancierFormulaire(); - $formulaire = instancierFormulaire(); - - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenirListCategories('',$_GET['id']); $champs['categorie'] = $champsRecup['categorie']; - $formulaire->setDefaults($champs); + $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } + $formulaire->addElement('hidden', 'id', $_GET['id']); + } -// partie saisie - $formulaire->addElement('header' , '' , ''); - $formulaire->addElement('text', 'nom_compte', 'Compte' , ['size' => 30, 'maxlength' => 40]); + // partie saisie + $formulaire->addElement('header' , '' , ''); + $formulaire->addElement('text', 'nom_compte', 'Compte' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); + $valeur = $formulaire->exportValues(); - if ($action == 'ajouter') { - $ok = $compta->ajouterConfig( - 'compta_compte', - 'nom_compte', - $valeur['nom_compte'] - ); + if ($action === 'ajouter') { + $ok = $compta->ajouterConfig( + 'compta_compte', + 'nom_compte', + $valeur['nom_compte'] + ); } else { - $ok = $compta->modifierConfig( - 'compta_compte', - $valeur['id'], - 'nom_compte', - $valeur['nom_compte'] - ); + $ok = $compta->modifierConfig( + 'compta_compte', + $valeur['id'], + 'nom_compte', + $valeur['nom_compte'] + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_compte&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_compte&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); - } diff --git a/htdocs/pages/administration/compta_conf_evenement.php b/htdocs/pages/administration/compta_conf_evenement.php index 462a6136d..bdd2d91e8 100644 --- a/htdocs/pages/administration/compta_conf_evenement.php +++ b/htdocs/pages/administration/compta_conf_evenement.php @@ -1,10 +1,11 @@ obtenirListEvenements(true); - $smarty->assign('data', $data); - + $data = $compta->obtenirListEvenements(true); + $smarty->assign('data', $data); } elseif ($action == 'ajouter' || $action == 'modifier') { + $formulaire = instancierFormulaire(); - $formulaire = instancierFormulaire(); - - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenirListEvenements('',$_GET['id']); $champs['evenement'] = $champsRecup['evenement']; - $formulaire->setDefaults($champs); + $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } + $formulaire->addElement('hidden', 'id', $_GET['id']); + } -// partie saisie - $formulaire->addElement('header' , '' , ''); - $formulaire->addElement('text', 'evenement', 'Nom Evenement' , ['size' => 30, 'maxlength' => 40]); + // partie saisie + $formulaire->addElement('header' , '' , ''); + $formulaire->addElement('text', 'evenement', 'Nom Evenement' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); + $valeur = $formulaire->exportValues(); - if ($action == 'ajouter') { - $ok = $compta->ajouterConfig( - 'compta_evenement', - 'evenement', - $valeur['evenement'] - ); + if ($action === 'ajouter') { + $ok = $compta->ajouterConfig( + 'compta_evenement', + 'evenement', + $valeur['evenement'] + ); } else { - $ok = $compta->modifierConfig( - 'compta_evenement', - $valeur['id'], - 'evenement', - $valeur['evenement'] - ); + $ok = $compta->modifierConfig( + 'compta_evenement', + $valeur['id'], + 'evenement', + $valeur['evenement'] + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_evenement&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_evenement&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); - } - -?> diff --git a/htdocs/pages/administration/compta_conf_operation.php b/htdocs/pages/administration/compta_conf_operation.php index ae9f47ef4..7d74ecf98 100644 --- a/htdocs/pages/administration/compta_conf_operation.php +++ b/htdocs/pages/administration/compta_conf_operation.php @@ -1,10 +1,11 @@ obtenirListOperations(true); - $smarty->assign('data', $data); - - + $data = $compta->obtenirListOperations(true); + $smarty->assign('data', $data); } elseif ($action == 'ajouter' || $action == 'modifier') { + $formulaire = instancierFormulaire(); - $formulaire = instancierFormulaire(); - - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenirListOperations('',$_GET['id']); $champs['operation'] = $champsRecup['operation']; - $formulaire->setDefaults($champs); + $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } + $formulaire->addElement('hidden', 'id', $_GET['id']); + } -// partie saisie - $formulaire->addElement('header' , '' , ''); - $formulaire->addElement('text', 'operation', 'Operation' , ['size' => 30, 'maxlength' => 40]); + // partie saisie + $formulaire->addElement('header' , '' , ''); + $formulaire->addElement('text', 'operation', 'Operation' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); + $valeur = $formulaire->exportValues(); - if ($action == 'ajouter') { - $ok = $compta->ajouterConfig( - 'compta_operation', - 'operation', - $valeur['operation'] - ); + if ($action === 'ajouter') { + $ok = $compta->ajouterConfig( + 'compta_operation', + 'operation', + $valeur['operation'] + ); } else { - $ok = $compta->modifierConfig( - 'compta_operation', - $valeur['id'], - 'operation', - $valeur['operation'] - ); + $ok = $compta->modifierConfig( + 'compta_operation', + $valeur['id'], + 'operation', + $valeur['operation'] + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_operation&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_operation&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); - } - -?> diff --git a/htdocs/pages/administration/compta_conf_regle.php b/htdocs/pages/administration/compta_conf_regle.php index 31f6e5236..fc710277b 100644 --- a/htdocs/pages/administration/compta_conf_regle.php +++ b/htdocs/pages/administration/compta_conf_regle.php @@ -1,5 +1,7 @@ obtenirListRegles('', (int)$_GET['id']); + $champs = $compta->obtenirListRegles('', (int) $_GET['id']); $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', (int)$_GET['id']); + $formulaire->addElement('hidden', 'id', (int) $_GET['id']); } // partie saisie @@ -59,18 +61,18 @@ if ($ok) { if ($action === 'ajouter') { - Logs::log('Ajout une règle '.$formulaire->exportValue('label')); + Logs::log('Ajout une règle ' . $formulaire->exportValue('label')); } else { - Logs::log('Modification une règle '.$formulaire->exportValue('label').' ('.$_GET['id'].')'); + Logs::log('Modification une règle ' . $formulaire->exportValue('label') . ' (' . $_GET['id'] . ')'); } afficherMessage( - 'La règle a été '.(($action === 'ajouter') ? 'ajoutée' : 'modifiée'), + 'La règle a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_regle&action=lister' ); } else { $smarty->assign( 'erreur', - 'Une erreur est survenue lors de '.(($action === 'ajouter') ? "l'ajout" : 'la modification').' de la règle' + 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de la règle' ); } } diff --git a/htdocs/pages/administration/compta_conf_reglement.php b/htdocs/pages/administration/compta_conf_reglement.php index f86fa563c..3af93eecf 100644 --- a/htdocs/pages/administration/compta_conf_reglement.php +++ b/htdocs/pages/administration/compta_conf_reglement.php @@ -1,10 +1,11 @@ obtenirListReglements(true); - $smarty->assign('data', $data); - + $data = $compta->obtenirListReglements(true); + $smarty->assign('data', $data); } elseif ($action == 'ajouter' || $action == 'modifier') { - $formulaire = instancierFormulaire(); + $formulaire = instancierFormulaire(); - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenirListReglements('',$_GET['id']); $champs['reglement'] = $champsRecup['reglement']; - $formulaire->setDefaults($champs); + $formulaire->setDefaults($champs); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } + $formulaire->addElement('hidden', 'id', $_GET['id']); + } -// partie saisie - $formulaire->addElement('header' , '' , ''); - $formulaire->addElement('text', 'reglement', 'Nom Reglement' , ['size' => 30, 'maxlength' => 40]); + // partie saisie + $formulaire->addElement('header' , '' , ''); + $formulaire->addElement('text', 'reglement', 'Nom Reglement' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); + $valeur = $formulaire->exportValues(); - if ($action == 'ajouter') { - $ok = $compta->ajouterConfig( - 'compta_reglement', - 'reglement', - $valeur['reglement'] - ); + if ($action === 'ajouter') { + $ok = $compta->ajouterConfig( + 'compta_reglement', + 'reglement', + $valeur['reglement'] + ); } else { - $ok = $compta->modifierConfig( - 'compta_reglement', - $valeur['id'], - 'reglement', - $valeur['reglement'] - ); + $ok = $compta->modifierConfig( + 'compta_reglement', + $valeur['id'], + 'reglement', + $valeur['reglement'] + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_reglement&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_conf_reglement&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); - } - -?> diff --git a/htdocs/pages/administration/compta_devis.php b/htdocs/pages/administration/compta_devis.php index f2c4b6faf..c4fb1dd0e 100644 --- a/htdocs/pages/administration/compta_devis.php +++ b/htdocs/pages/administration/compta_devis.php @@ -1,12 +1,13 @@ obtenirPeriodeEnCours($id_periode); - $ecritures = $comptaFact->obtenirDevis($id_periode); - $smarty->assign('id_periode', $id_periode); $smarty->assign('ecritures', $ecritures); $smarty->assign('listPeriode', $compta->obtenirListPeriode()); @@ -52,7 +44,8 @@ $formulaire = instancierFormulaire(); - function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { + function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) + { $champsRecup = $comptaFact->obtenir($id); $champs['date_devis'] = $champsRecup['date_devis']; @@ -80,60 +73,46 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { $champsRecup = $comptaFact->obtenir_details($id); $i=1; - foreach ($champsRecup as $row) - { - $champs['id'.$i] = $row['id']; - $champs['ref'.$i] = $row['ref']; - $champs['designation'.$i] = $row['designation']; - $champs['quantite'.$i] = $row['quantite']; - $champs['pu'.$i] = $row['pu']; - $champs['tva'.$i] = $row['tva']; + foreach ($champsRecup as $row) { + $champs['id' . $i] = $row['id']; + $champs['ref' . $i] = $row['ref']; + $champs['designation' . $i] = $row['designation']; + $champs['quantite' . $i] = $row['quantite']; + $champs['pu' . $i] = $row['pu']; + $champs['tva' . $i] = $row['tva']; $i++; } return $champs; } - if ($action == 'modifier') - { + if ($action === 'modifier') { $champs = prepareDefaultsFromComptaFacId($comptaFact, $_GET['id']); $formulaire->setDefaults($champs); $formulaire->addElement('hidden', 'id', $_GET['id']); - - } - else - { - if (strlen($_GET['from'])) { - $champsDefaults = prepareDefaultsFromComptaFacId($comptaFact, $_GET['from']); - } else { - $champsDefaults = []; - } - + } else { + $champsDefaults = strlen($_GET['from'] ?? '') !== 0 ? prepareDefaultsFromComptaFacId($comptaFact, $_GET['from']) : []; $champsDefaults['date_devis'] = date('d F Y'); $champsDefaults['id_pays'] = "FR"; - $champs['numero_devis'] = ""; $champs['numero_facture'] = ""; $formulaire->setDefaults($champsDefaults); - } -//detail devis - $formulaire->addElement('header' , '' , 'Détail Devis'); + //detail devis + $formulaire->addElement('header' , '' , 'Détail Devis'); -//$mois=10; - if ($action == 'modifier') - { - $formulaire->addElement('date' , 'date_devis' , 'Date devis', ['language' => 'fr', + //$mois=10; + if ($action === 'modifier') { + $formulaire->addElement('date' , 'date_devis' , 'Date devis', ['language' => 'fr', 'format' => 'd F Y', 'minYear' => date('Y')-3, 'maxYear' => date('Y')]); - } else { - $formulaire->addElement('date' , 'date_devis' , 'Date devis', ['language' => 'fr', + } else { + $formulaire->addElement('date' , 'date_devis' , 'Date devis', ['language' => 'fr', 'format' => 'd F Y', 'minYear' => date('Y'), 'maxYear' => date('Y')]); - - } + } $formulaire->addElement('header' , '' , 'Facturation'); $formulaire->addElement('static' , 'note' , '' , 'Ces informations concernent la personne ou la société qui sera facturée

'); $formulaire->addElement('text' , 'societe' , 'Société' , ['size' => 50, 'maxlength' => 100]); @@ -150,14 +129,15 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { $formulaire->addElement('text' , 'email' , 'Email (facture)', ['size' => 30, 'maxlength' => 100]); $formulaire->addElement('text' , 'tva_intra' , 'TVA intracommunautaire (facture)', ['size' => 30, 'maxlength' => 100]); - if (isset($champs['numero_devis']) || isset($champs['numero_facture']) ) - { + if (isset($champs['numero_devis']) || isset($champs['numero_facture'])) { $formulaire->addElement('header', null , 'Réservé à l\'administration'); $formulaire->addElement('static' , 'note' , '' , 'Numéro généré automatiquement et affiché en automatique'); - if ($champs['numero_devis']) + if ($champs['numero_devis']) { $formulaire->addElement('text' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); - if ($champs['numero_facture']) + } + if ($champs['numero_facture']) { $formulaire->addElement('text' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); + } } else { $formulaire->addElement('hidden' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('hidden' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); @@ -169,39 +149,38 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { $formulaire->addElement('text' , 'ref_clt2' , 'Référence client 2', ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('text' , 'ref_clt3' , 'Référence client 3' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('header' , '', 'Observation'); + $formulaire->addElement('header' , '', 'Observation'); $formulaire->addElement('static' , 'note' , '' , 'Ces informations seront écrites à la fin du document

'); - $formulaire->addElement('textarea', 'observation' , 'Observation', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('textarea', 'observation' , 'Observation', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('header' , '', 'Devise'); - $formulaire->addElement('select', 'devise_facture' , 'Monnaie de la facture', ['EUR' => 'Euro', + $formulaire->addElement('header' , '', 'Devise'); + $formulaire->addElement('select', 'devise_facture' , 'Monnaie de la facture', ['EUR' => 'Euro', 'DOL' => 'Dollar'], ['size' => 2]); - for ($i=1;$i<6;$i++) - { - $formulaire->addElement('header' , '', 'Contenu'); - $formulaire->addElement('static' , 'note' , '' , 'Ligne '.$i.'

'); - $formulaire->addElement('hidden' , 'id'.$i , 'id' ); - $formulaire->addElement('text' , 'ref'.$i , 'Référence' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('static' , 'note' , '' , 'Rappel : sponsoring 20%, place supplémentaire 10%.
'); - $formulaire->addElement('select' , 'tva' . $i , 'Taux de TVA' , ['0' => 'Non soumis', '5.50' => '5.5%', '10.00' => '10%', '20.00' => '20%']); - $formulaire->addElement('textarea', 'designation'.$i , 'Désignation', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('text' , 'quantite'.$i , 'Quantite' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('text' , 'pu'.$i , 'Prix Unitaire HT' , ['size' => 50, 'maxlength' => 100]); - } + for ($i=1;$i<6;$i++) { + $formulaire->addElement('header' , '', 'Contenu'); + $formulaire->addElement('static' , 'note' , '' , 'Ligne ' . $i . '

'); + $formulaire->addElement('hidden' , 'id' . $i , 'id'); + $formulaire->addElement('text' , 'ref' . $i , 'Référence' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('static' , 'note' , '' , 'Rappel : sponsoring 20%, place supplémentaire 10%.
'); + $formulaire->addElement('select' , 'tva' . $i , 'Taux de TVA' , ['0' => 'Non soumis', '5.50' => '5.5%', '10.00' => '10%', '20.00' => '20%']); + $formulaire->addElement('textarea', 'designation' . $i , 'Désignation', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('text' , 'quantite' . $i , 'Quantite' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'pu' . $i , 'Prix Unitaire HT' , ['size' => 50, 'maxlength' => 100]); + } -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); -// ajoute des regles -// $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); -// $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); + // ajoute des regles + // $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); + // $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); $formulaire->addRule('societe' , 'Société manquant' , 'required'); $formulaire->addRule('adresse' , 'Adresse manquant' , 'required'); $formulaire->addRule('email' , 'Email manquant' , 'required'); @@ -209,9 +188,9 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { if ($formulaire->validate()) { $valeur = $formulaire->exportValues(); -$date_devis= $valeur['date_devis']['Y']."-".$valeur['date_devis']['F']."-".$valeur['date_devis']['d'] ; + $date_devis= $valeur['date_devis']['Y'] . "-" . $valeur['date_devis']['F'] . "-" . $valeur['date_devis']['d'] ; - if ($action == 'ajouter') { + if ($action === 'ajouter') { $ok = $comptaFact->ajouter( $date_devis, $valeur['societe'], @@ -234,14 +213,13 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { $valeur['devise_facture'] ); - for ($i=1;$i<6;$i++) - { + for ($i=1;$i<6;$i++) { $ok = $comptaFact->ajouter_details( - $valeur['ref'.$i], - $valeur['designation'.$i], - (int)$valeur['quantite'.$i], - (float)$valeur['pu'.$i], - (int)$valeur['tva'.$i] + $valeur['ref' . $i], + $valeur['designation' . $i], + (int) $valeur['quantite' . $i], + (float) $valeur['pu' . $i], + (int) $valeur['tva' . $i] ); } } else { @@ -269,29 +247,27 @@ function prepareDefaultsFromComptaFacId(Facture $comptaFact, $id) { null, $valeur['devise_facture'] ); - for ($i=1;$i<6;$i++) - { - $ok = $comptaFact->modifier_details( - $valeur['id'.$i], - $valeur['ref'.$i], - $valeur['designation'.$i], - (int)$valeur['quantite'.$i], - (float)$valeur['pu'.$i], - (int)$valeur['tva'.$i] + for ($i=1;$i<6;$i++) { + $ok = $comptaFact->modifier_details( + $valeur['id' . $i], + $valeur['ref' . $i], + $valeur['designation' . $i], + (int) $valeur['quantite' . $i], + (float) $valeur['pu' . $i], + (int) $valeur['tva' . $i] ); } - } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout de l\'écriture pour ' . $valeur['societe']); } else { Logs::log('Modification de l\'écriture ' . $valeur['numero_devis'] . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_devis&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_devis&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } diff --git a/htdocs/pages/administration/compta_facture.php b/htdocs/pages/administration/compta_facture.php index ca357237d..7c8e91785 100644 --- a/htdocs/pages/administration/compta_facture.php +++ b/htdocs/pages/administration/compta_facture.php @@ -1,23 +1,24 @@ obtenirFacture(); - foreach ($ecritures as &$e) { - $e['link'] = urlencode(Utils::cryptFromText($e['id'])); - } - $smarty->assign('ecritures', $ecritures); + $ecritures = $comptaFact->obtenirFacture(); + foreach ($ecritures as &$e) { + $e['link'] = urlencode(Utils::cryptFromText($e['id'])); + } + $smarty->assign('ecritures', $ecritures); } elseif ($action == 'telecharger_facture') { - $comptaFact->genererFacture($_GET['ref']); -} elseif ($action == 'envoyer_facture'){ - if($comptaFact->envoyerfacture($_GET['ref'])){ - Logs::log('Envoi par email de la facture n°' . $_GET['ref']); - afficherMessage('La facture a été envoyée', 'index.php?page=compta_facture&action=lister'); - } else { - afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=compta_facture&action=lister', true); - } -} elseif ($action == 'envoyer_facture'){ - if($comptaFact->envoyerFacture($_GET['ref'])){ - Logs::log('Envoi par email de la facture n°' . $_GET['ref']); - afficherMessage('La facture a été envoyée', 'index.php?page=compta_facture&action=lister'); - } else { - afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=compta_facture&action=lister', true); - } + $comptaFact->genererFacture($_GET['ref']); +} elseif ($action == 'envoyer_facture') { + if ($comptaFact->envoyerfacture($_GET['ref'])) { + Logs::log('Envoi par email de la facture n°' . $_GET['ref']); + afficherMessage('La facture a été envoyée', 'index.php?page=compta_facture&action=lister'); + } else { + afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=compta_facture&action=lister', true); + } +} elseif ($action == 'envoyer_facture') { + if ($comptaFact->envoyerFacture($_GET['ref'])) { + Logs::log('Envoi par email de la facture n°' . $_GET['ref']); + afficherMessage('La facture a été envoyée', 'index.php?page=compta_facture&action=lister'); + } else { + afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=compta_facture&action=lister', true); + } } elseif ($action == 'ajouter' || $action == 'modifier') { $pays = new Pays($bdd); - $formulaire = instancierFormulaire(); + $formulaire = instancierFormulaire(); - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $comptaFact->obtenir($_GET['id']); $factureId = $champsRecup['id']; @@ -84,192 +84,185 @@ $champs['devise_facture'] = $champsRecup['devise_facture']; - $champsRecup = $comptaFact->obtenir_details($_GET['id']); + $champsRecup = $comptaFact->obtenir_details($_GET['id']); - $i=1; - foreach ($champsRecup as $row) - { - $champs['id'.$i] = $row['id']; - $champs['ref'.$i] = $row['ref']; - $champs['designation'.$i] = $row['designation']; - $champs['quantite'.$i] = $row['quantite']; - $champs['pu'.$i] = $row['pu']; - $champs['tva'.$i] = $row['tva']; - $i++; - } + $i=1; + foreach ($champsRecup as $row) { + $champs['id' . $i] = $row['id']; + $champs['ref' . $i] = $row['ref']; + $champs['designation' . $i] = $row['designation']; + $champs['quantite' . $i] = $row['quantite']; + $champs['pu' . $i] = $row['pu']; + $champs['tva' . $i] = $row['tva']; + $i++; + } - $formulaire->setDefaults($champs); - //$formulaire->setDefaults($champsRecup); - $formulaire->addElement('hidden', 'id', $_GET['id']); + $formulaire->setDefaults($champs); + //$formulaire->setDefaults($champsRecup); + $formulaire->addElement('hidden', 'id', $_GET['id']); - $smarty->assign('facture_id', $factureId); - } + $smarty->assign('facture_id', $factureId); + } -//detail devis - $formulaire->addElement('header' , '' , 'Détail Devis'); + //detail devis + $formulaire->addElement('header' , '' , 'Détail Devis'); -//$mois=10; - if ($action == 'modifier') - { - $formulaire->addElement('date' , 'date_facture' , 'Date facture', ['language' => 'fr', + //$mois=10; + if ($action === 'modifier') { + $formulaire->addElement('date' , 'date_facture' , 'Date facture', ['language' => 'fr', 'format' => 'd F Y', - 'minYear' => date('Y')-3, - 'maxYear' => date('Y')]); - } else { - $formulaire->addElement('date' , 'date_facture' , 'Date facture', ['language' => 'fr', + 'minYear' => date('Y')-3, + 'maxYear' => date('Y')]); + } else { + $formulaire->addElement('date' , 'date_facture' , 'Date facture', ['language' => 'fr', 'format' => 'd F Y', - 'minYear' => date('Y'), - 'maxYear' => date('Y')]); - - } - $formulaire->addElement('header' , '' , 'Facturation'); - $formulaire->addElement('static' , 'note' , '' , 'Ces informations concernent la personne ou la société qui sera facturée

'); - $formulaire->addElement('text' , 'societe' , 'Société' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('text' , 'service' , 'Service' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('textarea', 'adresse' , 'Adresse' , ['cols' => 42, 'rows' => 10]); - $formulaire->addElement('text' , 'code_postal', 'Code postal' , ['size' => 6, 'maxlength' => 10]); - $formulaire->addElement('text' , 'ville' , 'Ville' , ['size' => 30, 'maxlength' => 50]); - $formulaire->addElement('select' , 'id_pays' , 'Pays' , $pays->obtenirPays()); - - $formulaire->addElement('header', null , 'Contact'); - $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'prenom' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'tel' , 'Numero de tél' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'email' , 'Email (facture)', ['size' => 30, 'maxlength' => 100]); - $formulaire->addElement('text' , 'tva_intra' , 'TVA intracommunautaire (facture)', ['size' => 30, 'maxlength' => 100]); - - if ($champs['numero_devis'] || $champs['numero_facture'] ) - { - $formulaire->addElement('header', null , 'Réservé à l\'administration'); - $formulaire->addElement('static' , 'note' , '' , 'Numéro généré automatiquement et affiché en automatique'); - if ($champs['numero_devis']) - $formulaire->addElement('text' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); - if ($champs['numero_facture']) - $formulaire->addElement('text' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); - } else { - $formulaire->addElement('hidden' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('hidden' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); - } - - $formulaire->addElement('header', null , 'Référence client'); - $formulaire->addElement('static' , 'note' , '', 'Possible d\'avoir plusieurs références à mettre (obligation client)

'); - $formulaire->addElement('text' , 'ref_clt1' , 'Référence client' , ['size' => 50, 'maxlength' => 100]); + 'minYear' => date('Y'), + 'maxYear' => date('Y')]); + } + $formulaire->addElement('header' , '' , 'Facturation'); + $formulaire->addElement('static' , 'note' , '' , 'Ces informations concernent la personne ou la société qui sera facturée

'); + $formulaire->addElement('text' , 'societe' , 'Société' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'service' , 'Service' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('textarea', 'adresse' , 'Adresse' , ['cols' => 42, 'rows' => 10]); + $formulaire->addElement('text' , 'code_postal', 'Code postal' , ['size' => 6, 'maxlength' => 10]); + $formulaire->addElement('text' , 'ville' , 'Ville' , ['size' => 30, 'maxlength' => 50]); + $formulaire->addElement('select' , 'id_pays' , 'Pays' , $pays->obtenirPays()); + + $formulaire->addElement('header', null , 'Contact'); + $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'prenom' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'tel' , 'Numero de tél' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'email' , 'Email (facture)', ['size' => 30, 'maxlength' => 100]); + $formulaire->addElement('text' , 'tva_intra' , 'TVA intracommunautaire (facture)', ['size' => 30, 'maxlength' => 100]); + + if ($champs['numero_devis'] || $champs['numero_facture']) { + $formulaire->addElement('header', null , 'Réservé à l\'administration'); + $formulaire->addElement('static' , 'note' , '' , 'Numéro généré automatiquement et affiché en automatique'); + if ($champs['numero_devis']) { + $formulaire->addElement('text' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); + } + if ($champs['numero_facture']) { + $formulaire->addElement('text' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); + } + } else { + $formulaire->addElement('hidden' , 'numero_devis' , 'Numéro devis' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('hidden' , 'numero_facture' , 'Numéro facture' , ['size' => 50, 'maxlength' => 100]); + } + + $formulaire->addElement('header', null , 'Référence client'); + $formulaire->addElement('static' , 'note' , '', 'Possible d\'avoir plusieurs références à mettre (obligation client)

'); + $formulaire->addElement('text' , 'ref_clt1' , 'Référence client' , ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('text' , 'ref_clt2' , 'Référence client 2', ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('text' , 'ref_clt3' , 'Référence client 3' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('header' , '', 'Observation'); - $formulaire->addElement('static' , 'note' , '' , 'Ces informations seront écrites à la fin du document

'); - $formulaire->addElement('textarea', 'observation' , 'Observation', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('header' , '', 'Observation'); + $formulaire->addElement('static' , 'note' , '' , 'Ces informations seront écrites à la fin du document

'); + $formulaire->addElement('textarea', 'observation' , 'Observation', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('header' , '', 'Paiement'); - $formulaire->addElement('select', 'devise_facture' , 'Monnaie de la facture', ['EUR' => 'Euro', + $formulaire->addElement('header' , '', 'Paiement'); + $formulaire->addElement('select', 'devise_facture' , 'Monnaie de la facture', ['EUR' => 'Euro', 'DOL' => 'Dollar'], ['size' => 2]); - $formulaire->addElement('select', 'etat_paiement' , 'Etat paiement', ['En attente de paiement', 'Payé', 'Annulé'], ['size' => 3]); + $formulaire->addElement('select', 'etat_paiement' , 'Etat paiement', ['En attente de paiement', 'Payé', 'Annulé'], ['size' => 3]); $formulaire->addElement('date' , 'date_paiement' , 'Date paiement', ['language' => 'fr', 'format' => 'd F Y', 'minYear' => date('Y') - 5, 'maxYear' => date('Y')]); - $formulaire->addElement('header' , '', 'Contenu'); - $formulaire->addElement('text' , 'ref' , 'Référence' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('textarea', 'designation' , 'Désignation', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('text' , 'quantite' , 'Quantite' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('text' , 'pu' , 'Prix Unitaire' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('header' , '', 'Contenu'); + $formulaire->addElement('text' , 'ref' , 'Référence' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('textarea', 'designation' , 'Désignation', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('text' , 'quantite' , 'Quantite' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'pu' , 'Prix Unitaire' , ['size' => 50, 'maxlength' => 100]); - for ($i=1;$i<6;$i++) - { - $formulaire->addElement('header' , '', 'Contenu'); - $formulaire->addElement('static' , 'note' , '' , 'Ligne '.$i.'

'); - $formulaire->addElement('hidden' , 'id'.$i , 'id' ); - $formulaire->addElement('text' , 'ref'.$i , 'Référence' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('static' , 'note' , '' , 'Rappel : sponsoring 20%, place supplémentaire 10%.
'); - $formulaire->addElement('select' , 'tva' . $i , 'Taux de TVA' , ['0' => 'Non soumis', '5.50' => '5.5%', '10.00' => '10%', '20.00' => '20%']); - $formulaire->addElement('textarea', 'designation'.$i , 'Désignation', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('text' , 'quantite'.$i , 'Quantite' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('text' , 'pu'.$i , 'Prix Unitaire HT' , ['size' => 50, 'maxlength' => 100]); - } + for ($i=1;$i<6;$i++) { + $formulaire->addElement('header' , '', 'Contenu'); + $formulaire->addElement('static' , 'note' , '' , 'Ligne ' . $i . '

'); + $formulaire->addElement('hidden' , 'id' . $i , 'id'); + $formulaire->addElement('text' , 'ref' . $i , 'Référence' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('static' , 'note' , '' , 'Rappel : sponsoring 20%, place supplémentaire 10%.
'); + $formulaire->addElement('select' , 'tva' . $i , 'Taux de TVA' , ['0' => 'Non soumis', '5.50' => '5.5%', '10.00' => '10%', '20.00' => '20%']); + $formulaire->addElement('textarea', 'designation' . $i , 'Désignation', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('text' , 'quantite' . $i , 'Quantite' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'pu' . $i , 'Prix Unitaire HT' , ['size' => 50, 'maxlength' => 100]); + } -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); -// ajoute des regles -// $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); -// $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); - $formulaire->addRule('societe' , 'Société manquant' , 'required'); - $formulaire->addRule('adresse' , 'Adresse manquant' , 'required'); - $formulaire->addRule('email' , 'Email manquant' , 'required'); + // ajoute des regles + // $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); + // $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); + $formulaire->addRule('societe' , 'Société manquant' , 'required'); + $formulaire->addRule('adresse' , 'Adresse manquant' , 'required'); + $formulaire->addRule('email' , 'Email manquant' , 'required'); if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); - -$date_ecriture= $valeur['date_facture']['Y']."-".$valeur['date_facture']['F']."-".$valeur['date_facture']['d'] ; -$date_paiement= $valeur['date_paiement']['Y']."-".$valeur['date_paiement']['F']."-".$valeur['date_paiement']['d'] ; - if ($action == 'ajouter') { - // il faut passser obligatoirement par un devis - } else { - $ok = $comptaFact->modifier( - $_GET['id'], - $date_ecriture, - $valeur['societe'], - $valeur['service'], - $valeur['adresse'], - $valeur['code_postal'], - $valeur['ville'], - $valeur['id_pays'], - $valeur['nom'], - $valeur['prenom'], - $valeur['tel'], - $valeur['email'], - $valeur['tva_intra'], - $valeur['observation'], - $valeur['ref_clt1'], - $valeur['ref_clt2'], - $valeur['ref_clt3'], - $valeur['numero_devis'], - $valeur['numero_facture'], - $valeur['etat_paiement'], - $date_paiement, - $valeur['devise_facture'] - ); - - for ($i=1;$i<6;$i++) { - $ok = $comptaFact->modifier_details( - $valeur['id'.$i], - $valeur['ref'.$i], - $valeur['designation'.$i], - (int)$valeur['quantite'.$i], - (float)$valeur['pu'.$i], - (int)$valeur['tva'.$i] - ); - } - - - } + $valeur = $formulaire->exportValues(); + + $date_ecriture= $valeur['date_facture']['Y'] . "-" . $valeur['date_facture']['F'] . "-" . $valeur['date_facture']['d'] ; + $date_paiement= $valeur['date_paiement']['Y'] . "-" . $valeur['date_paiement']['F'] . "-" . $valeur['date_paiement']['d'] ; + if ($action === 'ajouter') { + // il faut passser obligatoirement par un devis + } else { + $ok = $comptaFact->modifier( + $_GET['id'], + $date_ecriture, + $valeur['societe'], + $valeur['service'], + $valeur['adresse'], + $valeur['code_postal'], + $valeur['ville'], + $valeur['id_pays'], + $valeur['nom'], + $valeur['prenom'], + $valeur['tel'], + $valeur['email'], + $valeur['tva_intra'], + $valeur['observation'], + $valeur['ref_clt1'], + $valeur['ref_clt2'], + $valeur['ref_clt3'], + $valeur['numero_devis'], + $valeur['numero_facture'], + $valeur['etat_paiement'], + $date_paiement, + $valeur['devise_facture'] + ); + + for ($i=1;$i<6;$i++) { + $ok = $comptaFact->modifier_details( + $valeur['id' . $i], + $valeur['ref' . $i], + $valeur['designation' . $i], + (int) $valeur['quantite' . $i], + (float) $valeur['pu' . $i], + (int) $valeur['tva' . $i] + ); + } + } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_facture&action=lister'); + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_facture&action=lister'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); } - -?> diff --git a/htdocs/pages/administration/compta_journal.php b/htdocs/pages/administration/compta_journal.php index d853f11e2..bdcee51b7 100755 --- a/htdocs/pages/administration/compta_journal.php +++ b/htdocs/pages/administration/compta_journal.php @@ -1,11 +1,15 @@ obtenirPeriodeEnCours($id_periode); $smarty->assign('id_periode', $id_periode); $listPeriode = $compta->obtenirListPeriode(); -$smarty->assign('listPeriode', $listPeriode ); +$smarty->assign('listPeriode', $listPeriode); $periode_debut=$listPeriode[$id_periode-1]['date_debut']; @@ -63,21 +63,16 @@ // Accounting lines for the selected period $journal = $compta->obtenirJournal('', $periode_debut, $periode_fin, !$alsoDisplayClassifed); $smarty->assign('journal', $journal); -} -elseif ($action == 'debit') { - $journal = $compta->obtenirJournal(1,$periode_debut,$periode_fin, !$alsoDisplayClassifed); - $smarty->assign('journal', $journal); -} -elseif ($action == 'credit') { - $journal = $compta->obtenirJournal(2,$periode_debut,$periode_fin, !$alsoDisplayClassifed); - $smarty->assign('journal', $journal); - +} elseif ($action == 'debit') { + $journal = $compta->obtenirJournal('1',$periode_debut,$periode_fin, !$alsoDisplayClassifed); + $smarty->assign('journal', $journal); +} elseif ($action == 'credit') { + $journal = $compta->obtenirJournal('2',$periode_debut,$periode_fin, !$alsoDisplayClassifed); + $smarty->assign('journal', $journal); } elseif ($action == 'ajouter' || $action == 'modifier') { + $formulaire = instancierFormulaire(); - $formulaire = instancierFormulaire(); - - if ($action == 'modifier') - { + if ($action === 'modifier') { $champsRecup = $compta->obtenir($_GET['id']); $champs['idcompte'] = $champsRecup['idcompte']; @@ -102,37 +97,37 @@ - //$formulaire->setDefaults($champsRecup); - $formulaire->addElement('hidden', 'id', $_GET['id']); - } else { - $champs['idcompte'] = 1; - $champs['date_saisie'] = date('Y-m-d'); - $champs['date_reglement'] = date('Y-m-d'); - } - $formulaire->setDefaults($champs); + //$formulaire->setDefaults($champsRecup); + $formulaire->addElement('hidden', 'id', $_GET['id']); + } else { + $champs['idcompte'] = 1; + $champs['date_saisie'] = date('Y-m-d'); + $champs['date_reglement'] = date('Y-m-d'); + } + $formulaire->setDefaults($champs); -// facture associé à un évènement - $formulaire->addElement('header' , '' , 'Sélectionner un Journal'); - $formulaire->addElement('select' , 'idoperation', 'Type d\'opération', $compta->obtenirListOperations()); - $formulaire->addElement('select' , 'idcompte' , 'Compte', $compta->obtenirListComptes()); - $formulaire->addElement('select' , 'idevenement', 'Evenement', $compta->obtenirListEvenements()); + // facture associé à un évènement + $formulaire->addElement('header' , '' , 'Sélectionner un Journal'); + $formulaire->addElement('select' , 'idoperation', 'Type d\'opération', $compta->obtenirListOperations()); + $formulaire->addElement('select' , 'idcompte' , 'Compte', $compta->obtenirListComptes()); + $formulaire->addElement('select' , 'idevenement', 'Evenement', $compta->obtenirListEvenements()); -//detail facture - $formulaire->addElement('header' , '' , 'Détail Facture'); + //detail facture + $formulaire->addElement('header' , '' , 'Détail Facture'); -//$mois=10; - $formulaire->addElement('date' , 'date_saisie' , 'Date saisie', ['language' => 'fr', + //$mois=10; + $formulaire->addElement('date' , 'date_saisie' , 'Date saisie', ['language' => 'fr', 'format' => 'd F Y', - 'minYear' => date('Y')-5, - 'maxYear' => date('Y')+1]); + 'minYear' => date('Y')-5, + 'maxYear' => date('Y')+1]); - $formulaire->addElement('select' , 'idcategorie', 'Type de compte', $compta->obtenirListCategories()); - $formulaire->addElement('text', 'nom_frs', 'Nom fournisseurs' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('select' , 'idcategorie', 'Type de compte', $compta->obtenirListCategories()); + $formulaire->addElement('text', 'nom_frs', 'Nom fournisseurs' , ['size' => 30, 'maxlength' => 40]); $formulaire->addElement('text' , 'tva_intra' , 'TVA intracommunautaire (facture)', ['size' => 30, 'maxlength' => 100]); $formulaire->addElement('text', 'numero', 'Numero facture' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('textarea', 'description', 'Description', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('text', 'montant', 'Montant' , ['size' => 30, 'maxlength' => 40, 'id' => 'compta_journal_montant']); - $formulaire->addElement('text', 'comment', 'Commentaire' , ['size' => 30, 'maxlength' => 255]); + $formulaire->addElement('textarea', 'description', 'Description', ['cols' => 42, 'rows' => 5]); + $formulaire->addElement('text', 'montant', 'Montant' , ['size' => 30, 'maxlength' => 40, 'id' => 'compta_journal_montant']); + $formulaire->addElement('text', 'comment', 'Commentaire' , ['size' => 30, 'maxlength' => 255]); $formulaire->addElement('header' , '' , 'TVA'); $formulaire->addElement('text', 'montant_ht_soumis_tva_5_5', 'Montant HT soumis à TVA 5.5%' , ['size' => 30, 'maxlength' => 40, 'id' => 'compta_journal_ht_5_5']); @@ -146,69 +141,69 @@ $formulaire->addElement('select' , 'tva_zone', 'Zone TVA', array_merge(['' => 'Non définie'], Comptabilite::TVA_ZONES)); -//reglement - $formulaire->addElement('header' , '' , 'Réglement'); - $formulaire->addElement('select' , 'idmode_regl', 'Réglement', $compta->obtenirListReglements()); - $formulaire->addElement('date' , 'date_reglement' , 'Date', ['language' => 'fr', + //reglement + $formulaire->addElement('header' , '' , 'Réglement'); + $formulaire->addElement('select' , 'idmode_regl', 'Réglement', $compta->obtenirListReglements()); + $formulaire->addElement('date' , 'date_reglement' , 'Date', ['language' => 'fr', 'format' => 'd F Y', - 'minYear' => date('Y')-5, - 'maxYear' => date('Y')+1]); - $formulaire->addElement('text', 'obs_regl', 'Info reglement' , ['size' => 30, 'maxlength' => 40]); + 'minYear' => date('Y')-5, + 'maxYear' => date('Y')+1]); + $formulaire->addElement('text', 'obs_regl', 'Info reglement' , ['size' => 30, 'maxlength' => 40]); -// boutons + // boutons $formulaire->addElement('header' , 'boutons' , ''); $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); - // 2012-02-18 A. Gendre - $passer = null; - if($action != 'ajouter'){ - $res = $compta->obtenirSuivantADeterminer($_GET['id']); - if(is_array($res)){ - $passer = $res['id']; - $formulaire->addElement('submit', 'soumettrepasser' , 'Soumettre & passer'); - $formulaire->addElement('submit', 'passer' , 'Passer'); - } - } - - // ajoute des regles - $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); - $formulaire->addRule('idcompte' , 'Compte manquant' , 'required'); - $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); - $formulaire->addRule('idevenement' , 'Evenement manquant' , 'required'); - $formulaire->addRule('idevenement' , 'Evenement manquant' , 'nonzero'); - $formulaire->addRule('idcategorie' , 'Type de compte manquant' , 'required'); - $formulaire->addRule('idcategorie' , 'Type de compte manquant' , 'nonzero'); - $formulaire->addRule('montant' , 'Montant manquant' , 'required'); - - - // 2012-02-18 A. Gendre - if (isset($_POST['passer']) && isset($passer)) { - afficherMessage('L\'écriture n\'a pas été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_journal&action=modifier&id=' . $passer); - return; - } + // 2012-02-18 A. Gendre + $passer = null; + if ($action !== 'ajouter') { + $res = $compta->obtenirSuivantADeterminer($_GET['id']); + if (is_array($res)) { + $passer = $res['id']; + $formulaire->addElement('submit', 'soumettrepasser' , 'Soumettre & passer'); + $formulaire->addElement('submit', 'passer' , 'Passer'); + } + } + + // ajoute des regles + $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'required'); + $formulaire->addRule('idcompte' , 'Compte manquant' , 'required'); + $formulaire->addRule('idoperation' , 'Type d\'opération manquant' , 'nonzero'); + $formulaire->addRule('idevenement' , 'Evenement manquant' , 'required'); + $formulaire->addRule('idevenement' , 'Evenement manquant' , 'nonzero'); + $formulaire->addRule('idcategorie' , 'Type de compte manquant' , 'required'); + $formulaire->addRule('idcategorie' , 'Type de compte manquant' , 'nonzero'); + $formulaire->addRule('montant' , 'Montant manquant' , 'required'); + + + // 2012-02-18 A. Gendre + if (isset($_POST['passer']) && isset($passer)) { + afficherMessage('L\'écriture n\'a pas été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=compta_journal&action=modifier&id=' . $passer); + return; + } if ($formulaire->validate()) { - $valeur = $formulaire->exportValues(); - -$date_ecriture= $valeur['date_saisie']['Y']."-".$valeur['date_saisie']['F']."-".$valeur['date_saisie']['d'] ; -$date_regl=$valeur['date_reglement']['Y']."-".$valeur['date_reglement']['F']."-".$valeur['date_reglement']['d'] ; - - if ($action == 'ajouter') { - $ok = $compta->ajouter( - $valeur['idoperation'], - $valeur['idcompte'], - $valeur['idcategorie'], - $date_ecriture, - $valeur['nom_frs'], - $valeur['tva_intra'], - $valeur['montant'], - $valeur['description'], - $valeur['numero'], - $valeur['idmode_regl'], - $date_regl, - $valeur['obs_regl'], - $valeur['idevenement'], + $valeur = $formulaire->exportValues(); + + $date_ecriture= $valeur['date_saisie']['Y'] . "-" . $valeur['date_saisie']['F'] . "-" . $valeur['date_saisie']['d'] ; + $date_regl=$valeur['date_reglement']['Y'] . "-" . $valeur['date_reglement']['F'] . "-" . $valeur['date_reglement']['d'] ; + + if ($action === 'ajouter') { + $ok = $compta->ajouter( + $valeur['idoperation'], + $valeur['idcompte'], + $valeur['idcategorie'], + $date_ecriture, + $valeur['nom_frs'], + $valeur['tva_intra'], + $valeur['montant'], + $valeur['description'], + $valeur['numero'], + $valeur['idmode_regl'], + $date_regl, + $valeur['obs_regl'], + $valeur['idevenement'], $valeur['comment'], 0, $valeur['montant_ht_soumis_tva_0'], @@ -217,23 +212,23 @@ $valeur['montant_ht_soumis_tva_20'], $valeur['tva_zone'] - ); + ); } else { - $ok = $compta->modifier( - $valeur['id'], - $valeur['idoperation'], - $valeur['idcompte'], - $valeur['idcategorie'], - $date_ecriture, - $valeur['nom_frs'], + $ok = $compta->modifier( + $valeur['id'], + $valeur['idoperation'], + $valeur['idcompte'], + $valeur['idcategorie'], + $date_ecriture, + $valeur['nom_frs'], $valeur['tva_intra'], - $valeur['montant'], - $valeur['description'], - $valeur['numero'], - $valeur['idmode_regl'], - $date_regl, - $valeur['obs_regl'], - $valeur['idevenement'], + $valeur['montant'], + $valeur['description'], + $valeur['numero'], + $valeur['idmode_regl'], + $date_regl, + $valeur['obs_regl'], + $valeur['idevenement'], $valeur['comment'], null, 0, @@ -242,23 +237,23 @@ $valeur['montant_ht_soumis_tva_10'], $valeur['montant_ht_soumis_tva_20'], $valeur['tva_zone'] - ); + ); } if ($ok) { - if ($action == 'ajouter') { + if ($action === 'ajouter') { Logs::log('Ajout une écriture ' . $formulaire->exportValue('titre')); } else { Logs::log('Modification une écriture ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); } - // 2012-02-18 A. Gendre - if (isset($_POST['soumettrepasser']) && isset($passer)) { - $urlredirect = 'index.php?page=compta_journal&action=modifier&id=' . $passer; - } else { - $urlredirect = 'index.php?page=compta_journal&action=lister#L' . $valeur['id']; - } - afficherMessage('L\'écriture a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), $urlredirect); + // 2012-02-18 A. Gendre + if (isset($_POST['soumettrepasser']) && isset($passer)) { + $urlredirect = 'index.php?page=compta_journal&action=modifier&id=' . $passer; + } else { + $urlredirect = 'index.php?page=compta_journal&action=lister#L' . $valeur['id']; + } + afficherMessage('L\'écriture a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), $urlredirect); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de l\'écriture'); } } @@ -376,7 +371,6 @@ * in order to improve utilization. */ elseif ($action === 'modifier_colonne') { - try { // Bad request? if (!isset($_POST['val']) || !isset($_GET['column']) || !isset($_GET['id']) || !($line = $compta->obtenir($_GET['id']))) { @@ -462,7 +456,6 @@ * the new one in the line. */ elseif ($action === 'upload_attachment') { - try { // Bad request? if (!isset($_GET['id']) || !($line = $compta->obtenir($_GET['id']))) { @@ -585,16 +578,13 @@ header("Content-disposition: attachment; filename=\"" . basename($filename) . "\""); readfile($filename); exit; - } catch (Exception $e) { header('HTTP/1.1 400 Bad Request'); header('X-Info: ' . $e->getMessage()); } exit; -} - -elseif ($action == 'supprimer') { - if ($compta->supprimerEcriture($_GET['id']) ) { +} elseif ($action == 'supprimer') { + if ($compta->supprimerEcriture($_GET['id'])) { Logs::log('Suppression de l\'écriture ' . $_GET['id']); afficherMessage('L\'écriture a été supprimée', 'index.php?page=compta_journal&action=lister'); } else { @@ -602,16 +592,16 @@ } } elseif ($action == 'importer') { $formulaire = instancierFormulaire(); - $formulaire->addElement('header', null , 'Import CSV'); - $formulaire->addElement('file', 'fichiercsv', 'Fichier banque' ); + $formulaire->addElement('header', null , 'Import CSV'); + $formulaire->addElement('file', 'fichiercsv', 'Fichier banque'); $formulaire->addElement('select', 'banque', 'Banque', [ - Importer\CaisseEpargne::CODE => "Caisse d'Épargne", - Importer\CreditMutuel::CODE => 'Crédit Mutuel - Compte Courant', - Importer\CreditMutuelLivret::CODE => 'Crédit Mutuel - Livret', + CaisseEpargne::CODE => "Caisse d'Épargne", + CreditMutuel::CODE => 'Crédit Mutuel - Compte Courant', + CreditMutuelLivret::CODE => 'Crédit Mutuel - Livret', ]); - $formulaire->addElement('header', 'boutons' , ''); - $formulaire->addElement('submit', 'soumettre', 'Soumettre'); + $formulaire->addElement('header', 'boutons' , ''); + $formulaire->addElement('submit', 'soumettre', 'Soumettre'); if ($formulaire->validate()) { $valeurs = $formulaire->exportValues(); @@ -619,7 +609,7 @@ $tmpDir = __DIR__ . '/../../../tmp'; if ($file->isUploadedFile()) { $file->moveUploadedFile($tmpDir, 'banque.csv'); - $importerFactory = new Importer\Factory(); + $importerFactory = new Factory(); $importer = $importerFactory->create( $tmpDir . '/banque.csv', $valeurs['banque'] @@ -636,7 +626,7 @@ } $smarty->assign('formulaire', genererFormulaire($formulaire)); } elseif ($action == 'ventiler') { - $idCompta = (int)$_GET['id']; + $idCompta = (int) $_GET['id']; $ligneCompta = $compta->obtenir($idCompta); $montantTotal = 0; diff --git a/htdocs/pages/administration/compta_recherche.php b/htdocs/pages/administration/compta_recherche.php index 0f5d8fe63..fe69339f2 100755 --- a/htdocs/pages/administration/compta_recherche.php +++ b/htdocs/pages/administration/compta_recherche.php @@ -1,9 +1,10 @@ obtenirListEvenements('liste'); - $smarty->assign('listEvenement', $listEvenement ); - - if (!isset($_GET['idevnt']) || intval($_GET['idevnt']) == 0) { - $idevnt= 8; - } - else - { - $idevnt=$_GET['idevnt']; - } +if ($action == 'lister') { + $listEvenement = $compta->obtenirListEvenements('liste'); + $smarty->assign('listEvenement', $listEvenement); + + $idevnt = !isset($_GET['idevnt']) || intval($_GET['idevnt']) == 0 ? 8 : $_GET['idevnt']; $smarty->assign('idevnt', $idevnt); - $debit = $compta->obtenirSyntheseEvenement(1,$idevnt); - $smarty->assign('debit', $debit); + $debit = $compta->obtenirSyntheseEvenement($idevnt, 1); + $smarty->assign('debit', $debit); - $credit = $compta->obtenirSyntheseEvenement(2,$idevnt); - $smarty->assign('credit', $credit); + $credit = $compta->obtenirSyntheseEvenement($idevnt, 2); + $smarty->assign('credit', $credit); - $totalDepense = $compta->obtenirTotalSyntheseEvenement(1,$idevnt); - $smarty->assign('totalDepense', $totalDepense); - - $totalRecette = $compta->obtenirTotalSyntheseEvenement(2,$idevnt); - $smarty->assign('totalRecette', $totalRecette); - - $difMontant = $totalRecette - $totalDepense ; - $smarty->assign('difMontant', $difMontant); + $totalDepense = $compta->obtenirTotalSyntheseEvenement($idevnt, 1); + $smarty->assign('totalDepense', $totalDepense); -} + $totalRecette = $compta->obtenirTotalSyntheseEvenement($idevnt, 2); + $smarty->assign('totalRecette', $totalRecette); -?> + $difMontant = $totalRecette - $totalDepense ; + $smarty->assign('difMontant', $difMontant); +} diff --git a/htdocs/pages/administration/cotisations.php b/htdocs/pages/administration/cotisations.php index 77862502f..83cad58e1 100644 --- a/htdocs/pages/administration/cotisations.php +++ b/htdocs/pages/administration/cotisations.php @@ -1,13 +1,17 @@ assign('cotisations', $cotisations->obtenirListe($_GET['type_personne'], $_GET['id_personne'])); -} elseif ($action == 'telecharger_facture'){ - $cotisations->genererFacture($_GET['id']); -} elseif ($action == 'envoyer_facture'){ - if($cotisations->envoyerFacture($_GET['id'], $this->get(\AppBundle\Email\Mailer\Mailer::class), $userRepository)){ - Logs::log('Envoi par email de la facture pour la cotisation n°' . $_GET['id']); - afficherMessage('La facture a été envoyée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' .$_GET['id_personne']); - } else { - afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' .$_GET['id_personne'], true); +} elseif ($action == 'telecharger_facture') { + $cotisations->genererFacture($_GET['id']); +} elseif ($action == 'envoyer_facture') { + if ($cotisations->envoyerFacture($_GET['id'], $this->get(Mailer::class), $userRepository)) { + Logs::log('Envoi par email de la facture pour la cotisation n°' . $_GET['id']); + afficherMessage('La facture a été envoyée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' . $_GET['id_personne']); + } else { + afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' . $_GET['id_personne'], true); } } elseif ($action == 'supprimer') { if ($cotisations->supprimer($_GET['id'])) { Logs::log('Suppression de la cotisation ' . $_GET['id']); - afficherMessage('La cotisation a été supprimée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' .$_GET['id_personne']); + afficherMessage('La cotisation a été supprimée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' . $_GET['id_personne']); } else { afficherMessage('Une erreur est survenue lors de la suppression de la personne morale', 'index.php?page=personnes_morales&action=lister', true); } @@ -99,9 +103,9 @@ if ($formulaire->validate()) { $nom = ($_GET['type_personne'] == AFUP_PERSONNES_PHYSIQUES) ? $personne['prenom'] . ' ' . $personne['nom'] : $personne['raison_sociale']; $date_debut = $formulaire->exportValue('date_debut'); - $date_debut = mktime(0, 0, 0, $date_debut['F'], $date_debut['d'], $date_debut['Y']); + $date_debut = mktime(0, 0, 0, (int) $date_debut['F'], (int) $date_debut['d'],(int) $date_debut['Y']); $date_fin = $formulaire->exportValue('date_fin'); - $date_fin = mktime(0, 0, 0, $date_fin['F'], $date_fin['d'], $date_fin['Y']); + $date_fin = mktime(0, 0, 0, (int) $date_fin['F'], (int) $date_fin['d'], (int) $date_fin['Y']); if ($action == 'ajouter') { if ($cotisations->ajouter( @@ -116,32 +120,28 @@ $formulaire->exportValue('reference_client') )) { Logs::log("Ajout de la cotisation jusqu'au " . date('d F Y', $date_fin) . ' pour ' . $nom); - afficherMessage("La cotisation jusqu'au " . date('d F Y', $date_fin) . ' pour ' . $nom . ' a bien été ajoutée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' .$_GET['id_personne']); + afficherMessage("La cotisation jusqu'au " . date('d F Y', $date_fin) . ' pour ' . $nom . ' a bien été ajoutée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' . $_GET['id_personne']); } else { $smarty->assign('erreur', "Une erreur est survenue lors de l'ajout de la cotisation jusqu'au " . date('d F Y', $date_fin) . ' pour ' . $nom); } + } elseif ($cotisations->modifier( + $_GET['id'], + $formulaire->exportValue('type_personne'), + $formulaire->exportValue('id_personne'), + $formulaire->exportValue('montant'), + $formulaire->exportValue('type_reglement'), + $formulaire->exportValue('informations_reglement'), + $date_debut, + $date_fin, + $formulaire->exportValue('commentaires'), + $formulaire->exportValue('reference_client') + )) { + Logs::log('Modification de la cotisation (' . $_GET['id'] . ') pour ' . $nom); + afficherMessage('La cotisation pour ' . $nom . ' a bien été modifiée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' . $_GET['id_personne']); } else { - if ($cotisations->modifier( - $_GET['id'], - $formulaire->exportValue('type_personne'), - $formulaire->exportValue('id_personne'), - $formulaire->exportValue('montant'), - $formulaire->exportValue('type_reglement'), - $formulaire->exportValue('informations_reglement'), - $date_debut, - $date_fin, - $formulaire->exportValue('commentaires'), - $formulaire->exportValue('reference_client') - )) { - Logs::log('Modification de la cotisation (' . $_GET['id'] . ') pour ' . $nom); - afficherMessage('La cotisation pour ' . $nom . ' a bien été modifiée', 'index.php?page=cotisations&action=lister&type_personne=' . $_GET['type_personne'] . '&id_personne=' .$_GET['id_personne']); - } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de la modification de la cotisation (' . $_GET['id'] . ') pour ' . $nom); - } + $smarty->assign('erreur', 'Une erreur est survenue lors de la modification de la cotisation (' . $_GET['id'] . ') pour ' . $nom); } } $smarty->assign('formulaire', genererFormulaire($formulaire)); } - -?> diff --git a/htdocs/pages/administration/forum_facturation.php b/htdocs/pages/administration/forum_facturation.php index 256b68120..c323dfc70 100644 --- a/htdocs/pages/administration/forum_facturation.php +++ b/htdocs/pages/administration/forum_facturation.php @@ -1,13 +1,16 @@ assign('forums', $forum->obtenirListe()); $smarty->assign('facturations', $forum_facturation->obtenirListe($_GET['id_forum'], $list_champs, $list_ordre, $list_associatif, $list_filtre)); } elseif ($action == 'telecharger_devis') { - $forum_facturation->genererDevis($_GET['ref']); + $forum_facturation->genererDevis($_GET['ref']); } elseif ($action == 'telecharger_facture') { - $forum_facturation->genererFacture($_GET['ref']); + $forum_facturation->genererFacture($_GET['ref']); // on évite de renvoyer du html en fin de PDF exit(0); -} elseif ($action == 'envoyer_facture'){ - if($forum_facturation->envoyerFacture($_GET['ref'])){ - Logs::log('Envoi par email de la facture n°' . $_GET['ref']); - afficherMessage('La facture a été envoyée', 'index.php?page=forum_facturation&action=lister'); - } else { - afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=forum_facturation&action=lister', true); - } -} elseif ($action == 'facturer_facture'){ - if($forum_facturation->estFacture($_GET['ref'])){ - Logs::log('Facturation => facture n°' . $_GET['ref']); - afficherMessage('La facture est prise en compte', 'index.php?page=forum_facturation&action=lister'); - } else { - afficherMessage("La facture n'a pas pu être prise en compte", 'index.php?page=forum_facturation&action=lister', true); - } -} elseif ($action == 'supprimer_facture'){ +} elseif ($action == 'envoyer_facture') { + if ($forum_facturation->envoyerFacture($_GET['ref'])) { + Logs::log('Envoi par email de la facture n°' . $_GET['ref']); + afficherMessage('La facture a été envoyée', 'index.php?page=forum_facturation&action=lister'); + } else { + afficherMessage("La facture n'a pas pu être envoyée", 'index.php?page=forum_facturation&action=lister', true); + } +} elseif ($action == 'facturer_facture') { + if ($forum_facturation->estFacture($_GET['ref'])) { + Logs::log('Facturation => facture n°' . $_GET['ref']); + afficherMessage('La facture est prise en compte', 'index.php?page=forum_facturation&action=lister'); + } else { + afficherMessage("La facture n'a pas pu être prise en compte", 'index.php?page=forum_facturation&action=lister', true); + } +} elseif ($action == 'supprimer_facture') { $invoice = $invoiceRepository->getByReference($_GET['ref']); if (null !== $invoice) { try { @@ -83,12 +86,11 @@ } } afficherMessage("La facture n'a pas pu être supprimée", 'index.php?page=forum_facturation&action=lister', true); -} elseif ($action == 'changer_date_reglement'){ +} elseif ($action == 'changer_date_reglement') { $reglement = strtotime(implode('-', array_reverse(explode('/', $_GET['reglement'])))); if ($forum_facturation->changerDateReglement($_GET['ref'], $reglement)) { - afficherMessage('La date de réglement a été changée', 'index.php?page=forum_facturation&action=lister'); + afficherMessage('La date de réglement a été changée', 'index.php?page=forum_facturation&action=lister'); } else { - afficherMessage('La date de réglement n\'a pas été changée', 'index.php?page=forum_facturation&action=lister', true); + afficherMessage('La date de réglement n\'a pas été changée', 'index.php?page=forum_facturation&action=lister', true); } } -?> diff --git a/htdocs/pages/administration/forum_gestion.php b/htdocs/pages/administration/forum_gestion.php index 4d3f7ee14..b5c118c09 100644 --- a/htdocs/pages/administration/forum_gestion.php +++ b/htdocs/pages/administration/forum_gestion.php @@ -1,12 +1,13 @@ addElement('file', 'mail_inscription_attachment', "Pièce jointe du mail d'inscription"); if (Event::hasInscriptionAttachment($forumPath)) { $formulaire->addElement('static', 'info', '', - "Un fichier joint au mail d'inscription est déjà présent."); + "Un fichier joint au mail d'inscription est déjà présent."); } $formulaire->addElement('checkbox', 'speakers_diner_enabled', "Activer le repas des speakers"); @@ -123,14 +124,14 @@ if (Event::hasSponsorFile($formulaire->exportValue('path'), 'fr')) { $publicPath = Event::getSponsorFilePublicPath($formulaire->exportValue('path'), 'fr'); $formulaire->addElement('static', 'info', '', - "Voir le dossier de sponsoring (FR)"); + "Voir le dossier de sponsoring (FR)"); } $fileSponsorENAttachment = $formulaire->addElement('file', 'file_sponsor_en', "Dossier de sponsoring (EN)"); if (Event::hasSponsorFile($formulaire->exportValue('path'), 'en')) { $publicPath = Event::getSponsorFilePublicPath($formulaire->exportValue('path'), 'en'); $formulaire->addElement('static', 'info', '', - "Voir le dossier de sponsoring (EN)"); + "Voir le dossier de sponsoring (EN)"); } diff --git a/htdocs/pages/administration/forum_inscriptions.php b/htdocs/pages/administration/forum_inscriptions.php index efa679737..842c704db 100644 --- a/htdocs/pages/administration/forum_inscriptions.php +++ b/htdocs/pages/administration/forum_inscriptions.php @@ -1,11 +1,14 @@ get($forumId); $ticketTypes = $ticketEventTypeRepository->getTicketsByEvent($event, false); @@ -123,7 +126,6 @@ function updateGlobalsForTarif( $smarty->assign('inscriptions', $forum_inscriptions->obtenirListe($_GET['id_forum'], $list_champs, $list_ordre, $list_associatif, $list_filtre)); $smarty->assign('finForum', (new \DateTime($forumData['date_fin']))->format('U')); $smarty->assign('now', (new \DateTime())->format('U')); - } elseif ($action == 'supprimer') { /** @var Invoice|null $invoice */ $invoice = $invoiceRepository->getByReference($_GET['id']); @@ -173,8 +175,8 @@ function updateGlobalsForTarif( $formulaire = instancierFormulaire(); if ($action == 'ajouter') { - $formulaire->setDefaults( - [ + $formulaire->setDefaults( + [ 'civilite' => 'M.', 'id_pays_facturation' => 'FR', 'type_inscription' => -1, @@ -207,7 +209,7 @@ function updateGlobalsForTarif( $champs['id_pays_facturation'] = $champs2['id_pays']; $champs['email_facturation'] = $champs2['email']; - /** @var \AppBundle\Event\Model\Ticket $ticket */ + /** @var Ticket $ticket */ $ticket = $ticketRepository->get($_GET['id']); if (null !== $ticket) { $champs['commentaires'] = $ticket->getComments(); @@ -215,70 +217,69 @@ function updateGlobalsForTarif( $formulaire->setDefaults($champs); - if (isset($champs) && isset($champs['id_forum'])) { - $_GET['id_forum'] = $champs['id_forum']; - } + if (isset($champs) && isset($champs['id_forum'])) { + $_GET['id_forum'] = $champs['id_forum']; + } } updateGlobalsForTarif($eventRepository, $ticketEventTypeRepository, $ticketTypeAvailability, $_GET['id_forum']); - $formulaire->addElement('hidden', 'old_reference', (isset($champs) ? $champs['reference'] : '')); - $formulaire->addElement('hidden', 'id_forum', $_GET['id_forum']); + $formulaire->addElement('hidden', 'old_reference', (isset($champs) ? $champs['reference'] : '')); + $formulaire->addElement('hidden', 'id_forum', $_GET['id_forum']); - $formulaire->addElement('header', null, 'Informations'); - $groupe = []; - foreach ($AFUP_Tarifs_Forum as $tarif_key => $tarifs) - { - $groupe[] = $formulaire->createElement('radio', 'type_inscription', null, $AFUP_Tarifs_Forum_Lib[$tarif_key] . ' (' . $AFUP_Tarifs_Forum[$tarif_key] . ' €)' , $tarif_key); - } + $formulaire->addElement('header', null, 'Informations'); + $groupe = []; + foreach ($AFUP_Tarifs_Forum as $tarif_key => $tarifs) { + $groupe[] = $formulaire->createElement('radio', 'type_inscription', null, $AFUP_Tarifs_Forum_Lib[$tarif_key] . ' (' . $AFUP_Tarifs_Forum[$tarif_key] . ' €)' , $tarif_key); + } - $formulaire->addGroup($groupe, 'groupe_type_inscription', 'Formule', '
', false); + $formulaire->addGroup($groupe, 'groupe_type_inscription', 'Formule', '
', false); - $formulaire->addElement('select', 'civilite' , 'Civilité' , ['M.' => 'M.', 'Mme' => 'Mme']); - $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'prenom' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'email' , 'Email' , ['size' => 30, 'maxlength' => 100]); - $formulaire->addElement('text' , 'telephone' , 'Tél.' , ['size' => 20, 'maxlength' => 20]); + $formulaire->addElement('select', 'civilite' , 'Civilité' , ['M.' => 'M.', 'Mme' => 'Mme']); + $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'prenom' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'email' , 'Email' , ['size' => 30, 'maxlength' => 100]); + $formulaire->addElement('text' , 'telephone' , 'Tél.' , ['size' => 20, 'maxlength' => 20]); $groupe = []; - $formulaire->addElement('header', null , 'Réservé à l\'administration'); - $formulaire->addElement('static' , 'note' , '' , 'La reference est utilisée comme numéro de facture. Elle peut être commune à plusieurs inscriptions...

'); + $formulaire->addElement('header', null , 'Réservé à l\'administration'); + $formulaire->addElement('static' , 'note' , '' , 'La reference est utilisée comme numéro de facture. Elle peut être commune à plusieurs inscriptions...

'); if ($action != 'ajouter') { - $formulaire->addElement('static', 'html', '', 'Rechercher la facture'); + $formulaire->addElement('static', 'html', '', 'Rechercher la facture'); } - $formulaire->addElement('text' , 'reference' , 'Référence' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'reference' , 'Référence' , ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('text' , 'autorisation', 'Autorisation', ['size' => 50, 'maxlength' => 100]); $formulaire->addElement('text' , 'transaction' , 'Transaction' , ['size' => 50, 'maxlength' => 100]); $state = [AFUP_FORUM_ETAT_CREE => 'Inscription créée', - AFUP_FORUM_ETAT_ANNULE => 'Inscription annulée', - AFUP_FORUM_ETAT_ERREUR => 'Paiement CB erreur', - AFUP_FORUM_ETAT_REFUSE => 'Paiement CB refusé', - AFUP_FORUM_ETAT_REGLE => 'Inscription réglée', - AFUP_FORUM_ETAT_INVITE => 'Invitation', - AFUP_FORUM_ETAT_ATTENTE_REGLEMENT => 'Attente règlement', - AFUP_FORUM_ETAT_CONFIRME => 'Inscription confirmée', - AFUP_FORUM_ETAT_A_POSTERIORI => 'Inscription à posteriori', - ]; - $formulaire->addElement('select', 'etat' , 'Etat' , $state); + AFUP_FORUM_ETAT_ANNULE => 'Inscription annulée', + AFUP_FORUM_ETAT_ERREUR => 'Paiement CB erreur', + AFUP_FORUM_ETAT_REFUSE => 'Paiement CB refusé', + AFUP_FORUM_ETAT_REGLE => 'Inscription réglée', + AFUP_FORUM_ETAT_INVITE => 'Invitation', + AFUP_FORUM_ETAT_ATTENTE_REGLEMENT => 'Attente règlement', + AFUP_FORUM_ETAT_CONFIRME => 'Inscription confirmée', + AFUP_FORUM_ETAT_A_POSTERIORI => 'Inscription à posteriori', + ]; + $formulaire->addElement('select', 'etat' , 'Etat' , $state); $facturation = [AFUP_FORUM_FACTURE_A_ENVOYER => 'Facture à envoyer', - AFUP_FORUM_FACTURE_ENVOYEE => 'Facture envoyée', - AFUP_FORUM_FACTURE_RECUE => 'Facture reçue', - ]; - $formulaire->addElement('select', 'facturation' , 'Facturation' , $facturation); - - $formulaire->addElement('header' , '' , 'Règlement'); - $groupe = []; - $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Carte bancaire', AFUP_FORUM_REGLEMENT_CARTE_BANCAIRE); - $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Chèque' , AFUP_FORUM_REGLEMENT_CHEQUE); + AFUP_FORUM_FACTURE_ENVOYEE => 'Facture envoyée', + AFUP_FORUM_FACTURE_RECUE => 'Facture reçue', + ]; + $formulaire->addElement('select', 'facturation' , 'Facturation' , $facturation); + + $formulaire->addElement('header' , '' , 'Règlement'); + $groupe = []; + $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Carte bancaire', AFUP_FORUM_REGLEMENT_CARTE_BANCAIRE); + $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Chèque' , AFUP_FORUM_REGLEMENT_CHEQUE); $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Virement' , AFUP_FORUM_REGLEMENT_VIREMENT); $groupe[] = $formulaire->createElement('radio', 'type_reglement', null, 'Aucun' , AFUP_FORUM_REGLEMENT_AUCUN); - $formulaire->addGroup($groupe, 'groupe_type_reglement', 'Règlement', ' ', false); + $formulaire->addGroup($groupe, 'groupe_type_reglement', 'Règlement', ' ', false); $formulaire->addElement('textarea' , 'informations_reglement', 'Informations règlement', ['cols' => 42, 'rows' => 4]); @@ -286,76 +287,75 @@ function updateGlobalsForTarif( $formulaire->addElement('date' , 'date_reglement' , 'Date', ['language' => 'fr', 'minYear' => $current['forum_annee']-2, 'maxYear' => $current['forum_annee']+2]); - $formulaire->addElement('header' , '' , 'Facturation'); - $formulaire->addElement('static' , 'note' , '' , 'Ces informations concernent la personne ou la société qui sera facturée

'); - $formulaire->addElement('text' , 'societe_facturation' , 'Société' , ['size' => 50, 'maxlength' => 100]); - $formulaire->addElement('text' , 'nom_facturation' , 'Nom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('text' , 'prenom_facturation' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); - $formulaire->addElement('textarea', 'adresse_facturation' , 'Adresse' , ['cols' => 42, 'rows' => 10]); - $formulaire->addElement('text' , 'code_postal_facturation', 'Code postal' , ['size' => 6, 'maxlength' => 10]); - $formulaire->addElement('text' , 'ville_facturation' , 'Ville' , ['size' => 30, 'maxlength' => 50]); - $formulaire->addElement('select' , 'id_pays_facturation' , 'Pays' , $pays->obtenirPays()); - $formulaire->addElement('text' , 'email_facturation' , 'Email (facture)', ['size' => 30, 'maxlength' => 100]); - $formulaire->addElement('text' , 'coupon' , 'Coupon' , ['size' => 30, 'maxlength' => 200]); + $formulaire->addElement('header' , '' , 'Facturation'); + $formulaire->addElement('static' , 'note' , '' , 'Ces informations concernent la personne ou la société qui sera facturée

'); + $formulaire->addElement('text' , 'societe_facturation' , 'Société' , ['size' => 50, 'maxlength' => 100]); + $formulaire->addElement('text' , 'nom_facturation' , 'Nom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('text' , 'prenom_facturation' , 'Prénom' , ['size' => 30, 'maxlength' => 40]); + $formulaire->addElement('textarea', 'adresse_facturation' , 'Adresse' , ['cols' => 42, 'rows' => 10]); + $formulaire->addElement('text' , 'code_postal_facturation', 'Code postal' , ['size' => 6, 'maxlength' => 10]); + $formulaire->addElement('text' , 'ville_facturation' , 'Ville' , ['size' => 30, 'maxlength' => 50]); + $formulaire->addElement('select' , 'id_pays_facturation' , 'Pays' , $pays->obtenirPays()); + $formulaire->addElement('text' , 'email_facturation' , 'Email (facture)', ['size' => 30, 'maxlength' => 100]); + $formulaire->addElement('text' , 'coupon' , 'Coupon' , ['size' => 30, 'maxlength' => 200]); - $formulaire->addElement('header', null, 'Divers'); + $formulaire->addElement('header', null, 'Divers'); $formulaire->addElement('textarea', 'commentaires' , 'Commentaires', ['cols' => 42, 'rows' => 5]); - $formulaire->addElement('static', 'label', null, "J'accepte que ma compagnie soit citée comme participant à la conférence"); - $groupe = []; - $groupe[] = $formulaire->createElement('radio', 'citer_societe', null, 'oui', 1); - $groupe[] = $formulaire->createElement('radio', 'citer_societe', null, 'non', 0); - $formulaire->addGroup($groupe, 'groupe_citer_societe', null, ' ', false); - $formulaire->addElement('static', 'label', null, "Je souhaite être tenu au courant des rencontres de l'AFUP sur des sujets afférents à PHP"); - $groupe = []; - $groupe[] = $formulaire->createElement('radio', 'newsletter_afup', null, 'oui', 1); - $groupe[] = $formulaire->createElement('radio', 'newsletter_afup', null, 'non', 0); - $formulaire->addGroup($groupe, 'groupe_newsletter_afup', null, ' ', false); - $formulaire->addElement('static', 'label', null, "Je souhaite être tenu au courant de l'actualité PHP via la newsletter de notre sponsor"); + $formulaire->addElement('static', 'label', null, "J'accepte que ma compagnie soit citée comme participant à la conférence"); + $groupe = []; + $groupe[] = $formulaire->createElement('radio', 'citer_societe', null, 'oui', 1); + $groupe[] = $formulaire->createElement('radio', 'citer_societe', null, 'non', 0); + $formulaire->addGroup($groupe, 'groupe_citer_societe', null, ' ', false); + $formulaire->addElement('static', 'label', null, "Je souhaite être tenu au courant des rencontres de l'AFUP sur des sujets afférents à PHP"); + $groupe = []; + $groupe[] = $formulaire->createElement('radio', 'newsletter_afup', null, 'oui', 1); + $groupe[] = $formulaire->createElement('radio', 'newsletter_afup', null, 'non', 0); + $formulaire->addGroup($groupe, 'groupe_newsletter_afup', null, ' ', false); + $formulaire->addElement('static', 'label', null, "Je souhaite être tenu au courant de l'actualité PHP via la newsletter de notre sponsor"); $groupe = []; $groupe[] = $formulaire->createElement('radio', 'newsletter_nexen', null, 'oui', 1); $groupe[] = $formulaire->createElement('radio', 'newsletter_nexen', null, 'non', 0); $formulaire->addGroup($groupe, 'groupe_newsletter_nexen', null, ' ', false); $formulaire->addElement('static', 'label', null, "Je souhaite recevoir des informations de la part de vos partenaires presse/media"); - $groupe = []; - $groupe[] = $formulaire->createElement('radio', 'mail_partenaire', null, 'oui', 1); - $groupe[] = $formulaire->createElement('radio', 'mail_partenaire', null, 'non', 0); - $formulaire->addGroup($groupe, 'groupe_mail_partenaire', null, ' ', false); + $groupe = []; + $groupe[] = $formulaire->createElement('radio', 'mail_partenaire', null, 'oui', 1); + $groupe[] = $formulaire->createElement('radio', 'mail_partenaire', null, 'non', 0); + $formulaire->addGroup($groupe, 'groupe_mail_partenaire', null, ' ', false); $formulaire->addElement('header', null, 'Transport'); $formulaire->addElement('select', 'transport_mode', 'Quel est votre mode de transport ?', Ticket::TRANSPORT_MODES); $formulaire->addElement('select', 'transport_distance', 'Quelle sera la distance parcourue ?', Ticket::TRANSPORT_DISTANCES); $formulaire->addElement('header', 'boutons' , ''); - $formulaire->addElement('submit', 'soumettre', 'Soumettre'); + $formulaire->addElement('submit', 'soumettre', 'Soumettre'); - // On ajoute les règles - $formulaire->addGroupRule('groupe_type_inscription', 'Formule non sélectionnée' , 'required', null, 1); - $formulaire->addGroupRule('groupe_type_reglement' , 'Règlement non sélectionné', 'required', null, 1); - $formulaire->addRule('civilite' , 'Civilité non sélectionnée', 'required'); - $formulaire->addRule('nom' , 'Nom manquant' , 'required'); - $formulaire->addRule('prenom' , 'Prénom manquant' , 'required'); - $formulaire->addRule('email' , 'Email manquant' , 'required'); - $formulaire->addRule('email' , 'Email invalide' , 'email'); + // On ajoute les règles + $formulaire->addGroupRule('groupe_type_inscription', 'Formule non sélectionnée' , 'required', null, 1); + $formulaire->addGroupRule('groupe_type_reglement' , 'Règlement non sélectionné', 'required', null, 1); + $formulaire->addRule('civilite' , 'Civilité non sélectionnée', 'required'); + $formulaire->addRule('nom' , 'Nom manquant' , 'required'); + $formulaire->addRule('prenom' , 'Prénom manquant' , 'required'); + $formulaire->addRule('email' , 'Email manquant' , 'required'); + $formulaire->addRule('email' , 'Email invalide' , 'email'); if ($formulaire->validate()) { - $valeurs = $formulaire->exportValues(); + $valeurs = $formulaire->exportValues(); // Date de réglement au 01/01 => non defini - if ($valeurs['date_reglement']['d'] == 1 && $valeurs['date_reglement']['M'] == 1) - { + if ($valeurs['date_reglement']['d'] == 1 && $valeurs['date_reglement']['M'] == 1) { $valeurs['date_reglement'] = null; } else { - $valeurs['date_reglement'] = mktime(0,0,0,$valeurs['date_reglement']['M'],$valeurs['date_reglement']['d'],$valeurs['date_reglement']['Y']); + $valeurs['date_reglement'] = mktime(0,0,0, (int) $valeurs['date_reglement']['M'], (int) $valeurs['date_reglement']['d'], (int) $valeurs['date_reglement']['Y']); } if ($action == 'ajouter') { - // On génére la référence si nécessaire + // On génére la référence si nécessaire if (empty($valeurs['reference'])) { $label = (empty($valeurs['societe_facturation']) ? (empty($valeurs['nom_facturation']) ? $valeurs['nom'] : $valeurs['nom_facturation']) : $valeurs['societe_facturation']); - $valeurs['reference'] = $forum_facturation->creerReference($valeurs['id_forum'], $label); + $valeurs['reference'] = $forum_facturation->creerReference($valeurs['id_forum'], $label); } - // On ajoute l'inscription dans la base de données + // On ajoute l'inscription dans la base de données $ticket = new Ticket(); $ticket->setDate(new DateTime()); $ticket->setAmount($GLOBALS['AFUP_Tarifs_Forum'][$valeurs['type_inscription']]); @@ -384,25 +384,25 @@ function updateGlobalsForTarif( } } else { $ok = $forum_inscriptions->modifierInscription($_GET['id'], - $valeurs['reference'], - $valeurs['type_inscription'], - $valeurs['civilite'], - $valeurs['nom'], - $valeurs['prenom'], - $valeurs['email'], - $valeurs['telephone'], - $valeurs['coupon'], - $valeurs['citer_societe'], - $valeurs['newsletter_afup'], + $valeurs['reference'], + $valeurs['type_inscription'], + $valeurs['civilite'], + $valeurs['nom'], + $valeurs['prenom'], + $valeurs['email'], + $valeurs['telephone'], + $valeurs['coupon'], + $valeurs['citer_societe'], + $valeurs['newsletter_afup'], $valeurs['newsletter_nexen'], - $valeurs['mail_partenaire'], + $valeurs['mail_partenaire'], $valeurs['commentaires'], - $valeurs['etat'], + $valeurs['etat'], $valeurs['facturation'], - (int)$valeurs['transport_mode'], - (int)$valeurs['transport_distance']); + (int) $valeurs['transport_mode'], + (int) $valeurs['transport_distance']); - /** @var \AppBundle\Event\Model\Ticket $ticket */ + /** @var Ticket $ticket */ $ticket = $ticketRepository->get($_GET['id']); if (null !== $ticket) { $ticket->setComments($valeurs['commentaires']); @@ -411,8 +411,13 @@ function updateGlobalsForTarif( } try { - $paymentDate = null !== $valeurs['date_reglement'] ? DateTime::createFromFormat('U', $valeurs['date_reglement']) : null; - $invoiceService->handleInvoicing($valeurs['reference'], + $paymentDate = null; + if (is_numeric($valeurs['date_reglement'])) { + $paymentDate = new DateTime('@' . $valeurs['date_reglement']); + } else { + $paymentDate = null !== $valeurs['date_reglement'] ? DateTime::createFromFormat('U', $valeurs['date_reglement']) : null; + } + $invoiceService->handleInvoicing($valeurs['reference'], $valeurs['type_reglement'], $valeurs['informations_reglement'], $paymentDate, diff --git a/htdocs/pages/administration/forum_joind_in.php b/htdocs/pages/administration/forum_joind_in.php index 0cb02a597..2601c9f02 100644 --- a/htdocs/pages/administration/forum_joind_in.php +++ b/htdocs/pages/administration/forum_joind_in.php @@ -1,9 +1,10 @@ obtenirDernier(); - } else { - $forum_id = $_GET['id_forum']; - } + $forum_id = !isset($_GET['id_forum']) || intval($_GET['id_forum']) == 0 ? $forum->obtenirDernier() : $_GET['id_forum']; $csv = $forum->obtenirCsvJoindIn($forum_id); diff --git a/htdocs/pages/administration/forum_partenaire.php b/htdocs/pages/administration/forum_partenaire.php index d18bce304..385b772df 100644 --- a/htdocs/pages/administration/forum_partenaire.php +++ b/htdocs/pages/administration/forum_partenaire.php @@ -1,12 +1,13 @@ setDefaults(['ranking' => 1]); + $formulaire->setDefaults(['ranking' => 1]); } else { $champs = $partenaires->obtenir($_GET['id']); $forum = $forums->obtenir($champs['id_forum']); $formulaire->setDefaults($champs); - if (isset($champs) && isset($champs['id'])) { - $_GET['id'] = $champs['id']; - } + if (isset($champs) && isset($champs['id'])) { + $_GET['id'] = $champs['id']; + } $formulaire->addElement('hidden', 'id', $_GET['id']); } @@ -60,9 +61,9 @@ $formulaire->addElement('static' , 'note' , '', 'Faire attention à la taille'); $formulaire->addElement('file' , 'logo' , 'Logo'); if ($action == 'modifier') { - $formulaire->addElement('static' , 'html' , '', '
'); - $chemin = realpath('../../templates/'.$forum['path'].'/images/'.$champs['logo']); - if ($champs['logo'] && file_exists($chemin)) { + $formulaire->addElement('static' , 'html' , '', '
'); + $chemin = realpath('../../templates/' . $forum['path'] . '/images/' . $champs['logo']); + if ($champs['logo'] && $chemin && file_exists($chemin)) { if ((function_exists('getimagesize'))) { $info = getimagesize($chemin); $formulaire->addElement('static' , 'html' , '', 'Taille actuelle : ' . $info[3]); @@ -88,7 +89,7 @@ $file = $formulaire->getElement('logo'); $data = $file->getValue(); if ($data['name']) { - $file->moveUploadedFile(realpath('../../templates/'.$forum['path'].'/images/')); + $file->moveUploadedFile(realpath('../../templates/' . $forum['path'] . '/images/')); $data = $file->getValue(); $valeurs['logo'] = $data['name']; } else { diff --git a/htdocs/pages/administration/forum_planning.php b/htdocs/pages/administration/forum_planning.php index a6ea267a2..3aabbd0df 100755 --- a/htdocs/pages/administration/forum_planning.php +++ b/htdocs/pages/administration/forum_planning.php @@ -1,15 +1,16 @@ obtenirDernier(); - } - $rs_forum = $forum->obtenir( $_GET['id_forum']); - $annee_forum = $rs_forum['forum_annee']; + if (!isset($_GET['id_forum']) || intval($_GET['id_forum']) == 0) { + $_GET['id_forum'] = $forum->obtenirDernier(); + } - $sessions = $forum_appel->obtenirListeSessionsPlannifies($_GET['id_forum']); - $salles = $forum_appel->obtenirListeSalles($_GET['id_forum'], true); + $rs_forum = $forum->obtenir($_GET['id_forum']); + $annee_forum = $rs_forum['forum_annee']; - $smarty->assign('agenda', $forum->genAgenda($annee_forum, true, false, $_GET['id_forum'])); - $smarty->assign('id_forum', $_GET['id_forum']); - $smarty->assign('forums', $forum->obtenirListe()); - $smarty->assign('sessions', $sessions); + $sessions = $forum_appel->obtenirListeSessionsPlannifies($_GET['id_forum']); + $salles = $forum_appel->obtenirListeSalles($_GET['id_forum'], true); + $smarty->assign('agenda', $forum->genAgenda($annee_forum, true, false, $_GET['id_forum'])); + $smarty->assign('id_forum', $_GET['id_forum']); + $smarty->assign('forums', $forum->obtenirListe()); + $smarty->assign('sessions', $sessions); } elseif ($action == 'supprimer') { - if ($forum_appel->supprimerSessionDuPlanning($_GET['id'])) { - Logs::log('Suppression de la programmation de la session ' . $_GET['id']); - afficherMessage('La programmation de la session a été supprimée', 'index.php?page=forum_planning&action=lister'); - } else { - afficherMessage('Une erreur est survenue lors de la suppression de la session', 'index.php?page=forum_planning&action=lister', true); - } - + if ($forum_appel->supprimerSessionDuPlanning($_GET['id'])) { + Logs::log('Suppression de la programmation de la session ' . $_GET['id']); + afficherMessage('La programmation de la session a été supprimée', 'index.php?page=forum_planning&action=lister'); + } else { + afficherMessage('Une erreur est survenue lors de la suppression de la session', 'index.php?page=forum_planning&action=lister', true); + } } else { - - $pays = new Pays($bdd); - $formulaire = instancierFormulaire(); - - $champs = $forum_appel->obtenirPlanningDeSession($_GET['id_session']); - $conferenciers = $forum_appel->obtenirConferenciersPourSession($_GET['id_session']); - - if (empty($champs['debut']) && empty($champs['fin'])) { - $id_forum = $forum->obtenirDernier(); - $forum_donnees = $forum->obtenir($id_forum); - $champs['debut'] = $forum_donnees['date_debut']; - $champs['fin'] = $forum_donnees['date_debut']; - } - - $formulaire->setDefaults($champs); - $id = $_GET['id'] ?? 0; - - $formulaire->addElement('hidden', 'id' , null); - $formulaire->addElement('hidden', 'id_session', $_GET['id_session']); - $formulaire->addElement('hidden', 'id_forum', $champs['id_forum']); - - $formulaire->addElement('header', null, 'Présentation'); - $formulaire->addElement('static', 'titre' , 'Titre' , ''.$champs['titre'].''); - $formulaire->addElement('static', 'abstract', 'Résumé', $champs['abstract']); - - foreach ($conferenciers as $conferencier) { - $formulaire->addElement('static', 'conferencier_id_'.$conferencier['conferencier_id'], 'Conférencier', $conferencier['nom'].' '.$conferencier['prenom'].' ('.$conferencier['societe'].')'); - } - - $formulaire->addElement('header', null, 'Plannification'); - $formulaire->addElement('date' , 'debut' , 'Début', ['language' => 'fr', 'format' => "dMY H:i", 'minYear' => date('Y'), 'maxYear' => date('Y') + 1, 'minHour' => 8, 'maxHour' => 18, 'optionIncrement' => ['i' => 5]]); - $formulaire->addElement('date' , 'fin' , 'Fin' , ['language' => 'fr', 'format' => "dMY H:i", 'minYear' => date('Y'), 'maxYear' => date('Y') + 1, 'optionIncrement' => ['i' => 5], 'minHour' => 8, 'maxHour' => 18]); - $formulaire->addElement('select' , 'id_salle', 'Salle', [null => '' ] + $forum_appel->obtenirListeSalles($champs['id_forum'], true)); + $pays = new Pays($bdd); + $formulaire = instancierFormulaire(); + + $champs = $forum_appel->obtenirPlanningDeSession($_GET['id_session']); + $conferenciers = $forum_appel->obtenirConferenciersPourSession($_GET['id_session']); + + if (empty($champs['debut']) && empty($champs['fin'])) { + $id_forum = $forum->obtenirDernier(); + $forum_donnees = $forum->obtenir($id_forum); + $champs['debut'] = $forum_donnees['date_debut']; + $champs['fin'] = $forum_donnees['date_debut']; + } + + $formulaire->setDefaults($champs); + $id = $_GET['id'] ?? 0; + + $formulaire->addElement('hidden', 'id' , null); + $formulaire->addElement('hidden', 'id_session', $_GET['id_session']); + $formulaire->addElement('hidden', 'id_forum', $champs['id_forum']); + + $formulaire->addElement('header', null, 'Présentation'); + $formulaire->addElement('static', 'titre' , 'Titre' , '' . $champs['titre'] . ''); + $formulaire->addElement('static', 'abstract', 'Résumé', $champs['abstract']); + + foreach ($conferenciers as $conferencier) { + $formulaire->addElement('static', 'conferencier_id_' . $conferencier['conferencier_id'], 'Conférencier', $conferencier['nom'] . ' ' . $conferencier['prenom'] . ' (' . $conferencier['societe'] . ')'); + } + + $formulaire->addElement('header', null, 'Plannification'); + $formulaire->addElement('date' , 'debut' , 'Début', ['language' => 'fr', 'format' => "dMY H:i", 'minYear' => date('Y'), 'maxYear' => date('Y') + 1, 'minHour' => 8, 'maxHour' => 18, 'optionIncrement' => ['i' => 5]]); + $formulaire->addElement('date' , 'fin' , 'Fin' , ['language' => 'fr', 'format' => "dMY H:i", 'minYear' => date('Y'), 'maxYear' => date('Y') + 1, 'optionIncrement' => ['i' => 5], 'minHour' => 8, 'maxHour' => 18]); + $formulaire->addElement('select' , 'id_salle', 'Salle', [null => '' ] + $forum_appel->obtenirListeSalles($champs['id_forum'], true)); $formulaire->addElement('text' , 'joindin' , 'Id de la conférence chez joind.in' , ['size' => 40, 'maxlength' => 10]); - $formulaire->addElement('header', 'boutons' , ''); - $formulaire->addElement('submit', 'soumettre', 'Soumettre'); - - $formulaire->addRule('debut' , 'Date et heure du début manquants', 'required'); - $formulaire->addRule('fin' , 'Date et heure de fin manquants' , 'required'); - $formulaire->addRule('id_salle', 'Nom de la salle manquant' , 'required'); - - if ($formulaire->validate()) { - $valeurs = $formulaire->exportValues(); - - if ($id == 0) { - $planning_id = $forum_appel->ajouterSessionDansPlanning($valeurs['id_forum'], - $valeurs['id_session'], - mktime($valeurs['debut']['H'], $valeurs['debut']['i'], 0, $valeurs['debut']['M'], $valeurs['debut']['d'], $valeurs['debut']['Y']), - mktime($valeurs['fin']['H'], $valeurs['fin']['i'], 0, $valeurs['fin']['M'], $valeurs['fin']['d'], $valeurs['fin']['Y']), - $valeurs['id_salle']); - - $ok = (bool)$planning_id; - } else { - $planning_id = (int)$_GET['id']; - $ok = $forum_appel->modifierSessionDuPlanning($planning_id, - $valeurs['id_forum'], - $valeurs['id_session'], - mktime($valeurs['debut']['H'], $valeurs['debut']['i'], 0, $valeurs['debut']['M'], $valeurs['debut']['d'], $valeurs['debut']['Y']), - mktime($valeurs['fin']['H'], $valeurs['fin']['i'], 0, $valeurs['fin']['M'], $valeurs['fin']['d'], $valeurs['fin']['Y']), - $valeurs['id_salle']); - $forum_appel->modifierJoindinSession($valeurs['id_session'], $valeurs['joindin']); - } - - if ($ok) { - if ($action == 'ajouter') { - Logs::log('Ajout du planning de la session de ' . $formulaire->exportValue('titre')); - } else { - Logs::log('Modification du planning de la session de ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); - } - afficherMessage('Le planning de la session a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=forum_planning&action=lister'); - } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' du planning de la session'); - } - } - - $current = $forum->obtenir($champs['id_forum'], 'titre'); - $smarty->assign('forum_name', $current['titre']); - $smarty->assign('formulaire', genererFormulaire($formulaire)); + $formulaire->addElement('header', 'boutons' , ''); + $formulaire->addElement('submit', 'soumettre', 'Soumettre'); + + $formulaire->addRule('debut' , 'Date et heure du début manquants', 'required'); + $formulaire->addRule('fin' , 'Date et heure de fin manquants' , 'required'); + $formulaire->addRule('id_salle', 'Nom de la salle manquant' , 'required'); + + if ($formulaire->validate()) { + $valeurs = $formulaire->exportValues(); + + if ($id == 0) { + $planning_id = $forum_appel->ajouterSessionDansPlanning($valeurs['id_forum'], + $valeurs['id_session'], + mktime($valeurs['debut']['H'], $valeurs['debut']['i'], 0, $valeurs['debut']['M'], $valeurs['debut']['d'], $valeurs['debut']['Y']), + mktime($valeurs['fin']['H'], $valeurs['fin']['i'], 0, $valeurs['fin']['M'], $valeurs['fin']['d'], $valeurs['fin']['Y']), + $valeurs['id_salle']); + + $ok = (bool) $planning_id; + } else { + $planning_id = (int) $_GET['id']; + $ok = $forum_appel->modifierSessionDuPlanning($planning_id, + $valeurs['id_forum'], + $valeurs['id_session'], + mktime((int) $valeurs['debut']['H'], (int) $valeurs['debut']['i'], 0, (int) $valeurs['debut']['M'], (int) $valeurs['debut']['d'], (int) $valeurs['debut']['Y']), + mktime((int) $valeurs['fin']['H'],(int) $valeurs['fin']['i'], 0,(int) $valeurs['fin']['M'],(int) $valeurs['fin']['d'], (int) $valeurs['fin']['Y']), + $valeurs['id_salle']); + $forum_appel->modifierJoindinSession($valeurs['id_session'], $valeurs['joindin']); + } + + if ($ok) { + if ($action == 'ajouter') { + Logs::log('Ajout du planning de la session de ' . $formulaire->exportValue('titre')); + } else { + Logs::log('Modification du planning de la session de ' . $formulaire->exportValue('titre') . ' (' . $_GET['id'] . ')'); + } + afficherMessage('Le planning de la session a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=forum_planning&action=lister'); + } else { + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' du planning de la session'); + } + } + + $current = $forum->obtenir($champs['id_forum'], 'titre'); + $smarty->assign('forum_name', $current['titre']); + $smarty->assign('formulaire', genererFormulaire($formulaire)); } diff --git a/htdocs/pages/administration/forum_sessions.php b/htdocs/pages/administration/forum_sessions.php index 7b183c641..7f889ffde 100644 --- a/htdocs/pages/administration/forum_sessions.php +++ b/htdocs/pages/administration/forum_sessions.php @@ -1,17 +1,20 @@ supprimerSession($_GET['id'])) { Logs::log('Suppression de la session ' . $_GET['id']); - afficherMessage('La session a été supprimée', 'index.php?page=forum_sessions&action=lister&type='.$list_type); + afficherMessage('La session a été supprimée', 'index.php?page=forum_sessions&action=lister&type=' . $list_type); } else { - afficherMessage('Une erreur est survenue lors de la suppression de la session', 'index.php?page=forum_sessions&action=lister&type='.$list_type, true); + afficherMessage('Une erreur est survenue lors de la suppression de la session', 'index.php?page=forum_sessions&action=lister&type=' . $list_type, true); } - } else { - $pays = new Pays($bdd); $talk = null; - + $formulaire = instancierFormulaire(); if ($action != 'ajouter') { $champs = $forum_appel->obtenirSession($_GET['id']); @@ -116,14 +117,14 @@ $formulaire->setDefaults($champs); - if (isset($champs) && isset($champs['id_forum'])) { - $_GET['id_forum'] = $champs['id_forum']; - } + if (isset($champs) && isset($champs['id_forum'])) { + $_GET['id_forum'] = $champs['id_forum']; + } } $id = $_GET['id'] ?? 0; - $formulaire->addElement('hidden', 'id' , $id); - $formulaire->addElement('hidden', 'id_forum', $_GET['id_forum']); + $formulaire->addElement('hidden', 'id' , $id); + $formulaire->addElement('hidden', 'id_forum', $_GET['id_forum']); $formulaire->addElement('header', null, 'Présentation'); @@ -138,10 +139,10 @@ } $formulaire->addElement('textarea', 'abstract' , 'Résumé', ['cols' => 40, 'rows' => 15,'class'=> $abstractClass]); - $formulaire->addElement('hidden', 'use_markdown', (int)$useMarkdown); + $formulaire->addElement('hidden', 'use_markdown', (int) $useMarkdown); - $typesLabelsByKey = \AppBundle\Event\Model\Talk::getTypeLabelsByKey(); + $typesLabelsByKey = Talk::getTypeLabelsByKey(); asort($typesLabelsByKey); $groupe = []; foreach ($typesLabelsByKey as $genreKey => $genreLabel) { @@ -190,15 +191,15 @@ Assertion::notNull($event); $conferenciers = [null => '']; foreach ($speakerRepository->searchSpeakers($event) as $speaker) { - $conferenciers[$speaker->getId()] = $speaker->getLastname().' '.$speaker->getFirstname(); + $conferenciers[$speaker->getId()] = $speaker->getLastname() . ' ' . $speaker->getFirstname(); } - $formulaire->addElement('select', 'conferencier_id_1' , 'N°1', $conferenciers); - $formulaire->addElement('select', 'conferencier_id_2' , 'N°2', $conferenciers); + $formulaire->addElement('select', 'conferencier_id_1' , 'N°1', $conferenciers); + $formulaire->addElement('select', 'conferencier_id_2' , 'N°2', $conferenciers); $formulaire->addElement('select', 'conferencier_id_3' , 'N°3', $conferenciers); - if ($action != 'ajouter') { + if ($action != 'ajouter') { $conferenciers = $forum_appel->obtenirConferenciersPourSession($id); - $smarty->assign('session_conferenciers', $conferenciers); + $smarty->assign('session_conferenciers', $conferenciers); } $commentaires = $forum_appel->obtenirCommentairesPourSession($id); @@ -209,7 +210,7 @@ $feed .= '
'; $feed .= '
'; $feed .= '
'; - $feed .= $commentaire['nom'].' '.$commentaire['prenom']; + $feed .= $commentaire['nom'] . ' ' . $commentaire['prenom']; $feed .= '
'; $feed .= date('d/m/Y h:i', $commentaire['date']); $feed .= '
'; @@ -224,39 +225,39 @@ $formulaire->addElement('static', 'note', '', $feed); } - $formulaire->addElement('header', 'boutons' , ''); - $formulaire->addElement('submit', 'soumettre', 'Soumettre'); + $formulaire->addElement('header', 'boutons' , ''); + $formulaire->addElement('submit', 'soumettre', 'Soumettre'); - // On ajoute les règles - $formulaire->addRule('titre' , 'Titre manquant' , 'required'); - $formulaire->addRule('conferencier_id_1', 'Conférencier n°1 manquant' , 'required'); + // On ajoute les règles + $formulaire->addRule('titre' , 'Titre manquant' , 'required'); + $formulaire->addRule('conferencier_id_1', 'Conférencier n°1 manquant' , 'required'); if ($formulaire->validate()) { - $valeurs = $formulaire->exportValues(); - $valeurs += ['needs_mentoring' => 0]; + $valeurs = $formulaire->exportValues(); + $valeurs += ['needs_mentoring' => 0]; - if ($action == 'ajouter') { - $session_id = $forum_appel->ajouterSession( - $valeurs['id_forum'], - $valeurs['date_soumission']['Y'].'-'.$valeurs['date_soumission']['M'].'-'.$valeurs['date_soumission']['d'], + if ($action == 'ajouter') { + $session_id = $forum_appel->ajouterSession( + $valeurs['id_forum'], + $valeurs['date_soumission']['Y'] . '-' . $valeurs['date_soumission']['M'] . '-' . $valeurs['date_soumission']['d'], $valeurs['titre'], $valeurs['abstract'], - (int)$valeurs['genre'], - (int)$valeurs['plannifie'], - isset($valeurs['needs_mentoring']) ? (int)$valeurs['needs_mentoring'] : 0, - (int)$valeurs['skill'], + (int) $valeurs['genre'], + (int) $valeurs['plannifie'], + isset($valeurs['needs_mentoring']) ? (int) $valeurs['needs_mentoring'] : 0, + (int) $valeurs['skill'], $valeurs['use_markdown'] ); - $ok = (bool)$session_id; + $ok = (bool) $session_id; } else { - $session_id = (int)$_GET['id']; + $session_id = (int) $_GET['id']; $ok = $forum_appel->modifierSession($session_id, - $valeurs['id_forum'], - $valeurs['date_soumission']['Y'].'-'.$valeurs['date_soumission']['M'].'-'.$valeurs['date_soumission']['d'], - $valeurs['titre'], - $valeurs['abstract'], - (int)$valeurs['genre'], - (int)$valeurs['plannifie'], + $valeurs['id_forum'], + $valeurs['date_soumission']['Y'] . '-' . $valeurs['date_soumission']['M'] . '-' . $valeurs['date_soumission']['d'], + $valeurs['titre'], + $valeurs['abstract'], + (int) $valeurs['genre'], + (int) $valeurs['plannifie'], $valeurs['joindin'], $valeurs['youtube_id'], $valeurs['slides_url'], @@ -264,12 +265,12 @@ $valeurs['blog_post_url'], $valeurs['interview_url'], $valeurs['language_code'], - (int)$valeurs['skill'], - (int)$valeurs['needs_mentoring'], + (int) $valeurs['skill'], + (int) $valeurs['needs_mentoring'], $valeurs['use_markdown'], $valeurs['video_has_fr_subtitles'], $valeurs['video_has_en_subtitles'], - $valeurs['date_publication']['Y'].'-'.$valeurs['date_publication']['M'].'-'.$valeurs['date_publication']['d'] . ' ' . $valeurs['date_publication']['H'] . ':' . $valeurs['date_publication']['i'] . ':' . $valeurs['date_publication']['s'], + $valeurs['date_publication']['Y'] . '-' . $valeurs['date_publication']['M'] . '-' . $valeurs['date_publication']['d'] . ' ' . $valeurs['date_publication']['H'] . ':' . $valeurs['date_publication']['i'] . ':' . $valeurs['date_publication']['s'], $valeurs['tweets'], $valeurs['transcript'], $valeurs['verbatim'] diff --git a/htdocs/pages/administration/index.php b/htdocs/pages/administration/index.php index f0481ad9e..a687583ba 100644 --- a/htdocs/pages/administration/index.php +++ b/htdocs/pages/administration/index.php @@ -1,6 +1,11 @@ getResponse(); $response->send(); diff --git a/htdocs/pages/administration/logs.php b/htdocs/pages/administration/logs.php index 5ce2f5dfd..3a4cc666b 100644 --- a/htdocs/pages/administration/logs.php +++ b/htdocs/pages/administration/logs.php @@ -1,20 +1,19 @@ assign('logs' , Logs::obtenirTous($_GET['numero_page'])); $smarty->assign('nombre_pages', Logs::obtenirNombrePages()); $smarty->assign('numero_page' , $_GET['numero_page']); $smarty->assign('url' , 'index.php?page=logs'); - -?> diff --git a/htdocs/pages/administration/membre_administrateur.php b/htdocs/pages/administration/membre_administrateur.php index f3eca4ba4..029d68578 100644 --- a/htdocs/pages/administration/membre_administrateur.php +++ b/htdocs/pages/administration/membre_administrateur.php @@ -1,9 +1,12 @@ exportValue('raison_sociale') . ' (' . $_GET['id'] . ')'); - afficherMessage('La personne morale a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=membre_personne_morale'); + afficherMessage('La personne morale a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=membre_personne_morale'); } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action == 'ajouter') ? "l'ajout" : 'la modification') . ' de la personne morale'); + $smarty->assign('erreur', 'Une erreur est survenue lors de ' . (($action === 'ajouter') ? "l'ajout" : 'la modification') . ' de la personne morale'); } } diff --git a/htdocs/pages/administration/message.php b/htdocs/pages/administration/message.php index 10b8bae81..0c2ae9320 100644 --- a/htdocs/pages/administration/message.php +++ b/htdocs/pages/administration/message.php @@ -1,7 +1,11 @@ assign('message', stripslashes($_GET['message'])); $smarty->assign('url' , $_GET['url']); $smarty->assign('erreur' , $_GET['erreur']); $smarty->display('message.html'); - + // On s'arrête là pour ne pas afficher le pied de page exit; -?> diff --git a/htdocs/pages/administration/paybox_annule.php b/htdocs/pages/administration/paybox_annule.php index 804de78c3..a67bf429b 100644 --- a/htdocs/pages/administration/paybox_annule.php +++ b/htdocs/pages/administration/paybox_annule.php @@ -1,5 +1,7 @@ Votre paiement a été annulé. Désolé.

"; $paybox .= "

Une questions ? N'hésitez pas à contacter le trésorier.

"; @@ -7,4 +9,3 @@ $smarty->assign('paybox', $paybox); $smarty->display('paybox.html'); -?> \ No newline at end of file diff --git a/htdocs/pages/administration/paybox_effectue.php b/htdocs/pages/administration/paybox_effectue.php index 2a8a34375..65a807084 100644 --- a/htdocs/pages/administration/paybox_effectue.php +++ b/htdocs/pages/administration/paybox_effectue.php @@ -1,6 +1,8 @@ Votre paiement a été enregistré. Merci et à bientôt.

"; $message .= "

Une questions ? N'hésitez pas à contacter le trésorier.

"; diff --git a/htdocs/pages/administration/paybox_erreur.php b/htdocs/pages/administration/paybox_erreur.php index 9320424e6..38b0a625c 100644 --- a/htdocs/pages/administration/paybox_erreur.php +++ b/htdocs/pages/administration/paybox_erreur.php @@ -1,5 +1,7 @@ Il y a eu une erreur lors de votre paiement. Désolé.

"; $paybox .= "

Une questions ? N'hésitez pas à contacter le trésorier.

"; diff --git a/htdocs/pages/administration/paybox_refuse.php b/htdocs/pages/administration/paybox_refuse.php index 8c58c2d3b..c712adcec 100644 --- a/htdocs/pages/administration/paybox_refuse.php +++ b/htdocs/pages/administration/paybox_refuse.php @@ -1,13 +1,17 @@ Votre paiement a été refusé. Désolé.

"; $paybox .= "

Une questions ? N'hésitez pas à contacter le trésorier.

"; @@ -15,4 +19,3 @@ $smarty->assign('paybox', $paybox); $smarty->display('paybox.html'); -?> diff --git a/htdocs/pages/administration/personnes_morales.php b/htdocs/pages/administration/personnes_morales.php index ae33fdfce..72f3c6c63 100644 --- a/htdocs/pages/administration/personnes_morales.php +++ b/htdocs/pages/administration/personnes_morales.php @@ -1,12 +1,15 @@ addElement('text' , 'email' , 'Email' , ['size' => 30, 'maxlength' => 100]); $formulaire->addElement('text' , 'telephone_fixe' , 'Tél. fixe' , ['size' => 20, 'maxlength' => 20]); $formulaire->addElement('text' , 'telephone_portable' , 'Tél. portable' , ['size' => 20, 'maxlength' => 20]); - if($action != 'ajouter') { + if ($action != 'ajouter') { $smarty->assign('personnes_physiques_associees', $users); } $formulaire->addElement('header' , '' , 'Paramètres'); diff --git a/htdocs/pages/administration/site_articles.php b/htdocs/pages/administration/site_articles.php index 3592294b1..137babe06 100755 --- a/htdocs/pages/administration/site_articles.php +++ b/htdocs/pages/administration/site_articles.php @@ -1,13 +1,17 @@ obtenirListe(null, '*', 'date_debut DESC') as $forum) { $forumLabelsById[$forum['id']] = $forum['titre']; @@ -59,7 +63,7 @@ function checkNoSpace($value): bool } $articlesList = []; - foreach ($articles->obtenirListe($list_champs, $list_ordre.' '.$list_sens, $list_filtre) as $article) { + foreach ($articles->obtenirListe($list_champs, $list_ordre . ' ' . $list_sens, $list_filtre) as $article) { $article['theme_label'] = Article::getThemeLabel($article['theme']); $article['forum_label'] = $forumLabelsById[$article['id_forum']] ?? ''; $articlesList[] = $article; @@ -67,7 +71,6 @@ function checkNoSpace($value): bool // Mise en place de la liste dans le scope de smarty $smarty->assign('articles', $articlesList); - } elseif ($action == 'supprimer') { $article = new Article($_GET['id']); if ($article->supprimer()) { @@ -76,14 +79,13 @@ function checkNoSpace($value): bool } else { afficherMessage('Une erreur est survenue lors de la suppression de l\'article', 'index.php?page=site_articles&action=lister', true); } - } else { - $id = isset($_GET['id']) ? (int)$_GET['id'] : 0; + $id = isset($_GET['id']) ? (int) $_GET['id'] : 0; $article = new Article($id); $rubriques = new Rubriques(); $users = [null => '']; foreach ($userRepository->search() as $user) { - $users[$user->getId()] = $user->getFirstName().' '.$user->getLastName(); + $users[$user->getId()] = $user->getFirstName() . ' ' . $user->getLastName(); } $formulaire = instancierFormulaire(); @@ -143,16 +145,12 @@ function checkNoSpace($value): bool $article->position = $formulaire->exportValue('position'); $date = $formulaire->exportValue('date'); - $article->date = mktime($date['H'], $date['i'], $date['s'], $date['M'], $date['d'], $date['Y']); + $article->date = mktime((int) $date['H'],(int) $date['i'], (int) $date['s'], (int) $date['M'], (int) $date['d'], (int) $date['Y']); $article->etat = $formulaire->exportValue('etat'); $article->theme = $formulaire->exportValue('theme'); $article->id_forum = $formulaire->exportValue('id_forum'); - if ($action == 'ajouter') { - $ok = $article->inserer(); - } else { - $ok = $article->modifier(); - } + $ok = $action == 'ajouter' ? $article->inserer() : $article->modifier(); if ($ok) { if ($action == 'ajouter') { diff --git a/htdocs/pages/administration/site_feuilles.php b/htdocs/pages/administration/site_feuilles.php index 45b68c3ae..404fda01b 100644 --- a/htdocs/pages/administration/site_feuilles.php +++ b/htdocs/pages/administration/site_feuilles.php @@ -1,14 +1,15 @@ assign('feuilles', $feuilles->obtenirListe($list_champs, $list_ordre.' '.$list_sens, $list_filtre)); + $f = []; + $list_champs = '*'; + $list_ordre = 'date'; + $list_sens = 'desc'; + $list_filtre = false; + + if (isset($_GET['sens']) && in_array($_GET['sens'], array_keys($sens_valides))) { + $list_sens = $_GET['sens']; + } else { + $_GET['sens'] = $list_sens; + } + if (isset($_GET['tri']) && in_array($_GET['tri'], array_keys($tris_valides))) { + $list_ordre = $_GET['tri']; + } else { + $_GET['tri'] = $list_ordre; + } + + // Mise en place de la liste dans le scope de smarty + $smarty->assign('feuilles', $feuilles->obtenirListe($list_champs, $list_ordre . ' ' . $list_sens, $list_filtre)); } elseif ($action == 'supprimer') { - $feuille = new Feuille($_GET['id']); - if ($feuille->supprimer()) { - Logs::log('Suppression de la feuille ' . $_GET['id']); - afficherMessage('La feuille a été supprimée', 'index.php?page=site_feuilles&action=lister'); - } else { - afficherMessage('Une erreur est survenue lors de la suppression de la feuille', 'index.php?page=site_feuilles&action=lister', true); - } + $feuille = new Feuille($_GET['id']); + if ($feuille->supprimer()) { + Logs::log('Suppression de la feuille ' . $_GET['id']); + afficherMessage('La feuille a été supprimée', 'index.php?page=site_feuilles&action=lister'); + } else { + afficherMessage('Une erreur est survenue lors de la suppression de la feuille', 'index.php?page=site_feuilles&action=lister', true); + } } else { // ajouter | modifier - $id = isset($_GET['id']) ? (int)$_GET['id'] : 0; - $feuille = new Feuille($id); - - $formulaire = instancierFormulaire(); - if ($action == 'ajouter') { - $formulaire->setDefaults(['date' => time(), - 'position' => 0, - 'id_personne_physique' => $droits->obtenirIdentifiant(), - 'etat' => 0]); - } else { - $feuille->charger(); - $formulaire->setDefaults($feuille->exportable()); - } - - $formulaire->addElement('header' , '' , 'feuille'); - $formulaire->addElement('select' , 'id_parent' , 'Parent' , [null => '' ] + $feuilles->obtenirListe('id, nom', 'nom', true)); - $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 60, 'maxlength' => 255]); - $formulaire->addElement('text' , 'lien' , 'Lien' , ['size' => 60, 'maxlength' => 255]); - $formulaire->addElement('text' , 'alt' , 'Description', ['size' => 60, 'maxlength' => 255]); + $id = isset($_GET['id']) ? (int) $_GET['id'] : 0; + $feuille = new Feuille($id); + + $formulaire = instancierFormulaire(); + if ($action == 'ajouter') { + $formulaire->setDefaults(['date' => time(), + 'position' => 0, + 'id_personne_physique' => $droits->obtenirIdentifiant(), + 'etat' => 0]); + } else { + $feuille->charger(); + $formulaire->setDefaults($feuille->exportable()); + } + + $formulaire->addElement('header' , '' , 'feuille'); + $formulaire->addElement('select' , 'id_parent' , 'Parent' , [null => '' ] + $feuilles->obtenirListe('id, nom', 'nom', true)); + $formulaire->addElement('text' , 'nom' , 'Nom' , ['size' => 60, 'maxlength' => 255]); + $formulaire->addElement('text' , 'lien' , 'Lien' , ['size' => 60, 'maxlength' => 255]); + $formulaire->addElement('text' , 'alt' , 'Description', ['size' => 60, 'maxlength' => 255]); $file =& $formulaire->addElement('file', 'nouvelle-image' , 'Image'); - $formulaire->addElement('static' , 'note' , '' , ''); - $formulaire->addElement('text' , 'image_alt' , 'Texte alternatif pour l\'image', ['size' => 60, 'maxlength' => 255]); + $formulaire->addElement('static' , 'note' , '' , ''); + $formulaire->addElement('text' , 'image_alt' , 'Texte alternatif pour l\'image', ['size' => 60, 'maxlength' => 255]); $formulaire->addElement('hidden' , 'image'); $formulaire->addElement('date' , 'date' , 'Date' , ['language' => 'fr', 'minYear' => 2001, 'maxYear' => date('Y')]); - $formulaire->addElement('select' , 'position' , 'Position' , $feuille->positionable()); - $formulaire->addElement('select' , 'etat' , 'Etat' , [-1 => 'Hors ligne', 0 => 'En attente', 1 => 'En ligne']); + $formulaire->addElement('select' , 'position' , 'Position' , $feuille->positionable()); + $formulaire->addElement('select' , 'etat' , 'Etat' , [-1 => 'Hors ligne', 0 => 'En attente', 1 => 'En ligne']); $formulaire->addElement('textarea' , 'patterns' , 'Patterns URL'); - $formulaire->addElement('header' , 'boutons' , ''); - $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); + $formulaire->addElement('header' , 'boutons' , ''); + $formulaire->addElement('submit' , 'soumettre' , ucfirst($action)); - $formulaire->addRule('nom' , 'Nom manquant' , 'required'); - $formulaire->addRule('contenu' , 'Contenu manquant' , 'required'); - $formulaire->addRule('lien' , 'Lien manquant' , 'required'); + $formulaire->addRule('nom' , 'Nom manquant' , 'required'); + $formulaire->addRule('contenu' , 'Contenu manquant' , 'required'); + $formulaire->addRule('lien' , 'Lien manquant' , 'required'); $formulaire->addRule('image' , 'Mimetype' , 'mimetype', ['jpg','jpeg','gif','png']); - if ($file->isUploadedFile()) { - $values = $file->getValue(); - if ($values['error'] == 0) { - $file->moveUploadedFile('../../templates/site/images/', $values['name']); - $feuille->image = $values['name']; - } else { - $feuille->image = $formulaire->exportValue('image'); - } - } + if ($file->isUploadedFile()) { + $values = $file->getValue(); + if ($values['error'] == 0) { + $file->moveUploadedFile('../../templates/site/images/', $values['name']); + $feuille->image = $values['name']; + } else { + $feuille->image = $formulaire->exportValue('image'); + } + } if ($formulaire->validate()) { - $feuille->id_parent = $formulaire->exportValue('id_parent'); - $feuille->nom = $formulaire->exportValue('nom'); - $feuille->lien = $formulaire->exportValue('lien'); - $feuille->alt = $formulaire->exportValue('alt'); + $feuille->id_parent = $formulaire->exportValue('id_parent'); + $feuille->nom = $formulaire->exportValue('nom'); + $feuille->lien = $formulaire->exportValue('lien'); + $feuille->alt = $formulaire->exportValue('alt'); $feuille->image_alt = $formulaire->exportValue('image_alt'); - $feuille->position = $formulaire->exportValue('position'); - $date = $formulaire->exportValue('date'); - $feuille->date = mktime(0, 0, 0, $date['M'], $date['d'], $date['Y']); - $feuille->etat = $formulaire->exportValue('etat'); - $feuille->patterns = $formulaire->exportValue('patterns'); - - if ($action == 'ajouter') { - if ($feuille->inserer()) { - Logs::log('Ajout de la feuille ' . $formulaire->exportValue('nom')); - afficherMessage('La feuille a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=site_feuilles&action=lister'); - } else { + $feuille->position = $formulaire->exportValue('position'); + $date = $formulaire->exportValue('date'); + $feuille->date = mktime(0, 0, 0, (int) $date['M'], (int) $date['d'], (int) $date['Y']); + $feuille->etat = $formulaire->exportValue('etat'); + $feuille->patterns = $formulaire->exportValue('patterns'); + + if ($action == 'ajouter') { + if ($feuille->inserer()) { + Logs::log('Ajout de la feuille ' . $formulaire->exportValue('nom')); + afficherMessage('La feuille a été ' . (($action === 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=site_feuilles&action=lister'); + } else { $smarty->assign('erreur', 'Une erreur est survenue lors de l\'ajout de la feuille'); - } - } else { - if ($feuille->modifier()) { - Logs::log('Ajout de la feuille ' . $formulaire->exportValue('nom')); - afficherMessage('La feuille a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=site_feuilles&action=lister'); - } else { - $smarty->assign('erreur', 'Une erreur est survenue lors de la modification de la feuille'); - } - } - } - - $smarty->assign('formulaire', genererFormulaire($formulaire)); + } + } elseif ($feuille->modifier()) { + Logs::log('Ajout de la feuille ' . $formulaire->exportValue('nom')); + afficherMessage('La feuille a été ' . (($action == 'ajouter') ? 'ajoutée' : 'modifiée'), 'index.php?page=site_feuilles&action=lister'); + } else { + $smarty->assign('erreur', 'Une erreur est survenue lors de la modification de la feuille'); + } + } + + $smarty->assign('formulaire', genererFormulaire($formulaire)); } -function process($values) { +function process($values): void +{ global $file; if ($file->isUploadedFile()) { $file->moveUploadedFile($path); diff --git a/htdocs/pages/event-payment/index.php b/htdocs/pages/event-payment/index.php index 508c4c204..69a2a4586 100644 --- a/htdocs/pages/event-payment/index.php +++ b/htdocs/pages/event-payment/index.php @@ -1,19 +1,20 @@ setUrlRetourErreur('https://' . $_SERVER['HTTP_HOST'] . dirname($_SERVER['REQUEST_URI']) . '/paybox_erreur.php') ; -$invoiceRepository = $symfonyKernel->getKernel()->getContainer()->get(\AppBundle\Event\Model\Repository\InvoiceRepository::class); -/** @var \AppBundle\Event\Model\Invoice $invoice */ +$invoiceRepository = $symfonyKernel->getKernel()->getContainer()->get(InvoiceRepository::class); +/** @var Invoice $invoice */ $invoice = $invoiceRepository->getByReference($inscription['reference']); if (null === $invoice) { throw new \RuntimeException(sprintf("Invoice %s not found", $inscription['reference'])); } -$payboxBilling = \AppBundle\Payment\PayboxBilling::createFromInvoice($invoice); +$payboxBilling = PayboxBilling::createFromInvoice($invoice); $smarty->assign('paybox', $paybox->generate(new \DateTime(), $payboxBilling)); $smarty->assign('inscription', $inscription); diff --git a/htdocs/pages/event-payment/paybox_annule.php b/htdocs/pages/event-payment/paybox_annule.php index d13db448b..b1285a8bc 100644 --- a/htdocs/pages/event-payment/paybox_annule.php +++ b/htdocs/pages/event-payment/paybox_annule.php @@ -1,3 +1,5 @@ display('paybox_annule.html'); diff --git a/htdocs/pages/event-payment/paybox_effectue.php b/htdocs/pages/event-payment/paybox_effectue.php index 696fa68eb..e5b187613 100644 --- a/htdocs/pages/event-payment/paybox_effectue.php +++ b/htdocs/pages/event-payment/paybox_effectue.php @@ -1,3 +1,5 @@ display('paybox_effectue.html'); \ No newline at end of file + +declare(strict_types=1); +require_once __DIR__ . '/../../../sources/Afup/Bootstrap/Http.php'; +$smarty->display('paybox_effectue.html'); diff --git a/htdocs/pages/event-payment/paybox_erreur.php b/htdocs/pages/event-payment/paybox_erreur.php index 9fe3384db..7fab36377 100644 --- a/htdocs/pages/event-payment/paybox_erreur.php +++ b/htdocs/pages/event-payment/paybox_erreur.php @@ -1,3 +1,5 @@ display('paybox_erreur.html'); diff --git a/htdocs/pages/event-payment/paybox_refuse.php b/htdocs/pages/event-payment/paybox_refuse.php index 8ec06be1c..4cef45449 100644 --- a/htdocs/pages/event-payment/paybox_refuse.php +++ b/htdocs/pages/event-payment/paybox_refuse.php @@ -1,3 +1,5 @@ display('paybox_refuse.html'); diff --git a/htdocs/pages/paiement/index.php b/htdocs/pages/paiement/index.php index 40c98da05..190705880 100644 --- a/htdocs/pages/paiement/index.php +++ b/htdocs/pages/paiement/index.php @@ -1,13 +1,17 @@ obtenir($ref); if ($facture) { @@ -37,7 +41,7 @@ ->setUrlRetourErreur('https://' . $_SERVER['HTTP_HOST'] . dirname($_SERVER['REQUEST_URI']) . '/paybox_erreur.php') ; - $payboxBilling = new \AppBundle\Payment\PayboxBilling($facture['prenom'], $facture['nom'], $facture['adresse'], $facture['code_postal'], $facture['ville'], $facture['id_pays']); + $payboxBilling = new PayboxBilling($facture['prenom'], $facture['nom'], $facture['adresse'], $facture['code_postal'], $facture['ville'], $facture['id_pays']); $now = new \DateTime(); $smarty->assign('paybox', $paybox->generate($now, $payboxBilling)); diff --git a/htdocs/pages/paiement/paybox_annule.php b/htdocs/pages/paiement/paybox_annule.php index d13db448b..b1285a8bc 100644 --- a/htdocs/pages/paiement/paybox_annule.php +++ b/htdocs/pages/paiement/paybox_annule.php @@ -1,3 +1,5 @@ display('paybox_annule.html'); diff --git a/htdocs/pages/paiement/paybox_effectue.php b/htdocs/pages/paiement/paybox_effectue.php index 696fa68eb..e5b187613 100644 --- a/htdocs/pages/paiement/paybox_effectue.php +++ b/htdocs/pages/paiement/paybox_effectue.php @@ -1,3 +1,5 @@ display('paybox_effectue.html'); \ No newline at end of file + +declare(strict_types=1); +require_once __DIR__ . '/../../../sources/Afup/Bootstrap/Http.php'; +$smarty->display('paybox_effectue.html'); diff --git a/htdocs/pages/paiement/paybox_erreur.php b/htdocs/pages/paiement/paybox_erreur.php index 9fe3384db..7fab36377 100644 --- a/htdocs/pages/paiement/paybox_erreur.php +++ b/htdocs/pages/paiement/paybox_erreur.php @@ -1,3 +1,5 @@ display('paybox_erreur.html'); diff --git a/htdocs/pages/paiement/paybox_refuse.php b/htdocs/pages/paiement/paybox_refuse.php index 8ec06be1c..4cef45449 100644 --- a/htdocs/pages/paiement/paybox_refuse.php +++ b/htdocs/pages/paiement/paybox_refuse.php @@ -1,3 +1,5 @@ display('paybox_refuse.html'); diff --git a/htdocs/pages/paiement/payment_tracking.php b/htdocs/pages/paiement/payment_tracking.php index 76705f45c..2e7303e12 100644 --- a/htdocs/pages/paiement/payment_tracking.php +++ b/htdocs/pages/paiement/payment_tracking.php @@ -1,13 +1,18 @@ boot(); -$container = $kernel->getContainer(); -/** @var FeedCrawler $crawler */ -$crawler = $container->get(FeedCrawler::class); -$crawler->crawl(); +boot(); +$container = $kernel->getContainer(); +/** @var FeedCrawler $crawler */ +$crawler = $container->get(FeedCrawler::class); +$crawler->crawl(); diff --git a/htdocs/templates/administration/compta_balance.html b/htdocs/templates/administration/compta_balance.html index 9014edc3f..1112bc547 100644 --- a/htdocs/templates/administration/compta_balance.html +++ b/htdocs/templates/administration/compta_balance.html @@ -32,8 +32,8 @@

Balance comptable

{foreach from=$balance item=ecriture} {$ecriture.evenement} - {$ecriture.debit|number_format:2:',':' '} - {$ecriture.credit|number_format:2:',':' '} + {$ecriture.debit|floatval|number_format:2:',':' '} + {$ecriture.credit|floatval|number_format:2:',':' '} Balance comptable {$detail.date_ecriture|date_format:"%d/%m/%Y"} {$detail.categorie} {$detail.description} - {if $detail.debit}{$detail.debit|number_format:2:',':' '}{else}0,00{/if} - {if $detail.credit}{$detail.credit|number_format:2:',':' '}{else}0,00{/if} + {if $detail.debit}{$detail.debit|floatval|number_format:2:',':' '}{else}0,00{/if} + {if $detail.credit}{$detail.credit|floatval|number_format:2:',':' '}{else}0,00{/if} Balance comptable Sous Total - {if $txt.debit}{$txt.debit|number_format:2:',':' '}{else}0,00{/if} + {if $txt.debit}{$txt.debit|floatval|number_format:2:',':' '}{else}0,00{/if} - {if $txt.credit}{$txt.credit|number_format:2:',':' '}{else}0,00{/if} + {if $txt.credit}{$txt.credit|floatval|number_format:2:',':' '}{else}0,00{/if} @@ -126,14 +126,14 @@

Balance comptable

- {$totalDepense|number_format:2:',':' '} + {$totalDepense|floatval|number_format:2:',':' '}
- {$totalRecette|number_format:2:',':' '} + {$totalRecette|floatval|number_format:2:',':' '}
@@ -185,7 +185,7 @@

Balance comptable

{$detail.description} {if $detail.debit} - {$detail.debit|number_format:2:',':' '} + {$detail.debit|floatval|number_format:2:',':' '} {/if} @@ -210,12 +210,12 @@

Balance comptable

{if $txt.debit} - {$txt.debit|number_format:2:',':' '} + {$txt.debit|floatval|number_format:2:',':' '} {/if} {if $txt.credit} - {$txt.credit|number_format:2:',':' '} + {$txt.credit|floatval|number_format:2:',':' '} {/if} diff --git a/htdocs/templates/administration/compta_banque.html b/htdocs/templates/administration/compta_banque.html index 1ca356708..35c8af527 100644 --- a/htdocs/templates/administration/compta_banque.html +++ b/htdocs/templates/administration/compta_banque.html @@ -57,11 +57,11 @@

Journal de banque

{$ecriture.reglement} {$ecriture.description} {if $ecriture.idoperation == 1} - {$ecriture.montant|number_format:2:',':' '} + {$ecriture.montant|floatval|number_format:2:',':' '} {else} - {$ecriture.montant|number_format:2:',':' '} + {$ecriture.montant|floatval|number_format:2:',':' '} {/if} Journal de banque Sous Total - {if $txt.debit}{$txt.debit|number_format:2:',':' '}{else}0,00{/if} + {if $txt.debit}{$txt.debit|floatval|number_format:2:',':' '}{else}0,00{/if} - {if $txt.credit}{$txt.credit|number_format:2:',':' '}{else}0,00{/if} + {if $txt.credit}{$txt.credit|floatval|number_format:2:',':' '}{else}0,00{/if} - {$txt.dif|number_format:2:',':' '} + {$txt.dif|floatval|number_format:2:',':' '} {assign var=compteurLigne value=0} @@ -107,7 +107,7 @@

Journal de banque

- {$total.debit|number_format:2:',':' '} + {$total.debit|floatval|number_format:2:',':' '}
Total débit @@ -117,7 +117,7 @@

Journal de banque

- {$total.credit|number_format:2:',':' '} + {$total.credit|floatval|number_format:2:',':' '}
Total crédit @@ -131,7 +131,7 @@

Journal de banque

- {$total.dif|number_format:2:',':' '} + {$total.dif|floatval|number_format:2:',':' '}
Solde diff --git a/htdocs/templates/administration/compta_bilan.html b/htdocs/templates/administration/compta_bilan.html index edc854d70..b8b00f091 100644 --- a/htdocs/templates/administration/compta_bilan.html +++ b/htdocs/templates/administration/compta_bilan.html @@ -41,7 +41,7 @@

Bilan

{foreach from=$debit item=debits} {$debits.evenement} - {$debits.montant|number_format:2:',':' '} + {$debits.montant|floatval|number_format:2:',':' '} Bilan {$detail.date_ecriture|date_format:"%d/%m/%Y"} {$detail.description} - {$detail.montant|number_format:2:',':' '} + {$detail.montant|floatval|number_format:2:',':' '} Bilan {foreach from=$credit item=credits} {$credits.evenement} - {$credits.montant|number_format:2:',':' '} + {$credits.montant|floatval|number_format:2:',':' '} Bilan {$detail.date_ecriture|date_format:"%d/%m/%Y"} {$detail.description} - {$detail.montant|number_format:2:',':' '} + {$detail.montant|floatval|number_format:2:',':' '} Bilan
- {$totalDepense|number_format:2:',':' '} + {$totalDepense|floatval|number_format:2:',':' '}
- {$totalRecette|number_format:2:',':' '} + {$totalRecette|floatval|number_format:2:',':' '}
@@ -201,7 +201,7 @@

Bilan

- {$difMontant|number_format:2:',':' '} + {$difMontant|floatval|number_format:2:',':' '}
@@ -244,7 +244,7 @@

Bilan

{$detail.date_ecriture|date_format:"%d/%m/%Y"} {$detail.description} - {$detail.montant|number_format:2:',':' '} + {$detail.montant|floatval|number_format:2:',':' '} {/foreach} @@ -272,7 +272,7 @@

Bilan

{$detail.date_ecriture|date_format:"%d/%m/%Y"} {$detail.description} - {$detail.montant|number_format:2:',':' '} + {$detail.montant|floatval|number_format:2:',':' '} {/foreach} @@ -290,14 +290,14 @@

Bilan

- {$sousTotalDebit|number_format:2:',':' '} + {$sousTotalDebit|floatval|number_format:2:',':' '}
- {$sousTotalCredit|number_format:2:',':' '} + {$sousTotalCredit|floatval|number_format:2:',':' '}
@@ -306,7 +306,7 @@

Bilan

- {$difMontant|number_format:2:',':' '} + {$difMontant|floatval|number_format:2:',':' '}
diff --git a/htdocs/templates/administration/compta_devis.html b/htdocs/templates/administration/compta_devis.html index 964832be1..8fea42d5a 100644 --- a/htdocs/templates/administration/compta_devis.html +++ b/htdocs/templates/administration/compta_devis.html @@ -53,7 +53,7 @@

Liste devis

{$ecriture.ville} {$ecriture.ref_clt1} {$ecriture.numero_devis} - {$ecriture.prix|number_format:2:',':' '} + {$ecriture.prix|floatval|number_format:2:',':' '} {if $ecriture.numero_facture == ''} non diff --git a/htdocs/templates/administration/compta_facture.html b/htdocs/templates/administration/compta_facture.html index 97d07cfad..a8bbc11ec 100644 --- a/htdocs/templates/administration/compta_facture.html +++ b/htdocs/templates/administration/compta_facture.html @@ -40,7 +40,7 @@

Factures

En attente {/if} - {$ecriture.prix|number_format:2:',':' '} + {$ecriture.prix|floatval|number_format:2:',':' '} Journal {$ecriture.description} {if $ecriture.idoperation == 1} - -{$ecriture.montant|number_format:2:',':' '} + -{$ecriture.montant|floatval|number_format:2:',':' '} {else} - +{$ecriture.montant|number_format:2:',':' '} + +{$ecriture.montant|floatval|number_format:2:',':' '} {/if} diff --git a/htdocs/templates/administration/compta_synthese.html b/htdocs/templates/administration/compta_synthese.html index 381ac05ec..497c72218 100644 --- a/htdocs/templates/administration/compta_synthese.html +++ b/htdocs/templates/administration/compta_synthese.html @@ -44,7 +44,7 @@

Synthese des évènements

{$debits.categorie} {$debits.description} - {$debits.montant|number_format:2:',':' '} + {$debits.montant|floatval|number_format:2:',':' '} {foreachelse} @@ -68,7 +68,7 @@

Synthese des évènements

{$credits.categorie} {$credits.description} - {$credits.montant|number_format:2:',':' '} + {$credits.montant|floatval|number_format:2:',':' '} {foreachelse} @@ -85,7 +85,7 @@

Synthese des évènements

- {$totalDepense|number_format:2:',':' '} + {$totalDepense|floatval|number_format:2:',':' '}
Total dépenses @@ -95,7 +95,7 @@

Synthese des évènements

- {$totalRecette|number_format:2:',':' '} + {$totalRecette|floatval|number_format:2:',':' '}
Total recettes @@ -108,7 +108,7 @@

Synthese des évènements

- {$difMontant|number_format:2:',':' '} + {$difMontant|floatval|number_format:2:',':' '}
Différence diff --git a/htdocs/templates/administration/forum_facturation.html b/htdocs/templates/administration/forum_facturation.html index 3b544b250..a19b60cfb 100644 --- a/htdocs/templates/administration/forum_facturation.html +++ b/htdocs/templates/administration/forum_facturation.html @@ -29,7 +29,7 @@

Factures d'évènement

- +
diff --git a/htdocs/templates/administration/forum_inscriptions.html b/htdocs/templates/administration/forum_inscriptions.html index 7298d668e..ee00d6214 100644 --- a/htdocs/templates/administration/forum_inscriptions.html +++ b/htdocs/templates/administration/forum_inscriptions.html @@ -131,7 +131,7 @@

Inscriptions

- +
diff --git a/htdocs/templates/administration/forum_sessions.html b/htdocs/templates/administration/forum_sessions.html index 9d4c923b3..9b11c2295 100644 --- a/htdocs/templates/administration/forum_sessions.html +++ b/htdocs/templates/administration/forum_sessions.html @@ -58,7 +58,7 @@

Conférences

- +
diff --git a/htdocs/templates/administration/personnes_morales.html b/htdocs/templates/administration/personnes_morales.html index 7e2fb3766..2ecbbcf87 100644 --- a/htdocs/templates/administration/personnes_morales.html +++ b/htdocs/templates/administration/personnes_morales.html @@ -19,7 +19,7 @@

Liste des personnes morales

- +
diff --git a/htdocs/templates/administration/site_articles.html b/htdocs/templates/administration/site_articles.html index fbad0e96f..c34849346 100755 --- a/htdocs/templates/administration/site_articles.html +++ b/htdocs/templates/administration/site_articles.html @@ -20,7 +20,7 @@

Liste des articles

- +
diff --git a/htdocs/templates/planete/images/cron.php b/htdocs/templates/planete/images/cron.php index 060af04c8..b47defc17 100644 --- a/htdocs/templates/planete/images/cron.php +++ b/htdocs/templates/planete/images/cron.php @@ -1,20 +1,20 @@ \ No newline at end of file diff --git a/rector.php b/rector.php index 9a0ab030a..f6381f1eb 100644 --- a/rector.php +++ b/rector.php @@ -17,13 +17,14 @@ __DIR__ . '/htdocs/cache', ]) ->withPhp74Sets() - //->withTypeCoverageLevel(0) - //->withDeadCodeLevel(0) - //->withCodeQualityLevel(0) - //->withSymfonyContainerXml(__DIR__ . '/var/cache/dev/appAppKernelDevDebugContainer.xml') - //->withSets([ - // SymfonySetList::SYMFONY_44, - // SymfonySetList::SYMFONY_CODE_QUALITY, - // SymfonySetList::SYMFONY_CONSTRUCTOR_INJECTION, - //]) + ->withTypeCoverageLevel(PHP_INT_MAX) + ->withDeadCodeLevel(PHP_INT_MAX) + ->withCodeQualityLevel(PHP_INT_MAX) + ->withImportNames(true, true, false) + ->withSymfonyContainerXml(__DIR__ . '/var/cache/dev/appDevDebugProjectContainer.xml') + ->withSets([ + SymfonySetList::SYMFONY_44, + SymfonySetList::SYMFONY_CODE_QUALITY, + SymfonySetList::SYMFONY_CONSTRUCTOR_INJECTION, + ]) ; diff --git a/sources/Afup/Association/Cotisations.php b/sources/Afup/Association/Cotisations.php index 04d2f7ad9..921b963ee 100644 --- a/sources/Afup/Association/Cotisations.php +++ b/sources/Afup/Association/Cotisations.php @@ -1,13 +1,16 @@ _bdd = $bdd; $this->_droits = $droits; @@ -69,10 +69,9 @@ function __construct(&$bdd, $droits = null) * @param string $champs Champs à renvoyer * @param string $ordre Tri des enregistrements * @param bool $associatif Renvoyer un tableau associatif ? - * @access public * @return array */ - function obtenirListe($type_personne, $id_personne, $champs = '*', $ordre = 'date_fin DESC', $associatif = false) + public function obtenirListe($type_personne, $id_personne, string $champs = '*', string $ordre = 'date_fin DESC', bool $associatif = false) { $requete = 'SELECT'; $requete .= ' ' . $champs . ' '; @@ -94,10 +93,9 @@ function obtenirListe($type_personne, $id_personne, $champs = '*', $ordre = 'dat * * @param int $id Identifiant de la cotisation * @param string $champs Champs à renvoyer - * @access public * @return array */ - function obtenir($id, $champs = '*') + public function obtenir($id, string $champs = '*') { $requete = 'SELECT'; $requete .= ' ' . $champs . ' '; @@ -111,10 +109,8 @@ function obtenir($id, $champs = '*') /** * Renvoit le numéro de la prochaine facture au format : {année}-{index depuis le début de l'année} * - * @access private - * @return string */ - function _genererNumeroFacture() + public function _genererNumeroFacture(): string { $requete = 'SELECT'; $requete .= " MAX(CAST(SUBSTRING_INDEX(numero_facture, '-', -1) AS UNSIGNED)) + 1 "; @@ -140,11 +136,10 @@ function _genererNumeroFacture() * @param int $date_fin Date de fin de la cotisation * @param string $commentaires Commentaires concernnant la cotisation * @param string $referenceClient Reference client à mentionner sur la facture - * @access public * @return bool Succès de l'ajout */ - function ajouter($type_personne, $id_personne, $montant, $type_reglement, - $informations_reglement, $date_debut, $date_fin, $commentaires, $referenceClient = null) + public function ajouter($type_personne, $id_personne, $montant, $type_reglement, + $informations_reglement, $date_debut, $date_fin, $commentaires, $referenceClient = null): bool { $requete = 'INSERT INTO '; $requete .= ' afup_cotisations (type_personne, id_personne, montant, type_reglement , informations_reglement,'; @@ -161,11 +156,7 @@ function ajouter($type_personne, $id_personne, $montant, $type_reglement, $requete .= $this->_bdd->echapper(base64_encode(random_bytes(30))) . ','; $requete .= $this->_bdd->echapper($commentaires) . ','; $requete .= $this->_bdd->echapper($referenceClient) . ')'; - - if ($this->_bdd->executer($requete) === false) { - return false; - } - return true; + return $this->_bdd->executer($requete) !== false; } /** @@ -182,11 +173,10 @@ function ajouter($type_personne, $id_personne, $montant, $type_reglement, * @param int $date_fin Date de fin de la cotisation * @param string $commentaires Commentaires concernnant la cotisation * @param string $referenceClient Reference client à mentionner sur la facture - * @access public * @return bool Succès de la modification */ - function modifier($id, $type_personne, $id_personne, $montant, $type_reglement, - $informations_reglement, $date_debut, $date_fin, $commentaires, $referenceClient) + public function modifier($id, $type_personne, $id_personne, $montant, $type_reglement, + $informations_reglement, $date_debut, $date_fin, $commentaires, $referenceClient): bool { $requete = 'UPDATE'; $requete .= ' afup_cotisations '; @@ -202,13 +192,10 @@ function modifier($id, $type_personne, $id_personne, $montant, $type_reglement, $requete .= ' reference_client=' . $this->_bdd->echapper($referenceClient) . ' '; $requete .= 'WHERE'; $requete .= ' id=' . $id; - if ($this->_bdd->executer($requete) === false) { - return false; - } - return true; + return $this->_bdd->executer($requete) !== false; } - function estDejaReglee($cmd) + public function estDejaReglee($cmd) { $requete = 'SELECT'; $requete .= ' 1 '; @@ -219,7 +206,7 @@ function estDejaReglee($cmd) return $this->_bdd->obtenirUn($requete); } - function notifierRegelementEnLigneAuTresorier($cmd, $total, $autorisation, $transaction, UserRepository $userRepository) + public function notifierRegelementEnLigneAuTresorier(string $cmd, string $total, string $autorisation, string $transaction, UserRepository $userRepository): ?bool { $account = $this->getAccountFromCmd($cmd); @@ -254,9 +241,10 @@ function notifierRegelementEnLigneAuTresorier($cmd, $total, $autorisation, $tran if (false === $ok) { return false; } + return null; } - function validerReglementEnLigne($cmd, $total, $autorisation, $transaction) + public function validerReglementEnLigne($cmd, $total, string $autorisation, string $transaction) { $reference = substr($cmd, 0, strlen($cmd) - 4); $verif = substr($cmd, strlen($cmd) - 3, strlen($cmd)); @@ -272,10 +260,9 @@ function validerReglementEnLigne($cmd, $total, $autorisation, $transaction) $cotisation['id'], AFUP_COTISATIONS_REGLEMENT_ENLIGNE, "autorisation : " . $autorisation . " / transaction : " . $transaction ); - - } elseif (substr(md5($reference), -3) == strtolower($verif) and !$this->estDejaReglee($cmd)) { + } elseif (substr(md5($reference), -3) === strtolower($verif) && !$this->estDejaReglee($cmd)) { [$ref, $date, $type_personne, $id_personne, $reste] = explode('-', $cmd, 5); - $date_debut = mktime(0, 0, 0, substr($date, 2, 2), substr($date, 0, 2), substr($date, 4, 4)); + $date_debut = mktime(0, 0, 0, (int) substr($date, 2, 2), (int) substr($date, 0, 2), (int) substr($date, 4, 4)); $cotisation = $this->obtenirDerniere($type_personne, $id_personne); $date_fin_precedente = $cotisation === false ? 0 : $cotisation['date_fin']; @@ -298,7 +285,10 @@ function validerReglementEnLigne($cmd, $total, $autorisation, $transaction) return $result; } - public function getAccountFromCmd($cmd) + /** + * @return int[]|string[] + */ + public function getAccountFromCmd($cmd): array { $arr = explode('-', $cmd, 5); // Personne morale : $cmd=FCOTIS-2023-202 @@ -316,10 +306,9 @@ public function getAccountFromCmd($cmd) * Supprime une cotisation * * @param int $id Identifiant de la cotisation à supprimer - * @access public * @return bool Succès de la suppression */ - function supprimer($id) + public function supprimer($id) { $requete = 'DELETE FROM afup_cotisations WHERE id=' . $id; return $this->_bdd->executer($requete); @@ -330,10 +319,9 @@ function supprimer($id) * * @param int $id_cotisation Identifiant de la cotisation * @param string $chemin Chemin du fichier PDF à générer. Si ce chemin est omi, le PDF est renvoyé au navigateur. - * @access public * @return int Le numero de la facture */ - function genererFacture($id_cotisation, $chemin = null) + public function genererFacture($id_cotisation, $chemin = null) { $requete = 'SELECT * FROM afup_cotisations WHERE id=' . $id_cotisation; $cotisation = $this->_bdd->obtenirEnregistrement($requete); @@ -397,7 +385,7 @@ function genererFacture($id_cotisation, $chemin = null) $pdf->Ln(); $pdf->SetFillColor(255, 255, 255); $pdf->Cell(50, 5, 'ADH', 1); - $pdf->Cell(100, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', $cotisation['date_fin'])), 1); + $pdf->Cell(100, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', (int) $cotisation['date_fin'])), 1); $pdf->Cell(40, 5, utf8_decode($cotisation['montant'] . ' €'), 1); $pdf->Ln(15); @@ -445,7 +433,7 @@ function genererFacture($id_cotisation, $chemin = null) $pdf->Cell(10, 5, utf8_decode('Lors de votre règlement, merci de préciser la mention : "Facture n°' . $cotisation['numero_facture']) . '"'); if (is_null($chemin)) { - $pattern = str_replace(' ', '', $patternPrefix) . '_' . $cotisation['numero_facture'] . '_' . date('dmY', $cotisation['date_debut']) . '.pdf'; + $pattern = str_replace(' ', '', $patternPrefix) . '_' . $cotisation['numero_facture'] . '_' . date('dmY', (int) $cotisation['date_debut']) . '.pdf'; $pdf->Output($pattern, 'D'); } else { @@ -455,13 +443,13 @@ function genererFacture($id_cotisation, $chemin = null) return $cotisation['numero_facture']; } - private function buildDetailsPersonneMorale(PDF_Facture $pdf, $montant, $dateFin) + private function buildDetailsPersonneMorale(PDF_Facture $pdf, $montant, $dateFin): array { $montantTtc = $montant * (1 + Utils::MEMBERSHIP_FEE_VAT_RATE); $pdf->Ln(); $pdf->SetFillColor(255, 255, 255); $pdf->Cell(20, 5, 'ADH', 1); - $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', $dateFin)), 1); + $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', (int) $dateFin)), 1); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montant) . ' €'), 1, 0, 'R'); $pdf->Cell(25, 5, utf8_decode((Utils::MEMBERSHIP_FEE_VAT_RATE * 100) . ' %'), 1, 0, 'R'); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montantTtc) . ' €'), 1, 0, 'R'); @@ -471,7 +459,7 @@ private function buildDetailsPersonneMorale(PDF_Facture $pdf, $montant, $dateFin return [$totalHt, $total]; } - private function buildDetailsPersonnePhysique(PDF_Facture $pdf, $montant, $dateFin) + private function buildDetailsPersonnePhysique(PDF_Facture $pdf, $montant, $dateFin): array { $montantFixeHt = 5 / 100 * $montant; $montantFixeTTc = $montantFixeHt * (1 + Utils::MEMBERSHIP_FEE_VAT_RATE); @@ -480,7 +468,7 @@ private function buildDetailsPersonnePhysique(PDF_Facture $pdf, $montant, $dateF $pdf->Ln(); $pdf->SetFillColor(255, 255, 255); $pdf->Cell(20, 5, 'ADH-var', 1); - $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', $dateFin) . ' - part variable'), 1); + $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', (int) $dateFin) . ' - part variable'), 1); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montantFixeHt) . ' €'), 1, 0, 'R'); $pdf->Cell(25, 5, utf8_decode((Utils::MEMBERSHIP_FEE_VAT_RATE * 100) . ' %'), 1, 0, 'R'); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montantFixeTTc) . ' €'), 1, 0, 'R'); @@ -488,9 +476,9 @@ private function buildDetailsPersonnePhysique(PDF_Facture $pdf, $montant, $dateF $pdf->Ln(); $pdf->SetFillColor(255, 255, 255); $pdf->Cell(20, 5, 'ADH-fixe', 1); - $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', $dateFin) . ' - part fixe'), 1); + $pdf->Cell(95, 5, utf8_decode("Adhésion AFUP jusqu'au " . date('d/m/Y', (int) $dateFin) . ' - part fixe'), 1); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montantVariable) . ' €'), 1, 0, 'R'); - $pdf->Cell(25, 5, utf8_decode('0' . ' %'), 1, 0, 'R'); + $pdf->Cell(25, 5, utf8_decode('0 %'), 1, 0, 'R'); $pdf->Cell(25, 5, utf8_decode($this->formatFactureValue($montantVariable) . ' €'), 1, 0, 'R'); $totalHt = $montantFixeHt + $montantVariable; @@ -499,7 +487,7 @@ private function buildDetailsPersonnePhysique(PDF_Facture $pdf, $montant, $dateF return [$totalHt, $total]; } - private function formatFactureValue($value) + private function formatFactureValue($value): string { return number_format($value, 2, ',', ' '); } @@ -508,13 +496,10 @@ private function formatFactureValue($value) * Envoi par mail d'une facture au format PDF * * @param int $id_cotisation Identifiant de la cotisation - * @access public * @return bool Succès de l'envoi */ public function envoyerFacture($id_cotisation, Mailer $mailer, UserRepository $userRepository) { - $configuration = $GLOBALS['AFUP_CONF']; - $personne = $this->obtenir($id_cotisation, 'type_personne, id_personne'); if ($personne['type_personne'] == AFUP_PERSONNES_MORALES) { @@ -563,12 +548,11 @@ public function envoyerFacture($id_cotisation, Mailer $mailer, UserRepository $u /** * Retourne la dernière cotisation d'une personne morale - * - * @param int $id_personne Identifiant de la personne - * @access public - * @return array|false + * @param int|string $type_personne + * @param int $id_personne Identifiant de la personne + * @return array|false */ - function obtenirDerniere($type_personne, $id_personne) + public function obtenirDerniere($type_personne, $id_personne) { $requete = 'SELECT'; $requete .= ' * '; @@ -591,10 +575,9 @@ function obtenirDerniere($type_personne, $id_personne) * * @param int $type_personne Identifiant du type de personne * @param int $id_personne Identifiant de la personne - * @access public * @return int Timestamp de la date de la cotisation */ - function obtenirDateDebut($type_personne, $id_personne) + public function obtenirDateDebut($type_personne, $id_personne) { $requete = 'SELECT'; $requete .= ' date_fin '; @@ -616,16 +599,12 @@ function obtenirDateDebut($type_personne, $id_personne) } /** - * @param array $cotisation from Afup_Personnes_Physiques::obtenirDerniereCotisation + * @param array|false $cotisation from Afup_Personnes_Physiques::obtenirDerniereCotisation * @return DateTime Date of end of next subscription */ - public function finProchaineCotisation($cotisation) + public function finProchaineCotisation($cotisation = false): DateTime { - if ($cotisation === false) { - $endSubscription = new DateTime(); - } else { - $endSubscription = DateTime::createFromFormat('U', $cotisation['date_fin']); - } + $endSubscription = $cotisation === false ? new DateTime() : DateTime::createFromFormat('U', $cotisation['date_fin']); $base = $now = new DateTime(); $year = new DateInterval('P1Y'); @@ -645,7 +624,7 @@ public function finProchaineCotisation($cotisation) * @param string|null $token Token de la facture. Si null, pas de vérification * @return array */ - public function getByInvoice($invoiceId, $token = null) + public function getByInvoice(string $invoiceId, string $token = null) { $requete = 'SELECT'; $requete .= ' * '; @@ -668,7 +647,7 @@ public function getByInvoice($invoiceId, $token = null) * @param string $informations_reglement Informations concernant le règlement (numéro de chèque, de virement etc.) * @return bool Succès de la modification */ - function updatePayment($id, $type_reglement, $informations_reglement) + public function updatePayment($id, $type_reglement, string $informations_reglement): bool { $requete = 'UPDATE'; $requete .= ' afup_cotisations '; @@ -677,13 +656,10 @@ function updatePayment($id, $type_reglement, $informations_reglement) $requete .= ' informations_reglement=' . $this->_bdd->echapper($informations_reglement); $requete .= ' WHERE'; $requete .= ' id=' . $id; - if ($this->_bdd->executer($requete) === false) { - return false; - } - return true; + return $this->_bdd->executer($requete) !== false; } - public function isCurrentUserAllowedToReadInvoice ($invoiceId) + public function isCurrentUserAllowedToReadInvoice(string $invoiceId) { if (!$this->_droits) { throw new \RuntimeException('La variable $_droits ne doit pas être null.'); diff --git a/sources/Afup/Association/Personnes_Morales.php b/sources/Afup/Association/Personnes_Morales.php index 2798a07f8..24b33c656 100644 --- a/sources/Afup/Association/Personnes_Morales.php +++ b/sources/Afup/Association/Personnes_Morales.php @@ -1,5 +1,7 @@ _bdd = $bdd; } @@ -35,16 +35,14 @@ function __construct(&$bdd) * @param string $champs Champs à renvoyer * @param string $ordre Tri des enregistrements * @param bool $associatif Renvoyer un tableau associatif ? - * @access public * @return array */ - function obtenirListe($champs = '*', - $ordre = 'raison_sociale', + public function obtenirListe(string $champs = '*', + string $ordre = 'raison_sociale', $associatif = false, $filtre = false, $isActive = null - ) - { + ) { $requete = 'SELECT'; $requete .= ' ' . $champs . ' '; $requete .= 'FROM'; @@ -77,22 +75,21 @@ function obtenirListe($champs = '*', * Obtenir le nombre d'actifs d'une personne morale * * @param integer $id ID de la personne morale - * @access public * @return int le nombre d'actifs */ - function obtenirActifs($id) + public function obtenirActifs($id): int { $requete = "SELECT count(*) FROM afup_personnes_morales m, afup_personnes_physiques p "; $requete .= "where m.id = $id and m.id = p.id_personne_morale and p.etat = 1"; $actifs = $this->_bdd->obtenirTous($requete); - return (int)current($actifs[0]); + return (int) current($actifs[0]); } public function obtenirMembresMaximum($id, $default = 3) { $requete = "SELECT max_members FROM afup_personnes_morales"; - $requete .= " where afup_personnes_morales.id = " . (int)$id; + $requete .= " where afup_personnes_morales.id = " . (int) $id; $row = $this->_bdd->obtenirTous($requete); if (!isset($row[0])) { @@ -111,10 +108,9 @@ public function obtenirMembresMaximum($id, $default = 3) * * @param int $id Identifiant de la personne * @param string $champs Champs à renvoyer - * @access public * @return array|false */ - function obtenir($id, $champs = '*') + public function obtenir($id, string $champs = '*') { $requete = 'SELECT'; $requete .= ' ' . $champs . ' '; @@ -133,10 +129,9 @@ function obtenir($id, $champs = '*') * @param string $ville Ville de la personne morale * @param int $id_pays Identifiant du pays de la personne morale * @param int $etat Etat de la personne morale - * @access public * @return bool Succès de l'ajout */ - function ajouter($civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adresse, $code_postal, $ville, $id_pays, $telephone_fixe, $telephone_portable, $etat, $max_members) + public function ajouter(string $civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adresse, $code_postal, $ville, $id_pays, $telephone_fixe, $telephone_portable, $etat, $max_members) { $requete = 'INSERT INTO afup_personnes_morales (civilite, nom, prenom, email, raison_sociale, siret, adresse, code_postal, ville, id_pays, telephone_fixe, telephone_portable, etat, max_members) @@ -155,7 +150,6 @@ function ajouter($civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adr $this->_bdd->echapper($etat) . ', ' . $this->_bdd->echapper($max_members) . ')'; return $this->_bdd->executer($requete); - } @@ -169,10 +163,9 @@ function ajouter($civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adr * @param string $ville Ville de la personne morale * @param int $id_pays Identifiant du pays de la personne morale * @param int $etat Etat de la personne morale - * @access public * @return bool Succès de l'ajout */ - function modifier($id, $civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adresse, $code_postal, $ville, $id_pays, $telephone_fixe, $telephone_portable, $etat, $max_members = null) + public function modifier($id, string $civilite, $nom, $prenom, $email, $raison_sociale, $siret, $adresse, $code_postal, $ville, $id_pays, $telephone_fixe, $telephone_portable, $etat, $max_members = null) { $requete = 'UPDATE afup_personnes_morales @@ -193,7 +186,7 @@ function modifier($id, $civilite, $nom, $prenom, $email, $raison_sociale, $siret if ($max_members !== null) { $requete .= ', max_members = ' . $this->_bdd->echapper($max_members); } - $requete .= 'WHERE id=' . (int)$id; + $requete .= 'WHERE id=' . (int) $id; return $this->_bdd->executer($requete); } @@ -201,10 +194,9 @@ function modifier($id, $civilite, $nom, $prenom, $email, $raison_sociale, $siret * Supprime une personne morale * * @param int $id Identifiant de la personne morale à supprimer - * @access public * @return bool Succès de la suppression */ - function supprimer($id, UserRepository $userRepository) + public function supprimer($id, UserRepository $userRepository) { $cotisation = new Cotisations($this->_bdd); $cotisation_personne_morale = $cotisation->obtenirListe(AFUP_PERSONNES_MORALES, $id, 'id'); @@ -221,10 +213,9 @@ function supprimer($id, UserRepository $userRepository) * Retourne le nombre de personnes morales. * * @param int $etat Etat des personnes à retourner - * @access public * @return int */ - function obtenirNombrePersonnesMorales($etat = NULL) + public function obtenirNombrePersonnesMorales($etat = null) { $requete = 'SELECT'; $requete .= ' COUNT(*) '; @@ -246,5 +237,3 @@ public function getMembershipFee($id) return ceil($membersCount / AFUP_PERSONNE_MORALE_SEUIL) * AFUP_COTISATION_PERSONNE_MORALE; } } - -?> diff --git a/sources/Afup/AuthentificationInterface.php b/sources/Afup/AuthentificationInterface.php index 24aa000bd..42be949b4 100644 --- a/sources/Afup/AuthentificationInterface.php +++ b/sources/Afup/AuthentificationInterface.php @@ -1,24 +1,21 @@ \ No newline at end of file diff --git a/sources/Afup/AuthentificationWiki.php b/sources/Afup/AuthentificationWiki.php index 0c587c4ac..1293f8940 100644 --- a/sources/Afup/AuthentificationWiki.php +++ b/sources/Afup/AuthentificationWiki.php @@ -1,4 +1,6 @@ \ No newline at end of file diff --git a/sources/Afup/Bootstrap/Cli.php b/sources/Afup/Bootstrap/Cli.php index c22a12b34..fbeaf2b98 100644 --- a/sources/Afup/Bootstrap/Cli.php +++ b/sources/Afup/Bootstrap/Cli.php @@ -1,18 +1,20 @@ * @author Olivier Hoareau * @copyright 2010 Association Française des Utilisateurs de PHP - * + * * @category AFUP * @package AFUP * @group Bootstraps @@ -25,4 +27,4 @@ require_once __DIR__ . '/_Common.php'; require_once __DIR__ . '/../../../vendor/autoload.php'; -Logs::initialiser($bdd, 0); \ No newline at end of file +Logs::initialiser($bdd, 0); diff --git a/sources/Afup/Bootstrap/Http.php b/sources/Afup/Bootstrap/Http.php index 69e071a0e..dc3fdceec 100644 --- a/sources/Afup/Bootstrap/Http.php +++ b/sources/Afup/Bootstrap/Http.php @@ -1,18 +1,20 @@ * @author Olivier Hoareau * @copyright 2010 Association Française des Utilisateurs de PHP - * + * * @category AFUP * @package AFUP * @group Bootstraps @@ -32,13 +34,13 @@ // mise à jour des paramétrages PHP en fonction de la configuration if (getenv('SYMFONY_ENV') === 'prod') { - ini_set('error_reporting', E_ALL ^ E_WARNING ^ E_NOTICE); - ini_set('display_errors', 0); + ini_set('error_reporting', (string) (E_ALL ^ E_WARNING ^ E_NOTICE)); + ini_set('display_errors', '0'); } else { - ini_set('error_reporting', E_ALL); - ini_set('display_errors', 1); + ini_set('error_reporting', (string) E_ALL); + ini_set('display_errors', '1'); } -ini_set('include_path', ini_get('include_path') . PATH_SEPARATOR . __DIR__.'/../../../dependencies/PEAR/'); +ini_set('include_path', ini_get('include_path') . PATH_SEPARATOR . __DIR__ . '/../../../dependencies/PEAR/'); header('Content-type: text/html; charset=UTF-8'); @@ -47,8 +49,8 @@ $serveur = ''; $url = $_SERVER['REQUEST_URI']; if (strrpos($url, '?') !== false) { - $position = strrpos($url, '?'); - $url = substr($url, 0, $position); + $position = strrpos($url, '?'); + $url = substr($url, 0, $position); } $position = strrpos($url, '/'); $url = substr($_SERVER['REQUEST_URI'], 0, $position); @@ -69,8 +71,8 @@ $smarty->php_handling = SMARTY_PHP_ALLOW; $smarty->assign('url_base', 'http://' . $_SERVER['HTTP_HOST'] . '/'); -$smarty->assign('chemin_template', $serveur.Site::WEB_PATH.'templates/' . $sous_site . '/'); -$smarty->assign('chemin_javascript', $serveur.Site::WEB_PATH.'javascript/'); +$smarty->assign('chemin_template', $serveur . Site::WEB_PATH . 'templates/' . $sous_site . '/'); +$smarty->assign('chemin_javascript', $serveur . Site::WEB_PATH . 'javascript/'); $GLOBALS['AFUP_DB']->executer("SET NAMES 'utf8'"); require_once(__DIR__ . '/commonStart.php'); diff --git a/sources/Afup/Bootstrap/OtherToolPlugin.php b/sources/Afup/Bootstrap/OtherToolPlugin.php index f54a580dd..d685e47a7 100644 --- a/sources/Afup/Bootstrap/OtherToolPlugin.php +++ b/sources/Afup/Bootstrap/OtherToolPlugin.php @@ -1,18 +1,20 @@ * @author Olivier Hoareau * @copyright 2010 Association Française des Utilisateurs de PHP - * + * * @category AFUP * @package AFUP * @group Bootstraps @@ -20,4 +22,4 @@ // chargement des paramétrages génériques / multi-contextuels de l'application -require_once __DIR__ . '/_Common.php'; \ No newline at end of file +require_once __DIR__ . '/_Common.php'; diff --git a/sources/Afup/Bootstrap/_Common.php b/sources/Afup/Bootstrap/_Common.php index 4dfcb5364..8340ec851 100644 --- a/sources/Afup/Bootstrap/_Common.php +++ b/sources/Afup/Bootstrap/_Common.php @@ -1,4 +1,7 @@ */ -set_include_path( - $root . '/dependencies' - . PATH_SEPARATOR . $root . '/sources' -); +set_include_path($root . '/dependencies' . PATH_SEPARATOR . $root . '/sources'); // préparation de la requête / session - -require_once 'Afup/fonctions.php'; - - +require_once __DIR__ . '/../fonctions.php'; // chargement de la configuration $conf = new Configuration(); // mets la configuration dans une 'clé de registre' accessible à tout moment - $GLOBALS['AFUP_CONF'] = $conf; // initialisation de la couche d'abstraction de la base de données - $bdd = new _Site_Base_De_Donnees(); // mets la connexion db dans une 'clé de registre' accessible à tout moment - $GLOBALS['AFUP_DB'] = $bdd; diff --git a/sources/Afup/Bootstrap/commonStart.php b/sources/Afup/Bootstrap/commonStart.php index cd250c0e6..87ae2f53b 100644 --- a/sources/Afup/Bootstrap/commonStart.php +++ b/sources/Afup/Bootstrap/commonStart.php @@ -1,5 +1,12 @@ addLoader('xliff', new \Symfony\Component\Translation\Loader\XliffFileLoader()); +$translator = new Translator($lang); +$translator->addLoader('xliff', new XliffFileLoader()); $translator->addResource('xliff', __DIR__ . '/../../../translations/inscription.en.xlf', 'en'); $translator->addResource('xliff', __DIR__ . '/../../../translations/cfp.en.xlf', 'en'); $translator->setFallbackLocales(['fr']); @@ -174,7 +181,7 @@ // Initialisation de ting -$services = new \CCMBenchmark\Ting\Services(); +$services = new Services(); $services->get('ConnectionPool')->setConfig([ 'main' => [ 'namespace' => '\CCMBenchmark\Ting\Driver\Mysqli', @@ -195,4 +202,4 @@ ['default' => ['database' => $GLOBALS['AFUP_CONF']->obtenir('database_name')]] ) ; -$services->set('security.csrf.token_manager', fn() => new Symfony\Component\Security\Csrf\CsrfTokenManager()); +$services->set('security.csrf.token_manager', fn (): CsrfTokenManager => new CsrfTokenManager()); diff --git a/sources/Afup/Comptabilite/Comptabilite.php b/sources/Afup/Comptabilite/Comptabilite.php index 5068037ef..fcf50d6e5 100755 --- a/sources/Afup/Comptabilite/Comptabilite.php +++ b/sources/Afup/Comptabilite/Comptabilite.php @@ -1,4 +1,6 @@ _bdd = $bdd; } @@ -43,12 +41,10 @@ function __construct(&$bdd) * Paypal * */ - function obtenirJournalBanque($compte = 1, + public function obtenirJournalBanque($compte = 1, $periode_debut = '', $periode_fin = '' - ) - { - + ) { $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut); $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin); $requete = 'SELECT '; @@ -74,27 +70,32 @@ function obtenirJournalBanque($compte = 1, $requete .= 'AND compta.date_regl <= \'' . $periode_fin . '\' '; $requete .= 'AND compta.montant != \'0.00\' '; $requete .= 'AND compta.idmode_regl = compta_reglement.id '; - $requete .= 'AND idcompte = ' . (int)$compte . ' '; + $requete .= 'AND idcompte = ' . (int) $compte . ' '; $requete .= 'ORDER BY '; $requete .= 'compta.date_regl '; return $this->_bdd->obtenirTous($requete); } - function obtenirSousTotalJournalBanque($compte = 1, $periode_debut, $periode_fin) + public function obtenirSousTotalJournalBanque($periode_debut, $periode_fin, $compte = 1) { - $data = $this->obtenirJournalBanque($compte, $periode_debut, $periode_fin); for ($i = 1; $i <= 12; $i++) { - $credit[$i] = ''; - $debit[$i] = ''; - $nligne[$i] = ''; + $credit[$i] = 0; + $debit[$i] = 0; + $nligne[$i] = 0; } - foreach ($data as $id => $row) { - if ($row['idoperation'] == "1") $debit[$row['mois']] += $row['montant']; - if ($row['idoperation'] == "2") $credit[$row['mois']] += $row['montant']; - if ($row['idoperation'] == "1" || $row['idoperation'] == "2") $nligne[$row['mois']]++; + foreach ($data as $row) { + if ($row['idoperation'] == "1") { + $debit[$row['mois']] += $row['montant']; + } + if ($row['idoperation'] == "2") { + $credit[$row['mois']] += $row['montant']; + } + if ($row['idoperation'] == "1" || $row['idoperation'] == "2") { + $nligne[$row['mois']]++; + } } $dif_old = 0; @@ -112,9 +113,8 @@ function obtenirSousTotalJournalBanque($compte = 1, $periode_debut, $periode_fin return $tableau; } - function obtenirTotalJournalBanque($compte = 1, $periode_debut, $periode_fin) + public function obtenirTotalJournalBanque($periode_debut, $periode_fin, $compte = 1): array { - $data = $this->obtenirJournalBanque($compte, $periode_debut, $periode_fin); /* echo "
";
        print_r($data);
@@ -122,22 +122,26 @@ function obtenirTotalJournalBanque($compte = 1, $periode_debut, $periode_fin)
         $credit = 0;
         $debit = 0;
 
-        foreach ($data as $id => $row) {
-            if ($row['idoperation'] == "1") $debit += $row['montant'];
-            if ($row['idoperation'] == "2") $credit += $row['montant'];
+        foreach ($data as $row) {
+            if ($row['idoperation'] == "1") {
+                $debit += $row['montant'];
+            }
+            if ($row['idoperation'] == "2") {
+                $credit += $row['montant'];
+            }
         }
-//print_r($credit);
-//$dif_old=0;
-//for ($i=1;$i<=12;$i++)
-//{
-//	$dif=$dif_old+$credit[$i]-$debit[$i];
+        //print_r($credit);
+        //$dif_old=0;
+        //for ($i=1;$i<=12;$i++)
+        //{
+        //	$dif=$dif_old+$credit[$i]-$debit[$i];
         $tableau = [
             "debit" => $debit,
             "credit" => $credit,
             "dif" => $credit - $debit
         ];
-//	$dif_old=$dif;
-//}
+        //	$dif_old=$dif;
+        //}
 
         return $tableau;
         /*		$total=0;
@@ -150,27 +154,21 @@ function obtenirTotalJournalBanque($compte = 1, $periode_debut, $periode_fin)
 
                 return $total;
                 */
-
     }
 
     /* Journal des opération
      *
      */
 
-    function obtenirJournal($debitCredit = '',
+    public function obtenirJournal(string $debitCredit = '',
                             $periode_debut = '',
                             $periode_fin = '',
                             $onlyUnclasifedEntries = true
-    )
-    {
-
+    ) {
         $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut);
         $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin);
 
-        if ($debitCredit == 1 || $debitCredit == 2)
-            $filtre = 'AND compta.idoperation =\'' . $debitCredit . '\'  ';
-        else
-            $filtre = "";
+        $filtre = $debitCredit == 1 || $debitCredit == 2 ? 'AND compta.idoperation =\'' . $debitCredit . '\'  ' : "";
 
         $requete = 'SELECT ';
         $requete .= 'compta.date_ecriture, compta.description, compta.montant, compta.idoperation,compta.id as idtmp, ';
@@ -221,15 +219,12 @@ function obtenirJournal($debitCredit = '',
     }
 
     // mise en forme du montant
-    function formatMontantCompta($valeur)
+    public function formatMontantCompta($valeur): string
     {
-        $prix_ok = number_format($valeur, 2, ',', ' ');
-
-        return $prix_ok;
-
+        return number_format($valeur, 2, ',', ' ');
     }
 
-    function periodeDebutFin($debutFin = 'debut', $date = '')
+    public function periodeDebutFin($debutFin = 'debut', $date = '')
     {
         // echo "=>$debutFin*$date*
";
         if ($date != '') {
@@ -242,8 +237,8 @@ function periodeDebutFin($debutFin = 'debut', $date = '')
                         {
                              $r=obtenirPeriodeEnCours($id_periode);
                         } else {*/
-            return DATE("Y") . "-01-01";
-            //		}
+            return date("Y") . "-01-01";
+        //		}
         } else {
             /*	if ($id_periode !='')
                 {
@@ -251,12 +246,12 @@ function periodeDebutFin($debutFin = 'debut', $date = '')
                      print_r($r);
                      return $r;
                 } else {*/
-            return DATE("Y") . "-12-31";
+            return date("Y") . "-12-31";
             //}
         }
     }
 
-    function obtenirPeriodeEnCours($id_periode)
+    public function obtenirPeriodeEnCours($id_periode)
     {
         // Si la periode existe
         if ($id_periode != "") {
@@ -270,18 +265,16 @@ function obtenirPeriodeEnCours($id_periode)
 
         if ($result) {
             return $result[0]['id'];
-        } else                // ajout d'une nouvelle periode
-        {
+        } else {                // ajout d'une nouvelle periode
             $result = $this->ajouterListPeriode();
             return $result[0]['id'];
         }
     }
 
-    function ajouterListPeriode()
+    public function ajouterListPeriode()
     {
-
-        $date_debut = DATE("Y") . '-01-01';
-        $date_fin = DATE("Y") . '-12-31';
+        $date_debut = date("Y") . '-01-01';
+        $date_fin = date("Y") . '-12-31';
 
         $requete = 'INSERT INTO ';
         $requete .= 'compta_periode (';
@@ -294,17 +287,16 @@ function ajouterListPeriode()
 
         $this->_bdd->executer($requete);
         return $this->obtenirListPeriode($date_debut, $date_fin);
-
     }
 
-    function obtenirListPeriode($date_debut = '', $date_fin = '')
+    public function obtenirListPeriode(?string $date_debut = '', ?string $date_fin = '')
     {
         $requete = 'SELECT ';
         $requete .= 'id, date_debut,date_fin, verouiller ';
         $requete .= 'FROM  ';
         $requete .= 'compta_periode  ';
 
-        if ($date_debut != '' AND $date_fin != '') {
+        if ($date_debut != '' && $date_fin != '') {
             $requete .= 'WHERE ';
             $requete .= 'compta_periode.date_debut= \'' . $date_debut . '\'  ';
             $requete .= 'AND compta_periode.date_fin= \'' . $date_fin . '\'  ';
@@ -313,13 +305,15 @@ function obtenirListPeriode($date_debut = '', $date_fin = '')
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirListOperations($filtre = '', $where = '')
+    public function obtenirListOperations($filtre = '', ?string $where = '')
     {
         $requete = 'SELECT ';
         $requete .= 'id, operation ';
         $requete .= 'FROM  ';
         $requete .= 'compta_operation  ';
-        if ($where) $requete .= 'WHERE id=' . $where . ' ';
+        if ($where) {
+            $requete .= 'WHERE id=' . $where . ' ';
+        }
 
         $requete .= 'ORDER BY ';
         $requete .= 'operation ';
@@ -339,13 +333,15 @@ function obtenirListOperations($filtre = '', $where = '')
         }
     }
 
-    function obtenirListComptes($filtre = '', $where = '')
+    public function obtenirListComptes($filtre = '', ?string $where = '')
     {
         $requete = 'SELECT ';
         $requete .= 'id, nom_compte ';
         $requete .= 'FROM  ';
         $requete .= 'compta_compte  ';
-        if ($where) $requete .= 'WHERE id=' . $where . ' ';
+        if ($where) {
+            $requete .= 'WHERE id=' . $where . ' ';
+        }
 
         $requete .= 'ORDER BY ';
         $requete .= 'nom_compte ';
@@ -365,7 +361,7 @@ function obtenirListComptes($filtre = '', $where = '')
         }
     }
 
-    function obtenirListCategories($filtre = '', $where = '', $usedInAccountingJournal = false)
+    public function obtenirListCategories($filtre = '', ?string $where = '', $usedInAccountingJournal = false)
     {
         $requete = 'SELECT ';
         $requete .= 'id, idevenement, categorie ';
@@ -379,7 +375,7 @@ function obtenirListCategories($filtre = '', $where = '', $usedInAccountingJourn
             $wheres[] = 'hide_in_accounting_journal_at IS NULL';
         }
 
-        if (count($wheres)) {
+        if ($wheres !== []) {
             $requete .= sprintf('WHERE %s ',implode(' AND ', $wheres));
         }
 
@@ -399,7 +395,6 @@ function obtenirListCategories($filtre = '', $where = '', $usedInAccountingJourn
 
             return $result;
         }
-
     }
 
     public function obtenirListCategoriesJournal()
@@ -410,7 +405,7 @@ public function obtenirListCategoriesJournal()
         return $categories;
     }
 
-    function obtenirListEvenements($filtre = '', $where = '', $usedInAccountingJournal = false)
+    public function obtenirListEvenements($filtre = '', ?string $where = '', $usedInAccountingJournal = false)
     {
         $requete = 'SELECT ';
         $requete .= 'id, evenement ';
@@ -424,7 +419,7 @@ function obtenirListEvenements($filtre = '', $where = '', $usedInAccountingJourn
             $wheres[] = 'hide_in_accounting_journal_at IS NULL';
         }
 
-        if (count($wheres)) {
+        if ($wheres !== []) {
             $requete .= sprintf('WHERE %s ',implode(' AND ', $wheres));
         }
 
@@ -454,7 +449,7 @@ public function obtenirListEvenementsJournal()
         return $events;
     }
 
-    function obtenirListReglements($filtre = '', $where = '', $usedInAccountingJournal = false)
+    public function obtenirListReglements($filtre = '', ?string $where = '', $usedInAccountingJournal = false)
     {
         $requete = 'SELECT ';
         $requete .= 'id, reglement ';
@@ -468,7 +463,7 @@ function obtenirListReglements($filtre = '', $where = '', $usedInAccountingJourn
             $wheres[] = 'hide_in_accounting_journal_at IS NULL';
         }
 
-        if (count($wheres)) {
+        if ($wheres !== []) {
             $requete .= sprintf('WHERE %s ',implode(' AND ', $wheres));
         }
 
@@ -498,11 +493,10 @@ public function obtenirListReglementsJournal()
         return $reglements;
     }
 
-    function ajouter($idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs, $tva_intra, $montant, $description,
+    public function ajouter($idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs, $tva_intra, $montant, $description,
                      $numero, $idmode_regl, $date_regl, $obs_regl, $idevenement, $numero_operation = null,
                      $attachmentRequired = 0, $montantHtSoumisTva0 = null, $montantHtSoumisTva55 = null, $montantHtSoumisTva10 = null, $montantHtSoumisTva20 = null, $tvaZone = null)
     {
-
         $requete = 'INSERT INTO ';
         $requete .= 'compta (';
         $requete .= 'idoperation,idcategorie,date_ecriture,nom_frs,tva_intra,montant,description,';
@@ -525,11 +519,11 @@ function ajouter($idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs
         $requete .= $this->_bdd->echapper($numero_operation) . ',';
         $requete .= $this->_bdd->echapper($idcompte) . ',';
         $requete .= $this->_bdd->echapper($attachmentRequired) . ',';
-        $requete .= (!$montantHtSoumisTva0 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva0)) . ',';
-        $requete .= (!$montantHtSoumisTva55 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva55)) . ',';
-        $requete .= (!$montantHtSoumisTva10 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva10)) . ',';
-        $requete .= (!$montantHtSoumisTva20 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva20)) . ',';
-        $requete .= (!$tvaZone ? 'NULL' : $this->_bdd->echapper($tvaZone)) . '';
+        $requete .= ($montantHtSoumisTva0 ? $this->_bdd->echapper($montantHtSoumisTva0) : 'NULL') . ',';
+        $requete .= ($montantHtSoumisTva55 ? $this->_bdd->echapper($montantHtSoumisTva55) : 'NULL') . ',';
+        $requete .= ($montantHtSoumisTva10 ? $this->_bdd->echapper($montantHtSoumisTva10) : 'NULL') . ',';
+        $requete .= ($montantHtSoumisTva20 ? $this->_bdd->echapper($montantHtSoumisTva20) : 'NULL') . ',';
+        $requete .= ($tvaZone ? $this->_bdd->echapper($tvaZone) : 'NULL') . '';
         $requete .= ');';
 
         $resultat = $this->_bdd->executer($requete);
@@ -539,13 +533,11 @@ function ajouter($idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs
         return $resultat;
     }
 
-    function modifier($id, $idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs, $tva_intra, $montant, $description,
+    public function modifier(string $id, $idoperation, $idcompte, $idcategorie, $date_ecriture, $nom_frs, $tva_intra, $montant, $description,
                       $numero, $idmode_regl, $date_regl, $obs_regl, $idevenement, $comment, $numero_operation = null, $attachmentRequired = 0,
                       $montantHtSoumisTva0 = null, $montantHtSoumisTva55 = null, $montantHtSoumisTva10 = null, $montantHtSoumisTva20 = null,
                       $tvaZone = null
-    )
-    {
-
+    ) {
         $requete = 'UPDATE ';
         $requete .= 'compta ';
         $requete .= 'SET ';
@@ -561,12 +553,12 @@ function modifier($id, $idoperation, $idcompte, $idcategorie, $date_ecriture, $n
         $requete .= 'date_regl=' . $this->_bdd->echapper($date_regl) . ',';
         $requete .= 'obs_regl=' . $this->_bdd->echapper($obs_regl) . ',';
         $requete .= 'idcompte=' . $this->_bdd->echapper($idcompte) . ',';
-        $requete .= 'montant_ht_soumis_tva_0=' . (!$montantHtSoumisTva0 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva0)) . ',';
-        $requete .= 'montant_ht_soumis_tva_5_5=' . (!$montantHtSoumisTva55 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva55)) . ',';
-        $requete .= 'montant_ht_soumis_tva_10=' . (!$montantHtSoumisTva10 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva10)) . ',';
-        $requete .= 'montant_ht_soumis_tva_20=' . (!$montantHtSoumisTva20 ? 'NULL' : $this->_bdd->echapper($montantHtSoumisTva20)) . ',';
-        $requete .= 'tva_zone=' . (!$tvaZone ? 'NULL' : $this->_bdd->echapper($tvaZone)) . ',';
-        $requete .= 'comment=' . (!$comment ? 'NULL' : $this->_bdd->echapper($comment)) . ',';
+        $requete .= 'montant_ht_soumis_tva_0=' . ($montantHtSoumisTva0 ? $this->_bdd->echapper($montantHtSoumisTva0) : 'NULL') . ',';
+        $requete .= 'montant_ht_soumis_tva_5_5=' . ($montantHtSoumisTva55 ? $this->_bdd->echapper($montantHtSoumisTva55) : 'NULL') . ',';
+        $requete .= 'montant_ht_soumis_tva_10=' . ($montantHtSoumisTva10 ? $this->_bdd->echapper($montantHtSoumisTva10) : 'NULL') . ',';
+        $requete .= 'montant_ht_soumis_tva_20=' . ($montantHtSoumisTva20 ? $this->_bdd->echapper($montantHtSoumisTva20) : 'NULL') . ',';
+        $requete .= 'tva_zone=' . ($tvaZone ? $this->_bdd->echapper($tvaZone) : 'NULL') . ',';
+        $requete .= 'comment=' . ($comment ? $this->_bdd->echapper($comment) : 'NULL') . ',';
         if ($numero_operation) {
             $requete .= 'numero_operation=' . $this->_bdd->echapper($numero_operation) . ',';
         }
@@ -612,7 +604,7 @@ public function modifierColonne($id, $columnName, $columnValue)
         return $this->_bdd->executer($requete);
     }
 
-    function ajouterConfig($table, $champ, $valeur)
+    public function ajouterConfig(string $table, string $champ, $valeur)
     {
         $requete = 'INSERT INTO ';
         $requete .= '' . $table . ' (';
@@ -624,9 +616,8 @@ function ajouterConfig($table, $champ, $valeur)
         return $this->_bdd->executer($requete);
     }
 
-    function modifierConfig($table, $id, $champ, $valeur)
+    public function modifierConfig(string $table, string $id, string $champ, $valeur)
     {
-
         $requete = 'UPDATE ';
         $requete .= '' . $table . ' ';
         $requete .= 'SET ';
@@ -637,7 +628,7 @@ function modifierConfig($table, $id, $champ, $valeur)
         return $this->_bdd->executer($requete);
     }
 
-    function obtenir($id)
+    public function obtenir(string $id)
     {
         $requete = 'SELECT';
         $requete .= '  * ';
@@ -648,8 +639,12 @@ function obtenir($id)
         return $this->_bdd->obtenirEnregistrement($requete);
     }
 
-
-    function obtenirSyntheseEvenement($idoperation = '1', $idevenement)
+    /**
+     * @param string|int $idevenement
+     * @param string|int $idoperation
+     * @return array|false|mixed
+     */
+    public function obtenirSyntheseEvenement($idevenement, $idoperation = 1)
     {
         $requete = 'SELECT ';
         $requete .= 'compta.*, ';
@@ -666,11 +661,15 @@ function obtenirSyntheseEvenement($idoperation = '1', $idevenement)
         $requete .= 'compta.date_ecriture ';
 
         return $this->_bdd->obtenirTous($requete);
-
     }
 
 
-    function obtenirTotalSyntheseEvenement($idoperation = '1', $idevenement)
+    /**
+     * @param string|int $idevenement
+     * @param string|int $idoperation
+     * @return int|mixed
+     */
+    public function obtenirTotalSyntheseEvenement($idevenement, $idoperation = 1)
     {
         $requete = 'SELECT ';
         $requete .= 'compta.montant ';
@@ -683,13 +682,13 @@ function obtenirTotalSyntheseEvenement($idoperation = '1', $idevenement)
         $data = $this->_bdd->obtenirTous($requete);
 
         $total = 0;
-        foreach ($data as $id => $row) {
+        foreach ($data as $row) {
             $total += $row['montant'];
         }
         return $total;
     }
 
-    function obtenirBilan($idoperation = '1', $periode_debut = '', $periode_fin = '')
+    public function obtenirBilan($idoperation = 1, $periode_debut = '', $periode_fin = '')
     {
         $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut);
         $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin);
@@ -713,21 +712,26 @@ function obtenirBilan($idoperation = '1', $periode_debut = '', $periode_fin = ''
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirTotalBilan($idoperation = '1', $periode_debut, $periode_fin)
+    public function obtenirTotalBilan($periode_debut, $periode_fin, $idoperation = '1')
     {
-
         $data = $this->obtenirBilan($idoperation, $periode_debut, $periode_fin);
 
         $total = 0;
-        foreach ($data as $id => $row) {
-
+        foreach ($data as $row) {
             $total += $row['montant'];
         }
 
         return $total;
     }
 
-    function obtenirBilanDetails($idoperation, $periode_debut = '', $periode_fin = '', $idevenement)
+    /**
+     * @param string|int $idoperation
+     * @param string|int $idevenement
+     * @param string $periode_debut
+     * @param string $periode_fin
+     * @return array|false|mixed
+     */
+    public function obtenirBilanDetails($idoperation, $idevenement, $periode_debut = '', $periode_fin = '')
     {
         $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut);
         $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin);
@@ -752,19 +756,16 @@ function obtenirBilanDetails($idoperation, $periode_debut = '', $periode_fin = '
         //$requete .= ' compta_evenement.evenement ';
         $requete .= 'ORDER BY ';
         $requete .= ' compta.date_ecriture ';
-//echo $requete."
";
+        //echo $requete."
";
         return $this->_bdd->obtenirTous($requete);
-
     }
 
-    function obtenirSousTotalBilan($idoperation = '1', $periode_debut, $periode_fin, $idevenement)
+    public function obtenirSousTotalBilan($periode_debut, $periode_fin, string $idevenement, string $idoperation = '1')
     {
-
-        $data = $this->obtenirBilanDetails($idoperation, $periode_debut, $periode_fin, $idevenement);
+        $data = $this->obtenirBilanDetails($idoperation, $idevenement, $periode_debut, $periode_fin);
 
         $total = 0;
-        foreach ($data as $id => $row) {
-
+        foreach ($data as $row) {
             $total += $row['montant'];
         }
 
@@ -772,7 +773,7 @@ function obtenirSousTotalBilan($idoperation = '1', $periode_debut, $periode_fin,
     }
 
 
-    function obtenirBalance($idoperation = '', $periode_debut = '', $periode_fin = '')
+    public function obtenirBalance($idoperation = null, $periode_debut = '', $periode_fin = '')
     {
         $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut);
         $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin);
@@ -789,8 +790,9 @@ function obtenirBalance($idoperation = '', $periode_debut = '', $periode_fin = '
         $requete .= ' compta.idevenement = compta_evenement.id ';
         $requete .= ' AND compta.date_ecriture >= \'' . $periode_debut . '\' ';
         $requete .= ' AND compta.date_ecriture <= \'' . $periode_fin . '\'  ';
-        if ($idoperation != '')
+        if ($idoperation != '') {
             $requete .= ' AND compta.idoperation = \'' . $idoperation . '\' ';
+        }
 
         $requete .= 'GROUP BY ';
         $requete .= ' compta_evenement.evenement ';
@@ -800,21 +802,23 @@ function obtenirBalance($idoperation = '', $periode_debut = '', $periode_fin = '
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirTotalBalance($idoperation = '1', $periode_debut, $periode_fin)
+    public function obtenirTotalBalance($periode_debut, $periode_fin, $idoperation = '1')
     {
-
         $data = $this->obtenirBalance($idoperation, $periode_debut, $periode_fin);
 
         $total = 0;
-        foreach ($data as $id => $row) {
-            if ($idoperation == 1) $total += $row['debit'];
-            if ($idoperation == 2) $total += $row['credit'];
-
+        foreach ($data as $row) {
+            if ($idoperation == 1) {
+                $total += $row['debit'];
+            }
+            if ($idoperation == 2) {
+                $total += $row['credit'];
+            }
         }
         return $total;
     }
 
-    function obtenirBalanceDetails($evenement, $periode_debut = '', $periode_fin = '')
+    public function obtenirBalanceDetails(string $evenement, $periode_debut = '', $periode_fin = '')
     {
         $periode_debut = $this->periodeDebutFin($debutFin = 'debut', $periode_debut);
         $periode_fin = $this->periodeDebutFin($debutFin = 'fin', $periode_fin);
@@ -840,23 +844,30 @@ function obtenirBalanceDetails($evenement, $periode_debut = '', $periode_fin = '
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirSousTotalBalance($evenement, $periode_debut, $periode_fin)
+    /**
+     * @return array{idevenement: int<1, 30>, debit: mixed, credit: mixed, nligne: int<0, max>}[]
+     */
+    public function obtenirSousTotalBalance(string $evenement, $periode_debut, $periode_fin): array
     {
         $tableau = [];
-//	    	echo $evenement."*".$periode_debut."*".$periode_fin;
+        //	    	echo $evenement."*".$periode_debut."*".$periode_fin;
         $data = $this->obtenirBalanceDetails($evenement, $periode_debut, $periode_fin);
 
         for ($i = 1; $i <= 30; $i++) {
             $credit[$i] = '';
             $debit[$i] = '';
             $nligne[$i] = 0;
-
         }
-        foreach ($data as $id => $row) {
-            if ($row['idoperation'] == "1") $debit[$row['id']] += $row['montant'];
-            if ($row['idoperation'] == "2") $credit[$row['id']] += $row['montant'];
-            if ($row['idoperation'] == "1" || $row['idoperation'] == "2") $nligne[$row['id']]++;
-
+        foreach ($data as $row) {
+            if ($row['idoperation'] == "1") {
+                $debit[$row['id']] += $row['montant'];
+            }
+            if ($row['idoperation'] == "2") {
+                $credit[$row['id']] += $row['montant'];
+            }
+            if ($row['idoperation'] == "1" || $row['idoperation'] == "2") {
+                $nligne[$row['id']]++;
+            }
         }
 
 
@@ -873,13 +884,13 @@ function obtenirSousTotalBalance($evenement, $periode_debut, $periode_fin)
         return $tableau;
     }
 
-    function supprimerEcriture($id)
+    public function supprimerEcriture(string $id)
     {
         $requete = 'DELETE FROM compta WHERE id=' . $id;
         return $this->_bdd->executer($requete);
     }
 
-    function obtenirParNumeroOperation($numero_operation)
+    public function obtenirParNumeroOperation($numero_operation)
     {
         $requete = 'SELECT';
         $requete .= '  * ';
@@ -890,7 +901,7 @@ function obtenirParNumeroOperation($numero_operation)
         return $this->_bdd->obtenirEnregistrement($requete);
     }
 
-    function obtenirSuivantADeterminer($numero_operation)
+    public function obtenirSuivantADeterminer($numero_operation)
     {
         $requete = 'SELECT';
         $requete .= '  id ';
@@ -907,7 +918,7 @@ function obtenirSuivantADeterminer($numero_operation)
         return $this->_bdd->obtenirEnregistrement($requete);
     }
 
-    function obtenirTous()
+    public function obtenirTous()
     {
         $requete = 'SELECT';
         $requete .= '  * ';
@@ -917,7 +928,7 @@ function obtenirTous()
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirEvenementParIdForum($id)
+    public function obtenirEvenementParIdForum($id)
     {
         $requete = 'SELECT ';
         $requete .= '  compta_evenement.id ';
@@ -926,16 +937,11 @@ function obtenirEvenementParIdForum($id)
         $requete .= 'INNER JOIN ';
         $requete .= '  afup_forum on afup_forum.titre = compta_evenement.evenement ';
         $requete .= 'WHERE ';
-        $requete .= '  afup_forum.id = ' . (int)$id;
+        $requete .= '  afup_forum.id = ' . (int) $id;
         return $this->_bdd->obtenirUn($requete);
     }
 
-    /**
-     *
-     * @param Importer $importer
-     * @return bool
-     */
-    function extraireComptaDepuisCSVBanque(Importer $importer)
+    public function extraireComptaDepuisCSVBanque(Importer $importer): bool
     {
         if (!$importer->validate()) {
             return false;
@@ -1012,7 +1018,7 @@ function extraireComptaDepuisCSVBanque(Importer $importer)
      * @param $query string String to search
      * @return array Results sorted by type
      */
-    public function rechercher($query)
+    public function rechercher($query): array
     {
         $like = $this->_bdd->echapper("%$query%");
         $results = [];
@@ -1130,7 +1136,7 @@ public function rechercher($query)
         return $results;
     }
 
-    function obtenirListRegles($filtre = '', $where = '')
+    public function obtenirListRegles($filtre = '', $where = '')
     {
         $requete = 'SELECT ';
         $requete .= '`id`, `label`, `condition`, `is_credit`, `vat`, `category_id`, `event_id`, `mode_regl_id`, `attachment_required` ';
@@ -1141,7 +1147,7 @@ function obtenirListRegles($filtre = '', $where = '')
             $wheres[] = 'id=' . intval($where) . ' ';
         }
 
-        if (count($wheres)) {
+        if ($wheres !== []) {
             $requete .= sprintf('WHERE %s ',implode(' AND ', $wheres));
         }
 
@@ -1153,9 +1159,10 @@ function obtenirListRegles($filtre = '', $where = '')
         } elseif ($filtre) {
             return $this->_bdd->obtenirTous($requete);
         }
+        return null;
     }
 
-    function ajouterRegle($label, $condition, $is_credit, $tva, $category_id, $event_id, $mode_regl_id, $attachment_required)
+    public function ajouterRegle($label, $condition, $is_credit, $tva, $category_id, $event_id, $mode_regl_id, $attachment_required)
     {
         $requete = 'INSERT INTO ';
         $requete .= 'compta_regle (';
@@ -1174,9 +1181,8 @@ function ajouterRegle($label, $condition, $is_credit, $tva, $category_id, $event
         return $this->_bdd->executer($requete);
     }
 
-    function modifierRegle($id, $label, $condition, $is_credit, $tva, $category_id, $event_id, $mode_regl_id, $attachment_required)
+    public function modifierRegle($id, $label, $condition, $is_credit, $tva, $category_id, $event_id, $mode_regl_id, $attachment_required)
     {
-
         $requete = 'UPDATE ';
         $requete .= 'compta_regle ';
         $requete .= 'SET ';
@@ -1203,4 +1209,3 @@ public static function getTvaZoneLabel($tvaZoneCode, $defaultValue = null)
         return self::TVA_ZONES[$tvaZoneCode];
     }
 }
-
diff --git a/sources/Afup/Comptabilite/Facture.php b/sources/Afup/Comptabilite/Facture.php
index f3497b8a1..e6a1e47f7 100644
--- a/sources/Afup/Comptabilite/Facture.php
+++ b/sources/Afup/Comptabilite/Facture.php
@@ -1,7 +1,10 @@
 _bdd = $bdd;
     }
@@ -27,9 +30,8 @@ function __construct(&$bdd)
     /* Journal des opération
      *
      */
-    function obtenirDevis($idPeriode = null)
+    public function obtenirDevis($idPeriode = null)
     {
-
         $requete = 'SELECT ';
         $requete .= ' acf.*, sum(quantite * pu) prix ';
         $requete .= 'FROM ';
@@ -54,9 +56,8 @@ function obtenirDevis($idPeriode = null)
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirDevisDetails($id)
+    public function obtenirDevisDetails($id)
     {
-
         $requete = 'SELECT ';
         $requete .= 'afup_compta_facture.*, ';
         $requete .= 'afup_compta_facture_details.ref,afup_compta_facture_details.designation,afup_compta_facture_details.quantite,afup_compta_facture_details.pu ';
@@ -72,9 +73,8 @@ function obtenirDevisDetails($id)
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirFacture()
+    public function obtenirFacture()
     {
-
         $requete = 'SELECT ';
         $requete .= ' acf.*, sum(quantite * pu) prix ';
         $requete .= 'FROM ';
@@ -93,9 +93,8 @@ function obtenirFacture()
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenirFactureDetails($id)
+    public function obtenirFactureDetails($id)
     {
-
         $requete = 'SELECT ';
         $requete .= 'afup_compta_facture.*, ';
         $requete .= 'afup_compta_facture_details.ref,afup_compta_facture_details.designation,afup_compta_facture_details.quantite,afup_compta_facture_details.pu ';
@@ -111,7 +110,7 @@ function obtenirFactureDetails($id)
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function obtenir($id)
+    public function obtenir(string $id)
     {
         $requete = 'SELECT';
         $requete .= '  * ';
@@ -133,7 +132,7 @@ public function obtenirParNumeroFacture($numerofacture)
         return $this->_bdd->obtenirEnregistrement($requete);
     }
 
-    function obtenir_details($id)
+    public function obtenir_details(string $id)
     {
         $requete = 'SELECT';
         $requete .= '  * ';
@@ -144,11 +143,10 @@ function obtenir_details($id)
         return $this->_bdd->obtenirTous($requete);
     }
 
-    function ajouter($date_devis, $societe, $service, $adresse, $code_postal, $ville, $id_pays,
+    public function ajouter($date_devis, $societe, $service, $adresse, $code_postal, $ville, $id_pays,
                      $nom, $prenom, $tel, $email, $tva_intra, $observation, $ref_clt1, $ref_clt2, $ref_clt3,
                      $etat_paiement = 0, $date_paiement = null, $devise = 'EUR')
     {
-
         $requete = 'INSERT INTO ';
         $requete .= 'afup_compta_facture (';
         $requete .= 'date_devis,societe,service,adresse,code_postal,ville,id_pays,';
@@ -180,7 +178,7 @@ function ajouter($date_devis, $societe, $service, $adresse, $code_postal, $ville
         return $this->_bdd->executer($requete);
     }
 
-    function ajouter_details($ref, $designation, int $quantite, float $pu, int $tva = 0)
+    public function ajouter_details($ref, $designation, int $quantite, float $pu, int $tva = 0)
     {
         $requete = 'INSERT INTO ';
         $requete .= 'afup_compta_facture_details (';
@@ -198,11 +196,10 @@ function ajouter_details($ref, $designation, int $quantite, float $pu, int $tva
         return $this->_bdd->executer($requete);
     }
 
-    function modifier($id, $date_devis, $societe, $service, $adresse, $code_postal, $ville, $id_pays,
+    public function modifier($id, $date_devis, $societe, $service, $adresse, $code_postal, $ville, $id_pays,
                       $nom, $prenom, $tel, $email, $tva_intra, $observation, $ref_clt1, $ref_clt2, $ref_clt3,
                       $numero_devis, $numero_facture, $etat_paiement, $date_paiement, $devise)
     {
-
         $requete = 'UPDATE ';
         $requete .= 'afup_compta_facture ';
         $requete .= 'SET ';
@@ -237,7 +234,7 @@ function modifier($id, $date_devis, $societe, $service, $adresse, $code_postal,
         return $this->_bdd->executer($requete);
     }
 
-    function modifier_details($id, $ref, $designation, int $quantite, float $pu, int $tva = 0)
+    public function modifier_details(string $id, $ref, $designation, int $quantite, float $pu, int $tva = 0)
     {
         $requete = 'UPDATE ';
         $requete .= 'afup_compta_facture_details ';
@@ -253,7 +250,7 @@ function modifier_details($id, $ref, $designation, int $quantite, float $pu, int
         return $this->_bdd->executer($requete);
     }
 
-    function obtenirDernier()
+    public function obtenirDernier()
     {
         /**
          * @TODO ne supporte pas les enregistrements concurrents !
@@ -266,7 +263,7 @@ function obtenirDernier()
     }
 
 
-    function transfertDevis($numero_devis)
+    public function transfertDevis($numero_devis)
     {
         $numero_facture = $this->genererNumeroFacture();
 
@@ -281,26 +278,26 @@ function transfertDevis($numero_devis)
         return $this->_bdd->executer($requete);
     }
 
-    function genererNumeroFacture()
+    public function genererNumeroFacture(): string
     {
         $year = (int) date('Y');
 
         $sql = "SELECT MAX(CAST(SUBSTRING_INDEX(numero_facture, '-', -1) AS UNSIGNED)) + 1
             FROM afup_compta_facture
             WHERE LEFT(numero_facture, 4)=";
-        $index = $this->_bdd->obtenirUn($sql.$year);
+        $index = $this->_bdd->obtenirUn($sql . $year);
 
         // index null = changement d'année
         // on va chercher l'index de l'année dernière
         if (null === $index) {
-            $index = $this->_bdd->obtenirUn($sql.($year-1));
+            $index = $this->_bdd->obtenirUn($sql . ($year-1));
             $index = (int) (is_null($index) ? 1 : $index);
         }
 
         return "$year-$index";
     }
 
-    function genererNumeroDevis()
+    public function genererNumeroDevis(): string
     {
         $requete = 'SELECT';
         $requete .= "  MAX(CAST(SUBSTRING_INDEX(numero_devis, '-', -1) AS UNSIGNED)) + 1 ";
@@ -314,7 +311,7 @@ function genererNumeroDevis()
     }
 
 
-    function genererDevis($reference, $chemin = null)
+    public function genererDevis(string $reference, $chemin = null): void
     {
         $requete = 'SELECT * FROM afup_compta_facture WHERE numero_devis=' . $this->_bdd->echapper($reference);
         $coordonnees = $this->_bdd->obtenirEnregistrement($requete);
@@ -443,22 +440,22 @@ function genererDevis($reference, $chemin = null)
                         $vatAmounts[$detail['tva']] = 0;
                     }
                     $vatAmounts[$detail['tva']] += ($detail['tva'] / 100) * $montantTtc;
-                    $montantTtc = $montantTtc * (1 + ($detail['tva'] / 100));
+                    $montantTtc *= 1 + ($detail['tva'] / 100);
 
                     $x += 20;
                     $pdf->SetXY($x, $y);
 
-                    $pdf->MultiCell(30, 5, utf8_decode($this->formatFactureValue($detail['pu'], $isSubjectedToVat)) . $devise, 'T', 0, "R");
+                    $pdf->MultiCell(30, 5, $this->formatFactureValue($detail['pu'], $isSubjectedToVat) . $devise, 'T', 0, "R");
 
                     $x += 30;
                     $pdf->SetXY($x, $y);
-                    $pdf->MultiCell(30, 5, utf8_decode($this->formatFactureValue($montantTtc, $isSubjectedToVat)) . $devise, 'T', 0, "R");
+                    $pdf->MultiCell(30, 5, $this->formatFactureValue($montantTtc, $isSubjectedToVat) . $devise, 'T', 0, "R");
                 } else {
                     $pdf->Cell(30, 5, $detail['ref'], 1);
                     $pdf->Cell(80, 5, utf8_decode($detail['designation']), 1);
                     $pdf->Cell(20, 5, utf8_decode($detail['quantite']), 1, 0, "C");
                     $pdf->Cell(30, 5, utf8_decode($detail['pu']) . $devise, 1, 0, "R");
-                    $pdf->Cell(30, 5, utf8_decode($montantHt) . $devise, 1, 0, "R");
+                    $pdf->Cell(30, 5, $montantHt . $devise, 1, 0, "R");
                 }
 
                 $totalHt += $montantHt;
@@ -511,7 +508,7 @@ function genererDevis($reference, $chemin = null)
     }
 
 
-    function genererFacture($reference, $chemin = null)
+    public function genererFacture(string $reference, $chemin = null): void
     {
         $requete = 'SELECT * FROM afup_compta_facture WHERE numero_facture=' . $this->_bdd->echapper($reference);
         $coordonnees = $this->_bdd->obtenirEnregistrement($requete);
@@ -554,7 +551,7 @@ function genererFacture($reference, $chemin = null)
             utf8_decode($coordonnees['code_postal']) . "\n" .
             utf8_decode($coordonnees['ville']) . "\n" .
             utf8_decode($pays->obtenirNom($coordonnees['id_pays'])) .
-            ($coordonnees['tva_intra'] ? ("\n" . utf8_decode('N° TVA Intracommunautaire : ' . $coordonnees['tva_intra'])) : null)        );
+            ($coordonnees['tva_intra'] ? ("\n" . utf8_decode('N° TVA Intracommunautaire : ' . $coordonnees['tva_intra'])) : null));
 
         $pdf->Ln(10);
         $pdf->SetFont('Arial', 'BU', 10);
@@ -607,11 +604,7 @@ function genererFacture($reference, $chemin = null)
                 break;
         }
         $yInitial = $pdf->getY();
-        if ($isSubjectedToVat) {
-            $columns = [0, 30, 90, 110, 130, 160, 190];
-        } else {
-            $columns = [0, 30, 110, 130, 160, 190];
-        }
+        $columns = $isSubjectedToVat ? [0, 30, 90, 110, 130, 160, 190] : [0, 30, 110, 130, 160, 190];
 
         $vatAmounts = [];
 
@@ -644,17 +637,17 @@ function genererFacture($reference, $chemin = null)
                         $vatAmounts[$detail['tva']] = 0;
                     }
                     $vatAmounts[$detail['tva']] += ($detail['tva'] / 100) * $montantTtc;
-                    $montantTtc = $montantTtc * (1 + ($detail['tva'] / 100));
+                    $montantTtc *= 1 + ($detail['tva'] / 100);
                 }
 
                 $x += 20;
                 $pdf->SetXY($x, $y);
 
-                $pdf->MultiCell(30, 5, utf8_decode($this->formatFactureValue($detail['pu'], $isSubjectedToVat)) . $devise, 'T', 0, "R");
+                $pdf->MultiCell(30, 5, $this->formatFactureValue($detail['pu'], $isSubjectedToVat) . $devise, 'T', 0, "R");
 
                 $x += 30;
                 $pdf->SetXY($x, $y);
-                $pdf->MultiCell(30, 5, utf8_decode($this->formatFactureValue($montantTtc, $isSubjectedToVat)) . $devise, 'T', 0, "R");
+                $pdf->MultiCell(30, 5, $this->formatFactureValue($montantTtc, $isSubjectedToVat) . $devise, 'T', 0, "R");
 
                 $totalHt += $montantHt;
                 $totalTtc += $montantTtc;
@@ -708,25 +701,23 @@ function genererFacture($reference, $chemin = null)
         }
     }
 
-    private function formatFactureValue($value, $isSubjectedToVat)
+    private function formatFactureValue($value, bool $isSubjectedToVat)
     {
         if (!$isSubjectedToVat) {
             return $value;
         }
 
-        return number_format($value, 2, ',', ' ');
+        return number_format((float) $value, 2, ',', ' ');
     }
 
     /**
      * Envoi par mail d'une facture au format PDF
      *
      * @param   string $reference Reference de la facturation
-     * @access public
      * @return bool Succès de l'envoi
      */
-    function envoyerFacture($reference)
+    public function envoyerFacture(string $reference)
     {
-        $configuration = $GLOBALS['AFUP_CONF'];
         $personne = $this->obtenirParNumeroFacture($reference);
 
         $sujet = "Facture AFUP\n";
@@ -745,7 +736,7 @@ function envoyerFacture($reference)
         $message = new Message($sujet, new MailUser(MailUser::DEFAULT_SENDER_EMAIL, MailUser::DEFAULT_SENDER_NAME), new MailUser($personne['email'], $personne['nom']));
         $message->addAttachment(new Attachment(
             $chemin_facture,
-            'facture-'.$reference.'.pdf',
+            'facture-' . $reference . '.pdf',
             'base64',
             'application/pdf'
         ));
diff --git a/sources/Afup/Comptabilite/PDF.php b/sources/Afup/Comptabilite/PDF.php
index 9e7def660..cf399245a 100644
--- a/sources/Afup/Comptabilite/PDF.php
+++ b/sources/Afup/Comptabilite/PDF.php
@@ -1,29 +1,30 @@
 Ln(1);              //Saut de ligne
     }
 
-//Pied de page
-    function Footer()
+    //Pied de page
+    public function Footer(): void
     {
 //    $this->SetY(-15);               //Positionnement à 1,5 cm du bas
 //    $this->SetFont('Arial','I',8);  //Police Arial italique 8
 //    $this->Cell(0,10,'Page '.$this->PageNo().'/{nb}',0,0,'C');      //Numéro de page
-
     }
 
 
-    function tableau($position, $header, $data)
+    public function tableau($position, $header, $data): void
     {
-        if ($position == 1) $position = 0; else $position = 150;
+        $position = $position == 1 ? 0 : 150;
         $y = 30;
 
         //Couleurs, épaisseur du trait et police grasse
@@ -66,6 +67,4 @@ function tableau($position, $header, $data)
         }
         $this->Cell(array_sum($w), 0, '', 'T');
     }
-
-
 }
diff --git a/sources/Afup/Corporate/Accueil.php b/sources/Afup/Corporate/Accueil.php
index 0f9b7c476..c01f9cc36 100644
--- a/sources/Afup/Corporate/Accueil.php
+++ b/sources/Afup/Corporate/Accueil.php
@@ -1,6 +1,7 @@
 bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
     }
 
-    function charger()
+    public function charger(): void
     {
-
     }
 
-    function afficher() {
+    public function afficher(): string
+    {
         return $this->colonne_de_gauche();
     }
 
-    function colonne_de_gauche() {
+    public function colonne_de_gauche(): string
+    {
         $articles = new Articles($this->bdd);
         $derniers_articles = $articles->chargerDerniersAjouts(10);
 
@@ -36,10 +34,10 @@ function colonne_de_gauche() {
 
         foreach ($derniers_articles as $article) {
             $chapeau = $article->chapeau;
-            $colonne .= '';
-            $colonne .= '';
-            $colonne .= '
'.$article->titre.'
';
-            $colonne .= '
'.strip_tags($chapeau, '
').'
';
+            $colonne .= '';
+            $colonne .= '';
+            $colonne .= '
' . $article->titre . '
';
+            $colonne .= '
' . strip_tags($chapeau, '
') . '
';
             $colonne .= '';
         }
 
diff --git a/sources/Afup/Corporate/Article.php b/sources/Afup/Corporate/Article.php
index 8b007dc26..f5184b112 100644
--- a/sources/Afup/Corporate/Article.php
+++ b/sources/Afup/Corporate/Article.php
@@ -1,11 +1,14 @@
  'Cycles de conférences',
@@ -52,7 +55,7 @@ public static function getThemeLabel($code)
     /**
      * @param mixed $position
      */
-    public function setPosition($position)
+    public function setPosition($position): void
     {
         $this->position = $position;
     }
@@ -64,28 +67,21 @@ public function setPosition($position)
      */
     protected $conf;
     /**
-     * @var \Afup\Site\Utils\Base_De_Donnees
+     * @var Base_De_Donnees
      */
     protected $bdd;
 
-    function __construct($id = 0, $bdd = false, $conf = false)
+    public function __construct($id = 0, $bdd = false, $conf = false)
     {
         $this->id = $id;
 
-        if ($bdd) {
-            $this->bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
 
-        if ($conf) {
-            $this->conf = $conf;
-        } else {
-            $this->conf = $GLOBALS['AFUP_CONF'];
-        }
+        $this->conf = $conf ?: $GLOBALS['AFUP_CONF'];
     }
 
-    function afficher() {
+    public function afficher(): string
+    {
         return '
             
' .
         '' .
@@ -95,17 +91,17 @@ function afficher() {
         '
';
     }
 
-    function titre()
+    public function titre()
     {
         return $this->titre;
     }
 
-    public function getCode()
+    public function getCode(): string
     {
         return $this->id . '-' . $this->raccourci;
     }
 
-    function teaser()
+    public function teaser()
     {
         switch (true) {
             case !empty($this->chapeau):
@@ -117,7 +113,7 @@ function teaser()
         return $teaser;
     }
 
-    function corps()
+    public function corps()
     {
         if ($this->etat <= 0) {
             return false;
@@ -138,12 +134,15 @@ function corps()
         return $corps;
     }
 
-    function date()
+    public function date(): string
     {
         return date("d/m/y", $this->date);
     }
 
-    function positionable()
+    /**
+     * @return int[]
+     */
+    public function positionable(): array
     {
         $positions = [];
         for ($i = 9; $i >= -9; $i--) {
@@ -153,7 +152,7 @@ function positionable()
         return $positions;
     }
 
-    function exportable()
+    public function exportable(): array
     {
         return [
             'id' => $this->id,
@@ -165,20 +164,20 @@ function exportable()
             'contenu' => $this->contenu,
             'type_contenu' => $this->type_contenu,
             'position' => $this->position,
-            'date' => date('Y-m-d H:i:s', $this->date),
+            'date' => date('Y-m-d H:i:s', (int) $this->date),
             'theme' => $this->theme,
             'id_forum' => $this->id_forum,
             'etat' => $this->etat,
         ];
     }
 
-    function supprimer()
+    public function supprimer()
     {
         $requete = 'DELETE FROM afup_site_article WHERE id = ' . $this->bdd->echapper($this->id);
         return $this->bdd->executer($requete);
     }
 
-    function charger()
+    public function charger(): void
     {
         $requete = 'SELECT *
                     FROM afup_site_article
@@ -186,7 +185,7 @@ function charger()
         $this->remplir($this->bdd->obtenirEnregistrement($requete));
     }
 
-    function chargerDepuisRaccourci($raccourci)
+    public function chargerDepuisRaccourci($raccourci): void
     {
         $requete = 'SELECT *
                     FROM afup_site_article
@@ -194,7 +193,7 @@ function chargerDepuisRaccourci($raccourci)
         $this->remplir($this->bdd->obtenirEnregistrement($requete));
     }
 
-    function charger_dernier_depuis_rubrique()
+    public function charger_dernier_depuis_rubrique(): void
     {
         $requete = 'SELECT *
                     FROM afup_site_article
@@ -203,7 +202,7 @@ function charger_dernier_depuis_rubrique()
         $this->remplir($this->bdd->obtenirEnregistrement($requete));
     }
 
-    function remplir($article)
+    public function remplir(array $article): void
     {
         $this->id = $article['id'];
         $this->id_site_rubrique = $article['id_site_rubrique'];
@@ -221,7 +220,7 @@ function remplir($article)
         $this->route = $this->route();
     }
 
-    function modifier()
+    public function modifier()
     {
         $requete = 'UPDATE afup_site_article
         			SET
@@ -243,7 +242,7 @@ function modifier()
         return $this->bdd->executer($requete);
     }
 
-    function inserer()
+    public function inserer()
     {
         $requete = 'INSERT INTO afup_site_article
         			SET
@@ -272,7 +271,7 @@ function inserer()
         return $resultat;
     }
 
-    function route()
+    public function route(): string
     {
         $rubrique = new Rubrique($this->id_site_rubrique, $this->bdd, $this->conf);
         $rubrique->charger();
@@ -280,10 +279,10 @@ function route()
             $rubrique->raccourci = 'rubrique';
         }
 
-        return Site::WEB_PATH.Site::WEB_PREFIX.Site::WEB_QUERY_PREFIX . $rubrique->raccourci . '/' . $this->id . '/' . $this->raccourci;
+        return Site::WEB_PATH . Site::WEB_PREFIX . Site::WEB_QUERY_PREFIX . $rubrique->raccourci . '/' . $this->id . '/' . $this->raccourci;
     }
 
-    function fil_d_ariane()
+    public function fil_d_ariane(): string
     {
         $fil = '';
 
@@ -296,14 +295,14 @@ function fil_d_ariane()
         return $fil;
     }
 
-    function autres_articles()
+    public function autres_articles(): array
     {
         $articles = new Articles($this->bdd);
 
         return $articles->chargerArticlesDeRubrique($this->id_site_rubrique);
     }
 
-    public function isTypeContenuMarkdown()
+    public function isTypeContenuMarkdown(): bool
     {
         return self::TYPE_CONTENU_MARKDOWN == $this->type_contenu;
     }
diff --git a/sources/Afup/Corporate/Articles.php b/sources/Afup/Corporate/Articles.php
index 04981851e..007eab56c 100644
--- a/sources/Afup/Corporate/Articles.php
+++ b/sources/Afup/Corporate/Articles.php
@@ -1,6 +1,7 @@
 bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
     }
 
-    function obtenirListe($champs = '*',
-                          $ordre = 'titre',
+    public function obtenirListe(string $champs = '*',
+                          string $ordre = 'titre',
                           $filtre = false,
                           $associatif = false)
     {
@@ -31,7 +28,7 @@ function obtenirListe($champs = '*',
         $requete .= '  afup_site_rubrique on afup_site_article.id_site_rubrique = afup_site_rubrique.id ';
         $requete .= 'WHERE 1 = 1 ';
         if ($filtre) {
-            $escapedFiltre = $this->bdd->echapper('%' .  $filtre . '%');
+            $escapedFiltre = $this->bdd->echapper('%' . $filtre . '%');
             $requete .= sprintf(' AND (afup_site_article.titre LIKE %s OR afup_site_article.contenu LIKE %s) ', $escapedFiltre, $escapedFiltre);
         }
         $requete .= ' ORDER BY ' . $ordre;
@@ -43,18 +40,21 @@ function obtenirListe($champs = '*',
         }
     }
 
-    function chargerArticlesDeRubrique($id_site_rubrique, $rowcount = null)
+    /**
+     * @return Article[]
+     */
+    public function chargerArticlesDeRubrique($id_site_rubrique, $rowcount = null): array
     {
         $requete = ' SELECT';
         $requete .= '  * ';
         $requete .= ' FROM';
         $requete .= '  afup_site_article ';
         $requete .= ' WHERE ';
-        $requete .= '  id_site_rubrique = ' . (int)$id_site_rubrique;
+        $requete .= '  id_site_rubrique = ' . (int) $id_site_rubrique;
         $requete .= '  AND etat = 1';
         $requete .= ' ORDER BY date DESC';
         if (is_int($rowcount)) {
-            $requete .= ' LIMIT 0, ' . (int)$rowcount;
+            $requete .= ' LIMIT 0, ' . $rowcount;
         }
         $elements = $this->bdd->obtenirTous($requete);
 
@@ -68,10 +68,12 @@ function chargerArticlesDeRubrique($id_site_rubrique, $rowcount = null)
         }
 
         return $articles;
-
     }
 
-    function chargerDerniersAjouts($rowcount = 10)
+    /**
+     * @return Article[]
+     */
+    public function chargerDerniersAjouts($rowcount = 10): array
     {
         $requete = ' SELECT';
         $requete .= '  afup_site_article.* ';
@@ -86,7 +88,7 @@ function chargerDerniersAjouts($rowcount = 10)
         $requete .= ' AND afup_site_rubrique.id <> ' . Rubrique::ID_RUBRIQUE_ANTENNES . ' '; // On affiche pas les articles de la page antennes
         $requete .= ' AND afup_site_rubrique.id <> ' . Rubrique::ID_RUBRIQUE_NOS_ACTIONS . ' '; // On affiche pas les articles de la page actions
         $requete .= ' ORDER BY date DESC';
-        $requete .= ' LIMIT 0, ' . (int)$rowcount;
+        $requete .= ' LIMIT 0, ' . (int) $rowcount;
 
         $ajouts = [];
         $elements = $this->bdd->obtenirTous($requete);
@@ -104,7 +106,10 @@ function chargerDerniersAjouts($rowcount = 10)
         return $ajouts;
     }
 
-    function chargerDernieresQuestions()
+    /**
+     * @return Article[]
+     */
+    public function chargerDernieresQuestions(): array
     {
         $requete = ' SELECT';
         $requete .= '  * ';
diff --git a/sources/Afup/Corporate/Branche.php b/sources/Afup/Corporate/Branche.php
index 771a5ff4a..93cf0aaed 100644
--- a/sources/Afup/Corporate/Branche.php
+++ b/sources/Afup/Corporate/Branche.php
@@ -1,35 +1,25 @@
 bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
-        if ($conf) {
-            $this->conf = $conf;
-        } else {
-            $this->conf = $GLOBALS['AFUP_CONF'];
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
     }
 
-    function navigation_avec_image($bool = false)
+    public function navigation_avec_image($bool = false): void
     {
         if ($bool) {
             $this->navigation = 'image';
@@ -40,7 +30,7 @@ public function feuillesEnfants($id)
     {
         $requete = 'SELECT *
                     FROM afup_site_feuille
-                    WHERE id_parent = '.$this->bdd->echapper($id).'
+                    WHERE id_parent = ' . $this->bdd->echapper($id) . '
                     AND etat = 1
                     ORDER BY position';
         return $this->bdd->obtenirTous($requete);
@@ -61,7 +51,7 @@ public function getNom($id)
         return $enregistrement['nom'];
     }
 
-    function naviguer($id, $profondeur = 1, $identification = "")
+    public function naviguer($id, $profondeur = 1, string $identification = ""): string
     {
         $requete = 'SELECT *
                     FROM afup_site_feuille
@@ -70,7 +60,7 @@ function naviguer($id, $profondeur = 1, $identification = "")
         $racine = $this->bdd->obtenirEnregistrement($requete);
 
         $feuilles = $this->extraireFeuilles($id, $profondeur);
-        if ($feuilles) {
+        if ($feuilles !== '' && $feuilles !== '0') {
             $navigation = '
    ' . $feuilles . '
';
         } else {
             $navigation = '';
@@ -79,12 +69,13 @@ function naviguer($id, $profondeur = 1, $identification = "")
         return $navigation;
     }
 
-    function extraireFeuilles($id, $profondeur) {
+    public function extraireFeuilles($id, $profondeur): string
+    {
         $extraction = '';
 
         $requete = 'SELECT *
                     FROM afup_site_feuille
-                    WHERE id_parent = '.$this->bdd->echapper($id).'
+                    WHERE id_parent = ' . $this->bdd->echapper($id) . '
                     AND etat = 1
                     ORDER BY position';
         $feuilles = $this->bdd->obtenirTous($requete);
@@ -101,22 +92,21 @@ function extraireFeuilles($id, $profondeur) {
                         $route = $feuille['lien'];
                         break;
                     default:
-                        $route = Site::WEB_PATH.Site::WEB_PREFIX.Site::WEB_QUERY_PREFIX.$feuille['lien'];
+                        $route = Site::WEB_PATH . Site::WEB_PREFIX . Site::WEB_QUERY_PREFIX . $feuille['lien'];
                         break;
                 }
-                $extraction .= '';
+                $extraction .= '';
                 if ($this->navigation == 'image' && $feuille['image'] !== null) {
-                    $extraction .= ''.$feuille['alt'].'
';
+                    $extraction .= '' . $feuille['alt'] . '
';
                     $extraction .= $feuille['nom'] . '
';
                     $extraction .= $feuille['alt'];
                 } else {
-                    $extraction .= '' . $feuille['nom'] . '';
+                    $extraction .= '' . $feuille['nom'] . '';
                 }
                 $extraction .= '';
                 if ($profondeur > 0) {
                     $extraction .= $this->naviguer($feuille['id'], $profondeur - 1);
                 }
-
             }
         }
 
diff --git a/sources/Afup/Corporate/Feuille.php b/sources/Afup/Corporate/Feuille.php
index 0810beb01..b28c407b6 100644
--- a/sources/Afup/Corporate/Feuille.php
+++ b/sources/Afup/Corporate/Feuille.php
@@ -1,7 +1,11 @@
 id = $id;
-        if ($bdd) {
-            $this->bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
     }
 
-    function inserer()
+    public function inserer()
     {
         if ($this->id > 0) {
             $this->supprimer();
         }
         $requete = 'INSERT INTO afup_site_feuille
         			SET
-        			id_parent = ' . $this->bdd->echapper(!$this->id_parent ? null : $this->id_parent) . ',
+        			id_parent = ' . $this->bdd->echapper($this->id_parent ?: null) . ',
         			nom       = ' . $this->bdd->echapper($this->nom) . ',
         			lien      = ' . $this->bdd->echapper($this->lien) . ',
         			alt       = ' . $this->bdd->echapper($this->alt) . ',
@@ -62,11 +62,11 @@ function inserer()
         return $this->bdd->executer($requete);
     }
 
-    function modifier()
+    public function modifier()
     {
         $requete = 'UPDATE afup_site_feuille
         			SET
-        			id_parent = ' . $this->bdd->echapper(!$this->id_parent ? null : $this->id_parent) . ',
+        			id_parent = ' . $this->bdd->echapper($this->id_parent ?: null) . ',
         			nom       = ' . $this->bdd->echapper($this->nom) . ',
         			lien      = ' . $this->bdd->echapper($this->lien) . ',
         			alt       = ' . $this->bdd->echapper($this->alt) . ',
@@ -76,12 +76,12 @@ function modifier()
         			date      = ' . $this->bdd->echapper($this->date) . ',
         			patterns  = ' . $this->bdd->echapper($this->patterns) . ',
         			etat      = ' . $this->bdd->echapper($this->etat) . '
-        			WHERE id  = ' . (int)$this->id;
+        			WHERE id  = ' . (int) $this->id;
 
         return $this->bdd->executer($requete);
     }
 
-    function remplir($f)
+    public function remplir(array $f): void
     {
         $this->id = $f['id'];
         $this->id_parent = $f['id_parent'];
@@ -96,7 +96,7 @@ function remplir($f)
         $this->patterns = $f['patterns'];
     }
 
-    function exportable()
+    public function exportable(): array
     {
         return [
             'id' => $this->id,
@@ -107,13 +107,13 @@ function exportable()
             'image' => $this->image,
             'image_alt' => $this->image_alt,
             'position' => $this->position,
-            'date' => date('Y-m-d', $this->date),
+            'date' => date('Y-m-d', (int) $this->date),
             'etat' => $this->etat,
             'patterns' => $this->patterns,
         ];
     }
 
-    function charger()
+    public function charger(): void
     {
         $requete = 'SELECT *
                     FROM afup_site_feuille
@@ -122,13 +122,16 @@ function charger()
         $this->remplir($f);
     }
 
-    function supprimer()
+    public function supprimer()
     {
         $requete = 'DELETE FROM afup_site_feuille WHERE id = ' . $this->bdd->echapper($this->id);
         return $this->bdd->executer($requete);
     }
 
-    function positionable()
+    /**
+     * @return int[]
+     */
+    public function positionable(): array
     {
         $positions = [];
         for ($i = 9; $i >= -9; $i--) {
@@ -136,5 +139,4 @@ function positionable()
         }
         return $positions;
     }
-
 }
diff --git a/sources/Afup/Corporate/Feuilles.php b/sources/Afup/Corporate/Feuilles.php
index 8188ec86a..12d2ff780 100644
--- a/sources/Afup/Corporate/Feuilles.php
+++ b/sources/Afup/Corporate/Feuilles.php
@@ -1,25 +1,23 @@
 bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
     }
 
-    function obtenirListe($champs = '*', $ordre = 'titre', $associatif = false)
+    public function obtenirListe(string $champs = '*', string $ordre = 'titre', $associatif = false)
     {
         $requete = 'SELECT ' . $champs . ' ';
         $requete .= 'FROM';
@@ -30,4 +28,4 @@ function obtenirListe($champs = '*', $ordre = 'titre', $associatif = false)
         }
         return $this->bdd->obtenirTous($requete);
     }
-}
\ No newline at end of file
+}
diff --git a/sources/Afup/Corporate/Footer.php b/sources/Afup/Corporate/Footer.php
index 51c4abda0..ca12f2a0e 100644
--- a/sources/Afup/Corporate/Footer.php
+++ b/sources/Afup/Corporate/Footer.php
@@ -1,23 +1,18 @@
 conf = $GLOBALS['AFUP_CONF'];
-    }
-
-    function logos()
+    public function logos(): string
     {
         $branche = new Branche();
         $branche->navigation_avec_image(true);
         return '
  • ' . $branche->naviguer(9, 0, "LogoElement") . '
';
     }
 
-    function questions()
+    public function questions(): string
     {
         $articles = new Articles();
         $questions = $articles->chargerDernieresQuestions();
@@ -26,12 +21,11 @@ function questions()
         foreach ($questions as $question) {
             $contenu .= '
  • ' . $question->titre() . '
    • ';
         }
-        $contenu .= '';
 
-        return $contenu;
+        return $contenu . '';
     }
 
-    function articles()
+    public function articles(): string
     {
         $articles = new Articles();
         $ajouts = $articles->chargerDerniersAjouts();
@@ -39,13 +33,12 @@ function articles()
         foreach ($ajouts as $ajout) {
             $contenu .= '
  • ' . $ajout->titre() . '
    • ';
         }
-        $contenu .= '';
 
-        return $contenu;
+        return $contenu . '';
     }
 
-    function render()
+    public function render(): string
     {
         return "";
     }
-}
\ No newline at end of file
+}
diff --git a/sources/Afup/Corporate/Page.php b/sources/Afup/Corporate/Page.php
index ee712235f..669d3ba9e 100644
--- a/sources/Afup/Corporate/Page.php
+++ b/sources/Afup/Corporate/Page.php
@@ -1,4 +1,6 @@
 bdd = $bdd;
-        } else {
-            $this->bdd = new _Site_Base_De_Donnees();
-        }
+        $this->bdd = $bdd ?: new _Site_Base_De_Donnees();
         $this->conf = $GLOBALS['AFUP_CONF'];
     }
 
-    function definirRoute($route)
+    public function definirRoute($route): void
     {
         $this->route = $route;
         switch (true) {
@@ -59,13 +57,13 @@ function definirRoute($route)
         }
     }
 
-    function community()
+    public function community(): string
     {
         $branche = new Branche($this->bdd);
         return $branche->naviguer(5, 2);
     }
 
-    function header($url = null, UserInterface $user = null)
+    public function header($url = null, UserInterface $user = null): string
     {
         $branche = new Branche($this->bdd);
         $url = urldecode($url);
@@ -73,7 +71,7 @@ function header($url = null, UserInterface $user = null)
 
         $feuillesEnfants = $branche->feuillesEnfants(Feuille::ID_FEUILLE_HEADER);
 
-        if (null !== $user) {
+        if ($user instanceof UserInterface) {
             $feuillesEnfants[] = [
                 'id' => PHP_INT_MAX,
                 'id_parent' => Feuille::ID_FEUILLE_HEADER,
@@ -86,8 +84,6 @@ function header($url = null, UserInterface $user = null)
                 'image' => null,
                 'patterns' => "#/admin/company#",
             ];
-
-
         } else {
             $feuillesEnfants[] = [
                 'id' => PHP_INT_MAX - 1,
@@ -106,14 +102,16 @@ function header($url = null, UserInterface $user = null)
 
         foreach ($feuillesEnfants as $feuille) {
             $isCurrent = false;
-            foreach (explode(PHP_EOL, $feuille['patterns']) as $pattern) {
-                $pattern = trim($pattern);
-                if (strlen($pattern) === 0) {
-                    continue;
-                }
+            if ($feuille['patterns']) {
+                foreach (explode(PHP_EOL, $feuille['patterns']) as $pattern) {
+                    $pattern = trim($pattern);
+                    if ($pattern === '') {
+                        continue;
+                    }
 
-                if (preg_match($pattern, $url)) {
-                    $isCurrent = true;
+                    if (preg_match($pattern, $url)) {
+                        $isCurrent = true;
+                    }
                 }
             }
 
@@ -129,7 +127,6 @@ function header($url = null, UserInterface $user = null)
                             $isCurrent = true;
                         }
                     }
-
                 }
             }
 
@@ -142,21 +139,20 @@ function header($url = null, UserInterface $user = null)
             $str .= sprintf("
  • %s
    • ", $class, $feuille['lien'], $feuille['nom']);
         }
 
-        $str .= '
      ';
-
-        return $str;
+        return $str . '
        ';
     }
 
-    function content()
+    public function content()
     {
         return $this->content;
     }
 
-    function social() {
+    public function social(): string
+    {
         return
             '