From: Greg Burri <greg.burri@gmail.com>
Date: Sat, 26 Jul 2008 18:21:26 +0000 (+0000)
Subject: FIX problème de gestion du profile, après ouverture d'une conversation puis enregistr...
X-Git-Tag: 1.1.0~43
X-Git-Url: https://git.euphorik.ch/?a=commitdiff_plain;h=11fe8f6d268a9b1c0a773294cc7d70ffdb6afb30;p=euphorik.git
FIX problème de gestion du profile, après ouverture d'une conversation puis enregistrement la conversation n'était pas gardée
FIX autres...
---
diff --git a/doc/protocole3.txt b/doc/protocole3.txt
index 55e2115..c8642ab 100644
--- a/doc/protocole3.txt
+++ b/doc/protocole3.txt
@@ -55,9 +55,10 @@ Le mot de passe est hashé en md5.
c -> s
{
- "header" : {action : "authentification", version : 3},
+ "header" : {action : "register", version : 3},
"login" : "paul",
- "password" : "IJKJDHHSAD9081238"
+ "password" : "IJKJDHHSAD9081238",
+ "profile_infos" : { .. } // voir <profile_infos>
}
ou
{
@@ -74,16 +75,7 @@ ou
s -> c
{
"reply" : "register" | "authentification",
- "status" : "auth_not_registered",
- "cookie" : "LKJDLAKSJBFLKASN",
- "id" : 193,
- "css" : "css/1/euphorik.css",
- "main_page" : 1
- }
-ou
- {
- "reply" : "register" | "authentification",
- "status" : "auth_registered",
+ "status" : "auth_registered" | "auth_not_registered",
"cookie" : "LKJDLAKSJBFLKASN",
"id" : 193,
"nick" : "Paul",
@@ -109,12 +101,8 @@ c -> s
=== Profile ===
-c -> s
+<profile_infos>
{
- "header" : {action : "set_profile", version : 3},
- "cookie" : "LKJDLAKSJBFLKASN",
- "login" : "paul49",
- "password" : "IJKJDHHSAD9081238",
"nick" : "Paul",
"email" : "paul@pierre.com",
"css" : "css/3/euphorik.css",
@@ -125,6 +113,15 @@ c -> s
"conversations" : [{"root" : 3, "minimized" : true},
"ostentatious_master" : "invisible" | "light" | "heavy"
}
+
+c -> s
+ {
+ "header" : {action : "set_profile", version : 3},
+ "cookie" : "LKJDLAKSJBFLKASN",
+ "login" : "paul49",
+ "password" : "IJKJDHHSAD9081238",
+ "profile_infos" : <profile_infos>
+ }
s -> c
<ok>
diff --git a/js/client.js b/js/client.js
index d7240a6..73622a4 100644
--- a/js/client.js
+++ b/js/client.js
@@ -180,13 +180,17 @@ euphorik.Client.prototype.getJSONLoginCookie = function() {
* de s'autentifier avec (login, password).
*/
euphorik.Client.prototype.getJSONEnregistrement = function(login, password) {
- var mess = { "header" : { "action" : "register", "version" : euphorik.conf.versionProtocole } };
+ var mess = {
+ "header" : { "action" : "register","version" : euphorik.conf.versionProtocole }
+ };
if (login && password) {
mess.login = login;
mess.password = password;
}
+ mess.profile_infos = this.getJSONProfileInfos();
+
return mess;
};
@@ -204,6 +208,12 @@ euphorik.Client.prototype.getJSONProfile = function() {
"cookie" : this.cookie,
"login" : this.login,
"password" : this.password,
+ "profile_infos" : this.getJSONProfileInfos()
+ };
+};
+
+euphorik.Client.prototype.getJSONProfileInfos = function() {
+ return {
"nick" : this.pseudo,
"email" : this.email,
"css" : this.css,
@@ -302,7 +312,7 @@ euphorik.Client.prototype.enregistrement = function(login, password) {
};
/**
- * Connexion. Réalisé de manière synchrone.
+ * Connexion. Réalisée de manière synchrone.
*/
euphorik.Client.prototype.connexion = function(messageJson) {
var thisClient = this;
diff --git a/js/pageMinichat/conversation.js b/js/pageMinichat/conversation.js
index 3948629..238976b 100644
--- a/js/pageMinichat/conversation.js
+++ b/js/pageMinichat/conversation.js
@@ -238,7 +238,6 @@ euphorik.Conversation.prototype.flush = function() {
var thisConversation = this;
var reverse = this.client.chatOrder === "reverse";
- // est-ce que le prochain message est pair ? (permet d'alterner le style des messages)
var messagePair = (this.idDernierMessageAffiche === 0 ? true :
($("#" + this.getId() + " .messages div:" + (reverse ? "first" : "last")).attr("class").search("messagePair") === -1)
);
diff --git a/js/pageMinichat/message.js b/js/pageMinichat/message.js
index 67ba7e6..a3d4d32 100644
--- a/js/pageMinichat/message.js
+++ b/js/pageMinichat/message.js
@@ -111,7 +111,9 @@ euphorik.Message.prototype.getConversation = function(messages) {
* Aucun callback n'est affecté.
*/
euphorik.Message.prototype.XHTML = function(messagePair, pre) {
- messagePair = messagePair || true;
+ if (messagePair === undefined) {
+ messagePair = true;
+ }
pre = pre || "";
thisMessage = this;
diff --git a/js/pageProfile.js b/js/pageProfile.js
index fa7235b..b272400 100755
--- a/js/pageProfile.js
+++ b/js/pageProfile.js
@@ -113,6 +113,7 @@ euphorik.PageProfile.prototype.getHTML = function() {
' <tr>' +
' <td>login</td>' +
' <td><input class="login" type="text" size="20" maxlength="20" ' + (this.client.authentifie() ? 'readonly="readonly"' : '') + ' /></td>' +
+ (this.client.authentifie() ? '' : ' <td>(sensible à la <a href="http://fr.wikipedia.org/wiki/Casse_(informatique)">casse</a>)</td>') +
' </tr>' +
' <tr>' +
' <td>password</td>' +
diff --git a/js/pageRegister.js b/js/pageRegister.js
index b422f5d..f1abd70 100755
--- a/js/pageRegister.js
+++ b/js/pageRegister.js
@@ -14,64 +14,65 @@
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
-// along with Euphorik. If not, see <http://www.gnu.org/licenses/>.
-
-euphorik.PageRegister = function(client, formateur, util) {
- this.nom = "register";
-
- this.client = client;
- this.formateur = formateur;
- this.util = util;
-};
-
-euphorik.PageRegister.prototype.contenu = function() {
- return '<form action="" id="register" >' +
- ' <table>' +
- ' <tr>' +
- ' <td>login</td>' +
- ' <td><input class="login" type="text" size="20" maxlength="20"/><input class="captcha" name="captcha" type="text" size="12"></input></td>' +
- ' </tr>' +
- ' <tr>' +
- ' <td>password</td>' +
- ' <td><input class="password" type="password" size="20" maxlength="20"/></td>' +
- ' </tr>' +
- ' <tr>' +
- ' <td>password re</td>' +
- ' <td><input class="passwordRe" type="password" size="20" maxlength="20"/></td>' +
- ' </tr>' +
- ' <tr>' +
- ' <td></td>' +
- ' <td><button>valider</button>' +
- ' </tr>' +
- ' </table>' +
- '</form>';
-};
-
-euphorik.PageRegister.prototype.charger = function() {
- $("#page form#register").submit(function(){ return false; });
-
- var thisPage = this;
-
- $("#page form#register button").click(
+// along with Euphorik. If not, see <http://www.gnu.org/licenses/>.
+
+euphorik.PageRegister = function(client, formateur, util) {
+ this.nom = "register";
+
+ this.client = client;
+ this.formateur = formateur;
+ this.util = util;
+};
+
+euphorik.PageRegister.prototype.contenu = function() {
+ return '<form action="" id="register" >' +
+ ' <table>' +
+ ' <tr>' +
+ ' <td>login</td>' +
+ ' <td><input class="login" type="text" size="20" maxlength="20"/><input class="captcha" name="captcha" type="text" size="12"></input></td>' +
+ ' <td>(sensible à la <a href="http://fr.wikipedia.org/wiki/Casse_(informatique)">casse</a>)</td>' +
+ ' </tr>' +
+ ' <tr>' +
+ ' <td>password</td>' +
+ ' <td><input class="password" type="password" size="20" maxlength="20"/></td>' +
+ ' </tr>' +
+ ' <tr>' +
+ ' <td>password re</td>' +
+ ' <td><input class="passwordRe" type="password" size="20" maxlength="20"/></td>' +
+ ' </tr>' +
+ ' <tr>' +
+ ' <td></td>' +
+ ' <td><button>valider</button>' +
+ ' </tr>' +
+ ' </table>' +
+ '</form>';
+};
+
+euphorik.PageRegister.prototype.charger = function() {
+ $("#page form#register").submit(function(){ return false; });
+
+ var thisPage = this;
+
+ $("#page form#register button").click(
function() {
if ($("#page form#register input.captcha").val() !== "") {
return;
}
-
- var login = $("#page form#register input.login").val().trim();
- var password = $("#page form#register input.password").val();
- var passwordRe = $("#page form#register input.passwordRe").val();
-
- if (login === "") {
- thisPage.util.messageDialogue("Le login ne doit pas être vide");
- } else if (password === "" && passwordRe === "") {
- thisPage.util.messageDialogue("Un mot de passe est obligatoire");
- } else if (password !== passwordRe) {
- thisPage.util.messageDialogue("Les mots de passes ne correspondent pas");
- } else if(thisPage.client.enregistrement(login, thisPage.util.md5(password))) {
- thisPage.util.messageDialogue("Enregistrement réussi");
- thisPage.pages.afficherPage("minichat");
- }
- }
- );
+
+ var login = $("#page form#register input.login").val().trim();
+ var password = $("#page form#register input.password").val();
+ var passwordRe = $("#page form#register input.passwordRe").val();
+
+ if (login === "") {
+ thisPage.util.messageDialogue("Le login ne doit pas être vide");
+ } else if (password === "" && passwordRe === "") {
+ thisPage.util.messageDialogue("Un mot de passe est obligatoire");
+ } else if (password !== passwordRe) {
+ thisPage.util.messageDialogue("Les mots de passes ne correspondent pas");
+ } else if(thisPage.client.enregistrement(login, thisPage.util.md5(password))) {
+ thisPage.util.messageDialogue("Enregistrement réussi");
+ thisPage.pages.afficherPage("minichat");
+ }
+ }
+ );
};
diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl
index 0539910..f5fa258 100755
--- a/modules/erl/euphorik_bd.erl
+++ b/modules/erl/euphorik_bd.erl
@@ -25,9 +25,9 @@
-module(euphorik_bd).
-export([
% users :
- nouveau_user/2,
nouveau_user/3,
- set_profile/12,
+ nouveau_user/4,
+ set_profile/1,
update_date_derniere_connexion/1,
update_ip/2,
update_pseudo_user/2,
@@ -90,10 +90,11 @@
% Ajoute un nouveau user et le renvoie
-nouveau_user(Pseudo, Cookie) ->
+% User_infos représente le profile de l'utilisateur, il est possibe qu'il soit vide
+nouveau_user(Pseudo, Cookie, User_infos) ->
F = fun() ->
Id = nouvel_id(user),
- User = #user{id = Id, cookie = Cookie, pseudo = Pseudo, date_creation = now(), date_derniere_connexion = now()},
+ User = User_infos#user{id = Id, cookie = Cookie, pseudo = Pseudo, date_creation = now(), date_derniere_connexion = now()},
mnesia:write(User),
User
end,
@@ -101,55 +102,40 @@ nouveau_user(Pseudo, Cookie) ->
% Ajoute un nouveau user et le renvoie
-nouveau_user(Login, Password, Cookie) ->
+% User_infos représente le profile de l'utilisateur, il est possibe qu'il soit vide
+nouveau_user(Login, Password, Cookie, User_infos) ->
F = fun() ->
Id = nouvel_id(user),
- User = #user{id = Id, cookie = Cookie, pseudo = Login, login = Login, password = Password, date_creation = now(), date_derniere_connexion = now()},
+ User = User_infos#user{id = Id, cookie = Cookie, pseudo = Login, login = Login, password = Password, date_creation = now(), date_derniere_connexion = now()},
mnesia:write(User),
User
end,
resultat_transaction(mnesia:transaction(F)).
+
-
-% Mise à par Cookie les autres peuvent être undefined ce qui veut dire qu'ils ne seront pas modifié.
-% Conversation est de type [{int(), bool()}] où l'entier est la racine, le booléen indique si la conversation est réduite ou non
-% Ostentatious_master peut valoir invisible, light ou heavy
-set_profile(Cookie, Login, Password, Pseudo, Email, Css, Chat_order, Nick_format, View_times, View_tooltips, Conversations, Ostentatious_master) ->
- if Nick_format =:= nick; Nick_format =:= login; Nick_format =:= nick_login,
- Ostentatious_master =:= invisible; Ostentatious_master =:= light; Ostentatious_master =:= heavy ->
- resultat_transaction(mnesia:transaction(
- fun() ->
- case user_by_cookie(Cookie) of
- {ok, User} ->
- case user_by_login(Login) of
- {ok, U} when Login =/= [], U#user.id =/= User#user.id ->
- login_deja_pris;
- _ ->
- User_modifie = User#user{
- % TODO : pourquoi ne pas tester avec la valeur "undefined" plutôt qu'avec "is_list" ?
- % TODO : validation plus strict des données (pas de page négative dans les conv par exemple)
- login = if is_list(Login) -> Login; true -> User#user.login end,
- password = if is_list(Password) andalso Password =/= [] -> Password; true -> User#user.password end,
- pseudo = if is_list(Pseudo) -> Pseudo; true -> User#user.pseudo end,
- email = if is_list(Email) -> Email; true -> User#user.email end,
- css = if is_list(Css) -> Css; true -> User#user.css end,
- chat_order = Chat_order,
- nick_format = Nick_format,
- view_times = View_times,
- view_tooltips = View_tooltips,
- conversations = if is_list(Conversations) -> Conversations; true -> User#user.conversations end,
- ostentatious_master = Ostentatious_master
- },
- mnesia:write(User_modifie),
- ok
- end;
- _ -> erreur
- end
- end
- ));
- true ->
- erreur
- end.
+% Définit les données d'une utilisateur.
+set_profile(User) ->
+ resultat_transaction(mnesia:transaction(
+ fun() ->
+ case user_by_cookie(User#user.cookie) of
+ {ok, User_existant} ->
+ case user_by_login(User#user.login) of
+ {ok, U} when User#user.login =/= [], U#user.id =/= User_existant#user.id ->
+ login_deja_pris;
+ _ ->
+ mnesia:write(
+ User#user{
+ id = User_existant#user.id,
+ 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 !
+ password = if User#user.password =:= [] -> User_existant#user.password; true -> User#user.password end
+ }
+ ),
+ ok
+ end;
+ _ -> erreur
+ end
+ end
+ )).
% Met à jour la date de la dernière connexion d'un utilisateur à maintenant
@@ -163,7 +149,7 @@ update_date_derniere_connexion(User_id) ->
mnesia:abort("update_date_derniere_connexion: User inconnu")
end
end
- ).
+ ).
% Met à jour l'ip d'un user
diff --git a/modules/erl/euphorik_protocole.erl b/modules/erl/euphorik_protocole.erl
index d8bfa89..c3dfbbe 100755
--- a/modules/erl/euphorik_protocole.erl
+++ b/modules/erl/euphorik_protocole.erl
@@ -43,14 +43,14 @@
% Une utilisateur s'enregistre avec un tuple {Login, Password}.
-register([{login, Login}, {password, Password}], IP) ->
+register([{login, Login}, {password, Password}, {profile_infos, Profile_infos}], IP) ->
Can_register = euphorik_bd:can_register(IP),
if Can_register ->
case euphorik_bd:user_by_login(Login) of
{ok, _} ->
erreur("Login déjà existant");
_ ->
- User = euphorik_bd:nouveau_user(Login, Password, generer_cookie()),
+ User = euphorik_bd:nouveau_user(Login, Password, generer_cookie(), user_from_json(Profile_infos)),
euphorik_bd:update_ip(User#user.id, IP),
json_reponse_login_ok(User)
end;
@@ -58,10 +58,10 @@ register([{login, Login}, {password, Password}], IP) ->
erreur_register_flood()
end;
% Enregistrement sans {Login, Password}
-register([], IP) ->
+register([{profile_infos, Profile_infos}], IP) ->
Can_register = euphorik_bd:can_register(IP),
if Can_register ->
- User = euphorik_bd:nouveau_user("<nick>", generer_cookie()),
+ User = euphorik_bd:nouveau_user("<nick>", generer_cookie(), user_from_json(Profile_infos)),
euphorik_bd:update_ip(User#user.id, IP),
json_reponse_login_ok(User);
true ->
@@ -115,16 +115,44 @@ profile(
{cookie, Cookie},
{login, Login},
{password, Password},
- {nick, Pseudo},
- {email, Email},
- {css, Css},
- {chat_order, Chat_order_str},
- {nick_format, Nick_format_str},
- {view_times, View_times},
- {view_tooltips, View_tooltips},
- {conversations, {array, Conversations_json}},
- {ostentatious_master, Ostentatious_master}
+ {profile_infos, Profile_infos}
]
+) ->
+ case user_from_json(Profile_infos) of
+ {erreur, E} -> E;
+ UserInfos ->
+ User = UserInfos#user {
+ cookie = Cookie,
+ login = Login,
+ password = Password
+ },
+ % TODO : pas très beau, mieux vaut construire un #user
+ case euphorik_bd:set_profile(User) of
+ ok ->
+ json_reponse_ok();
+ login_deja_pris ->
+ erreur("Login déjà pris");
+ _ ->
+ erreur("Impossible de mettre à jour le profile")
+ end
+ end.
+
+
+% Construit un #user à partir des données JSON
+user_from_json(
+ {struct,
+ [
+ {nick, Pseudo},
+ {email, Email},
+ {css, Css},
+ {chat_order, Chat_order_str},
+ {nick_format, Nick_format_str},
+ {view_times, View_times},
+ {view_tooltips, View_tooltips},
+ {conversations, {array, Conversations_json}},
+ {ostentatious_master, Ostentatious_master_str}
+ ]
+ }
) ->
% décomposition de la strucure JSON
Conversations = lists:foldr(
@@ -140,26 +168,35 @@ profile(
[],
Conversations_json
),
- % TODO : pas très beau, mieux vaut construire un #user
- case euphorik_bd:set_profile(
- Cookie,
- Login,
- Password,
- Pseudo,
- Email,
- Css,
- list_to_atom(Chat_order_str),
- list_to_atom(Nick_format_str),
- View_times,
- View_tooltips,
- Conversations,
- list_to_atom(Ostentatious_master)) of
- ok ->
- json_reponse_ok();
- login_deja_pris ->
- erreur("Login déjà pris");
- _ ->
- erreur("Impossible de mettre à jour le profile")
+ % vérification des données JSON
+ Chat_order = list_to_atom(Chat_order_str),
+ Chat_order_valide = lists:any(fun(E) -> E =:= Chat_order end, [reverse, chrono]),
+ if not Chat_order_valide ->
+ {erreur, Chat_order_str ++ " n'est pas une valeur acceptée pour 'chat_order'"};
+ true ->
+ Nick_format = list_to_atom(Nick_format_str),
+ Nick_format_valide = lists:any(fun(E) -> E =:= Nick_format end, [nick, login, nick_login]),
+ if not Nick_format_valide ->
+ {erreur, Nick_format_str ++ " n'est pas une valeur acceptée pour 'nick_format'"};
+ true ->
+ Ostentatious_master = list_to_atom(Ostentatious_master_str),
+ Ostentatious_master_valide = lists:any(fun(E) -> E =:= Ostentatious_master end, [invisible, light, heavy]),
+ if not Ostentatious_master_valide ->
+ {erreur, Ostentatious_master_str ++ " n'est pas une valeur acceptée pour 'ostentatious_master'"};
+ true ->
+ #user{
+ pseudo = Pseudo,
+ email = Email,
+ css = Css,
+ chat_order = Chat_order,
+ nick_format = Nick_format,
+ view_times = View_times,
+ view_tooltips = View_tooltips,
+ conversations = Conversations,
+ ostentatious_master = Ostentatious_master
+ }
+ end
+ end
end.
diff --git a/modules/include/euphorik_bd.hrl b/modules/include/euphorik_bd.hrl
index 053c046..038dae8 100755
--- a/modules/include/euphorik_bd.hrl
+++ b/modules/include/euphorik_bd.hrl
@@ -64,9 +64,9 @@
{
id,
cookie, % string()
- pseudo = [], % string()
login = [], % string()
password = [], % string() (md5)
+ pseudo = [], % string()
email = [], % string()
date_creation, % erlang:now()
date_derniere_connexion, % erlang:now(), est mis à jour lors de n'importe quelle activitée (envoie de message par exemple)
diff --git a/styles/1/euphorik.css b/styles/1/euphorik.css
index 0383346..76cce82 100755
--- a/styles/1/euphorik.css
+++ b/styles/1/euphorik.css
@@ -17,6 +17,10 @@ body {
background-color: #f6dfc2;
}
+td {
+ padding-right: 4px
+}
+
#container {
height: auto;
margin: 0px;