[
https://issues.apache.org/jira/browse/SHINDIG-58?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jamey Wood updated SHINDIG-58:
------------------------------
Attachment: shindig-58.patch
A patch which changes the opensocial.Activity constructor (in
features/opensocial-reference/activity.js) to expect the title property as its
own separate parameter (similar to how other opensocial.* constructors appear
to work).
> features/opensocial-reference/container.js passes incorrect parameters to
> opensocial.Activity constructor
> ---------------------------------------------------------------------------------------------------------
>
> Key: SHINDIG-58
> URL: https://issues.apache.org/jira/browse/SHINDIG-58
> Project: Shindig
> Issue Type: Bug
> Components: Features
> Reporter: Jamey Wood
> Assignee: Kevin Brown
> Attachments: shindig-58.patch
>
> Time Spent: 0.02h
> Remaining Estimate: 0h
>
> These lines in container.js:
> 303 opensocial.Container.prototype.newActivity = function(title,
> 304 opt_params) {
> 305 return new opensocial.Activity(title, opt_params);
> 306 };
> do not match up with the opensocial.Activity constructor defined in
> activity.js:
> 147 opensocial.Activity = function(params) {
> 148 this.fields_ = params;
> 149 };
> (because the latter does not expect the "title" property to be passed in a
> separate parameter).
> One or the other should be changed. Without such a change, the internal
> state of activity objects is being mishandled.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
