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}Balance comptable
Balance comptable
Balance comptable
Journal de banque
Journal de banque
Journal de banque
Journal de banque
Bilan
{foreach from=$debit item=debits}Bilan
Bilan
Bilan
Bilan
Bilan
Liste devis
Factures
En attente {/if}Synthese des évènements
Synthese des évènements
Synthese des évènements
Synthese des évènements
Synthese des évènements
Factures d'évènement
Inscriptions
Conférences
Liste des personnes morales
Liste des articles
"; 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 . '
'; + $extraction .= '
'; $extraction .= $feuille['nom'] . '
'; $extraction .= $feuille['alt']; } else { - $extraction .= '' . $feuille['nom'] . ''; + $extraction .= '' . $feuille['nom'] . ''; } $extraction .= '
- ' . $branche->naviguer(9, 0, "LogoElement") . '
- ';
-
- return $str;
+ return $str . '
- ';
}
- function content()
+ public function content()
{
return $this->content;
}
- function social() {
+ public function social(): string
+ {
return
'