From: Greg Burri <greg.burri@gmail.com>
Date: Thu, 19 Jun 2008 20:26:26 +0000 (+0000)
Subject: MOD modification de la manière d'afficher les messages auquels l'utilisateur souhaite... 
X-Git-Tag: 1.1.0~121
X-Git-Url: https://git.euphorik.ch/?a=commitdiff_plain;h=e82b363f65838828948a62ef4452e752743ef661;p=euphorik.git

MOD modification de la manière d'afficher les messages auquels l'utilisateur souhaite répondre (première partie)
---

diff --git a/css/1/pageMinichat.css b/css/1/pageMinichat.css
index 722c3fb..9e9f601 100755
--- a/css/1/pageMinichat.css
+++ b/css/1/pageMinichat.css
@@ -202,6 +202,9 @@
 #page.minichat #conversations div.message.proprietaire .repondA {
 	background-image: url(../../img/css1/fleche_proprietaire.png);
 }
+#page.minichat #conversations div.message.repondEnEvidence {
+	background-color: #fce476
+}
 #page.minichat #conversations div.message.systeme {
 	background-color: #f0df95
 }
diff --git a/doc/TODO.txt b/doc/TODO.txt
index 1181d7f..7b784ec 100755
--- a/doc/TODO.txt
+++ b/doc/TODO.txt
@@ -1,11 +1,6 @@
 == TODO ==
 
-=== v1.1 ===
-* Faire l'update vers la version 2 pour les changements de la BD :
-   * suppression de #user.page_principale
-   * modification de #user.conversations (les pages ne sont plus mémorisées)
-   * ajout de #minichat.racine_id
-* La validation du profil fait revenir sur la page de chat
+=== v1.1 ===
 * Gestion de l'historique au niveau du navigateur (pouvoir revenir aux pages précédentes). Utiliser un plugin jQuery si possible. Voir du coté des ancres (fragment d'url) : #ancre.
    * http://www.euphorik.ch/#page=chat
 * Revoir le système de conversation : 
@@ -13,8 +8,7 @@
    * Tous les messages auquels on répond doivent faire partie de la conversation !
    * L'entête de la conversation contient le premier message (la racine)
    * Possibilité de fermer temporairement une conversation (la réduire sous la forme d'un onglet)
-   * lien vers une conversation : http://www.euphorik.ch/#conv=45
-   * faire un backup de la base v1 avant d'écrire la mise à jour 1->2, permet de revenir à la 1 si des modifs supplémentaires sont nécessaires
+   * lien vers une conversation : http://www.euphorik.ch/#conv=45
 * Mettre un icon (genre sablier ou truc qui tourne à la apple) lorsque le chat se charge (également lors d'un changement de page par exemple)
    * L'icon apparait tout en haut (absolute)
    * Simuler un réseau lent
@@ -29,8 +23,7 @@
    * Un élément se place entre le pseudo et la ligne de saisie du message et affiche le nombre de message auquel on répond
       * Lorsque l'on passe le curseur sur cet élément les messages sont affichés en dessous du bouton
    * Les messages auquels ont répond sont mis en évidence
-      * lors de la mise à jour des messages la liste est interrogé pour savoir s'il faut le mettre en évidence
-* Ne plus mémoriser le numéro de page dans le profile
+      * lors de la mise à jour des messages la liste est interrogé pour savoir s'il faut le mettre en évidence
 === v1.2 ===
 * Possibilité d'ajouter des messages dans son profil
    * Les messages sont soient publics soient privées (Par défaut privées)
@@ -223,6 +216,9 @@
 [ok] Elaborer une stratégie de mise à jour de la structure de la BD quand celle ci est modifiée (voir euphorik_bd:vers_version())   
 === v1.0.1 ===
 [ok] Ne pas pouvoir poster avec "<nick>"
+[ok] La validation du profil fait revenir sur la page de chat
+[ok] Ne plus mémoriser le numéro de page dans le profile
+[ok] faire un backup de la base v1 avant d'écrire la mise à jour 1->2, permet de revenir à la 1 si des modifs supplémentaires sont nécessaires
 
 === Bugs ===
 1 : Critique
diff --git a/js/euphorik.js b/js/euphorik.js
index 4e70844..cb7f71a 100755
--- a/js/euphorik.js
+++ b/js/euphorik.js
@@ -1080,7 +1080,7 @@ PageEvent.prototype.waitEvent = function(funSend, funsReceive)
       type: "POST",
       url: "request",
       dataType: "json",
-      // Obsolète (voir TODO)
+      // TODO : doit disparaitre
       timeout: 180000, // timeout de 3min. Gros HACK pas beau. FIXME problème décrit ici : http://groups.google.com/group/jquery-en/browse_thread/thread/8724e64af3333a76
       data: this.util.jsonVersAction(dataToSend),
       success:
diff --git a/js/pageMinichat.js b/js/pageMinichat.js
index ec745f3..b09b4ca 100755
--- a/js/pageMinichat.js
+++ b/js/pageMinichat.js
@@ -165,8 +165,12 @@ PageMinichat.prototype.decharger = function()
    $("body #smiles").remove()
 }
 
-PageMinichat.prototype.getJSONMessage = function(pseudo, message, repondA)
+PageMinichat.prototype.getJSONMessage = function(pseudo, message)
 {
+   repondA = []
+   for (var id in this.messages.messagesRepond)
+      repondA.push(parseInt(id)) // FIXME : une propriété ne peut pas être de type int ?
+      
    return {
       "action" : "put_message",
       "cookie" : this.client.cookie,
@@ -182,16 +186,6 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message)
 
    // (un pseudo vide est autorisé)
    pseudo = this.formateur.filtrerInputPseudo(pseudo)
-
-   // extraction des id des messages (en base 36 évidemment) auquels le user répond
-   var repondA = []
-   var tags = message.match(this.regexMessageTagMatch)
-   if (tags != null)
-   {
-      for(var i = 0; i < tags.length; i++)
-         repondA.push(parseInt(/\{(.*?)\}>/.exec(tags[i])[1], 36))
-      message = message.replace(this.regexMessageTagReplace, "")
-   }
    
    if (pseudo == conf.nickDefaut)
    {
@@ -223,12 +217,13 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message)
    }
    this.envoieMessageEnCours = true
    
-   ;; dumpObj(this.getJSONMessage(pseudo, message, repondA))
+   ;; dumpObj(this.getJSONMessage(pseudo, message))
    jQuery.ajax(
+         
       {
          url : "request", 
          type: "POST",
-         data : this.util.jsonVersAction(this.getJSONMessage(pseudo, message, repondA)),
+         data : this.util.jsonVersAction(this.getJSONMessage(pseudo, message)),
          dataType : "json",
          beforeSend : function(xmlHttpRequest)
          {
@@ -242,6 +237,7 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message)
             if(data["reply"] == "ok")
             {  
                $("form input.message").val("")
+               thisPageMinichat.messages.enleverMessagesRepond()
                         
                // met à jour la classe des messages auquel repond celui ci (c'est un peu de la triche) TODO : ya mieux ?
                for (var i = 0; i < repondA.length; i++)
@@ -293,10 +289,13 @@ function Reponse(id, pseudo, login)
   * @param pseudo
   * @param contenu
   */
-function Message(id, auteurId, date, pseudo, login, contenu)
+function Message(client, formateur, id, auteurId, racineId, date, pseudo, login, contenu)
 {
+   this.client = client
+   this.formateur = formateur
    this.id = id
    this.auteurId = auteurId
+   this.racineId = racineId
    this.date = date
    this.pseudo = pseudo
    this.login = login
@@ -312,6 +311,11 @@ function Message(id, auteurId, date, pseudo, login, contenu)
    this.repondA = {} // un ensemble de reponse (voir Reponse) indexé par l'id du message de la reponses
 }
 
+Message.prototype.getId = function()
+{
+   return "mess" + this.id.toString(36)
+}
+
 /**
   *
   */
@@ -369,6 +373,43 @@ Message.prototype.getConversation = function(messages)
    return messagesEnEvidence
 }
 
+Message.prototype.XHTML = function(messagePair)
+{
+   if (messagePair == undefined)
+      messagePair = true
+      
+   // construit l'identifiant de la personne
+   var identifiant = 
+      this.client.nickFormat == "nick" || this.login == "" ? this.formateur.traitementComplet(this.pseudo) : 
+      (this.client.nickFormat == "login" ? this.formateur.traitementComplet(message.login) : 
+      this.formateur.traitementComplet(this.pseudo) + "<span class=\"login\">(" + this.formateur.traitementComplet(message.login) +")</span>" )
+
+   var XHTMLrepondA = ""
+   var debut = true
+   for (var id in this.repondA)
+   {
+      if (!debut) XHTMLrepondA += ", "
+      XHTMLrepondA += this.formateur.traitementComplet(this.repondA[id].pseudo)
+      debut = false
+   }
+   if (XHTMLrepondA != "")
+      XHTMLrepondA = "<span class=\"repondA\">" + XHTMLrepondA + "</span><span class=\"delimitationRepondA\"></span>"
+   
+   return "<div id=\"" + this.getId() + "\" class=\"" + (messagePair ? "messagePair" : "messageImpair") + " message" +
+         (this.appartientAuClient ? " proprietaire" : "")  +
+         (this.clientARepondu ? " repondu" : "") +
+         (this.estUneReponse ? " reponse" : "") +
+         (this.systeme ? " systeme" : "") +
+         (this.ekMaster ? " ekMaster" : "") +
+      "\">" +
+         "<div class=\"outilsMess\"><div class=\"extraire\"></div><div class=\"extraireCompletement\"></div></div><span class=\"entete\">" +
+         "<span class=\"dateComplete\">[<span class=\"date\">" + this.date + "</span>]</span>" +
+         "<span class=\"pseudo\"><span class=\"id\" style=\"display: none\">" + this.auteurId + "</span class=\"ident\">" + identifiant + "</span></span><span class=\"delimitationEntete\"></span>" +
+         XHTMLrepondA +
+         "<span class=\"contenu\">" + this.formateur.traitementComplet(this.contenu, this.pseudo) + "</span>" +
+      "</div>"
+}
+
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 /**
@@ -409,9 +450,6 @@ function Conversation(num, util, formateur, client)
    }
 }
 
-/**
-  *
-  */
 Conversation.prototype.enleverMiseEnEvidence = function()
 {
    $("#conversations .message").removeClass("cache")
@@ -425,11 +463,11 @@ Conversation.prototype.colorerEntetes = function()
    for (var i = 0; i < this.messages.length; i++)
    {
       if (this.messages[i].appartientAuClient)
-         messagesProprietaire += "#mess" + this.messages[i].id.toString(36) + ","
+         messagesProprietaire += "#" + this.messages[i].getId() + ","
       else if (this.messages[i].estUneReponse)
-         messagesReponse += "#mess" + this.messages[i].id.toString(36) + ","
+         messagesReponse += "#" + this.messages[i].getId() + ","
       else if (this.messages[i].clientARepondu)
-         messagesRepondu += "#mess" + this.messages[i].id.toString(36) + ","
+         messagesRepondu += "#" + this.messages[i].getId() + ","
    }
    $(messagesReponse).addClass("reponse")
    $(messagesRepondu).addClass("repondu")
@@ -536,9 +574,10 @@ Conversation.prototype.viderMessages = function()
   * Après l'ajout d'un ou plusieurs message cette méthode est appelée afin
   * d'afficher les messages non-affichés.
   * FIXME : méthode super lourde, à optimiser.
-  * @param funClickExtract fonction (fun(numMess)) appellée lors du clic sur un bouton "extraire"
+  * @param funClickExtract fonction (fun(numMess)) appelée lors du clic sur un bouton "extraire"
+  * @param funClickMessage fonction (fun(numMess)) appelée lors du clic sur un message
   */
-Conversation.prototype.flush = function(funClickOuvrirConv)
+Conversation.prototype.flush = function(funClickOuvrirConv, funClickMessage)
 {
    var thisConversation = this
 
@@ -551,42 +590,8 @@ Conversation.prototype.flush = function(funClickOuvrirConv)
    var XHTML = ""
    for (var i = 0; i < this.messages.length; i++)
       if (this.messages[i].id > this.idDernierMessageAffiche)
-      {      
-         var message = this.messages[i]
-        
-         // construit l'identifiant de la personne
-         var identifiant = 
-            this.client.nickFormat == "nick" || message.login == "" ? this.formateur.traitementComplet(message.pseudo) : 
-            (this.client.nickFormat == "login" ? this.formateur.traitementComplet(message.login) : 
-            this.formateur.traitementComplet(message.pseudo) + "<span class=\"login\">(" + this.formateur.traitementComplet(message.login) +")</span>" )
-         
-         var XHTMLrepondA = ""
-         var debut = true
-         for (var id in message.repondA)
-         {
-            if (!debut) XHTMLrepondA += ", "
-            ;; dumpObj(message.repondA.count)
-            XHTMLrepondA += this.formateur.traitementComplet(message.repondA[id].pseudo)
-            debut = false
-         }
-         if (XHTMLrepondA != "")
-            XHTMLrepondA = "<span class=\"repondA\">" + XHTMLrepondA + "</span><span class=\"delimitationRepondA\"></span>"
-         
-         XHTML += 
-            "<div id=\"mess" + message.id.toString(36) + "\" class=\"" + (messagePair ? "messagePair" : "messageImpair") + " message" +
-               (this.messages[i].appartientAuClient ? " proprietaire" : "")  +
-               (this.messages[i].clientARepondu ? " repondu" : "") +
-               (this.messages[i].estUneReponse ? " reponse" : "") +
-               (this.messages[i].systeme ? " systeme" : "") +
-               (this.messages[i].ekMaster ? " ekMaster" : "") +
-            "\">" +
-               "<div class=\"outilsMess\"><div class=\"extraire\"></div><div class=\"extraireCompletement\"></div></div><span class=\"entete\">" +
-               "<span class=\"dateComplete\">[<span class=\"date\">" + message.date + "</span>]</span>" +
-               "<span class=\"pseudo\"><span class=\"id\" style=\"display: none\">" + message.auteurId + "</span class=\"ident\">" + identifiant + "</span></span><span class=\"delimitationEntete\"></span>" +
-               XHTMLrepondA +
-               "<span class=\"contenu\">" + this.formateur.traitementComplet(message.contenu, message.pseudo) + "</span>" +
-            "</div>"
-            
+      {
+         XHTML += this.messages[i].XHTML(messagePair)
          messagePair = !messagePair
       }
    
@@ -608,8 +613,7 @@ Conversation.prototype.flush = function(funClickOuvrirConv)
          thisConversation.util.infoBulle("Extraction de la conversation complète", $(".extraireCompletement", this))
          
          // l'id du message
-         var idMess36 = $(this).attr("id").substr(4)
-         var idMess = parseInt(idMess36, 36)
+         var idMess = parseInt($(this).attr("id").substr(4), 36)
          
          $(this).click(
             function(event)
@@ -622,13 +626,14 @@ Conversation.prototype.flush = function(funClickOuvrirConv)
                   funClickOuvrirConv(idMess)
                   return
                }
-            
-               var valCourant = $("input.message").val()
-               if (valCourant == undefined) valCourant = ""
-               var tag = $(".pseudo span.ident", this).text()  + "{" + idMess36 + "}" + ">"
-               if (valCourant.indexOf(tag, 0) == -1)
-                  $("input.message").val(tag + " " + valCourant)
-               thisConversation.util.setCaretToEnd($("form input.message")[0])
+               
+               if ($(event.target).is(".extraireCompletement"))
+               {
+                  funClickOuvrirConv(thisConversation.messagesParId[idMess].racineId)
+                  return
+               }
+               
+               funClickMessage(thisConversation.messagesParId[idMess])
             }
          )
          
@@ -763,15 +768,17 @@ Conversation.prototype.supprimer = function()
 
 /**
   * Représente l'ensemble des messages affichés.
+  * TODO : renommer cette "classe" en 'Conversations'
   */
 function Messages(client, formateur, util)
 {
-   var thisMessages = this
-
    this.client = client
    this.formateur = formateur
    this.util = util
    
+   // un ensemble des messages (id) auquel l'utilisateur répond (vider après l'envoie du message courant)
+   this.messagesRepond = {}
+   
    this.conversations = new Array() // les conversations, la première représente la conversation principale
    
    this.nouvelleConversation(0)
@@ -781,6 +788,61 @@ function Messages(client, formateur, util)
    this.pageEvent = new PageEvent("chat", this.util)
 }
 
+/**
+  * Si le message 
+  */
+Messages.prototype.toggleMessageRepond = function(mess)
+{   
+   // est-ce que l'on répond déjà à ce message ? si oui alors on l'enlève de la liste
+   if (mess.id in this.messagesRepond)
+   {
+      $("#" + mess.getId()).removeClass("repondEnEvidence")
+      delete this.messagesRepond[mess.id]
+      return
+   }
+   
+   // est-ce que le message fait partie de la même conversation que les autres messages ?
+   // TODO : solution plus élégante pour prendre un mess parmis messagesRepond !?
+   var mess2
+   for(mess2 in this.messagesRepond){ break; }
+   mess2 = this.messagesRepond[mess2]
+   
+   if (mess2 != undefined && mess2.racineId != mess.racineId)
+   {
+      this.util.messageDialogue("Impossible de répondre à deux messages ne faisant pas partie de la même conversation")
+      return
+   }
+   
+   this.messagesRepond[mess.id] = mess
+   $("#" + mess.getId()).addClass("repondEnEvidence")
+}
+
+/**
+  * Enlève tous les messages auquel l'utilisateur souhaite répond.
+  */
+Messages.prototype.enleverMessagesRepond = function()
+{
+   this.messagesRepond = {}
+   $(".message").removeClass("repondEnEvidence")
+}
+
+
+/**
+  * Définit un message comme n'y répondant plus.
+  */
+/*Messages.prototype.enleverMessageRepond = function(id)
+{
+
+}*/
+
+/**
+  * Définit un message comme y répondant.
+  */
+/*Messages.prototype.ajouterMessageRepond = function(id)
+{
+
+}*/
+
 /**
   * Crée un message JSON contenant le message demandant un rafraichissement.
   */
@@ -854,8 +916,11 @@ Messages.prototype.ajouterMessage = function(element, numConversation)
    var id = element["id"]
    
    var message = new Message(
+      this.client,
+      this.formateur,
       id,      
       element["user_id"],
+      element["root"],
       element["date"],
       element["nick"],
       element["login"],
@@ -973,9 +1038,15 @@ Messages.prototype.flush = function(numConv)
    var thisMessages = this
    
    this.conversations[numConv].flush(
-      function(racine) // appelé lorsqu'un utilisateur click sur un lien vers une conversation
+      function(racine) // appelé lors d'un clic sur un lien vers une conversation
       {
          thisMessages.ouvrirConversation(racine)
+      },
+      // 'Mess' correspond à un objet de type 'Message'
+      function(Mess) // appelé lors d'un clic sur un message
+      {
+         // met ou enlève la mise en evidence du message
+         thisMessages.toggleMessageRepond(Mess)
       }
    )
 }
diff --git a/js/pageProfile.js b/js/pageProfile.js
index 5c78846..14ca09c 100755
--- a/js/pageProfile.js
+++ b/js/pageProfile.js
@@ -82,7 +82,10 @@ PageProfile.prototype.chargerProfile = function()
          if(!thisPage.client.flush())
             thisPage.util.messageDialogue("Impossible de mettre à jour votre profile, causes inconnues", messageType.erreur)
          else
-            thisPage.util.messageDialogue("Votre profile a été mis à jour")
+         {
+            thisPage.util.messageDialogue("Votre profile a été mis à jour")
+            thisPage.pages.afficherPage("minichat")
+         }
       }
    )
 }
diff --git a/js/pageRegister.js b/js/pageRegister.js
index 4d107e9..ca1ddb3 100755
--- a/js/pageRegister.js
+++ b/js/pageRegister.js
@@ -73,7 +73,6 @@ PageRegister.prototype.charger = function()
             thisPage.util.messageDialogue("Les mots de passes ne correspondent pas")
          else if(thisPage.client.enregistrement(login, thisPage.util.md5(password)))
          {   
-            // TODO : avertir que l'enregistrement s'est bien déroulé
             thisPage.util.messageDialogue("Enregistrement réussi")
             thisPage.pages.afficherPage("minichat")
          }