MOD fin du système de page et cleanage du code

diff --git a/css/1/pageMinichat.css b/css/1/pageMinichat.css
index d965d6a..e24092c 100755 (executable)
--- a/css/1/pageMinichat.css
+++ b/css/1/pageMinichat.css
@@ -176,11 +176,26 @@
 }      
 
 #page.minichat #conversations .titre .next,
 }      
 
 #page.minichat #conversations .titre .next,

-#page.minichat #conversations .titre .prev
-{
-       display:none
+#page.minichat #conversations .titre .prev {
+       display: none;

 }
 }

-       \r
+
+#page.minichat #conversations .titre .next,
+#page.minichat #conversations .titre .prev,
+#page.minichat #conversations .titre .numPage {
+       background-color: #42380b;
+       cursor: pointer;
+       padding-right: 5px;
+       padding-left: 5px;
+}
+
+#page.minichat #conversations .titre .next:hover,
+#page.minichat #conversations .titre .prev:hover,
+#page.minichat #conversations .titre .numPage:hover {
+       background-color: #7c6e2e
+}
+
+\r

 #page.minichat #pages {\r
        margin-top: 10px;\r
 }\r
 #page.minichat #pages {\r
        margin-top: 10px;\r
 }\r

diff --git a/js/euphorik.js b/js/euphorik.js
index 8487f77..4b76b93 100755 (executable)
--- a/js/euphorik.js
+++ b/js/euphorik.js
@@ -53,11 +53,12 @@ String.prototype.rtrim = function()
        return this.replace(/\s+$/, "");\r
 }\r
 \r
        return this.replace(/\s+$/, "");\r
 }\r
 \r

-String.prototype.dump = function()\r
-{\r
+String.prototype.dump = function(titre)\r
+{
+   titre = titre == undefined ? "" : titre\r

    if (typeof dump != "undefined")\r
    {\r
    if (typeof dump != "undefined")\r
    {\r

-      dump("\n--- EUPHORIK.CH ---\n")\r
+      dump("\n--- EUPHORIK.CH : " + titre + " ---\n")\r

       dump(this)\r
       dump("\n------\n")\r
    }\r
       dump(this)\r
       dump("\n------\n")\r
    }\r


@@ -123,25 +124,29 @@ Util.prototype.serializeXML = function(documentXML)
       return documentXML.xml\r
 }\r
 
       return documentXML.xml\r
 }\r
 

+var documentXMLBase = undefined // singleton 

 Util.prototype.creerDocumentXMLAction = function()
 Util.prototype.creerDocumentXMLAction = function()

-{\r
-   if (document.implementation && document.implementation.createDocument)\r
-   {\r
-      // var doc = document.implementation.createDocument("", "action", null)
-      var parser = new DOMParser();
-      var doc =  parser.parseFromString("<?xml version=\"1.0\" encoding=\"utf-8\" ?>\n<action/>", "text/xml")
-      //alert(this.serializeXML(doc))
-      return doc\r
-   }\r
-   else if (window.ActiveXObject)\r
+{
+   // FIXME : essayer de garder le doc de base en cache (singleton) et d'en retourner une copie
+   if (true)//documentXMLBase == undefined)

    {\r
    {\r

-      var doc = new ActiveXObject("MSXML2.DOMDocument") //("Microsoft.XMLDOM")\r
-      doc.appendChild(doc.createElement("action"));\r
-      //doc.loadXML("<action></action>")\r
-      //alert(doc.documentElement)\r
-      //doc.createElement("action")\r
-      return doc\r
-   }   
+      if (document.implementation && document.implementation.createDocument)\r
+      {\r
+         // var doc = document.implementation.createDocument("", "action", null)
+         var parser = new DOMParser();
+         documentXMLBase =  parser.parseFromString("<?xml version=\"1.0\" encoding=\"utf-8\" ?>\n<action/>", "text/xml")
+         //alert(this.serializeXML(doc))\r
+      }\r
+      else if (window.ActiveXObject)\r
+      {\r
+         documentXMLBase = new ActiveXObject("MSXML2.DOMDocument") //("Microsoft.XMLDOM")\r
+         documentXMLBase.appendChild(doc.createElement("action"));\r
+         //doc.loadXML("<action></action>")\r
+         //alert(doc.documentElement)\r
+         //doc.createElement("action")\r
+      }
+   }
+   return documentXMLBase

 }
 
 Util.prototype.xmlVersAction = function(xml)
 }
 
 Util.prototype.xmlVersAction = function(xml)


@@ -298,7 +303,7 @@ Formateur.prototype.traitementComplet = function(M, pseudo)
 }
 \r
 /**\r
 }
 \r
 /**\r

-  * FIXME : Cette méthode est attrocement lourde !!\r
+  * FIXME : Cette méthode est attrocement lourde ! A optimiser.\r

   */
 Formateur.prototype.traiterSmiles = function(M)
 {  
   */
 Formateur.prototype.traiterSmiles = function(M)
 {  


@@ -434,21 +439,43 @@ Client.prototype.setCss = function(css)
 }
 
 Client.prototype.pageSuivante = function(numConv)
 }
 
 Client.prototype.pageSuivante = function(numConv)

+{
+   if (numConv < 0 && this.pagePrincipale > 1)
+      this.pagePrincipale -= 1
+   else if (this.conversations[numConv].page > 1)
+      this.conversations[numConv].page -= 1
+   this.flush(false)
+}
+
+Client.prototype.pagePrecedente = function(numConv)

 {
    if (numConv < 0)
       this.pagePrincipale += 1
 {
    if (numConv < 0)
       this.pagePrincipale += 1

-   else
+   else 

       this.conversations[numConv].page += 1
    this.flush(false)
 }
 
       this.conversations[numConv].page += 1
    this.flush(false)
 }
 

-Client.prototype.pagePrecedente = function(numConv)
+/**
+  * Définit la première page pour la conversation donnée.
+  * @return true si la page a changé sinon false
+  */
+Client.prototype.goPremierePage = function(numConv)

 {
    if (numConv < 0)
 {
    if (numConv < 0)

-      this.pagePrincipale -= 1
-   else 
-      this.conversations[numConv].page -= 1
+   {
+      if (this.pagePrincipale == 1)
+         return false
+      this.pagePrincipale = 1
+   }
+   else
+   {
+      if (this.conversations[numConv].page == 1)
+         return false
+      this.conversations[numConv].page = 1
+   }

    this.flush(false)
    this.flush(false)

+   return true

 }
 
 /**
 }
 
 /**


@@ -551,6 +578,10 @@ Client.prototype.getXMLEnregistrement = function(login, password)
    return XMLDocument   
 }
 
    return XMLDocument   
 }
 

+/**
+  * Sérialize le profile en XML.
+  * TODO : méthode assez lourde, 3.25ms de moyenne
+  */

 Client.prototype.getXMLProfile = function()
 {
    var XMLDocument = this.util.creerDocumentXMLAction()
 Client.prototype.getXMLProfile = function()
 {
    var XMLDocument = this.util.creerDocumentXMLAction()


@@ -581,7 +612,7 @@ Client.prototype.getXMLProfile = function()
    XMLDocument.documentElement.appendChild(nodeCSS)
    
    var nodePagePrincipale = XMLDocument.createElement("pagePrincipale")
    XMLDocument.documentElement.appendChild(nodeCSS)
    
    var nodePagePrincipale = XMLDocument.createElement("pagePrincipale")

-   nodePagePrincipale.appendChild(XMLDocument.createTextNode(this.pagePrincipale))
+   nodePagePrincipale.appendChild(XMLDocument.createTextNode(this.pagePrincipale < 1 ? 1 : this.pagePrincipale))

    XMLDocument.documentElement.appendChild(nodePagePrincipale)
    
    // mémorise les conversations affichées
    XMLDocument.documentElement.appendChild(nodePagePrincipale)
    
    // mémorise les conversations affichées


@@ -729,7 +760,7 @@ Client.prototype.connexion = function(action)
          success:
             function(data)
             {
          success:
             function(data)
             {

-               //thisClient.util.serializer.serializeToString(data).dump()
+               //thisClient.util.serializer.serializeToString(data).dump("Charger client")

                thisClient.chargerDonnees(data)
             }
       }
                thisClient.chargerDonnees(data)
             }
       }


@@ -762,7 +793,7 @@ Client.prototype.chargerDonnees = function(data)
       
       // la page de la conversation principale
       var tmp = jQuery("pagePrincipale", data.documentElement)
       
       // la page de la conversation principale
       var tmp = jQuery("pagePrincipale", data.documentElement)

-      this.pagePrincipale = tmp.length < 1 ? 1 : tmp.text()
+      this.pagePrincipale = tmp.length < 1 ? 1 : parseInt(tmp.text())

       
       // met à jour la css
       if (this.css != "")
       
       // met à jour la css
       if (this.css != "")


@@ -792,7 +823,7 @@ Client.prototype.flush = function(async)
       async = true
 
    thisClient = this
       async = true
 
    thisClient = this

-   //thisClient.util.log(this.util.xmlVersAction(this.getXMLProfile()).action)      
+   //this.util.xmlVersAction(this.getXMLProfile()).action.dump("Flush client")      

    jQuery.ajax(
       {
          async: async,
    jQuery.ajax(
       {
          async: async,

diff --git a/js/pageMinichat.js b/js/pageMinichat.js
index c41aa6a..ed1a46c 100755 (executable)
--- a/js/pageMinichat.js
+++ b/js/pageMinichat.js
@@ -175,19 +175,7 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message)
       {\r
          this.util.messageDialogue("login impossible")\r
          return\r
       {\r
          this.util.messageDialogue("login impossible")\r
          return\r

-      }
-
-   //this.util.log(this.util.xmlVersAction(this.getXMLmessage(pseudo, message, repondA)).action)\r
-   //alert(this.util.xmlVersAction(this.getXMLMessage(pseudo, message, repondA)).action)
-   /* Obsolète\r
-   jQuery.post("request",  this.util.xmlVersAction(this.getXMLMessage(pseudo, message, repondA)),
-      function(data, textStatus)
-      {
-         // TODO : traiter les erreurs
-         //alert(data)
-         jQuery("form input.message").val("")
-      }
-   )*/\r
+      }\r

    \r
    jQuery.ajax(\r
       {\r
    \r
    jQuery.ajax(\r
       {\r


@@ -305,7 +293,6 @@ Message.prototype.getConversation = function(messages)
   */
 function Conversation(num, util, formateur, funFermer)
 {
   */
 function Conversation(num, util, formateur, funFermer)
 {

-   //alert("ok")

    var thisConversation = this
 
    this.num = num // peut changer au cours de la vie de la conversation
    var thisConversation = this
 
    this.num = num // peut changer au cours de la vie de la conversation


@@ -320,7 +307,7 @@ function Conversation(num, util, formateur, funFermer)
       '<div id="' + this.getId() + '" class="conversation">\
       <div class="titre">' +
          (funFermer == undefined ? '' : '<div class="fermer">x</div>') +
       '<div id="' + this.getId() + '" class="conversation">\
       <div class="titre">' +
          (funFermer == undefined ? '' : '<div class="fermer">x</div>') +

-         '<span class="prev">&lt;</span><span class="numPage">1</span><span class="next">&gt;</span>\
+         '<span class="next">&lt;</span><span class="numPage">1</span><span class="prev">&gt;</span>\

          </div>\
       </div>'
    )
          </div>\
       </div>'
    )


@@ -347,24 +334,34 @@ function Conversation(num, util, formateur, funFermer)
 
 /**
   * Défini les fonctions (callback) appelées lorsque l'on change de page.
 
 /**
   * Défini les fonctions (callback) appelées lorsque l'on change de page.

-  * @funNext appelé lorsque l'on revient une page en arrière
-  * @funPrev appelé lorsque l'on passe à la page suisvante

   * @pageCourante la page courante
   * @dernierePage true si c'est la dernière page sinon false
   */
   * @pageCourante la page courante
   * @dernierePage true si c'est la dernière page sinon false
   */

-Conversation.prototype.initPage = function(funNext, funPrev, pageCourante, dernierePage)
+Conversation.prototype.setPage = function(pageCourante, dernierePage)

 {
 {

-   var thisConversation = this
+   jQuery("#conversations #" + this.getId() + " .numPage").text(pageCourante)
+   jQuery("#conversations #" + this.getId() + " .next").css("display", pageCourante == 1 ? "none" : "inline")
+   jQuery("#conversations #" + this.getId() + " .prev").css("display", dernierePage ? "none" : "inline")
+}

 
 

+/**
+  * @funNext appelé lorsque l'on passe à la page suivante (de 2 à 1 par exemple)
+  * @funPrev appelé lorsque l'on passe à la page précédente (de 1 à 2 par exemple)
+  * @funReset appelé lorsque l'on souhaite revenir à la page une
+  */
+Conversation.prototype.setFunPage = function(funNext, funPrev, funReset)
+{
+   var thisConversation = this
+   

    jQuery("#conversations #" + this.getId() + " .next").click(
       function() { funNext(thisConversation.num) }
    )
    jQuery("#conversations #" + this.getId() + " .prev").click(
       function() { funPrev(thisConversation.num) }
    )
    jQuery("#conversations #" + this.getId() + " .next").click(
       function() { funNext(thisConversation.num) }
    )
    jQuery("#conversations #" + this.getId() + " .prev").click(
       function() { funPrev(thisConversation.num) }
    )

-   
-   jQuery("#conversations #" + this.getId() + " .next").css("display", pageCourante == 1 ? "none" : "inline")
-   jQuery("#conversations #" + this.getId() + " .prev").css("display", dernierePage ? "none" : "inline")
+   jQuery("#conversations #" + this.getId() + " .numPage").click(
+      function() { funReset(thisConversation.num) }
+   )

 }
 
 /**
 }
 
 /**


@@ -414,7 +411,7 @@ Conversation.prototype.flush = function(funClickExtract)
          XHTML += 
             "<div id=\"" + message.id + "\" class=\"" + (parseInt(message.id, 36) % 2 == 0 ? "messagePair" : "messageImpair") + " message" +
                (this.messages[i].appartientAuClient ? " proprietaire" : "")  + (this.messages[i].clientARepondu ? " repondu" : "") + (this.messages[i].estUneReponse ? " reponse" : "") + (this.messages[i].systeme ? " systeme" : "") +
          XHTML += 
             "<div id=\"" + message.id + "\" class=\"" + (parseInt(message.id, 36) % 2 == 0 ? "messagePair" : "messageImpair") + " message" +
                (this.messages[i].appartientAuClient ? " proprietaire" : "")  + (this.messages[i].clientARepondu ? " repondu" : "") + (this.messages[i].estUneReponse ? " reponse" : "") + (this.messages[i].systeme ? " systeme" : "") +

-            "\" >" +
+            "\" &gt;" +

                "<div class=\"extraire\">&gt;</div>" +
                "[<span class=\"date\">" + message.date + "</span>]" +
                "<span class=\"pseudo\">" + this.formateur.traitementComplet(message.pseudo) + "</span>:" +
                "<div class=\"extraire\">&gt;</div>" +
                "[<span class=\"date\">" + message.date + "</span>]" +
                "<span class=\"pseudo\">" + this.formateur.traitementComplet(message.pseudo) + "</span>:" +


@@ -422,9 +419,8 @@ Conversation.prototype.flush = function(funClickExtract)
                "<span class=\"contenu\">" + (message.systeme ? this.formateur.remplacerBalisesHTML(message.contenu) : this.formateur.traitementComplet(message.contenu, message.pseudo)) + "</span>" +
             "</div>"
       }
                "<span class=\"contenu\">" + (message.systeme ? this.formateur.remplacerBalisesHTML(message.contenu) : this.formateur.traitementComplet(message.contenu, message.pseudo)) + "</span>" +
             "</div>"
       }

-   //alert(this.getId())
+      

    jQuery("#conversations #" + this.getId()).prepend(XHTML)
    jQuery("#conversations #" + this.getId()).prepend(XHTML)

-   //alert(jQuery("#conversations").text())

    
    // mise à jour des images (LightBox) après l'ajout de message
    if (myLightbox != null)
    
    // mise à jour des images (LightBox) après l'ajout de message
    if (myLightbox != null)


@@ -438,13 +434,6 @@ Conversation.prototype.flush = function(funClickExtract)
    jQuery("#conversations #" + this.getId() + " .message").filter(function(){return parseInt(jQuery(this).attr("id"), 36) > parseInt(idDernierMessageAffiche, 36)}).each(
       function()
       {
    jQuery("#conversations #" + this.getId() + " .message").filter(function(){return parseInt(jQuery(this).attr("id"), 36) > parseInt(idDernierMessageAffiche, 36)}).each(
       function()
       {

-         /*jQuery(".extraire", this).click(
-            function(event)
-            {
-               funClickExtract()
-               return false
-            }
-         )*/

          jQuery(this).click(
             function(event)
             {
          jQuery(this).click(
             function(event)
             {


@@ -531,7 +520,7 @@ function Messages(client, formateur, util)
    this.util = util
    
    this.conversations = new Array() // les conversations, la première représente la conversation principale
    this.util = util
    
    this.conversations = new Array() // les conversations, la première représente la conversation principale

-   this.conversations[0] = new Conversation(0, this.util, this.formateur)
+   this.nouvelleConversation(0)

 
    this.idDernierMessage = null // l'id du dernier message connu
    
 
    this.idDernierMessage = null // l'id du dernier message connu
    


@@ -546,8 +535,6 @@ Messages.prototype.getXMLrafraichirMessages = function()
 {
    var XMLDocument = this.util.creerDocumentXMLAction()
    XMLDocument.documentElement.setAttribute("name", "refreshMessages")
 {
    var XMLDocument = this.util.creerDocumentXMLAction()
    XMLDocument.documentElement.setAttribute("name", "refreshMessages")

-   \r
-   //alert(this.util.serializer.serializeToString(XMLDocument))\r

    
    if (this.client.identifie())
    {
    
    if (this.client.identifie())
    {


@@ -567,26 +554,6 @@ Messages.prototype.getXMLrafraichirMessages = function()
    nodeNombreMessage.appendChild(XMLDocument.createTextNode(conf.nbMessageAffiche))
    XMLDocument.documentElement.appendChild(nodeNombreMessage)
    
    nodeNombreMessage.appendChild(XMLDocument.createTextNode(conf.nbMessageAffiche))
    XMLDocument.documentElement.appendChild(nodeNombreMessage)
    

-   /*var nodePage = XMLDocument.createElement("page")
-   nodePage.appendChild(XMLDocument.createTextNode(this.conversations[0].page))
-   XMLDocument.documentElement.appendChild(nodePage)*/
-   
-   // les conversations
-   /* Obsolète, le serveur les recupères directement auprès des informations utilisateurs
-   for (var i = 0; i < this.client.conversations.length; i++)
-   {
-      var nodeConversation = XMLDocument.createElement("conversation")
-      XMLDocument.documentElement.appendChild(nodeConversation)
-      
-      var nodeRacine = XMLDocument.createElement("racine")
-      nodeRacine.appendChild(XMLDocument.createTextNode(this.client.conversations[i].racine))
-      nodeConversation.appendChild(nodeRacine)
-      
-      var nodePageConv = XMLDocument.createElement("page")
-      nodePageConv.appendChild(XMLDocument.createTextNode(this.client.conversations[i].page))
-      nodeConversation.appendChild(nodePageConv)
-   }*/
-   

    return XMLDocument;
 }
 
    return XMLDocument;
 }
 


@@ -614,10 +581,8 @@ Messages.prototype.ajouterMessage = function(element, numConversation)
 
    // pas d'utilisation de jquery pour des raisons de performance
    var id = element.getAttribute("id")
 
    // pas d'utilisation de jquery pour des raisons de performance
    var id = element.getAttribute("id")

-   //alert(parseInt(null, 36) +" " + parseInt(this.idDernierMessage, 36))

    if (this.idDernierMessage == null || parseInt(id, 36) > parseInt(this.idDernierMessage, 36))
       this.idDernierMessage = id
    if (this.idDernierMessage == null || parseInt(id, 36) > parseInt(this.idDernierMessage, 36))
       this.idDernierMessage = id

-   //alert(this.idDernierMessage)

    
    var message = new Message(
       id,      \r
    
    var message = new Message(
       id,      \r


@@ -634,19 +599,45 @@ Messages.prototype.ajouterMessage = function(element, numConversation)
    
    if (this.conversations[numConversation] == null)
    {
    
    if (this.conversations[numConversation] == null)
    {

-      this.conversations[numConversation] = new Conversation(numConversation, this.util, this.formateur,
+      this.nouvelleConversation(
+         numConversation,

          function(num) // fermeture de la conversation
          {
             thisMessages.supprimerConversation(num)
          }
       )
          function(num) // fermeture de la conversation
          {
             thisMessages.supprimerConversation(num)
          }
       )

-      
-      this.ajusterLargeurConversations()

    }
 
    this.conversations[numConversation].ajouterMessage(message)
 }
 
    }
 
    this.conversations[numConversation].ajouterMessage(message)
 }
 

+Messages.prototype.nouvelleConversation = function(num, funFermer)
+{
+   var thisMessages = this
+
+   this.conversations[num] = new Conversation(num, this.util, this.formateur, funFermer)
+
+   this.conversations[num].setFunPage(
+      function(num) // page suivante
+      {
+         thisMessages.client.pageSuivante(num - 1)
+         thisMessages.rafraichirMessages(true)
+      },
+      function(num) // page précédente
+      {
+         thisMessages.client.pagePrecedente(num - 1)
+         thisMessages.rafraichirMessages(true)
+      },
+      function(num) // retour à la page une
+      {
+         if (thisMessages.client.goPremierePage(num - 1))
+            thisMessages.rafraichirMessages(true)
+      }
+   )
+   
+   this.ajusterLargeurConversations()
+}
+

 /**
   * Enlève une conversation.
   */
 /**
   * Enlève une conversation.
   */


@@ -730,8 +721,7 @@ Messages.prototype.rafraichirMessages = function(vider)
    if (vider)
       this.idDernierMessage = null\r
       
    if (vider)
       this.idDernierMessage = null\r
       

-   this.util.xmlVersAction(this.getXMLrafraichirMessages()).action.dump()
-   //alert(this.util.xmlVersAction(this.getXMLrafraichirMessages()).action)
+   //this.util.xmlVersAction(this.getXMLrafraichirMessages()).action.dump()

    this.attenteCourante = jQuery.ajax({
          type: "POST",
          url: "request",
    this.attenteCourante = jQuery.ajax({
          type: "POST",
          url: "request",


@@ -740,13 +730,11 @@ Messages.prototype.rafraichirMessages = function(vider)
          success:
             function(data)
             {            \r
          success:
             function(data)
             {            \r

-               thisMessages.util.serializer.serializeToString(data).dump()\r
+               //thisMessages.util.serializer.serializeToString(data).dump()\r

             
                if (vider)
                   thisMessages.viderMessages()
                
             
                if (vider)
                   thisMessages.viderMessages()
                

-               //thisMessages.MAJPages(parseInt(jQuery("nbPage", data.documentElement).text()))
-               

                // ajoute les messages reçu à leur conversation respective
                var numConv = 0
                jQuery("conversation", data.documentElement).each(
                // ajoute les messages reçu à leur conversation respective
                var numConv = 0
                jQuery("conversation", data.documentElement).each(


@@ -760,18 +748,10 @@ Messages.prototype.rafraichirMessages = function(vider)
                            messages.push(this)
                         }
                      )
                            messages.push(this)
                         }
                      )

-                     //alert("ajoutMess : " + messages.length + " " + numConv)
+                     

                      thisMessages.ajouterMessages(messages, numConv)
                      
                      thisMessages.ajouterMessages(messages, numConv)
                      

-                     thisMessages.conversations[numConv].initPage(
-                        function(num) // page suivante
-                        {
-                           thisMessages.client.pageSuivante(num - 1)
-                        },
-                        function(num) // page précédente
-                        {
-                           thisMessages.client.pagePrecedente(num - 1)
-                        },
+                     thisMessages.conversations[numConv].setPage(

                         numConv == 0 ? thisMessages.client.pagePrincipale : thisMessages.client.conversations[numConv - 1].page,
                         jQuery("autresPages", this).text() != "true"
                      )
                         numConv == 0 ? thisMessages.client.pagePrincipale : thisMessages.client.conversations[numConv - 1].page,
                         jQuery("autresPages", this).text() != "true"
                      )


@@ -779,9 +759,6 @@ Messages.prototype.rafraichirMessages = function(vider)
                   }
                )
                
                   }
                )
                

-               // demande à toutes les conversations d'afficher les messages non-affichés
-               //thisMessages.flush()
-               

                // rappel de la fonction dans 100 ms
                setTimeout(function(){ thisMessages.rafraichirMessages() }, 100);
             },
                // rappel de la fonction dans 100 ms
                setTimeout(function(){ thisMessages.rafraichirMessages() }, 100);
             },


@@ -793,52 +770,3 @@ Messages.prototype.rafraichirMessages = function(vider)
       }
    )
 }
       }
    )
 }

-
-/* Osbolète
-Messages.prototype.MAJPages = function(nbPage)
-{
-   //alert(nbPage)
-   
-   var thisMessages = this
-   
-   var nbPageActuel = jQuery("#pages span").size()
-
-   for(var p = nbPageActuel + 1; p <= nbPage && p <= 3; p++)
-   {  
-      jQuery("#pages").append("<span " + (this.page == p ? "class=\"pageCourante\"" : "" ) + ">" + p + "</span>").click(
-         function(event)
-         {
-            var target = jQuery(event.target)
-            
-            if(!target.is("span"))
-               return
-            
-            thisMessages.allerSurLaPage(parseInt(jQuery(event.target).text()))
-         }
-      )
-   }
-}*/
-
-/* Obsolète
-Messages.prototype.allerSurLaPage = function(page)
-{            
-   if (page == this.page)
-      return
-   this.page = page
-   
-   var thisMessages = this
-      
-   //jQuery("#pages span").removeClass("pageCourante")
-   jQuery("#pages span").each(
-      function(i)
-      {
-         if (jQuery(this).text() == thisMessages.page)
-            jQuery(this).addClass("pageCourante")
-         else
-            jQuery(this).removeClass("pageCourante")
-      }
-   )
-
-   this.rafraichirMessages(true)
-}*/
-

diff --git a/modules/erl/euphorik_minichat.erl b/modules/erl/euphorik_minichat.erl
index 53fe1e3..4268fef 100755 (executable)
--- a/modules/erl/euphorik_minichat.erl
+++ b/modules/erl/euphorik_minichat.erl
@@ -205,13 +205,14 @@ set_profile(Cookie, Login, Password, Pseudo, Email, Css, Page_principale, Conver
                      login_deja_pris;\r
                   _ ->               \r
                      User_modifie = User#user{
                      login_deja_pris;\r
                   _ ->               \r
                      User_modifie = User#user{

-                        % TODO : pourquoi ne pas tester avec la valeur "undefined" plutôt qu'avec "is_list" ?\r
+                        % 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)\r

                         login = if is_list(Login) -> Login; true -> User#user.login end,\r
                         password = if is_list(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,
                         login = if is_list(Login) -> Login; true -> User#user.login end,\r
                         password = if is_list(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,

-                        page_principale = if is_list(Page_principale) -> Page_principale; true -> User#user.page_principale end,
+                        page_principale = if is_integer(Page_principale), Page_principale > 0 -> Page_principale; true -> User#user.page_principale end,

                         conversations = if is_list(Conversations) -> Conversations; true -> User#user.conversations end\r
                      },\r
                      mnesia:write(User_modifie),\r
                         conversations = if is_list(Conversations) -> Conversations; true -> User#user.conversations end\r
                      },\r
                      mnesia:write(User_modifie),\r

diff --git a/modules/erl/euphorik_protocole.erl b/modules/erl/euphorik_protocole.erl
index 19932f2..251d4c1 100755 (executable)
--- a/modules/erl/euphorik_protocole.erl
+++ b/modules/erl/euphorik_protocole.erl
@@ -199,7 +199,7 @@ refreshMessage(Action) ->
                            )
                         ]
                      }\r
                            )
                         ]
                      }\r

-                  end,            
+                  end,

                   euphorik_minichat_conversation:conversations(User#user.conversations, Nb_message, Dernier_id, User#user.page_principale)\r
                )
             }];
                   euphorik_minichat_conversation:conversations(User#user.conversations, Nb_message, Dernier_id, User#user.page_principale)\r
                )
             }];