Doesn't this change break the sample container? Around line 196 of statefileparser.js, the "name" property is set to a simple string:

   'name' : $(xmlNode).attr(opensocial.Person.Field.NAME),

...so when the new "name.getField in not a function" errors, when the new getDisplayName code comes along, it complains that "name.getField is not a function".

Not sure if this is the proper way to fix this, but changing that line to this gets things working for me:

'name' : new opensocial.Name({'unstructured' : $(xmlNode).attr(opensocial.Person.Field.NAME)}),

--Jamey

Cassie Doll (JIRA) wrote:
     [ 
https://issues.apache.org/jira/browse/SHINDIG-59?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Cassie Doll closed SHINDIG-59.
------------------------------

    Resolution: Fixed

opensocial.Person.getDisplayName returns an object instead of a string
----------------------------------------------------------------------

                Key: SHINDIG-59
                URL: https://issues.apache.org/jira/browse/SHINDIG-59
            Project: Shindig
         Issue Type: Bug
           Reporter: Paul Lindner
           Assignee: Paul Lindner

The specification says that getDisplayName() should return a string. Instead right now it returns a opensocial.Name object:
opensocial.Person.prototype.getDisplayName = function() {
  return this.getField(opensocial.Person.Field.NAME);
}
To return a string we should construct a value based off the opensocial.Name 
object.


Reply via email to