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
