FIX problème de gestion du profile, après ouverture d'une conversation puis enregistr...

[euphorik.git] / modules / erl / euphorik_bd.erl

diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl
index 0539910..f5fa258 100755 (executable)
--- a/modules/erl/euphorik_bd.erl
+++ b/modules/erl/euphorik_bd.erl
@@ -25,9 +25,9 @@
 -module(euphorik_bd).\r
 -export([   \r
    % users :\r
-   nouveau_user/2,\r
    nouveau_user/3,\r
-   set_profile/12,\r
+   nouveau_user/4,\r
+   set_profile/1,\r
    update_date_derniere_connexion/1,\r
    update_ip/2,\r
    update_pseudo_user/2,\r
@@ -90,10 +90,11 @@
 \r
 \r
 % Ajoute un nouveau user et le renvoie\r
-nouveau_user(Pseudo, Cookie) ->\r
+% User_infos représente le profile de l'utilisateur, il est possibe qu'il soit vide\r
+nouveau_user(Pseudo, Cookie, User_infos) ->\r
    F = fun() ->\r
       Id = nouvel_id(user),\r
-      User = #user{id = Id, cookie = Cookie, pseudo = Pseudo, date_creation = now(), date_derniere_connexion = now()},\r
+      User = User_infos#user{id = Id, cookie = Cookie, pseudo = Pseudo, date_creation = now(), date_derniere_connexion = now()},\r
       mnesia:write(User),\r
       User\r
    end,\r
@@ -101,55 +102,40 @@ nouveau_user(Pseudo, Cookie) ->
   \r
   \r
 % Ajoute un nouveau user et le renvoie\r
-nouveau_user(Login, Password, Cookie) ->\r
+% User_infos représente le profile de l'utilisateur, il est possibe qu'il soit vide\r
+nouveau_user(Login, Password, Cookie, User_infos) ->\r
    F = fun() ->\r
       Id = nouvel_id(user),\r
-      User = #user{id = Id, cookie = Cookie, pseudo = Login, login = Login, password = Password, date_creation = now(), date_derniere_connexion = now()},\r
+      User = User_infos#user{id = Id, cookie = Cookie, pseudo = Login, login = Login, password = Password, date_creation = now(), date_derniere_connexion = now()},\r
       mnesia:write(User),\r
       User\r
    end,\r
   resultat_transaction(mnesia:transaction(F)).\r
+  \r
 \r
-\r
-% Mise à par Cookie les autres peuvent être undefined ce qui veut dire qu'ils ne seront pas modifié.\r
-% Conversation est de type [{int(), bool()}] où l'entier est la racine, le booléen indique si la conversation est réduite ou non\r
-% Ostentatious_master peut valoir invisible, light ou heavy\r
-set_profile(Cookie, Login, Password, Pseudo, Email, Css, Chat_order, Nick_format, View_times, View_tooltips, Conversations, Ostentatious_master) ->\r
-   if Nick_format =:= nick; Nick_format =:= login; Nick_format =:= nick_login,\r
-      Ostentatious_master =:= invisible; Ostentatious_master =:= light; Ostentatious_master =:= heavy ->\r
-         resultat_transaction(mnesia:transaction(\r
-            fun() ->\r
-               case user_by_cookie(Cookie) of\r
-                  {ok, User} ->\r
-                     case user_by_login(Login) of\r
-                        {ok, U} when Login =/= [], U#user.id =/= User#user.id ->\r
-                           login_deja_pris;\r
-                        _ ->\r
-                           User_modifie = User#user{\r
-                              % TODO : pourquoi ne pas tester avec la valeur "undefined" plutôt qu'avec "is_list" ?\r
-                              % TODO : validation plus strict des données (pas de page négative dans les conv par exemple)\r
-                              login = if is_list(Login) -> Login; true -> User#user.login end,\r
-                              password = if is_list(Password) andalso Password =/= [] -> Password; true -> User#user.password end,\r
-                              pseudo = if is_list(Pseudo) -> Pseudo; true -> User#user.pseudo end,\r
-                              email = if is_list(Email) -> Email; true -> User#user.email end,\r
-                              css = if is_list(Css) -> Css; true -> User#user.css end,\r
-                              chat_order = Chat_order,\r
-                              nick_format = Nick_format,\r
-                              view_times = View_times,\r
-                              view_tooltips = View_tooltips,\r
-                              conversations = if is_list(Conversations) -> Conversations; true -> User#user.conversations end,\r
-                              ostentatious_master = Ostentatious_master\r
-                           },\r
-                           mnesia:write(User_modifie),\r
-                           ok\r
-                     end;\r
-                  _ -> erreur\r
-               end\r
-            end\r
-         ));\r
-      true ->\r
-         erreur\r
-   end.\r
+% Définit les données d'une utilisateur.\r
+set_profile(User) ->\r
+   resultat_transaction(mnesia:transaction(\r
+      fun() ->\r
+         case user_by_cookie(User#user.cookie) of\r
+            {ok, User_existant} ->\r
+               case user_by_login(User#user.login) of\r
+                  {ok, U} when User#user.login =/= [], U#user.id =/= User_existant#user.id ->\r
+                     login_deja_pris;\r
+                  _ ->\r
+                     mnesia:write(\r
+                        User#user{\r
+                           id = User_existant#user.id,\r
+                           login = if User_existant#user.login =:= [] -> User#user.login; true -> User_existant#user.login end, % on ne peut pas changer de login sauf si on en a pas !\r
+                           password = if User#user.password =:= [] -> User_existant#user.password; true -> User#user.password end\r
+                        }\r
+                     ),\r
+                     ok\r
+               end;\r
+            _ -> erreur\r
+         end\r
+      end\r
+   )).\r
 \r
 \r
 % Met à jour la date de la dernière connexion d'un utilisateur à maintenant\r
@@ -163,7 +149,7 @@ update_date_derniere_connexion(User_id) ->
                mnesia:abort("update_date_derniere_connexion: User inconnu")\r
           end\r
       end\r
-   ).   \r
+   ).\r
 \r
 \r
 % Met à jour l'ip d'un user\r