You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by ch...@apache.org on 2010/03/17 05:28:16 UTC

svn commit: r924150 - /shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java

Author: chirag
Date: Wed Mar 17 04:28:16 2010
New Revision: 924150

URL: http://svn.apache.org/viewvc?rev=924150&view=rev
Log:
SHINDIG-1306 | Correctly apply SortOrder.descending when we're sorting by name.

Modified:
    shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java

Modified: shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java?rev=924150&r1=924149&r2=924150&view=diff
==============================================================================
--- shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java (original)
+++ shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java Wed Mar 17 04:28:16 2010
@@ -285,10 +285,10 @@ public class JsonDbOpensocialService imp
       // We can pretend that by default the people are in top friends order
       if (options.getSortBy().equals(Person.Field.NAME.toString())) {
         Collections.sort(result, NAME_COMPARATOR);
-      }
 
-      if (options.getSortOrder() == SortOrder.descending) {
-        Collections.reverse(result);
+        if (options.getSortOrder() == SortOrder.descending) {
+          Collections.reverse(result);
+        }
       }
 
       // TODO: The samplecontainer doesn't really have the concept of HAS_APP so