You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by ie...@apache.org on 2008/12/11 06:03:06 UTC

svn commit: r725577 - /incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java

Author: ieb
Date: Wed Dec 10 21:03:05 2008
New Revision: 725577

URL: http://svn.apache.org/viewvc?rev=725577&view=rev
Log:
   SHINDIG-769

Patch from Chico Charlesworth
 Fixes
q.setFirstResult() in PersonServiceDb.getPerson() is being set to 1 where it should be set to 0. This bug ensures that q.getResultList() will always return an empty list, which results in PersonServiceDb.getPerson() always returning null.

Thanks

Modified:
    incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java

Modified: incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java?rev=725577&r1=725576&r2=725577&view=diff
==============================================================================
--- incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java (original)
+++ incubator/shindig/trunk/java/samples/src/main/java/org/apache/shindig/social/opensocial/jpa/spi/PersonServiceDb.java Wed Dec 10 21:03:05 2008
@@ -141,7 +141,7 @@
     String uid = id.getUserId(token);
     Query q = entiyManager.createNamedQuery(PersonDb.FINDBY_PERSONID);
     q.setParameter(PersonDb.PARAM_PERSONID, uid);
-    q.setFirstResult(1);
+    q.setFirstResult(0);
     q.setMaxResults(1);
     List<?> plist = q.getResultList();
     Person person = null;