-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2572/
-----------------------------------------------------------
(Updated 2011-10-27 12:06:49.082022)
Review request for shindig and Brian Lillie.
Changes
-------
Sorry Ryan, I thought I had added JIRA. Patch included in the JIRA and linked
to in the review.
Summary
-------
If a gadget uses OAuth but requires no other features that depend on the
"security-token" feature, a security token is not returned on the iframeurl
when it should be. This stems from the fact that ModulePrefs.getViewFeatures()
doesn't pick up the artificially added "security-token" feature that gets added
by the FeatureVisitor if the OAuthVisitors found oauth tags.
This addresses bug SHINDIG-1651.
https://issues.apache.org/jira/browse/SHINDIG-1651
Diffs
-----
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/spec/ModulePrefs.java
1189151
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/spec/ModulePrefsTest.java
1189151
Diff: https://reviews.apache.org/r/2572/diff
Testing
-------
Updated existing JUnits and added new JUnits.
Thanks,
Stanton
