X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fprofile.php;h=9416557cbaf3056c88aa6fa920503f027e75e8f1;hb=911075226546c6b058eca1e0b889097f79fa6232;hp=f055c30eaaaa238dbd885a2b516832b0cb273b73;hpb=e12ab28529594b7814568ca282974b620cde3eb1;p=platal.git diff --git a/modules/profile.php b/modules/profile.php index f055c30..9416557 100644 --- a/modules/profile.php +++ b/modules/profile.php @@ -1,6 +1,6 @@ $this->make_hook('fiche', AUTH_PUBLIC), 'profile' => $this->make_hook('profile', AUTH_PUBLIC), 'profile/private' => $this->make_hook('profile', AUTH_COOKIE), - 'profile/ax' => $this->make_hook('ax', AUTH_COOKIE, 'admin'), + 'profile/ax' => $this->make_hook('ax', AUTH_COOKIE, 'admin,edit_directory'), 'profile/edit' => $this->make_hook('p_edit', AUTH_MDP), 'profile/ajax/address' => $this->make_hook('ajax_address', AUTH_COOKIE, 'user', NO_AUTH), 'profile/ajax/tel' => $this->make_hook('ajax_tel', AUTH_COOKIE, 'user', NO_AUTH), @@ -56,30 +56,30 @@ class ProfileModule extends PLModule 'groupes-x/logo' => $this->make_hook('xnetlogo', AUTH_PUBLIC), 'vcard' => $this->make_hook('vcard', AUTH_COOKIE, 'user', NO_HTTPS), - 'admin/binets' => $this->make_hook('admin_binets', AUTH_MDP, 'admin'), - 'admin/medals' => $this->make_hook('admin_medals', AUTH_MDP, 'admin'), - 'admin/education' => $this->make_hook('admin_education', AUTH_MDP, 'admin'), - 'admin/education_field' => $this->make_hook('admin_education_field', AUTH_MDP, 'admin'), - 'admin/education_degree' => $this->make_hook('admin_education_degree', AUTH_MDP, 'admin'), - 'admin/education_degree_set' => $this->make_hook('admin_education_degree_set', AUTH_MDP, 'admin'), - 'admin/sections' => $this->make_hook('admin_sections', AUTH_MDP, 'admin'), - 'admin/networking' => $this->make_hook('admin_networking', AUTH_MDP, 'admin'), - 'admin/trombino' => $this->make_hook('admin_trombino', AUTH_MDP, 'admin'), - 'admin/corps_enum' => $this->make_hook('admin_corps_enum', AUTH_MDP, 'admin'), - 'admin/corps_rank' => $this->make_hook('admin_corps_rank', AUTH_MDP, 'admin'), - 'admin/names' => $this->make_hook('admin_names', AUTH_MDP, 'admin'), + 'admin/binets' => $this->make_hook('admin_binets', AUTH_MDP, 'admin'), + 'admin/medals' => $this->make_hook('admin_medals', AUTH_MDP, 'admin'), + 'admin/education' => $this->make_hook('admin_education', AUTH_MDP, 'admin'), + 'admin/education_field' => $this->make_hook('admin_education_field', AUTH_MDP, 'admin'), + 'admin/education_degree' => $this->make_hook('admin_education_degree', AUTH_MDP, 'admin'), + 'admin/education_degree_set' => $this->make_hook('admin_education_degree_set', AUTH_MDP, 'admin'), + 'admin/sections' => $this->make_hook('admin_sections', AUTH_MDP, 'admin'), + 'admin/networking' => $this->make_hook('admin_networking', AUTH_MDP, 'admin'), + 'admin/trombino' => $this->make_hook('admin_trombino', AUTH_MDP, 'admin'), + 'admin/corps_enum' => $this->make_hook('admin_corps_enum', AUTH_MDP, 'admin'), + 'admin/corps_rank' => $this->make_hook('admin_corps_rank', AUTH_MDP, 'admin'), + 'admin/names' => $this->make_hook('admin_names', AUTH_MDP, 'admin'), ); } /* Function needed for compatibility reasons. * TODO: removes calls to fiche.php?user=blah.machin.2083 and then removes this. */ - function handler_fiche(&$page) + function handler_fiche($page) { return $this->handler_profile($page, Env::v('user')); } - function handler_photo(&$page, $x = null, $req = null) + function handler_photo($page, $x = null, $req = null) { if (!$x || !($profile = Profile::get($x))) { return PL_NOT_FOUND; @@ -97,7 +97,7 @@ class ProfileModule extends PLModule $photo->send(); } - function handler_medal(&$page, $mid) + function handler_medal($page, $mid) { $thumb = ($mid == 'thumb'); $mid = $thumb ? @func_get_arg(2) : $mid; @@ -114,7 +114,7 @@ class ProfileModule extends PLModule exit; } - function handler_name_info(&$page) + function handler_name_info($page) { pl_content_headers("text/html"); $page->changeTpl('profile/name_info.tpl', SIMPLE); @@ -127,7 +127,7 @@ class ProfileModule extends PLModule $page->assign('types', $res); } - function handler_networking(&$page, $mid) + function handler_networking($page, $mid) { $res = XDB::query("SELECT icon FROM profile_networking_enum @@ -139,36 +139,39 @@ class ProfileModule extends PLModule exit; } - /** Tries to return the correct user from given hrpid - * Will redirect to $returnurl$hrpid if $hrpid was empty + /** Tries to return the correct profile from a given hrpid. */ - private function findProfile($returnurl, $hrpid = null) + private function findProfile($hrpid = null) { if (is_null($hrpid)) { $user = S::user(); if (!$user->hasProfile()) { return PL_NOT_FOUND; } else { - pl_redirect($returnurl . $user->profile()->hrid()); + $profile = $user->profile(); } } else { $profile = Profile::get($hrpid); - if (!$profile) { - return PL_NOT_FOUND; - } else if (!S::user()->canEdit($profile) && Platal::notAllowed()) { - return PL_FORBIDDEN; - } + } + + if (!$profile) { + return PL_NOT_FOUND; + } else if (!S::user()->canEdit($profile) && Platal::notAllowed()) { + return PL_FORBIDDEN; } return $profile; } - function handler_photo_change(&$page, $hrpid = null) + function handler_photo_change($page, $hrpid = null) { global $globals; - $profile = $this->findProfile('photo/change/', $hrpid); + $profile = $this->findProfile($hrpid); if (! ($profile instanceof Profile) && ($profile == PL_NOT_FOUND || $profile == PL_FORBIDDEN)) { return $profile; } + if (is_null($hrpid)) { + pl_redirect('photo/change/' . $profile->hrid()); + } $page->changeTpl('profile/trombino.tpl'); $page->assign('hrpid', $profile->hrid()); @@ -225,24 +228,13 @@ class ProfileModule extends PLModule $page->assign('has_trombi_x', file_exists($trombi_x)); } - function handler_profile(&$page, $id = null) + function handler_profile($page, $id = null) { // Checks if the identifier corresponds to an actual profile. Numeric // identifiers canonly be user by logged users. if (is_null($id)) { return PL_NOT_FOUND; } - $pid = (!is_numeric($id) || S::admin()) ? Profile::getPID($id) : null; - if (is_null($pid)) { - if (S::logged()) { - $page->trigError($id . " inconnu dans l'annuaire."); - } - return PL_NOT_FOUND; - } - - // Now that we know this is an existing profile, we can switch to the - // appropriate template. - $page->changeTpl('profile/profile.tpl', SIMPLE); // Determines the access level at which the profile will be displayed. if (!S::logged() || !S::user()->checkPerms('directory_ax') || Env::v('view') == 'public') { @@ -258,13 +250,30 @@ class ProfileModule extends PLModule $page->assign('with_pending_pic', true); } - // Fetches profile's and profile's owner information and redirects to - // marketing if the owner has not subscribed and the requirer has logged in. - $profile = Profile::get($pid, Profile::FETCH_ALL, $view); - $owner = $profile->owner(); - if (S::logged() && !is_null($owner) && $owner->state == 'pending') { - pl_redirect('marketing/public/' . $profile->hrid()); + $pid = (!is_numeric($id) || S::admin()) ? Profile::getPID($id) : null; + if (is_null($pid)) { + $owner = User::getSilent($id); + if ($owner) { + $profile = $owner->profile(true, Profile::FETCH_ALL, $view); + if ($profile) { + $pid = $profile->id(); + } + } + } else { + // Fetches profile's and profile's owner information and redirects to + // marketing if the owner has not subscribed and the requirer has logged in. + $profile = Profile::get($pid, Profile::FETCH_ALL, $view); + $owner = $profile->owner(); + } + if (is_null($pid)) { + if (S::logged()) { + $page->kill($id . " inconnu dans l'annuaire."); + } + return PL_NOT_FOUND; } + // Now that we know this is an existing profile, we can switch to the + // appropriate template. + $page->changeTpl('profile/profile.tpl', SIMPLE); // Profile view are logged. if (S::logged()) { @@ -284,11 +293,10 @@ class ProfileModule extends PLModule $page->assign('view', $view); $page->assign('logged', S::logged()); - $page->addJsLink('close_on_esc.js'); header('Last-Modified: ' . date('r', strtotime($profile->last_change))); } - function handler_ax(&$page, $user = null) + function handler_ax($page, $user = null) { $user = Profile::get($user); if (!$user) { @@ -300,21 +308,30 @@ class ProfileModule extends PLModule http_redirect("http://www.polytechniciens.com/?page=AX_FICHE_ANCIEN&ancc_id=" . $user->ax_id); } - function handler_p_edit(&$page, $hrpid = null, $opened_tab = null, $mode = null, $success = null) + function handler_p_edit($page, $hrpid = null, $opened_tab = null, $mode = null, $success = null) { global $globals; - $profile = $this->findProfile('profile/edit/', $hrpid); + if (in_array($hrpid, array('general', 'adresses', 'emploi', 'poly', 'deco', 'skill', 'mentor'))) { + $aux = $opened_tab; + $opened_tab = $hrpid; + $hrpid = $aux; + $url_error = true; + } else { + $url_error = false; + } + $profile = $this->findProfile($hrpid); if (! ($profile instanceof Profile) && ($profile == PL_NOT_FOUND || $profile == PL_FORBIDDEN)) { return $profile; } + if (is_null($hrpid) || $url_error) { + pl_redirect('profile/edit/' . $profile->hrid() . (is_null($opened_tab) ? '' : '/' . $opened_tab)); + } // Build the page - $page->addJsLink('ajax.js'); $page->addJsLink('education.js', false); /* dynamic content */ $page->addJsLink('grades.js', false); /* dynamic content */ $page->addJsLink('profile.js'); - $page->addJsLink('jquery.autocomplete.js'); $wiz = new PlWizard('Profil', PlPage::getCoreTpl('plwizard.tpl'), true, true, false); $wiz->addUserData('profile', $profile); $wiz->addUserData('owner', $profile->owner()); @@ -343,14 +360,14 @@ class ProfileModule extends PLModule } } - function handler_education_js(&$page) + function handler_education_js($page) { pl_cached_content_headers("text/javascript", "utf-8"); $page->changeTpl('profile/education.js.tpl', NO_SKIN); require_once 'education.func.inc.php'; } - function handler_grades_js(&$page) + function handler_grades_js($page) { pl_cached_content_headers("text/javascript", "utf-8"); $page->changeTpl('profile/grades.js.tpl', NO_SKIN); @@ -373,7 +390,7 @@ class ProfileModule extends PLModule $page->assign('medal_list', $mlist); } - function handler_ajax_address(&$page, $id) + function handler_ajax_address($page, $id) { pl_content_headers("text/html"); $page->changeTpl('profile/adresses.address.tpl', NO_SKIN); @@ -381,7 +398,7 @@ class ProfileModule extends PLModule $page->assign('address', array()); } - function handler_ajax_tel(&$page, $prefid, $prefname, $telid) + function handler_ajax_tel($page, $prefid, $prefname, $telid, $subField, $mainField, $mainId) { pl_content_headers("text/html"); $page->changeTpl('profile/phone.tpl', NO_SKIN); @@ -390,9 +407,14 @@ class ProfileModule extends PLModule $page->assign('telid', $telid); $phone = new Phone(); $page->assign('tel', $phone->toFormArray()); + if ($mainField) { + $page->assign('subField', $subField); + $page->assign('mainField', $mainField); + $page->assign('mainId', $mainId); + } } - function handler_ajax_edu(&$page, $eduid, $class) + function handler_ajax_edu($page, $eduid, $class) { pl_content_headers("text/html"); $page->changeTpl('profile/general.edu.tpl', NO_SKIN); @@ -405,15 +427,15 @@ class ProfileModule extends PLModule require_once "education.func.inc.php"; } - function handler_ajax_medal(&$page, $id) + function handler_ajax_medal($page, $i, $id) { pl_content_headers("text/html"); $page->changeTpl('profile/deco.medal.tpl', NO_SKIN); - $page->assign('id', $id); - $page->assign('medal', array('valid' => 0, 'grade' => 0)); + $page->assign('id', $i); + $page->assign('medal', array('id' => $id, 'grade' => 0, 'valid' => 0)); } - function handler_ajax_job(&$page, $id) + function handler_ajax_job($page, $id) { pl_content_headers("text/html"); $page->changeTpl('profile/jobs.job.tpl', NO_SKIN); @@ -421,7 +443,7 @@ class ProfileModule extends PLModule $page->assign('job', array()); $page->assign('new', true); require_once "emails.combobox.inc.php"; - fill_email_combobox($page); + fill_email_combobox($page, array('redirect', 'job', 'stripped_directory')); } /** @@ -436,12 +458,12 @@ class ProfileModule extends PLModule * @param Env::v('treeid') tree id that will be given as first argument of attrfunc function * the second argument will be the chosen job term id and the third one the chosen job full name. */ - function handler_ajax_tree_job_terms(&$page, $filter = JobTerms::ALL) + function handler_ajax_tree_job_terms($page, $filter = JobTerms::ALL) { - JobTerms::ajaxGetBranch(&$page, $filter); + JobTerms::ajaxGetBranch($page, $filter); } - function handler_ajax_skill(&$page, $cat, $id) + function handler_ajax_skill($page, $cat, $id) { pl_content_headers("text/html"); $page->changeTpl('profile/skill.skill.tpl', NO_SKIN); @@ -461,7 +483,7 @@ class ProfileModule extends PLModule } } - function handler_ajax_searchname(&$page, $id, $isFemale) + function handler_ajax_searchname($page, $id, $isFemale) { pl_content_headers("text/html"); $page->changeTpl('profile/general.searchname.tpl', NO_SKIN); @@ -474,7 +496,7 @@ class ProfileModule extends PLModule $page->assign('i', $id); } - function handler_ajax_buildnames(&$page, $data, $isFemale) + function handler_ajax_buildnames($page, $data, $isFemale) { pl_content_headers("text/html"); $page->changeTpl('profile/general.buildnames.tpl', NO_SKIN); @@ -482,7 +504,7 @@ class ProfileModule extends PLModule $page->assign('names', build_javascript_names($data, $isFemale)); } - function handler_referent(&$page, $pf) + function handler_referent($page, $pf) { $page->changeTpl('profile/fiche_referent.tpl', SIMPLE); @@ -495,25 +517,23 @@ class ProfileModule extends PLModule // Retrieves referents' countries. $res = XDB::query( - "SELECT gc.countryFR + "SELECT gc.country FROM profile_mentor_country AS m LEFT JOIN geoloc_countries AS gc ON (m.country = gc.iso_3166_1_a2) WHERE pid = {?}", $pf->id()); $page->assign('pays', $res->fetchColumn()); - - $page->addJsLink('close_on_esc.js'); } - function handler_ref_country(&$page) + function handler_ref_country($page) { pl_content_headers("text/html"); $page->changeTpl('include/field.select.tpl', NO_SKIN); $page->assign('name', 'pays_sel'); - $it = XDB::iterator("SELECT gc.iso_3166_1_a2 AS id, gc.countryFR AS field + $it = XDB::iterator("SELECT gc.iso_3166_1_a2 AS id, gc.country AS field FROM geoloc_countries AS gc INNER JOIN profile_mentor_country AS mp ON (mp.country = gc.iso_3166_1_a2) GROUP BY iso_3166_1_a2 - ORDER BY countryFR"); + ORDER BY country"); $page->assign('list', $it); } @@ -522,9 +542,9 @@ class ProfileModule extends PLModule * separated by "|" chars) for jobterms in referent (mentor) search. * @see handler_jobterms */ - function handler_ref_autocomplete(&$page) + function handler_ref_autocomplete($page) { - $this->handler_jobterms(&$page, 'mentor'); + $this->handler_jobterms($page, 'mentor'); } /** @@ -537,7 +557,7 @@ class ProfileModule extends PLModule * * @param Env::v('q') the text that has been typed and to complete automatically */ - function handler_jobterms(&$page, $type = 'nomentor') + function handler_jobterms($page, $type = 'nomentor') { pl_content_headers("text/plain"); @@ -589,14 +609,15 @@ class ProfileModule extends PLModule } $res .= "\n"; } - XDB::query('REPLACE INTO search_autocomplete - VALUES ({?}, {?}, {?}, NOW())', + XDB::query('INSERT INTO search_autocomplete (name, query, result, generated) + VALUES ({?}, {?}, {?}, NOW()) + ON DUPLICATE KEY UPDATE result = VALUES(result), generated = VALUES(generated)', $type, $q_normalized, $res); echo $res; exit(); } - function handler_xnet(&$page) + function handler_xnet($page) { $page->changeTpl('profile/groupesx.tpl'); $page->setTitle('Promo, Groupes X, Binets'); @@ -611,7 +632,7 @@ class ProfileModule extends PLModule $page->assign('assos', $req->fetchAllAssoc()); } - function handler_xnetlogo(&$page, $id) + function handler_xnetlogo($page, $id) { if (is_null($id)) { return PL_NOT_FOUND; @@ -633,7 +654,7 @@ class ProfileModule extends PLModule exit; } - function handler_vcard(&$page, $x = null) + function handler_vcard($page, $x = null) { if (is_null($x)) { return PL_NOT_FOUND; @@ -646,11 +667,11 @@ class ProfileModule extends PLModule } $vcard = new VCard(); - $vcard->addProfile(Profile::get($x)); + $vcard->addProfile(Profile::get($x, Profile::FETCH_ALL)); $vcard->show(); } - function handler_admin_trombino(&$page, $login = null, $action = null) { + function handler_admin_trombino($page, $login = null, $action = null) { $page->changeTpl('profile/admin_trombino.tpl'); $page->setTitle('Administration - Trombino'); @@ -669,13 +690,14 @@ class ProfileModule extends PLModule S::assert_xsrf_token(); $data = file_get_contents($_FILES['userfile']['tmp_name']); - list($x, $y) = getimagesize($_FILES['userfile']['tmp_name']); - $mimetype = substr($_FILES['userfile']['type'], 6); - unlink($_FILES['userfile']['tmp_name']); - XDB::execute( - "REPLACE INTO profile_photos SET pid={?}, attachmime = {?}, attach={?}, x={?}, y={?}", - $user->profile()->id(), $mimetype, $data, $x, $y); - break; + list($x, $y) = getimagesize($_FILES['userfile']['tmp_name']); + $mimetype = substr($_FILES['userfile']['type'], 6); + unlink($_FILES['userfile']['tmp_name']); + XDB::execute('INSERT INTO profile_photos (pid, attachmime, attach, x, y) + VALUES ({?}, {?}, {?}, {?}, {?}) + ON DUPLICATE KEY UPDATE attachmime = VALUES(attachmime), attach = VALUES(attach), x = VALUES(x), y = VALUES(y)', + $user->profile()->id(), $mimetype, $data, $x, $y); + break; case "delete": S::assert_xsrf_token(); @@ -684,7 +706,7 @@ class ProfileModule extends PLModule break; } } - function handler_admin_names(&$page, $action = 'list', $id = null) { + function handler_admin_names($page, $action = 'list', $id = null) { $page->setTitle('Administration - Types de noms'); $page->assign('title', 'Gestion des types de noms'); $table_editor = new PLTableEditor('admin/names', 'profile_name_enum', 'id', true); @@ -695,7 +717,7 @@ class ProfileModule extends PLModule $table_editor->describe('score', 'Score', true); $table_editor->apply($page, $action, $id); } - function handler_admin_binets(&$page, $action = 'list', $id = null) { + function handler_admin_binets($page, $action = 'list', $id = null) { $page->setTitle('Administration - Binets'); $page->assign('title', 'Gestion des binets'); $table_editor = new PLTableEditor('admin/binets', 'profile_binet_enum', 'id'); @@ -703,7 +725,7 @@ class ProfileModule extends PLModule $table_editor->describe('text','intitulé',true); $table_editor->apply($page, $action, $id); } - function handler_admin_education(&$page, $action = 'list', $id = null) { + function handler_admin_education($page, $action = 'list', $id = null) { $page->setTitle('Administration - Formations'); $page->assign('title', 'Gestion des formations'); $table_editor = new PLTableEditor('admin/education', 'profile_education_enum', 'id'); @@ -713,7 +735,7 @@ class ProfileModule extends PLModule $table_editor->describe('url', 'site web', false); $table_editor->apply($page, $action, $id); } - function handler_admin_education_field(&$page, $action = 'list', $id = null) { + function handler_admin_education_field($page, $action = 'list', $id = null) { $page->setTitle('Administration - Domaines de formation'); $page->assign('title', 'Gestion des domaines de formation'); $table_editor = new PLTableEditor('admin/education_field', 'profile_education_field_enum', 'id', true); @@ -721,7 +743,7 @@ class ProfileModule extends PLModule $table_editor->describe('field', 'domaine', true); $table_editor->apply($page, $action, $id); } - function handler_admin_education_degree(&$page, $action = 'list', $id = null) { + function handler_admin_education_degree($page, $action = 'list', $id = null) { $page->setTitle('Administration - Niveau de formation'); $page->assign('title', 'Gestion des niveau de formation'); $table_editor = new PLTableEditor('admin/education_degree', 'profile_education_degree_enum', 'id'); @@ -730,10 +752,10 @@ class ProfileModule extends PLModule $table_editor->describe('degree', 'niveau', true); $table_editor->apply($page, $action, $id); } - function handler_admin_education_degree_set(&$page, $action = 'list', $id = null) { + function handler_admin_education_degree_set($page, $action = 'list', $id = null, $id2 = null) { $page->setTitle('Administration - Correspondances formations - niveau de formation'); $page->assign('title', 'Gestion des correspondances formations - niveau de formation'); - $table_editor = new PLTableEditor('admin/education_degree_set', 'profile_education_degree', 'eduid', true); + $table_editor = new PLTableEditor('admin/education_degree_set', 'profile_education_degree', 'eduid', true, 'degreeid'); $table_editor->describe('eduid', 'id formation', true); $table_editor->describe('degreeid', 'id niveau', true); @@ -744,16 +766,16 @@ class ProfileModule extends PLModule $table_editor->add_option_table('profile_education_degree_enum','profile_education_degree_enum.id = t.degreeid'); $table_editor->add_option_field('profile_education_degree_enum.degree', 'degree_name', 'niveau'); - $table_editor->apply($page, $action, $id); + $table_editor->apply($page, $action, $id, $id2); } - function handler_admin_sections(&$page, $action = 'list', $id = null) { + function handler_admin_sections($page, $action = 'list', $id = null) { $page->setTitle('Administration - Sections'); $page->assign('title', 'Gestion des sections'); $table_editor = new PLTableEditor('admin/sections','profile_section_enum','id'); $table_editor->describe('text','intitulé',true); $table_editor->apply($page, $action, $id); } - function handler_admin_networking(&$page, $action = 'list', $id = null) { + function handler_admin_networking($page, $action = 'list', $id = null) { $page->assign('xorg_title', 'Polytechnique.org - Administration - Networking'); $page->assign('title', 'Gestion des types de networking'); $table_editor = new PLTableEditor('admin/networking', 'profile_networking_enum', 'nwid'); @@ -763,7 +785,7 @@ class ProfileModule extends PLModule $table_editor->describe('link', 'lien web', true); $table_editor->apply($page, $action, $id); } - function handler_admin_corps_enum(&$page, $action = 'list', $id = null) { + function handler_admin_corps_enum($page, $action = 'list', $id = null) { $page->setTitle('Administration - Corps'); $page->assign('title', 'Gestion des Corps'); $table_editor = new PLTableEditor('admin/corps_enum', 'profile_corps_enum', 'id'); @@ -772,7 +794,7 @@ class ProfileModule extends PLModule $table_editor->describe('still_exists', 'existe encore ?', true); $table_editor->apply($page, $action, $id); } - function handler_admin_corps_rank(&$page, $action = 'list', $id = null) { + function handler_admin_corps_rank($page, $action = 'list', $id = null) { $page->setTitle('Administration - Grade dans les Corps'); $page->assign('title', 'Gestion des grade dans les Corps'); $table_editor = new PLTableEditor('admin/corps_rank', 'profile_corps_rank_enum', 'id'); @@ -780,7 +802,7 @@ class ProfileModule extends PLModule $table_editor->describe('abbreviation', 'abbréviation', true); $table_editor->apply($page, $action, $id); } - function handler_admin_medals(&$page, $action = 'list', $id = null) { + function handler_admin_medals($page, $action = 'list', $id = null) { $page->setTitle('Administration - Distinctions'); $page->assign('title', 'Gestion des Distinctions'); $table_editor = new PLTableEditor('admin/medals','profile_medal_enum','id');