From: Christophe Romain <christophe.romain@process-one.net>
Date: Wed, 22 Jun 2016 10:25:41 +0000 (+0200)
Subject: Fix PubSub RSM on get_items (#1147)
X-Git-Tag: 16.06~16
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=95ff94b0549d30c3be26f266f4bf1094c9927500;p=ejabberd

Fix PubSub RSM on get_items (#1147)
---

diff --git a/src/node_flat_sql.erl b/src/node_flat_sql.erl
index 7b8ee552c..0869f531c 100644
--- a/src/node_flat_sql.erl
+++ b/src/node_flat_sql.erl
@@ -686,13 +686,10 @@ get_items(Nidx, _From,
 	    #rsm_in{max = M, direction = Direction, id = I, index = IncIndex}) ->
     Max = ejabberd_sql:escape(jlib:i2l(M)),
     {Way, Order} = case Direction of
-	%     aft -> {<<"<">>, <<"desc">>};
-	%     before when I == <<>> -> {<<"is not">>, <<"asc">>};
-	%     before -> {<<">">>, <<"asc">>};
-	%     _ when IncIndex =/= undefined ->
-	%         {<<"<">>, <<"desc">>}; % using index
-	_ ->
-	    {<<"is not">>, <<"desc">>}% Can be better
+	     aft when I == <<>> -> {<<"is not">>, <<"desc">>};
+	     aft -> {<<"<">>, <<"desc">>};
+	     before when I == <<>> -> {<<"is not">>, <<"asc">>};
+	     before -> {<<">">>, <<"asc">>}
     end,
     SNidx = integer_to_binary(Nidx),
     [AttrName, Id] = case I of