Improves email combobox (Closes #1110).
[platal.git] / modules / profile.php
index f055c30..9416557 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /***************************************************************************
- *  Copyright (C) 2003-2010 Polytechnique.org                              *
+ *  Copyright (C) 2003-2011 Polytechnique.org                              *
  *  http://opensource.polytechnique.org/                                   *
  *                                                                         *
  *  This program is free software; you can redistribute it and/or modify   *
@@ -30,7 +30,7 @@ class ProfileModule extends PLModule
             'fiche.php'                  => $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');