----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4965/#review7579 -----------------------------------------------------------
Ship it! i am going to apply the changes with minor change to add same change to getUniqueListOfConnectedPeople().. i still need to findout Rave service to friends data - Raminder On 2012-05-04 17:15:00, Fernando Pinhati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4965/ > ----------------------------------------------------------- > > (Updated 2012-05-04 17:15:00) > > > Review request for rave. > > > Summary > ------- > > Issue: > https://issues.apache.org/jira/browse/RAVE-599 > > Fixed passing username instead of userId to PersonRepository findFriends > method. > > > Diffs > ----- > > > /trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultPersonService.java > 1333183 > > /trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/test/java/org/apache/rave/opensocial/service/PersonServiceTest.java > 1333183 > > Diff: https://reviews.apache.org/r/4965/diff > > > Testing > ------- > > > Thanks, > > Fernando > >
