From: Stéphane Jacob <sj@m4x.org>
Date: Wed, 21 Sep 2011 10:58:16 +0000 (+0200)
Subject: Last fix was not good enough.
X-Git-Tag: xorg/1.1.4~55^2~8
X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=b3b00d585c426ee6420cc3f497fb1be869982191;p=platal.git

Last fix was not good enough.

Signed-off-by: Stéphane Jacob <sj@m4x.org>
---

diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php
index 864cb0a..0dc178b 100644
--- a/modules/xnetgrp.php
+++ b/modules/xnetgrp.php
@@ -1182,16 +1182,14 @@ class XnetGrpModule extends PLModule
                            $full_name, $directory_name, Post::t('display_name'), $firstname, $lastname,
                            (Post::t('sex') == 'male') ? 'male' : 'female', Post::t('email'),
                            (Post::t('type') == 'xnet') ? 'xnet' : 'virtual', $user->id());
-            } else if (!$user->perms) {
+            } else if (!$user->perms && Post::has('email') && require_email_update($user, Post::t('email'))) {
                 XDB::query('UPDATE  accounts
                                SET  email = {?}
                              WHERE  uid = {?}',
                            Post::t('email'), $user->id());
-                if (Post::has('email') && require_email_update($user, Post::t('email'))) {
-                    $listClient = new MMList(S::user());
-                    $listClient->change_user_email($user->forlifeEmail(), Post::t('email'));
-                    update_alias_user($user->forlifeEmail(), Post::t('email'));
-                }
+                $listClient = new MMList(S::user());
+                $listClient->change_user_email($user->forlifeEmail(), Post::t('email'));
+                update_alias_user($user->forlifeEmail(), Post::t('email'));
             }
             if (XDB::affectedRows()) {
                 $page->trigSuccess('Données de l\'utilisateur mises à jour.');