From e8c61abe37cc6961ff83b579a6f18d76c1c8a933 Mon Sep 17 00:00:00 2001
From: Florent Bruneau <florent.bruneau@polytechnique.org>
Date: Mon, 11 Oct 2010 15:44:03 +0200
Subject: [PATCH 1/1] Use a dropdown instead of checkbox to select the sex of a
 new account.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>
---
 modules/admin.php            | 2 +-
 templates/admin/accounts.tpl | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/modules/admin.php b/modules/admin.php
index f203d78..633a683 100644
--- a/modules/admin.php
+++ b/modules/admin.php
@@ -1207,7 +1207,7 @@ class AdminModule extends PLModule
             S::assert_xsrf_token();
             $firstname = Post::t('firstname');
             $lastname = strtoupper(Post::t('lastname'));
-            $sex = Post::b('sex') ? User::GENDER_FEMALE : User::GENDER_MALE;
+            $sex = Post::s('sex');
             $email = Post::t('email');
             $type = Post::s('type');
             $login = PlUser::makeHrid($firstname, $lastname, $type);
diff --git a/templates/admin/accounts.tpl b/templates/admin/accounts.tpl
index 52af2e5..d7b1dd9 100644
--- a/templates/admin/accounts.tpl
+++ b/templates/admin/accounts.tpl
@@ -92,7 +92,12 @@ function add_user_to_url(f) {
         </tr>
         <tr>
           <td class="titre">Sexe</td>
-          <td><input type="checkbox" name="sex" checked="checked" /></td>
+          <td>
+            <select name="sex">
+              <option value="female">Femme</option>
+              <option value="male">Homme</option>
+            </select>
+          </td>
         </tr>
         <tr>
           <td class="titre">Email</td>
-- 
2.1.4