Comment: | Élimination des $_GET pour utiliser qg et qv() |
---|---|
Downloads: | Tarball | ZIP archive | SQL archive |
Timelines: | family | ancestors | descendants | both | dev |
Files: | files | file ages | folders |
SHA1: |
da2139e72ecab6053fd6edda86b6044d |
User & Date: | bohwaz on 2017-07-14 06:34:55 |
Other Links: | branch diff | manifest | tags |
2017-07-14
| ||
07:25 | Modernisation: suppression des derniers appels aux méthodes simple* check-in: 5c80b8c6f8 user: bohwaz tags: dev | |
06:34 | Élimination des $_GET pour utiliser qg et qv() check-in: da2139e72e user: bohwaz tags: dev | |
06:24 | Modernisation code: passage de Utils::get à qg() check-in: 9103d3b2c1 user: bohwaz tags: dev | |
Modified src/www/admin/compta/categories/index.php from [9b123882d0] to [0cf8dc5306].
1 2 3 4 5 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; | | < < < | < < | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('compta', Membres::DROIT_ADMIN); $cats = new Compta\Categories; if (null !== qg('depenses')) $type = Compta\Categories::DEPENSES; else $type = Compta\Categories::RECETTES; $tpl->assign('type', $type); $tpl->assign('liste', $cats->getList($type)); $tpl->display('admin/compta/categories/index.tpl'); |
Modified src/www/admin/compta/operations/cotisation.php from [ed97743915] to [92b0f6ecfd].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $journal = new Compta\Journal; | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $journal = new Compta\Journal; qv(['id' => 'required|numeric']); $id = (int) qg('id'); $m_cotisations = new Membres\Cotisations; $cotisations = new Cotisations; $mco = $m_cotisations->get($id); if (!$mco) |
︙ | ︙ |
Modified src/www/admin/compta/operations/index.php from [717885fe51] to [ef889a1c1a].
︙ | ︙ | |||
15 16 17 18 19 20 21 | throw new UserException("La catégorie demandée n'existe pas."); } $type = $cat['type']; } else { | | | | 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 | throw new UserException("La catégorie demandée n'existe pas."); } $type = $cat['type']; } else { if (null !== qg('autres')) $type = Compta\Categories::AUTRES; elseif (null !== qg('depenses')) $type = Compta\Categories::DEPENSES; else $type = Compta\Categories::RECETTES; } $journal = new Compta\Journal; |
︙ | ︙ |
Modified src/www/admin/compta/operations/membre.php from [a1ca776a55] to [da8d2a3389].
︙ | ︙ | |||
9 10 11 12 13 14 15 | $exercice = qg('exercice') ?: $exercices->getCurrentId(); if (!$exercice) { throw new UserException('Exercice inconnu.'); } | | | < < < | | 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 | $exercice = qg('exercice') ?: $exercices->getCurrentId(); if (!$exercice) { throw new UserException('Exercice inconnu.'); } qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Le membre demandé n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/compta/operations/saisir.php from [8d05da860f] to [f05e543934].
1 2 3 4 5 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; | | < < < | | | | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('compta', Membres::DROIT_ECRITURE); $journal = new Compta\Journal; $journal->checkExercice(); $cats = new Compta\Categories; $banques = new Compta\Comptes_Bancaires; if (null !== qg('depense')) $type = Compta\Categories::DEPENSES; elseif (null !== qg('virement')) $type = 'virement'; elseif (null !== qg('dette')) $type = 'dette'; elseif (null !== qg('avance')) $type = null; else $type = Compta\Categories::RECETTES; $error = false; if (!empty($_POST['save'])) |
︙ | ︙ |
Modified src/www/admin/config/membres.php from [a7dd2bb69f] to [8a6c35e7ca].
︙ | ︙ | |||
18 19 20 21 22 23 24 | // l'enregistrement des modifs ne marchera pas car les deux instances seront identiques. // Càd si on utilise directement l'instance de $config, elle sera modifiée directement // du coup quand on essaiera de comparer si ça a changé ça comparera deux fois la même chose // donc ça n'aura pas changé forcément. $champs = new Membres\Champs($config->get('champs_membres')); } | | | 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 | // l'enregistrement des modifs ne marchera pas car les deux instances seront identiques. // Càd si on utilise directement l'instance de $config, elle sera modifiée directement // du coup quand on essaiera de comparer si ça a changé ça comparera deux fois la même chose // donc ça n'aura pas changé forcément. $champs = new Membres\Champs($config->get('champs_membres')); } if (null !== qg('ok')) { $error = 'OK'; } if (!empty($_POST['save']) || !empty($_POST['add']) || !empty($_POST['review']) || !empty($_POST['reset'])) { if (!Utils::CSRF_check('config_membres')) |
︙ | ︙ | |||
119 120 121 122 123 124 125 | $error = $e->getMessage(); } } } } $tpl->assign('error', $error); | | | | 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 | $error = $e->getMessage(); } } } } $tpl->assign('error', $error); $tpl->assign('review', null !== qg('review') ? true : false); $types = $champs->getTypes(); $tpl->assign('champs', $champs->getAll()); $tpl->assign('types', $types); $tpl->assign('presets', Membres\Champs::listUnusedPresets($champs)); $tpl->assign('new', Utils::post('new')); $tpl->register_modifier('get_type', function ($type) use ($types) { return $types[$type]; }); $tpl->assign('csrf_name', Utils::CSRF_field_name('config_membres')); $tpl->assign('csrf_value', Utils::CSRF_create('config_membres')); $tpl->assign('title', 'Configuration — ' . (null !== qg('review') ? 'Confirmer les changements' : 'Fiche membres')); $tpl->display('admin/config/membres.tpl'); |
Modified src/www/admin/config/site.php from [112b75f1e2] to [3c72b37ce7].
︙ | ︙ | |||
72 73 74 75 76 77 78 | { $error = 'Une erreur est survenue, merci de renvoyer le formulaire.'; } else { if (Squelette::editSource(qg('edit'), Utils::post('content'))) { | | | 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 | { $error = 'Une erreur est survenue, merci de renvoyer le formulaire.'; } else { if (Squelette::editSource(qg('edit'), Utils::post('content'))) { $fullscreen = null !== qg('fullscreen') ? '#fullscreen' : ''; Utils::redirect('/admin/config/site.php?edit='.rawurlencode(qg('edit')).'&ok'.$fullscreen); } else { $error = "Impossible d'enregistrer le squelette."; } } |
︙ | ︙ |
Modified src/www/admin/membres/cat_modifier.php from [ac24a4dd6a] to [e550802be2].
1 2 3 4 5 6 7 8 9 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $cats = new Membres\Categories; | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $cats = new Membres\Categories; qv(['id' => 'required|numeric']); $id = (int) qg('id'); $cat = $cats->get($id); if (!$cat) { throw new UserException("Cette catégorie n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/cat_supprimer.php from [9693b6dec4] to [43e89df1f9].
1 2 3 4 5 6 7 8 9 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $cats = new Membres\Categories; | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $cats = new Membres\Categories; qv(['id' => 'required|numeric']); $id = (int) qg('id'); $cat = $cats->get($id); if (!$cat) { throw new UserException("Cette catégorie n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/cotisations.php from [54685c9fe6] to [32cdb67ed5].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/cotisations/ajout.php from [14398346e2] to [4b74270ede].
1 2 3 4 5 6 7 8 9 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); $membre = false; | | | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); $membre = false; if (($id = qg('id')) && is_numeric($id)) { $membre = $membres->get((int) $id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } $cats = new Membres\Categories; |
︙ | ︙ |
Modified src/www/admin/membres/cotisations/rappels.php from [bbdea97b0c] to [6b9cc31b63].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } |
︙ | ︙ | |||
47 48 49 50 51 52 53 | { $error = $e->getMessage(); } } } $tpl->assign('error', $error); | | | 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 | { $error = $e->getMessage(); } } } $tpl->assign('error', $error); $tpl->assign('ok', null !== qg('ok')); $tpl->assign('membre', $membre); $tpl->assign('cotisations', $cm->listSubscriptionsForMember($membre->id)); $tpl->assign('default_date', date('Y-m-d')); $tpl->assign('rappels', $re->listForMember($membre->id)); $tpl->assign('rappels_envoyes', $re); $tpl->display('admin/membres/cotisations/rappels.tpl'); |
Modified src/www/admin/membres/cotisations/supprimer.php from [5883297a40] to [01b29d1116].
1 2 3 4 5 6 7 8 9 10 11 12 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); $membre = false; $cotisations = new Cotisations; $m_cotisations = new Membres\Cotisations; | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); $membre = false; $cotisations = new Cotisations; $m_cotisations = new Membres\Cotisations; qv(['id' => 'required|numeric']); $id = (int) qg('id'); $co = $m_cotisations->get($id); if (!$co) { throw new UserException("Cette cotisation membre n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/cotisations/voir.php from [2fa458fb02] to [40bc6d371e].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); | | | < < < | | | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 | <?php namespace Garradin; require_once __DIR__ . '/../_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); qv(['id' => 'required|numeric']); $id = (int) qg('id'); $cotisations = new Cotisations; $m_cotisations = new Membres\Cotisations; $co = $cotisations->get($id); if (!$co) { throw new UserException("Cette cotisation n'existe pas."); } $page = (int) qg('p') ?: 1; $tpl->assign('page', $page); $tpl->assign('bypage', Membres\Cotisations::ITEMS_PER_PAGE); $tpl->assign('total', $m_cotisations->countMembersForCotisation($co->id)); $tpl->assign('pagination_url', Utils::getSelfUrl(true) . '?id=' . $co->id . '&p=[ID]'); $tpl->assign('cotisation', $co); $tpl->assign('order', qg('o') ?: 'date'); $tpl->assign('desc', !null !== qg('a')); $tpl->assign('liste', $m_cotisations->listMembersForCotisation( $co->id, $page, qg('o'), null !== qg('a') ? false : true)); $tpl->display('admin/membres/cotisations/voir.tpl'); |
Modified src/www/admin/membres/fiche.php from [1a6a56f8e5] to [8ea2bd6528].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/import.php from [af2f9a0078] to [058273a22b].
1 2 3 4 5 6 7 8 9 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $import = new Membres\Import; | | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ADMIN); $import = new Membres\Import; if (null !== qg('export')) { header('Content-type: application/csv'); header('Content-Disposition: attachment; filename="Export membres - ' . $config->get('nom_asso') . ' - ' . date('Y-m-d') . '.csv"'); $import->toCSV(); exit; } |
︙ | ︙ | |||
53 54 55 56 57 58 59 | { $error = $e->getMessage(); } } } $tpl->assign('error', $error); | | | 53 54 55 56 57 58 59 60 61 62 63 64 65 66 | { $error = $e->getMessage(); } } } $tpl->assign('error', $error); $tpl->assign('ok', null !== qg('ok') ? true : false); $tpl->assign('garradin_champs', $champs); $tpl->assign('galette_champs', $import->galette_fields); $tpl->assign('translate', Utils::post('galette_translate')); $tpl->display('admin/membres/import.tpl'); |
Modified src/www/admin/membres/index.php from [8d17730d0b] to [f61218542f].
︙ | ︙ | |||
39 40 41 42 43 44 45 | // Par défaut le champ de tri c'est l'identité $order = $config->get('champ_identite'); $desc = false; if (qg('o')) $order = qg('o'); | | | 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 | // Par défaut le champ de tri c'est l'identité $order = $config->get('champ_identite'); $desc = false; if (qg('o')) $order = qg('o'); if (null !== qg('d')) $desc = true; $fields = $champs->getListedFields(); // Vérifier que le champ de tri existe bien dans la table if ($order != 'id' && !isset($fields->$order)) { |
︙ | ︙ | |||
70 71 72 73 74 75 76 | $tpl->assign('current_cat', $cat_id); $tpl->assign('page', $page); $tpl->assign('bypage', Membres::ITEMS_PER_PAGE); } | | | 70 71 72 73 74 75 76 77 78 79 | $tpl->assign('current_cat', $cat_id); $tpl->assign('page', $page); $tpl->assign('bypage', Membres::ITEMS_PER_PAGE); } $tpl->assign('sent', null !== qg('sent')); $tpl->display('admin/membres/index.tpl'); |
Modified src/www/admin/membres/message.php from [0c98a0bb58] to [a5ef4b0e94].
1 2 3 4 5 6 7 8 9 10 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; if (empty($user->email)) { throw new UserException("Vous devez renseigner l'adresse e-mail dans vos informations pour pouvoir contacter les autres membres."); } | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; if (empty($user->email)) { throw new UserException("Vous devez renseigner l'adresse e-mail dans vos informations pour pouvoir contacter les autres membres."); } qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/membres/modifier.php from [c5f6388ed3] to [cc405b398e].
1 2 3 4 5 6 7 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); | | | < < < | | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php namespace Garradin; require_once __DIR__ . '/_inc.php'; $session->requireAccess('membres', Membres::DROIT_ECRITURE); qv(['id' => 'required|numeric']); $id = (int) qg('id'); $membre = $membres->get($id); if (!$membre) { throw new UserException("Ce membre n'existe pas."); } |
︙ | ︙ |
Modified src/www/admin/password.php from [7da509eab2] to [1b8d4026c7].
︙ | ︙ | |||
29 30 31 32 33 34 35 | Utils::redirect('/admin/password.php?sent'); } $error = 'MAIL'; } } | | | | 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 | Utils::redirect('/admin/password.php?sent'); } $error = 'MAIL'; } } if (!$error && null !== qg('sent')) { $tpl->assign('sent', true); } elseif (!$error && null !== qg('new_sent')) { $tpl->assign('new_sent', true); } $champs = $config->get('champs_membres'); |
︙ | ︙ |