From: Greg Burri <greg.burri@gmail.com>
Date: Thu, 1 May 2008 07:54:32 +0000 (+0000)
Subject: MOD Correction de petits bugs
X-Git-Tag: 1.0.0^2~132
X-Git-Url: https://git.euphorik.ch/?a=commitdiff_plain;h=46b6cab14594802cdb4db9e3d77082815182073b;p=euphorik.git

MOD Correction de petits bugs
MOD maj TODO
---

diff --git a/doc/TODO.txt b/doc/TODO.txt
index a203be5..0b32e09 100755
--- a/doc/TODO.txt
+++ b/doc/TODO.txt
@@ -152,12 +152,20 @@
 3 : Peu grave
 
 [1] Lors de l'extraction d'une conv il arrive que la conv extracté soit bien créée mais vide, le bouton ne ferme pas la conv (très étrange, bug de firefox?)
+[1] Comme le json du client est encapsulé dans de l'xml il faut utiliser des xml entities pour les charactères <, > et &. Il faudrait, absolument éviter cette encapsulation moisie.
 [2] Quand on revient en arrière dans firefox le message en rédaction est perdu
 [2] En changeant de page puis en revenant sur la page principale les smiles ne sont plus highlightés lorsque le curseur les survol
 [2] Dans certains cas (à déterminer) les message-box (message d'information affiché tout en haut de la page) ne s'affiche plus (on ne voit que un petit bout dépassé)
 [2] Après un register le pseudo est effacé - le pseudo n'est pas mémorisé dans le profil lors d'un envoie de mess en l'ayant changé
 [2] cliquer sur les smiles ne marche pas sous IE
 [2] le return ne marche pas sous IE
+[2] Lors d'un login ou d'un logout il faut faire un full refresh
+[3] Un message envoyé sans être authentifié ne sera pas taggé comme appartenant à l'utilisateur.
+   a) L'utilisateur attend des messages SANS donné de cookie car il n'est pas authentifié
+   b) L'utilisateur envoie un message
+      i) Il se register sans login/pass (réalisé automatiquement)
+      ii) Il envoie le message (put_message)
+   c) Le serveur réagie au nouveau message et débloque la connexion, à ce moment le serveur n'a pas le cookie car pas donné, voir a)
 [3] Traiter le cas ou le cookie n'existe pas coté serveur (et plus généralement traiter tous les cas d'inputs exeptionnels)
 [3] "Return" ne marche pas sous safari
 [3] Amélioration des requêtes MNESIA, voir : http://mail.google.com/mail/#label/Erlang+mailing-list/117f688280569a58
diff --git a/doc/protocole3.txt b/doc/protocole3.txt
index f8b4c00..8b16d6e 100644
--- a/doc/protocole3.txt
+++ b/doc/protocole3.txt
@@ -107,7 +107,7 @@ c -> s
       "cookie" : "LKJDLAKSJBFLKASN",
       "login" : "paul49",
       "password" : "IJKJDHHSAD9081238",
-      "pseudo" : "Paul",
+      "nick" : "Paul",
       "email" : "paul@pierre.com",
       "css" : "css/3/euphorik.css",
       "main_page" : 1,
diff --git a/js/euphorik.js b/js/euphorik.js
index 88ef6e7..c37ee4c 100755
--- a/js/euphorik.js
+++ b/js/euphorik.js
@@ -525,6 +525,7 @@ Client.prototype.getJSONProfile = function()
       "cookie" : this.cookie,
       "login" : this.login,
       "password" : this.password,
+      "nick" : this.pseudo,
       "email" : this.email,
       "css" : this.css,
       "main_page" : this.pagePrincipale < 1 ? 1 : this.pagePrincipale,
@@ -688,7 +689,7 @@ Client.prototype.flush = function(async)
       return false
 
    thisClient = this
-   //this.util.jsonVersAction(this.getJSONProfile()).action.dump("Flush client")      
+   ;;; dumpObj(this.getJSONProfile())
    jQuery.ajax(
       {
          async: async,
@@ -765,11 +766,12 @@ jQuery(document).ready(
          })
       }
 
-      jQuery(document).unload(
+      // TODO : pourquoi jQuery(document).unload ne fonctionne pas ?
+      jQuery(window).unload(
          function()
          {
-            alert("ok")
-            client.flush()
+            //alert("ok")
+            client.flush(false)
          }
       )
       
diff --git a/modules/erl/euphorik_protocole.erl b/modules/erl/euphorik_protocole.erl
index ff53a1c..548f4af 100755
--- a/modules/erl/euphorik_protocole.erl
+++ b/modules/erl/euphorik_protocole.erl
@@ -66,7 +66,7 @@ profile(
       {cookie, Cookie},
       {login, Login},
       {password, Password},
-      {pseudo, Pseudo},
+      {nick, Pseudo},
       {email, Email},
       {css, Css},
       {main_page, Main_page},
@@ -183,11 +183,12 @@ put_message(
          if (Strip_content =:= []) ->
                erreur("Message vide");
             true ->
+               % TODO : non-atomique (update_pseudo+nouveau_message)
+               euphorik_minichat:update_pseudo_user(User#user.id, Nick),
                case euphorik_minichat:nouveau_message(Strip_content, User#user.id, Answer_to) of
-                  erreur -> erreur("Impossible d'ajouter un nouveau message");
+                     erreur -> erreur("Impossible d'ajouter un nouveau message");
                   _ ->
-                  euphorik_minichat:update_pseudo_user(User#user.id, Nick),
-                  json_reponse_ok()
+                     json_reponse_ok()
                end
          end;
       _ ->