]> granicus.if.org Git - ejabberd/commitdiff
Fix get_queue_length argument type
authorBadlop <badlop@process-one.net>
Fri, 27 May 2011 09:23:44 +0000 (11:23 +0200)
committerBadlop <badlop@process-one.net>
Fri, 27 May 2011 09:54:52 +0000 (11:54 +0200)
src/mod_offline.erl
src/web/ejabberd_web_admin.erl

index 2202a4241bfdf6947a58862d8bd057ad0b0f3b81..67f98813ae9a06a56598ad422b35f64d3d2f842c 100644 (file)
@@ -693,11 +693,9 @@ user_queue_parse_query(US, Query) ->
 us_to_list({User, Server}) ->
     exmpp_jid:to_list(User, Server).
 
-%% @spec (User::string(), Host::string()) -> integer()
+%% @spec (User::binary(), Host::binary()) -> integer()
 get_queue_length(User, Host) ->
-    UserB = list_to_binary(User),
-    HostB = list_to_binary(Host),
-    gen_storage:dirty_count_records(HostB, offline_msg, [{'=', user_host, {UserB, HostB}}]).
+    gen_storage:dirty_count_records(Host, offline_msg, [{'=', user_host, {User, Host}}]).
 
 get_messages_subset(User, Host, MsgsAll) ->
     Access = gen_mod:get_module_opt(Host, ?MODULE, access_max_user_messages,
index 3e44be216359497d60245d82bfd828eb06a8677b..5adefa3b3cb949cbd8580c5dd4dee1ed375fc1a5 100644 (file)
@@ -1710,7 +1710,7 @@ list_given_users(Host, Users, Prefix, Lang, URLFunc) ->
 get_offlinemsg_length(ModOffline, User, Server) ->
     case ModOffline of
        none -> "disabled";
-       _ -> pretty_string_int(ModOffline:get_queue_length(User, Server))
+       _ -> pretty_string_int(ModOffline:get_queue_length(list_to_binary(User), list_to_binary(Server)))
     end.
 
 get_offlinemsg_module(Server) ->