MOD avancement dans la Grande Restructuration
authorGreg Burri <greg.burri@gmail.com>
Thu, 24 Jul 2008 12:08:18 +0000 (12:08 +0000)
committerGreg Burri <greg.burri@gmail.com>
Thu, 24 Jul 2008 12:08:18 +0000 (12:08 +0000)
js/euphorik.js
js/pageMinichat/conversation.js
js/pageMinichat/conversations.js

index 34888c3..1d6f3e8 100755 (executable)
@@ -77,6 +77,16 @@ var objectEach = function(o, f) {
       }
    }
 };
+\r
+var objectMemberCount = function(o) {\r
+   var nb = 0;\r
+   for (var k in o) {\r
+      if (o.hasOwnProperty(k)) {\r
+         nb += 1;\r
+      }\r
+   }\r
+   return nb;\r
+};\r
 
 Array.prototype.each = function(f) {
    for (var i = 0; i < this.length; i++) {
index c3082b8..77ee3a2 100644 (file)
@@ -16,7 +16,8 @@
 // You should have received a copy of the GNU General Public License\r
 // along with Euphorik.  If not, see <http://www.gnu.org/licenses/>.\r
  \r
\r
+ /*jslint laxbreak:true */\r
+\r
 /**\r
   * Représente une conversation.\r
   * Une conversation, au niveau XHTML, est formé de deux partie, le titre et les messages.\r
index 0c0f3d0..2b78fc5 100644 (file)
 // You should have received a copy of the GNU General Public License\r
 // along with Euphorik.  If not, see <http://www.gnu.org/licenses/>.\r
  \r
+/*jslint laxbreak:true */\r
+\r
 /**\r
   * Représente l'ensemble des conversations affichés.\r
   */\r
-function Conversations(client, formateur, util)\r
-{\r
-   this.client = client\r
-   this.formateur = formateur\r
-   this.util = util\r
+euphorik.Conversations = function(client, formateur, util) {\r
+   this.client = client;\r
+   this.formateur = formateur;\r
+   this.util = util;\r
    \r
    // un ensemble des messages (id) auquel l'utilisateur répond (vider après l'envoie du message courant)\r
-   this.messagesRepond = {}\r
+   this.messagesRepond = {};\r
    \r
-   this.conversations = new Array() // les conversations, la première représente la conversation principale\r
+   this.conversations = []; // les conversations, la première représente la conversation principale\r
    \r
-   this.nouvelleConversation(0)\r
+   this.nouvelleConversation(0);\r
    \r
-   this.trollIdCourant = 0\r
+   this.trollIdCourant = 0;\r
    \r
-   this.pageEvent = new euphorik.PageEvent("chat", this.util)\r
-}\r
+   this.pageEvent = new euphorik.PageEvent("chat", this.util);\r
+};\r
 \r
 // les messages insérés dans le document XHTML on leur id prefixé par cette valeur\r
 // cela permet de les distinguer des "vrais" messages\r
-Conversations.prototype.prefixIdMessage = "rep"\r
+euphorik.Conversations.prototype.prefixIdMessage = "rep";\r
 \r
 /**\r
   * Permet de définir un message comme étant ou n'étant plus un message auquel l'utilisateur\r
   * répond.\r
   */\r
-Conversations.prototype.toggleMessageRepond = function(mess)\r
-{   \r
+euphorik.Conversations.prototype.toggleMessageRepond = function(mess) {   \r
    // est-ce que l'on répond déjà à ce message ? si oui alors on l'enlève de la liste\r
-   if (mess.id in this.messagesRepond)\r
-   {\r
-      this.enleverMessageRepond(mess)\r
-      return\r
+   if (mess.id in this.messagesRepond) {\r
+      this.enleverMessageRepond(mess);\r
+      return;\r
    }\r
    \r
-   this.ajouterMessageRepond(mess)\r
-}\r
+   this.ajouterMessageRepond(mess);\r
+};\r
 \r
 /**\r
   * Enlève tous les messages auquel l'utilisateur souhaite répond.\r
   */\r
-Conversations.prototype.enleverMessagesRepond = function()\r
-{\r
-   for (var messId in this.messagesRepond)\r
-      this.enleverMessageRepond(this.messagesRepond[messId])\r
+euphorik.Conversations.prototype.enleverMessagesRepond = function() {\r
+   var thisConversations = this;\r
+   \r
+   objectEach(function(messId, mess) {\r
+      thisConversations.enleverMessageRepond(mess);\r
+   });\r
     \r
    // on réinitialise pour être sur que tout est bien enlevé\r
-   this.messagesRepond = {}\r
-   $("#conversations .message").removeClass("repondEnEvidence")\r
-   $("form#posterMessage #repondA .messages").empty()\r
-}\r
+   this.messagesRepond = {};\r
+   $("#conversations .message").removeClass("repondEnEvidence");\r
+   $("form#posterMessage #repondA .messages").empty();\r
+};\r
 \r
 /**\r
   * Définit un message comme n'y répondant plus.\r
   */\r
-Conversations.prototype.enleverMessageRepond = function(mess)\r
-{\r
-   $(this.exprIdsPotentiels(mess)).removeClass("repondEnEvidence")\r
-   $("#" + mess.getId(this.prefixIdMessage)).remove()\r
-   delete this.messagesRepond[mess.id]\r
-   this.rafraichireNombreMessagesRepond()\r
-}\r
+euphorik.Conversations.prototype.enleverMessageRepond = function(mess) {\r
+   $(this.exprIdsPotentiels(mess)).removeClass("repondEnEvidence");\r
+   $("#" + mess.getId(this.prefixIdMessage)).remove();\r
+   delete this.messagesRepond[mess.id];\r
+   this.rafraichireNombreMessagesRepond();\r
+};\r
 \r
 /**\r
   * Définit un message comme y répondant.\r
   */\r
-Conversations.prototype.ajouterMessageRepond = function(mess)\r
-{\r
-   var thisMessages = this\r
+euphorik.Conversations.prototype.ajouterMessageRepond = function(mess) {\r
+   var thisMessages = this;\r
     \r
    // est-ce que le message fait partie de la même conversation que les autres messages ?\r
    // TODO : solution plus élégante pour prendre un mess parmis messagesRepond !?\r
-   var mess2\r
-   for(mess2 in this.messagesRepond){ break; }\r
-   mess2 = this.messagesRepond[mess2]\r
+   var mess2;\r
+   for (mess2 in this.messagesRepond) {\r
+      if (this.messagesRepond.hasOwnProperty(mess2)) {\r
+         break;\r
+      }\r
+   }\r
+   mess2 = this.messagesRepond[mess2];\r
    \r
-   if (mess2 != undefined && mess2.racineId != mess.racineId)\r
-   {\r
-      this.util.messageDialogue("Impossible de répondre à deux messages ne faisant pas partie de la même conversation")\r
-      return\r
+   if (!mess2 && mess2.racineId !== mess.racineId) {\r
+      this.util.messageDialogue("Impossible de répondre à deux messages ne faisant pas partie de la même conversation");\r
+      return;\r
    }\r
    \r
-   $("form#posterMessage #repondA .messages").append(mess.XHTML(undefined, this.prefixIdMessage))\r
-   this.messagesRepond[mess.id] = mess\r
+   $("form#posterMessage #repondA .messages").append(mess.XHTML(undefined, this.prefixIdMessage));\r
+   this.messagesRepond[mess.id] = mess;\r
    \r
    // ajout la classe 'repondEnEvidence' au message. comme il peut se trouver potentiellement dans\r
    // chaque conversation on construit tous les id potentiels\r
-   $(mess.getId(this.prefixIdMessage) + ", " + this.exprIdsPotentiels(mess)).addClass("repondEnEvidence")\r
+   $(mess.getId(this.prefixIdMessage) + ", " + this.exprIdsPotentiels(mess)).addClass("repondEnEvidence");\r
    \r
    $("#" + mess.getId(this.prefixIdMessage)).click(\r
-      function()\r
-      {\r
-         $(this).fadeOut("normal", function(){\r
-            thisMessages.enleverMessageRepond(mess)\r
-            $("form#posterMessage #repondA .messages").hide()\r
-         })\r
+      function() {\r
+         $(this).fadeOut("normal", function() {\r
+            thisMessages.enleverMessageRepond(mess);\r
+            $("form#posterMessage #repondA .messages").hide();\r
+         });\r
       }\r
-   )\r
-   this.rafraichireNombreMessagesRepond()\r
-}\r
+   );\r
+   this.rafraichireNombreMessagesRepond();\r
+};\r
 \r
 /**\r
   * Construit tous les id potentiels d'un message, renvoie par exemple :\r
   * "conv9b28mess1h, conv9b2amess1h, conv9b32mess1h"\r
   */\r
-Conversations.prototype.exprIdsPotentiels = function(mess)\r
-{\r
-   var exprMess = ""\r
-   for(var i = 0; i < this.conversations.length; i++)\r
-   {\r
-      exprMess += (mess != "" ? ", " : "") + "#" + mess.getId(this.conversations[i].getId())\r
-   }\r
-   return exprMess\r
-}\r
+euphorik.Conversations.prototype.exprIdsPotentiels = function(mess) {\r
+   var exprMess = "";\r
+   this.conversations.each(function(i, conv) {\r
+      exprMess += (mess !== "" ? ", " : "") + "#" + mess.getId(conv.getId());\r
+   });\r
+   return exprMess;\r
+};\r
 \r
 /**\r
   * Met à jour le nombre qui indique à l'utilisateur à combien de messages il répond.\r
   */\r
-Conversations.prototype.rafraichireNombreMessagesRepond = function()\r
-{\r
+euphorik.Conversations.prototype.rafraichireNombreMessagesRepond = function() {\r
    // TODO : ya pas mieux pour trouver le nombre d'objet ?\r
-   var nb = 0\r
-   for (var m in this.messagesRepond)\r
-      nb += 1\r
-   $("#posterMessage #repondA .nb").text(nb)\r
+   var nb = objectMemberCount(this.messagesRepond);\r
    \r
-   var boite = $("#posterMessage #repondA")\r
-   if (nb > 0) boite.show()\r
-   else boite.hide()\r
-}\r
+   $("#posterMessage #repondA .nb").text(nb);\r
+   \r
+   var boite = $("#posterMessage #repondA");\r
+   if (nb > 0) {\r
+      boite.show();\r
+   } else {\r
+      boite.hide();\r
+   }\r
+};\r
 \r
 /**\r
   * Affiche les messages auquel l'utilisateur souhaite répondre au sein des messages des conversations.\r
   * Utilisé lorsqu'une conversation est extraite.\r
   */\r
-Conversations.prototype.afficherMessagesRepondConversations = function()\r
-{\r
-   var expr = ""\r
-   for(var messId in this.messagesRepond)\r
-      expr += "#" + this.messagesRepond[messId].getId() + ","\r
-   $(expr).addClass("repondEnEvidence")\r
-}\r
+euphorik.Conversations.prototype.afficherMessagesRepondConversations = function() {\r
+   var expr = "";\r
+   objectEach(function(messId, mess) {\r
+      expr += "#" + mess.getId() + ",";\r
+   });\r
+   $(expr).addClass("repondEnEvidence");\r
+};\r
 \r
 /**\r
   * Crée un message JSON contenant le message demandant un rafraichissement.\r
   */\r
-Conversations.prototype.getJSONrafraichirMessages = function()\r
-{\r
+euphorik.Conversations.prototype.getJSONrafraichirMessages = function() {\r
    var mess =  {\r
       "message_count" : euphorik.conf.nbMessageAffiche,\r
       "main_page" : this.client.pagePrincipale,\r
       "conversations" : this.getJSONConversations(),\r
       "troll_id" : this.trollIdCourant\r
-   }\r
+   };\r
    \r
-   if (this.client.cookie != null) mess["cookie"] = this.client.cookie\r
-   mess["last_message_id"] = this.conversations[0].idDernierMessageAffiche\r
+   if (this.client.cookie) {\r
+      mess.cookie = this.client.cookie;\r
+   }\r
+   mess.last_message_id = this.conversations[0].idDernierMessageAffiche;\r
    \r
-   return mess\r
-}\r
+   return mess;\r
+};\r
 \r
-Conversations.prototype.getJSONConversations = function()\r
-{\r
-   var clientConv = []\r
+euphorik.Conversations.prototype.getJSONConversations = function() {\r
+   var clientConv = [];\r
    \r
-   for (var i = 0; i < this.client.conversations.length; i++)\r
-   {\r
-      clientConv.push(\r
-         {\r
-            root : this.client.conversations[i].root,\r
-            page : this.client.conversations[i].page,\r
-            last_message_id : this.conversations[i + 1] == undefined ? 0 : this.conversations[i + 1].idDernierMessageAffiche\r
-         }\r
-      )\r
-   }\r
-   return clientConv\r
-}\r
+   this.client.conversations.each(function(i, conv) {\r
+      clientConv.push({\r
+         root : conv.root,\r
+         page : conv.page,\r
+         last_message_id : this.conversations[i + 1] ? this.conversations[i + 1].idDernierMessageAffiche : 0\r
+      });\r
+   });\r
+   return clientConv;\r
+};\r
 \r
 /**\r
   * Ajoute un ensemble de messages puis les affiches.\r
@@ -203,26 +200,28 @@ Conversations.prototype.getJSONConversations = function()
   * @param numConversation le numéro de la conversation auquel appartiennent les messages\r
   * @return true si les messages on été ajoutés, false si la conversation n'existe pas et qu'il n'y a pas de message\r
   */\r
-Conversations.prototype.ajouterMessages = function(elements, numConversation)\r
-{\r
-   if (elements["messages"].length == 0)\r
-      return this.conversations[numConversation] != undefined\r
+euphorik.Conversations.prototype.ajouterMessages = function(elements, numConversation) {\r
+   if (!elements.messages.length) {\r
+      return this.conversations[numConversation] !== undefined;\r
+   }\r
 \r
-   for (var i = 0; i < elements["messages"].length; i++)\r
+   for (var i = 0; i < elements.messages.length; i++) {\r
       // si une nouvelle conversation a été créée alors on lui donne la racine\r
-      if(this.ajouterMessage(elements["messages"][i], numConversation))\r
-         this.conversations[numConversation].setRacine(elements["first"])\r
+      if (this.ajouterMessage(elements.messages[i], numConversation)) {\r
+         this.conversations[numConversation].setRacine(elements.first);\r
+      }\r
+   }\r
          \r
-   this.flush(numConversation)\r
+   this.flush(numConversation);\r
    \r
    // renseigne la conversation sur la page courante et si c'est la dernière\r
    this.conversations[numConversation].setPage(\r
-      numConversation == 0 ? this.client.pagePrincipale : this.client.conversations[numConversation - 1].page,\r
-      elements["last_page"]\r
-   )\r
+      numConversation === 0 ? this.client.pagePrincipale : this.client.conversations[numConversation - 1].page,\r
+      elements.last_page\r
+   );\r
    \r
-   return true\r
-}\r
+   return true;\r
+};\r
 \r
 /**\r
   * Création d'un nouveau message.\r
@@ -231,191 +230,188 @@ Conversations.prototype.ajouterMessages = function(elements, numConversation)
   * @param numConversation le numéro de la conversation, 0 = principale\r
   * @return true si une nouvelle conversation a été créée sinon false\r
   */\r
-Conversations.prototype.ajouterMessage = function(element, numConversation)\r
-{\r
-   var thisMessages = this\r
+euphorik.Conversations.prototype.ajouterMessage = function(element, numConversation) {\r
+   var thisMessages = this;\r
 \r
    // pas d'utilisation de jquery pour des raisons de performance      \r
-   var message = new Message(\r
-      this.client,\r
-      this.formateur,\r
-      element\r
-   )   \r
+   var message = \r
+      new euphorik.Message(\r
+         this.client,\r
+         this.formateur,\r
+         element\r
+      );\r
    \r
-   var nouvelleConversation = false\r
+   var nouvelleConversation = false;\r
    \r
-   if (this.conversations[numConversation] == null)\r
-   {\r
-      nouvelleConversation = true\r
+   if (!this.conversations[numConversation]) {\r
+      nouvelleConversation = true;\r
       this.nouvelleConversation(\r
          numConversation,\r
-         function(num) // fermeture de la conversation\r
-         {\r
-            thisMessages.supprimerConversation(num)\r
+         function(num) { // fermeture de la conversation\r
+            thisMessages.supprimerConversation(num);\r
          },\r
-         function(num) // insertion du lien vers la conversation\r
-         {\r
-               thisPage.util.replaceSelection(\r
-                  $("form#posterMessage input.message")[0],\r
-                  "{" + thisMessages.client.conversations[num-1].root.toString(36) + "}"\r
-               )\r
+         function(num) { // insertion du lien vers la conversation\r
+            thisPage.util.replaceSelection(\r
+               $("form#posterMessage input.message")[0],\r
+               "{" + thisMessages.client.conversations[num-1].root.toString(36) + "}"\r
+            );\r
          }\r
-      )\r
+      );\r
    }\r
 \r
-   this.conversations[numConversation].ajouterMessage(message)\r
-   return nouvelleConversation\r
-}\r
+   this.conversations[numConversation].ajouterMessage(message);\r
+   return nouvelleConversation;\r
+};\r
 \r
-Conversations.prototype.nouvelleConversation = function(num, funFermer, funLien)\r
+euphorik.Conversations.prototype.nouvelleConversation = function(num, funFermer, funLien)\r
 {\r
-   var thisMessages = this\r
+   var thisMessages = this;\r
 \r
-   this.conversations[num] = new Conversation(this, num)   \r
+   this.conversations[num] = new euphorik.Conversation(this, num);\r
+   \r
+   if (funFermer) {\r
+      this.conversations[num].eventFermer(funFermer);\r
+   }\r
    \r
-   if (funFermer != undefined)\r
-      this.conversations[num].eventFermer(funFermer)\r
-   if (funLien != undefined)\r
-      this.conversations[num].eventLien(funLien)\r
+   if (funLien) {\r
+      this.conversations[num].eventLien(funLien);\r
+   }\r
 \r
    this.conversations[num].setFunPage(\r
-      function(num) // page suivante\r
-      {\r
-         thisMessages.client.pageSuivante(num - 1)\r
-         thisMessages.rafraichirMessages(true)\r
+      function(num) { // page suivante\r
+         thisMessages.client.pageSuivante(num - 1);\r
+         thisMessages.rafraichirMessages(true);\r
       },\r
-      function(num) // page précédente\r
-      {\r
-         thisMessages.client.pagePrecedente(num - 1)\r
-         thisMessages.rafraichirMessages(true)\r
+      function(num) { // page précédente\r
+         thisMessages.client.pagePrecedente(num - 1);\r
+         thisMessages.rafraichirMessages(true);\r
       },\r
-      function(num) // retour à la page une\r
-      {\r
-         if (thisMessages.client.goPremierePage(num - 1))\r
-            thisMessages.rafraichirMessages(true)\r
+      function(num) // retour à la page une\r
+         if (thisMessages.client.goPremierePage(num - 1)) {\r
+            thisMessages.rafraichirMessages(true);\r
+         }\r
       }\r
-   )\r
+   );\r
    \r
-   this.ajusterLargeurConversations()\r
-}\r
+   this.ajusterLargeurConversations();\r
+};\r
 \r
 /**\r
   * Enlève une conversation.\r
   */\r
-Conversations.prototype.supprimerConversation = function(num)\r
-{\r
-   if (num <= 0 || num >= this.conversations.length) return // la numéro 0 ne peut être supprimé\r
-   this.conversations[num].supprimer()\r
+euphorik.Conversations.prototype.supprimerConversation = function(num) {\r
+   if (num <= 0 || num >= this.conversations.length) {\r
+      return; // la numéro 0 ne peut être supprimé\r
+   }\r
+   this.conversations[num].supprimer();\r
    \r
    // décalage TODO : supprimer le dernier élément \r
-   for (var i = num; i < this.conversations.length - 1; i++)\r
-   {\r
-      this.conversations[i] = this.conversations[i+1]\r
-      this.conversations[i].num -= 1\r
+   for (var i = num; i < this.conversations.length - 1; i++) {\r
+      this.conversations[i] = this.conversations[i+1];\r
+      this.conversations[i].num -= 1;\r
    }\r
-   this.conversations.pop()\r
-   this.ajusterLargeurConversations()\r
+   this.conversations.pop();\r
+   this.ajusterLargeurConversations();\r
       \r
-   this.client.supprimerConversation(num-1)   \r
+   this.client.supprimerConversation(num - 1);\r
       \r
-   this.rafraichirMessages(true)\r
-}\r
+   this.rafraichirMessages(true);\r
+};\r
 \r
 /**\r
   * Ajuste la largeur des conversations en fonction de leur nombre. modifie l'attribut CSS 'width'.\r
   */\r
-Conversations.prototype.ajusterLargeurConversations = function()\r
-{\r
+euphorik.Conversations.prototype.ajusterLargeurConversations = function() {\r
    // TODO : trouver mieux !\r
-   var largeurPourcent = (100 / this.conversations.length)   \r
+   var largeurPourcent = (100 / this.conversations.length);\r
    // obsolète !?\r
    // le "- 0.01" evite que IE se chie dessus lamentablement et affiche les conversations les unes au dessus des autres\r
    //if($.browser["msie"])\r
    //    largeurPourcent -= 0.05\r
-   $("#conversations td").css("width", largeurPourcent + "%")\r
-}\r
+   \r
+   $("#conversations td").css("width", largeurPourcent + "%");\r
+};\r
 \r
 /**\r
   * Demande à toutes les conversations de se flusher (afficher les messages non-affichés).\r
   */\r
-Conversations.prototype.flushAll = function()\r
-{\r
-   for (var i = 0; i < this.conversations.length; i++)\r
-      this.flush(i)\r
-}\r
+euphorik.Conversations.prototype.flushAll = function() {\r
+   for (var i = 0; i < this.conversations.length; i++) {\r
+      this.flush(i);\r
+   }\r
+};\r
 \r
 /**\r
   * Demande à une conversation de se flusher.\r
   */\r
-Conversations.prototype.flush = function(numConv)\r
-{   \r
-   this.conversations[numConv].flush()\r
-}\r
+euphorik.Conversations.prototype.flush = function(numConv) {   \r
+   this.conversations[numConv].flush();\r
+};\r
   \r
-Conversations.prototype.ouvrirConversation = function(racine)\r
-{\r
-   if (this.client.ajouterConversation(racine))\r
-      this.rafraichirMessages(true)\r
-   else\r
-      this.util.messageDialogue("Cette conversation est déjà ouverte")\r
-}\r
+euphorik.Conversations.prototype.ouvrirConversation = function(racine) {\r
+   if (this.client.ajouterConversation(racine)) {\r
+      this.rafraichirMessages(true);\r
+   } else {\r
+      this.util.messageDialogue("Cette conversation est déjà ouverte");\r
+   }\r
+};\r
 \r
-Conversations.prototype.viderMessages = function()\r
-{\r
-   for (var i = 0; i < this.conversations.length; i++)\r
-      this.conversations[i].viderMessages()\r
-}\r
+euphorik.Conversations.prototype.viderMessages = function() {\r
+   this.conversations.each(function(i, conv) {\r
+      conv.viderMessages();\r
+   });\r
+};\r
 \r
 /**\r
   * Met à jour les messages de manière continue.\r
   * (AJAX-Comet-style proof)\r
   * @param vider vide tous les messages avant d'afficher les nouveaux\r
   */\r
-Conversations.prototype.rafraichirMessages = function(vider)\r
-{\r
-   var thisMessages = this\r
+euphorik.Conversations.prototype.rafraichirMessages = function(vider) {\r
+   var thisMessages = this;\r
    \r
-   if (vider == undefined)\r
-      vider = false\r
+   vider = vider || false;\r
    \r
-   if (vider)\r
-      for (var i = 0; i < this.conversations.length; i++)\r
-         this.conversations[i].idDernierMessageAffiche = 0\r
+   if (vider) {\r
+      this.conversations.each(function(i, conv) {\r
+         conv.idDernierMessageAffiche = 0;\r
+      });\r
+   }\r
    \r
    this.pageEvent.waitEvent(\r
-      function() { return thisMessages.getJSONrafraichirMessages() },\r
+      function() { return thisMessages.getJSONrafraichirMessages(); },\r
       {\r
          "new_troll" :\r
-            function(data)\r
-            {   \r
-               thisMessages.trollIdCourant = data["troll_id"]\r
-               $("#trollCourant .troll").html(thisMessages.formateur.traitementComplet(data["content"])).unbind("click").click(\r
-                  function()\r
-                  {\r
-                     thisMessages.ouvrirConversation(data["message_id"])\r
+            function(data) {   \r
+               thisMessages.trollIdCourant = data.troll_id;\r
+               $("#trollCourant .troll").html(thisMessages.formateur.traitementComplet(data.content)).unbind("click").click(\r
+                  function() {\r
+                     thisMessages.ouvrirConversation(data.message_id);\r
                   }\r
-               )\r
+               );\r
                \r
-               $("#trollCourant .troll a[@rel*=lightbox]").lightBox()\r
+               $("#trollCourant .troll a[@rel*=lightbox]").lightBox();\r
             },\r
-         "new_messages" :   \r
-            function(data)\r
-            {\r
-               if (vider)\r
-                  thisMessages.viderMessages()\r
+         "new_messages" :\r
+            function(data) {\r
+               if (vider) {\r
+                  thisMessages.viderMessages();\r
+               }\r
+               \r
                // ajoute les messages reçus à leur conversation respective\r
-               for (var numConv = 0; numConv < data["conversations"].length; numConv++)\r
-               {\r
-                  if (!thisMessages.ajouterMessages(data["conversations"][numConv], numConv))\r
-                  {\r
-                     thisMessages.util.messageDialogue("La conversation {" + thisMessages.client.conversations[numConv -1].root.toString(36) + "} n'existe pas")\r
-                     thisMessages.client.supprimerConversation(numConv - 1) \r
+               data.conversations.each(function(numConv, conv) {\r
+                  if (!thisMessages.ajouterMessages(conv, numConv)) {\r
+                     thisMessages.util.messageDialogue("La conversation {" + thisMessages.client.conversations[numConv - 1].root.toString(36) + "} n'existe pas");\r
+                     thisMessages.client.supprimerConversation(numConv - 1);\r
                   }\r
+               });\r
+               \r
+               if (vider) {\r
+                  thisMessages.afficherMessagesRepondConversations();\r
                }\r
-               if (vider)\r
-                  thisMessages.afficherMessagesRepondConversations()\r
-               vider = false\r
+               \r
+               vider = false;\r
             }\r
       }\r
-   )\r
-}\r
+   );\r
+};\r