X-Git-Url: http://git.euphorik.ch/?a=blobdiff_plain;f=modules%2Ferl%2Feuphorik_minichat_conversation.erl;h=0ce113c9111fb639be02d8ae26639ad6d73fe772;hb=8bec0dac79e750d0040de8a009c6ae864479642e;hp=a280e1fd5c499d4ec28603042912730e84e4bdb3;hpb=5f49b7e6af9be3e2415136013370af4ac99d59da;p=euphorik.git

diff --git a/modules/erl/euphorik_minichat_conversation.erl b/modules/erl/euphorik_minichat_conversation.erl
index a280e1f..0ce113c 100755
--- a/modules/erl/euphorik_minichat_conversation.erl
+++ b/modules/erl/euphorik_minichat_conversation.erl
@@ -17,7 +17,7 @@
 -include_lib("stdlib/include/qlc.hrl").
 
 -import(lists, [reverse/1, any/2, map/2, sublist/3, filter/2]).
--import(euphorik_minichat, [resultat_transaction/1]).
+-import(euphorik_bd, [resultat_transaction/1]).
 -import(qlc, [e/1, q/1, keysort/3]).
 -import(mnesia, [table/1, transaction/1]).
   
@@ -26,84 +26,71 @@
 % Chaque conversation est un tuple {[{Message, Parents}], Plus} où
 % Message est le message de type #minichat et Parents une liste d'Id.
 % Plus est un bool. Si Plus vaut true alors il y a encore des messages.
-% Si il n'y a pas de nouveaux message alors la fonction est bloquante et attend un nouveau message.
+% Si il n'y a pas de nouveaux message alors vide est renvoyé.
 % @spec conversations([{integer(), integer(), integer()}], integer(), integer(), integer()) -> [Conversation()]
 conversations(Racines, N, D, P) ->
-   % écoute des nouveaux messages
-   case subscribe(minichat, 2) of
-      erreur = E ->
-         E;
-      _ ->
-         % demande des conversations
-         Conversations = conversations_detailees(Racines, N, D, P),
-         %io:format("Conversations : ~p~n", [Conversations]),
-         % si les conversations sont vides alors on attend un nouveau message
-         Vide = not any(
-            fun(C) ->
-               case C of
-                  {[], _} -> false;
-                  {_, [], _, _} -> false;
-                  _ -> true
-               end
-            end,
-            Conversations
-         ),
-         Conversations_mises_en_forme = mise_en_forme_conversations(
-            if Vide ->
-               attend_nouveau_message(),
-               %io:format("Racines = ~p, N = ~p, D = ~p, P = ~p~n", [Racines, N, D, P]),
-               %io:format("Conversations = ~p~n", [conversations_detailees(Racines, N, D, P)]),
-               conversations_detailees(Racines, N, D, P);
-            true ->
-               Conversations
+      Conversations = conversations_detailees(Racines, N, D, P),
+      % si les conversations sont vides alors on attend un nouveau message
+      Vide = not any(
+         fun(C) ->
+            case C of
+               {[], _} -> false;
+               {_, [], _, _} -> false;
+               _ -> true
             end
-         ),
-         %io:format("Conversations_mises_en_forme = ~p~n", [Conversations_mises_en_forme]),
-         unsubscribe(minichat),
-         Conversations_mises_en_forme
-   end.
+         end,
+         Conversations
+      ),
+      if Vide ->
+            vide;
+         true ->
+            mise_en_forme_conversations(Conversations)
+      end.
    
    
 % Ecoute les événements de modification d'une table.
 % @spec subscribe(atom(), integer()) -> ok | erreur
-subscribe(_Table, 0) ->
-   erreur;
-subscribe(Table, C) ->
-   case mnesia:subscribe({table, Table, simple}) of 
-   	{error, {not_active_local, Table}} ->	    
-         mnesia:add_table_copy(minichat, node(), ram_copies),
-         subscribe(Table, C - 1);
-      {error, _}->
-         erreur;
-      _ ->
-         ok
-   end.   
+% Obsolète
+%~ subscribe(_Table, 0) ->
+   %~ erreur;
+%~ subscribe(Table, C) ->
+   %~ case mnesia:subscribe({table, Table, simple}) of 
+   	%~ {error, {not_active_local, Table}} ->
+         %~ mnesia:add_table_copy(minichat, node(), ram_copies),
+         %~ subscribe(Table, C - 1);
+      %~ {error, _}->
+         %~ erreur;
+      %~ _ ->
+         %~ ok
+   %~ end.   
    
    
 % Arrête d'écouter les modifications d'une table.
 % @spec unsubscribe(atom()) -> term()
-unsubscribe(Table) -> 
-   mnesia:unsubscribe({table, Table, simple}).
+% Obsolète
+%~ unsubscribe(Table) -> 
+   %~ mnesia:unsubscribe({table, Table, simple}).
  
  
 % Attend qu'un nouveau message arrive, function bloquante.
 % Renvoie le nouveau message.
+% Obsolète
 % @spec attend_nouveau_message() -> #minichat | timeout
-attend_nouveau_message() ->
-   receive % attente d'un post
-      {mnesia_table_event, {write, Message, _}} ->
-         Message;
-         %file:close(F);
+%~ attend_nouveau_message() ->
+   %~ receive % attente d'un post
+      %~ {mnesia_table_event, {write, Message, _}} ->
+         %~ Message;
+         %~ %file:close(F);
       %~ {tcp_closed, _} ->
          %~ mnesia:unsubscribe({table, minichat, simple});      
-      _ ->
-         attend_nouveau_message()
+      %~ _ ->
+         %~ attend_nouveau_message()
    % 60 minutes de timeout (le cas ou il n'y a que des consultations et jamais de post)
    % Après 60 minutes de connexion, le client doit donc reétablir une connexion
    % TODO : pour être mieux : quand le socket se ferme alors un message devrait être envoyé et débloquer ce receive (demande en cours sur la mailing list de yaws)
-   after 1000 * 60 * 60 -> 
-      timeout
-   end.   
+   %~ after 1000 * 60 * 60 -> 
+      %~ timeout
+   %~ end.   
      
 
 % Mise en forme des conversations pour l'utilisateur du module.
@@ -119,7 +106,7 @@ mise_en_forme_conversations([{Principale, Plus_principale} | Conversations]) ->
 mise_en_forme_conversation(Messages) ->
    lists:foldr(
       fun(Id, Acc) ->
-         case euphorik_minichat:message_by_id(Id) of
+         case euphorik_bd:message_by_id(Id) of
             {ok, Message} ->
                [{Message, parents(Id)} | Acc];
             _ ->