From 39d4d474dc011f29b3060729f7ff09f446093070 Mon Sep 17 00:00:00 2001 From: Greg Burri Date: Fri, 6 Jun 2008 19:42:53 +0000 Subject: [PATCH] FIX de plein de truc, j'ai plus la force de regarder toutes les modifs (vive les commits offline :/) --- css/1/pageAbout.css | 10 +++ css/2/pageAbout.css | 10 +++ css/2/pageMinichat.css | 6 +- doc/TODO.txt | 11 +-- doc/technique.txt | 9 +++ js/euphorik.js | 30 ++++++-- js/pageMinichat.js | 110 +++++++++++++++++------------- modules/erl/euphorik_bd.erl | 18 +++-- modules/erl/euphorik_daemon.erl | 3 +- modules/erl/euphorik_requests.erl | 15 ++-- modules/erl/euphorik_test.erl | 14 ++-- pages/about.html | 8 ++- 12 files changed, 159 insertions(+), 85 deletions(-) diff --git a/css/1/pageAbout.css b/css/1/pageAbout.css index f21cd61..687e6d1 100644 --- a/css/1/pageAbout.css +++ b/css/1/pageAbout.css @@ -2,3 +2,13 @@ #page.about { } + +#page.about .faqCouleurProprietaire { + color: #31732f; +} +#page.about .faqCouleurReponse { + color: #bf2911; +} +#page.about .faqCouleurRepondu { + color: #84196c; +} diff --git a/css/2/pageAbout.css b/css/2/pageAbout.css index f21cd61..10eeae3 100644 --- a/css/2/pageAbout.css +++ b/css/2/pageAbout.css @@ -2,3 +2,13 @@ #page.about { } + +#page.about .faqCouleurProprietaire { + color: #31732f; +} +#page.about .faqCouleurReponse { + color: #bf2911; +} +#page.about .faqCouleurRepondu { + color: #84196c; +} \ No newline at end of file diff --git a/css/2/pageMinichat.css b/css/2/pageMinichat.css index 0c5b96a..f0e4785 100755 --- a/css/2/pageMinichat.css +++ b/css/2/pageMinichat.css @@ -139,13 +139,13 @@ zoom: 1 } #page.minichat #conversations div.reponse { - border-color: #bd7a11 + border-color: #bf2911 } #page.minichat #conversations div.repondu { - border-color: #b711bd + border-color: #84196c } #page.minichat #conversations div.proprietaire { - border-color: #bd1129 + border-color: #31732f } #page.minichat #conversations div.systeme { background-color: #555555 diff --git a/doc/TODO.txt b/doc/TODO.txt index fb3d59a..2e98211 100755 --- a/doc/TODO.txt +++ b/doc/TODO.txt @@ -2,8 +2,7 @@ === v1.0.1 === * Pouvoir afficher les utilisateurs (print_users(admin)) qui sont admin -* Mettre dans la FAQ la signification des couleurs associées aux messages. -* Ne pas pouvoir poster avec "" +* Ne pas pouvoir poster avec "" -> mettre en constante * Ajouter euphorik_common.erl au repo !! * Compilation : * Compiler avec le flag +debug_info pour le developpement @@ -199,7 +198,8 @@ [ok] Cleaner le code (erl, js, xhtml, css) et eventuellement profiler un peu (le refresh est lent sous opera) [ok] Afficher l'ip dans le print_users(). [ok] Enlever le petit carré mis en couleur et mettre le pseudo + date en couleur à la place - +[ok] Mettre dans la FAQ la signification des couleurs associées aux messages. + === Bugs === 1 : Critique 2 : Urgent @@ -222,12 +222,10 @@ [2] Le changement de skin n'est pas mémorisé lorsque l'on est pas connecté (normal puisque le style est mémorisé dans le profil) * solution 1 : permettre le changement de skin uniquement pour les personnes enregistré ? * solution 2 : mémoriser le skin courant dans un cookie -[2] Il est possible d'envoyer plusieurs fois le même message en pressant très rapidement plusieurs fois sur ENTER... (à vérifier) [2] Après l'ajout d'une image il n'est pas possible de naviger depuis celle ci vers les autres images en utilisant Next et Prev de lightbox après l'avoir affich [2] Plein de bugs sous MS internet explorer 7 * click sur le lien du conv insère sont id systématiquement au début du message * le changement de skin foire complétement -[2] griser le login dans le profil pour montrer qu'on ne peut pas l'éditer [3] Quand on revient en arrière dans firefox le message en rédaction est perdu * Pas sous Opera, apparemment Firefox recharge toute la page (donc impossible qu'il puisse remettre le message) * Eventuellement sauvegarder le message en rédaction dans le profile... @@ -280,6 +278,9 @@ [ok] Lors d'un logout il faut faire un full refresh (pour mettre à jour les messages auquel on répond par exemple) [ok] Après être passé de la page Admin à Chat le client continu de réaliser de temps en temp des requêtes "lists_banned_ips" * action=%7B%22action%22%3A%22list_banned_ips%22%2C%22cookie%22%3A%22<>%22%7D +[ok] Le widget select qui permet la sélection des css n'est pas initialisé correctement au chargement du site +[ok] Il est possible d'envoyer plusieurs fois le même message en pressant très rapidement plusieurs fois sur ENTER... (à vérifier) +[ok] griser le login dans le profil pour montrer qu'on ne peut pas l'éditer === Idées === Une fois l'idée validée elle est déplacée dans une version à venir. diff --git a/doc/technique.txt b/doc/technique.txt index 1d45db1..74a6360 100644 --- a/doc/technique.txt +++ b/doc/technique.txt @@ -12,6 +12,15 @@ Sequences : * Conversation * Message +=== Compilation avec +native === +Mesure du temps d'execution pour : + * euphorik_test:start(20, 20) : 20 personnes postants 20 messages + sans +native : 3:39 + avec +native : 3.41 + +Conclusion : + l'ajout de +native n'a pas de répercussions significatives sur les performances, cela provient + surement du fait que le gros du travail est fait du coté de la base de donnée Mnesia. === Séquences === * Attente de nouveaux messages diff --git a/js/euphorik.js b/js/euphorik.js index 32b8758..901915a 100755 --- a/js/euphorik.js +++ b/js/euphorik.js @@ -158,12 +158,17 @@ Util.prototype.infoBulle = function(message, element) { if (!thisUtil.bulleActive) return - + var m = $("#messageBulle") var f = $("#flecheBulle") + // remplie le paragraphe de la bulle avec le message $("p", m).html(message) - + + // réinitialise la position, évite le cas ou la boite est collé à droite et remplie avec un texte la faisant dépassé + // dans ce cas la hauteur n'est pas calculé correctement + m.css("top", 0).css("left", 0) + var positionFleche = { left : element.offset().left + element.width() / 2 - f.width() / 2, top : element.offset().top - f.height() @@ -565,9 +570,7 @@ Client.prototype.setCss = function(css) return this.css = css - $("link#cssPrincipale").attr("href", this.css) - this.majMenu() - + $("link#cssPrincipale").attr("href", this.css) if (this.autoflush) this.flush(true) } @@ -848,6 +851,7 @@ Client.prototype.chargerDonnees = function(data) this.conversations = data["conversations"] this.majBulle() + this.majCssSelectionee() } } @@ -897,8 +901,6 @@ Client.prototype.flush = function(async) Client.prototype.majMenu = function() { - // TODO : à virer : ne plus changer de style de display ... spa beau .. ou trouver une autre méthode - // var displayType = this.css == "css/3/euphorik.css" ? "block" : "inline" //this.client displayType = "block" $("#menu .admin").css("display", this.ekMaster ? displayType : "none") @@ -932,6 +934,20 @@ Client.prototype.majBulle = function() this.util.bulleActive = this.viewTooltips } +/** + * Met à jour la css sélectionnée, lors du chargement des données. + */ +Client.prototype.majCssSelectionee = function() +{ + // extraction du numéro de la css courante + var numCssCourante = this.css.match(/^.*?\/(\d)\/.*$/) + if (numCssCourante[1] != undefined) + { + $("#menuCss option").removeAttr("selected") + $("#menuCss option[value=" + numCssCourante[1]+ "]").attr("selected", "selected") + } +} + Client.prototype.slap = function(userId, raison) { var thisClient = this diff --git a/js/pageMinichat.js b/js/pageMinichat.js index 03024bc..7282a43 100755 --- a/js/pageMinichat.js +++ b/js/pageMinichat.js @@ -26,6 +26,10 @@ function PageMinichat(client, formateur, util) this.regexMessageTagMatch = /\{.*?\}>/g this.regexMessageTagReplace = /^(.*?\{.*?\}>)*/ + + // permet d'éviter d'envoyer plusieurs messages simultanément en pressant + // rapidement sur "enter" par exemple + this.envoieMessageEnCours = false } PageMinichat.prototype.contenu = function() @@ -173,7 +177,7 @@ PageMinichat.prototype.getJSONMessage = function(pseudo, message, repondA) } PageMinichat.prototype.envoyerMessage = function(pseudo, message) -{ +{ var thisPageMinichat = this // (un pseudo vide est autorisé) @@ -205,6 +209,14 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message) this.client.pseudo = pseudo + // évite le double post + if (this.envoieMessageEnCours) + { + this.util.messageDialogue("Message en cours d'envoie...") + return + } + this.envoieMessageEnCours = true + ;; dumpObj(this.getJSONMessage(pseudo, message, repondA)) jQuery.ajax( { @@ -214,6 +226,7 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message) dataType : "json", beforeSend : function(xmlHttpRequest) { + // TODO : ça marche ça ?? xmlHttpRequest.setRequestHeader("X-Requested-With", "") }, success : function(data, textStatus) @@ -231,7 +244,12 @@ PageMinichat.prototype.envoyerMessage = function(pseudo, message) else if (data["reply"] == "error") { thisPageMinichat.util.messageDialogue(data["error_message"]) - } + } + thisPageMinichat.envoieMessageEnCours = false + }, + error : function() + { + thisPageMinichat.envoieMessageEnCours = false } } ) @@ -561,7 +579,7 @@ Conversation.prototype.flush = function(funClickOuvrirConv) var DOM = $(XHTML) DOM.each( function() - { + { $(".lienConv", this).click( function(event) { @@ -570,6 +588,49 @@ Conversation.prototype.flush = function(funClickOuvrirConv) funClickOuvrirConv(parseInt(racine.substring(1, racine.length - 1), 36)) return false } + ) + + thisConversation.util.infoBulle("Extraction de la conversation", $(".extraire", this)) + + // l'id du message + var idMess36 = $(this).attr("id").substr(4) + var idMess = parseInt(idMess36, 36) + + $(this).click( + function(event) + { + if ($(event.target).is("a") || $(event.target).parents("#outilsBan").length > 0) return + + // extraction d'une conversation + if ($(event.target).is(".extraire")) + { + 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]) + } + ) + + // mise en évidence de la conversation + $(".entete", this).hover( + function() + { + thisConversation.decolorerEntetes() + thisConversation.afficherConversation(idMess) + }, + // quand on sort de l'entête du message la mise en évidence est enlevée + function() + { + thisConversation.enleverMiseEnEvidence() + thisConversation.decolorerEntetes() + thisConversation.colorerEntetes() + } ) if (thisConversation.client.viewTimes) @@ -620,49 +681,6 @@ Conversation.prototype.flush = function(funClickOuvrirConv) $("#outilsBan", this).hide() } ) - - thisConversation.util.infoBulle("Extraction de la conversation", $(".extraire", this)) - - // l'id du message - var idMess36 = $(this).attr("id").substr(4) - var idMess = parseInt(idMess36, 36) - - $(this).click( - function(event) - { - if ($(event.target).is("a") || $(event.target).parents("#outilsBan").length > 0) return - - // extraction d'une conversation - if ($(event.target).is(".extraire")) - { - 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]) - } - ) - - // mise en évidence de la conversation - $(".entete", this).hover( - function() - { - thisConversation.decolorerEntetes() - thisConversation.afficherConversation(idMess) - }, - // quand on sort de l'entête du message la mise en évidence est enlevée - function() - { - thisConversation.enleverMiseEnEvidence() - thisConversation.decolorerEntetes() - thisConversation.colorerEntetes() - } - ) } ) DOM.prependTo("#conversations #" + this.getId()) diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl index 2b18d5b..b72d7a8 100755 --- a/modules/erl/euphorik_bd.erl +++ b/modules/erl/euphorik_bd.erl @@ -303,12 +303,19 @@ update_pseudo_user(UserId, Pseudo) -> % Affiche N user trié par leur date de dernière connexion. -print_users(N) -> +% Opt est une liste d'option d'affichage : +% * ekmaster : n'affiche que les admins +print_users(N, Opt) -> + AfficheQueLesEkMaster = lists:any(fun(O) -> O =:= ekmaster end, Opt), resultat_transaction(mnesia:transaction(fun() -> C = cursor( qlc:keysort( #user.date_derniere_connexion, - q([E || E <- mnesia:table(user)]), + if AfficheQueLesEkMaster -> + q([E || E <- mnesia:table(user), E#user.ek_master =:= true]); + true -> + q([E || E <- mnesia:table(user)]) + end, [{order, descending}] ), [{tmpdir, ?KEY_SORT_TEMP_DIR}] @@ -324,10 +331,13 @@ print_users(N) -> end)). +% Affiche tous les users. +print_users(Opt) -> + print_users(all_remaining, Opt). + % Affiche tous les users. print_users() -> - print_users(all_remaining). - + print_users(all_remaining, []). print_user(User) when is_record(User, user) -> #user{id = Id, pseudo = Pseudo, login = Login, ek_master = Ek_master, date_derniere_connexion = Date, last_ip = IP} = User, diff --git a/modules/erl/euphorik_daemon.erl b/modules/erl/euphorik_daemon.erl index dc33a7f..1813e0e 100755 --- a/modules/erl/euphorik_daemon.erl +++ b/modules/erl/euphorik_daemon.erl @@ -60,5 +60,4 @@ reload_euphorik() -> end, [euphorik_minichat_conversation, euphorik_protocole, euphorik_requests, euphorik_bd, euphorik_daemon] ). - - + \ No newline at end of file diff --git a/modules/erl/euphorik_requests.erl b/modules/erl/euphorik_requests.erl index 524f735..28019da 100755 --- a/modules/erl/euphorik_requests.erl +++ b/modules/erl/euphorik_requests.erl @@ -23,27 +23,20 @@ -module(euphorik_requests). --export([ - tester/0, - out/1 -]). +-export([out/1]). -include_lib("xmerl/include/xmerl.hrl"). -include_lib("yaws/include/yaws_api.hrl"). -% Test du module (TODO) -tester() -> - que_dal. - - out(A) -> %io:format("~p~n", [A]), % utilisé parfois pendant le debug IP = case inet:peername(A#arg.clisock) of {ok, {Adresse, _Port}} -> Adresse; _ -> inconnue end, - % passive -> active, permet de recevoir {tcp_closed, _} lorsque le socket se ferme - inet:setopts(A#arg.clisock, [{active, true}]), + % passive -> active, permet de recevoir {tcp_closed, _} lorsque le socket se ferme + % keepalive -> true, evite que des firewalls coupe la connexion TCP sans prévenir + inet:setopts(A#arg.clisock, [{active, true}, {keepalive, true}]), {value, {_, Contenu}} = lists:keysearch("action", 1, yaws_api:parse_post(A)), Ret = traiter_donnees(Contenu, IP), {content, "application/json", Ret}. diff --git a/modules/erl/euphorik_test.erl b/modules/erl/euphorik_test.erl index c602292..d319997 100644 --- a/modules/erl/euphorik_test.erl +++ b/modules/erl/euphorik_test.erl @@ -21,12 +21,13 @@ -module(euphorik_test). --export([start/1, stop/1]). +-export([start/2, stop/1]). -include("../include/euphorik_bd.hrl"). % N est le nombre d'utilisateur -start(N) -> +% M est le nombre de message que chaque utilisateur va poster +start(N, M) -> Ids = creer_users(N), lists:map( fun(Id) -> @@ -35,7 +36,7 @@ start(N) -> fun() -> {A1, A2, A3} = now(), random:seed(A1, A2, A3), - loop(Id) + loop(Id, M) end ) end, @@ -102,13 +103,14 @@ tire_element_rand(N, L, Elements) -> tire_element_rand(N-1, L, [E | Elements]) end. - -loop(User_id) -> +loop(User_id, 0) -> + io:format("~p a fini~n", [User_id]); +loop(User_id, M) -> % attend un temp aléatoire compris entre 1 sec et 5 sec timer:sleep(1000 * random:uniform(5)), % poste un message aléatoire par une personne aléatoire répondant à des messages aléatoires {Message, Repond_a} = {message_rand(), messages_id_rand()}, io:format("~p poste ~p et repond a ~w~n", [User_id, Message, Repond_a]), euphorik_bd:nouveau_message(Message, User_id, Repond_a), - loop(User_id). + loop(User_id, M - 1). \ No newline at end of file diff --git a/pages/about.html b/pages/about.html index c142b72..364c31e 100644 --- a/pages/about.html +++ b/pages/about.html @@ -16,7 +16,13 @@

C'est quoi un "troll de la semaine" ?

Simplement un sujet de débat à tendance trollifique. Chaque lundi à 3 heure du matin un nouveau troll est choisi au hasard parmis ceux proposés par les administrateurs.

- + +

A quoi correspondent les couleurs liés aux messages ?

+
    +
  • Vert : messages appartenant à l'utilisateur courant.
    • +
  • Rouge : messages répondant à l'utilisateur courant.
    • +
  • Bleu : messages auquels l'utilisateur courant à répondu.
    • +

Quels-sont les navigateurs supportés ?

Le site a été testé sous "Firefox 2 et 3", "Safari 3" et "Opera 9". -- 2.43.0