----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4965/#review7485 -----------------------------------------------------------
You need to change the unit test also. Current unit test are broken after i applied the patch. Share a test gadget also if you have it hosted some place. - Raminder On 2012-05-02 13:51:02, Fernando Pinhati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4965/ > ----------------------------------------------------------- > > (Updated 2012-05-02 13:51:02) > > > 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 > 1332769 > > Diff: https://reviews.apache.org/r/4965/diff > > > Testing > ------- > > > Thanks, > > Fernando > >
