X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=classes%2Fuser.php;h=8b9e1418c23df78404909ef44aeed795fd3444ca;hb=f869356b6135fab05539b9021f4c740bd980f6a7;hp=b99b2ddd826ef446badaf99193fe0ad4fb3007f1;hpb=6f0ed746468397c663783a89f5b3b43f6fff77e2;p=platal.git diff --git a/classes/user.php b/classes/user.php index b99b2dd..8b9e141 100644 --- a/classes/user.php +++ b/classes/user.php @@ -1,6 +1,6 @@ hruid != null && $this->forlife != null && - $this->bestalias != null && $this->display_name != null && - $this->full_name != null && $this->promo != NULL) { + if ($this->hruid !== null && $this->forlife !== null + && $this->bestalias !== null && $this->display_name !== null + && $this->full_name !== null && $this->promo !== null && $this->perms !== null + && $this->gender !== null && $this->email_format !== null) { return; } @@ -119,9 +120,13 @@ class User extends PlUser $res = XDB::query("SELECT u.hruid, u.promo, CONCAT(af.alias, '@{$globals->mail->domain}') AS forlife, CONCAT(ab.alias, '@{$globals->mail->domain}') AS bestalias, - CONCAT(u.prenom, ' ', u.nom) AS full_name, - IF(u.prenom != '', u.prenom, u.nom) AS display_name + CONCAT(u.prenom, ' ', IF(u.nom_usage <> '', u.nom_usage, u.nom)) AS full_name, + IF(u.prenom != '', u.prenom, u.nom) AS display_name, + FIND_IN_SET('femme', u.flags) AS gender, + q.core_mail_fmt AS email_format, + u.perms FROM auth_user_md5 AS u + LEFT JOIN auth_user_quick AS q ON (q.user_id = u.user_id) LEFT JOIN aliases AS af ON (af.id = u.user_id AND af.type = 'a_vie') LEFT JOIN aliases AS ab ON (ab.id = u.user_id AND FIND_IN_SET('bestalias', ab.flags)) WHERE u.user_id = {?}", $this->user_id); @@ -130,6 +135,8 @@ class User extends PlUser // Specialization of the fillFromArray method, to implement hacks to enable // lazy loading of user's main properties from the session. + // TODO(vzanotti): remove the conversion hacks once the old codebase will + // stop being used actively. protected function fillFromArray(array $values) { // It might happen that the 'user_id' field is called uid in some places @@ -150,9 +157,57 @@ class User extends PlUser } } + // We also need to convert the gender (usually named "femme"), and the + // email format parameter (valued "texte" instead of "text"). + if (isset($values['femme'])) { + $values['gender'] = (bool) $values['femme']; + } + if (isset($values['mail_fmt'])) { + $values['email_format'] = $values['mail_fmt']; + } + if (isset($values['email_format'])) { + $values['email_format'] = ($values['email_format'] ? self::FORMAT_HTML : self::FORMAT_TEXT); + } + parent::fillFromArray($values); } + // Specialization of the buildPerms method + // This function build 'generic' permissions for the user. It does not take + // into account page specific permissions (e.g X.net group permissions) + protected function buildPerms() + { + if (!is_null($this->perm_flags)) { + return; + } + if ($this->perms === null) { + $this->loadMainFields(); + } + $this->perm_flags = self::makePerms($this->perms); + } + + // Return the password of the user + public function password() + { + return XDB::fetchOneCell('SELECT u.password + FROM auth_user_md5 AS u + WHERE u.user_id = {?}', $this->id()); + } + + // Return permission flags for a given permission level. + public static function makePerms($perms) + { + $flags = new PlFlagSet(); + if (is_null($flags) || $perms == 'disabled' || $perms == 'ext') { + return $flags; + } + $flags->addFlag(PERMS_USER); + if ($perms == 'admin') { + $flags->addFlag(PERMS_ADMIN); + } + return $flags; + } + // Implementation of the default user callback. public static function _default_user_callback($login, $results) {