Author: mfranklin
Date: Mon Feb 25 21:37:37 2013
New Revision: 1449909

URL: http://svn.apache.org/r1449909
Log:
Fixed ActivityStream target not detecting person object (RAVE-880)

Modified:
    
rave/trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultActivityStreamsService.java

Modified: 
rave/trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultActivityStreamsService.java
URL: 
http://svn.apache.org/viewvc/rave/trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultActivityStreamsService.java?rev=1449909&r1=1449908&r2=1449909&view=diff
==============================================================================
--- 
rave/trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultActivityStreamsService.java
 (original)
+++ 
rave/trunk/rave-providers/rave-opensocial-provider/rave-opensocial-core/src/main/java/org/apache/rave/opensocial/service/impl/DefaultActivityStreamsService.java
 Mon Feb 25 21:37:37 2013
@@ -411,12 +411,15 @@ public class DefaultActivityStreamsServi
 
     private void populatePersonObjects(ActivityStreamsEntry entry, Map<String, 
Person> peopleById) {
         ActivityStreamsObject actor = entry.getActor();
-        if(OBJECT_TYPE_PERSON.equals(actor.getObjectType())) {
+        if(entry.getActor() != null && 
OBJECT_TYPE_PERSON.equals(actor.getObjectType())) {
             populatePerson(peopleById, actor);
         }
-        if(OBJECT_TYPE_PERSON.equals(entry.getObject().getObjectType())) {
+        if(entry.getObject() != null && 
OBJECT_TYPE_PERSON.equals(entry.getObject().getObjectType())) {
             populatePerson(peopleById, entry.getObject());
         }
+        if(entry.getTarget() != null && 
OBJECT_TYPE_PERSON.equals(entry.getTarget().getObjectType())) {
+            populatePerson(peopleById, entry.getTarget());
+        }
     }
 
     private void populatePerson(Map<String, Person> peopleById, 
ActivityStreamsObject actor) {


Reply via email to