[ 
https://issues.apache.org/jira/browse/SHINDIG-1592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13090621#comment-13090621
 ] 

[email protected] commented on SHINDIG-1592:
--------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1616/#review1620
-----------------------------------------------------------

Ship it!


Committed revision 1161323. Please close the JIRA as well.  Thanks.

- Ryan


On 2011-08-24 20:14:29, li xu wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1616/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-08-24 20:14:29)
bq.  
bq.  
bq.  Review request for shindig, Paul Lindner, Henry Saputra, and Ryan Baxter.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This fix enables EE sample page to work out of box even if it's deployed 
with a context root other than root or port other than 8080
bq.  
bq.  we added two tokens at sample db: canonicaldb.json
bq.  %contextroot%
bq.  %origin%
bq.  it's resolved by JsonDBOpensocialService.java based on injected 
hostProvider
bq.  This is a generic change... any other opensocial object could use the same 
token in sample json db.
bq.  
bq.  Opensocial sample context is updated to include an additional attribute 
since photogadget.xml need to refer to AlbumViewer.xml
bq.              "eeGadget" : 
"%origin%%contextroot%/samplecontainer/examples/embeddedexperiences/AlbumViewer.xml",
bq.  fixed related junit tests.
bq.  
bq.  Thanks to Ryan's help on this! 
bq.  
bq.  
bq.  This addresses bug shindig-1592.
bq.      https://issues.apache.org/jira/browse/shindig-1592
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
/trunk/content/samplecontainer/examples/embeddedexperiences/PhotoList.xml 
1160372 
bq.    /trunk/content/samplecontainer/examples/embeddedexperiences/index.html 
1160372 
bq.    /trunk/content/sampledata/canonicaldb.json 1160372 
bq.    
/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java
 1160372 
bq.    
/trunk/java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryAtomIds.xml
 1160372 
bq.    
/trunk/java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonGroup.json
 1160372 
bq.    
/trunk/java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonId.json
 1160372 
bq.    
/trunk/java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonIds.json
 1160372 
bq.    
/trunk/java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryXmlIds.xml
 1160372 
bq.  
bq.  Diff: https://reviews.apache.org/r/1616/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Passed JUnit tests.
bq.  Tested with /embeddedexperiences/index.html
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  li
bq.  
bq.



> EE sample gadget doesn't work out of box if deployed with context other than 
> root or deployed on port other than 8080
> ---------------------------------------------------------------------------------------------------------------------
>
>                 Key: SHINDIG-1592
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-1592
>             Project: Shindig
>          Issue Type: Bug
>          Components: Java
>    Affects Versions: 3.0.0
>            Reporter: li xu
>            Priority: Minor
>              Labels: shindig
>             Fix For: 3.0.0
>
>         Attachments: patch0824
>
>
> This fix enables EE sample page to work out of box even if it's deployed with 
> a context root other than root or port other than 8080
> we added two tokens at sample db: canonicaldb.json
> %contextroot%
> %origin%
> it's resolved by JsonDBOpensocialService.java based on injected hostProvider
> This is a generic change... any other opensocial object could use the same 
> token in sample json db.
> Opensocial sample context is updated to include an additional attribute since 
> photogadget.xml need to refer to AlbumViewer.xml
>             "eeGadget" : 
> "%origin%%contextroot%/samplecontainer/examples/embeddedexperiences/AlbumViewer.xml",
> fixed related junit tests.
> Thanks to Ryan's help on this!

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to