MOD amélioration de la complexité de la fonction euphorik_bd:nouveau_message/3 mais...

[euphorik.git] / modules / erl / euphorik_bd.erl

diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl
index e5afe74..81a36a0 100755 (executable)
--- a/modules/erl/euphorik_bd.erl
+++ b/modules/erl/euphorik_bd.erl
@@ -276,11 +276,14 @@ user_by_mess(Id) ->
 % retourne soit l'id du message soit {erreur, <raison>}.\r
 nouveau_message(Mess, Auteur_id, Repond_A_ids) ->\r
    % regarde si les id 'Repond_A' existent\r
-   F = fun() ->   \r
-      Repond_a = e(\r
-         q([M || M <- mnesia:table(minichat), lists:member(M#minichat.id, Repond_A_ids)]),\r
-         [{tmpdir, ?KEY_SORT_TEMP_DIR}]\r
-      ), \r
+   F = fun() ->         \r
+      Repond_a = lists:map(\r
+         fun(Repond_a_id) ->\r
+            [M] = mnesia:wread({minichat, Repond_a_id}),\r
+            M\r
+         end,\r
+         Repond_A_ids\r
+      ),      \r
       Racine_id = case Repond_a of\r
          [] -> undefined;\r
          [M | _] -> \r
@@ -297,37 +300,38 @@ nouveau_message(Mess, Auteur_id, Repond_A_ids) ->
          {erreur, E} -> {erreur, E};\r
          _ ->\r
             % est-ce que l'auteur existe ?\r
-            case e(q([E || E <- mnesia:table(user), E#user.id =:= Auteur_id]), [{tmpdir, ?KEY_SORT_TEMP_DIR}]) of\r
+            case mnesia:wread({user, Auteur_id}) of\r
                [#user{profile = Profile} = Auteur] ->\r
                   if length(Repond_a) =/= length(Repond_A_ids) ->\r
                         {erreur, "Un ou plusieurs messages introuvable"};\r
                      true ->\r
                         % comparaison entre la date du dernier poste et maintenant (gestion du flood)\r
-                        Delta = euphorik_common:delta_date_ms(Auteur#user.date_derniere_connexion, now()),\r
+                        Now = now(),\r
+                        Delta = euphorik_common:delta_date_ms(Auteur#user.date_derniere_connexion, Now),\r
                         Nouvel_indice_flood = Auteur#user.indice_flood + if Delta =< ?DUREE_SPAM -> 2; true -> -1 end,\r
                         Auteur_maj = Auteur#user{\r
                            indice_flood = if Nouvel_indice_flood > ?INDICE_SPAM_MAX -> ?INDICE_SPAM_MAX; Nouvel_indice_flood < 0 -> 0; true -> Nouvel_indice_flood end,\r
-                           date_derniere_connexion = now()\r
+                           date_derniere_connexion = Now\r
                         },\r
                         % est-ce que l'auteur à trop floodé ?\r
                         if Auteur#user.indice_flood =/= ?INDICE_SPAM_MAX, Auteur_maj#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
-                           mnesia:write(Auteur#user{indice_flood = Auteur_maj#user.indice_flood}),\r
-                           nouveau_message_sys("\"" ++ Profile#profile.pseudo ++ if Auteur#user.login =/= [] -> " (" ++ Auteur#user.login ++ ")"; true -> "" end ++ "\" est bloqué pour " ++ integer_to_list(trunc(?DUREE_BLOCAGE_SPAM / 1000)) ++ " secondes pour cause de flood.");\r
-                        Auteur#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
-                           {erreur, "Bloqué pour cause de flood"};\r
-                        true ->     \r
-                           mnesia:write(Auteur_maj),\r
-                           Id = nouvel_id(minichat),\r
-                           inserer_reponses(Id, Repond_A_ids),\r
-                           mnesia:write(#minichat{\r
-                              id = Id,\r
-                              auteur_id = Auteur#user.id,\r
-                              date = now(),\r
-                              pseudo = Profile#profile.pseudo,\r
-                              contenu = Mess,\r
-                              racine_id = if Racine_id =:= undefined -> Id; true -> Racine_id end\r
-                           }),\r
-                           Id\r
+                              mnesia:write(Auteur#user{indice_flood = Auteur_maj#user.indice_flood}),\r
+                              nouveau_message_sys("\"" ++ Profile#profile.pseudo ++ if Auteur#user.login =/= [] -> " (" ++ Auteur#user.login ++ ")"; true -> "" end ++ "\" est bloqué pour " ++ integer_to_list(trunc(?DUREE_BLOCAGE_SPAM / 1000)) ++ " secondes pour cause de flood.");\r
+                           Auteur#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
+                              {erreur, "Bloqué pour cause de flood"};\r
+                           true ->     \r
+                              mnesia:write(Auteur_maj),\r
+                              Id = nouvel_id(minichat),\r
+                              inserer_reponses(Id, Repond_A_ids),\r
+                              mnesia:write(#minichat{\r
+                                 id = Id,\r
+                                 auteur_id = Auteur#user.id,\r
+                                 date = Now,\r
+                                 pseudo = Profile#profile.pseudo,\r
+                                 contenu = Mess,\r
+                                 racine_id = if Racine_id =:= undefined -> Id; true -> Racine_id end\r
+                              }),\r
+                              Id\r
                         end\r
                   end;\r
                _ ->\r