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

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



bq.  On 2011-09-16 18:47:49, Stanton Sievers wrote:
bq.  > 
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/spec/LinkSpec.java, 
line 74
bq.  > <https://reviews.apache.org/r/1928/diff/2/?file=41488#file41488line74>
bq.  >
bq.  >     Is there a particular reason getMethod() isn't being added to 
GadgetsHandlerApi.LinkSpec?
bq.  
bq.  Henry Saputra wrote:
bq.      No reason other than forget cause its too many duplicate classes for 
different purpose =(
bq.      
bq.      I will add it in the code.
bq.  
bq.  Henry Saputra wrote:
bq.      Actually the GadgetHandler code no longer has reference to the 
GadgetsHandlerApi.LinkSpec. Do you know where its used now?

Never mind its refactored to GadgetsHandlerService =)


- Henry


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


On 2011-09-16 18:16:57, Henry Saputra wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1928/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-09-16 18:16:57)
bq.  
bq.  
bq.  Review request for shindig.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  The <Link> representation LinkSpec class missing "method" optional 
property as defined in the OpenSocial specs: 
http://opensocial-resources.googlecode.com/svn/spec/2.0/Core-Gadget.xml#LifeCycleEvents
bq.  
bq.  
bq.  This addresses bug SHINDIG-1622.
bq.      https://issues.apache.org/jira/browse/SHINDIG-1622
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/spec/LinkSpec.java 
1171202 
bq.    
trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/spec/LinkSpecTest.java
 1171202 
bq.  
bq.  Diff: https://reviews.apache.org/r/1928/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Updated the unit test for LinkSpec and pass.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Henry
bq.  
bq.



> Add support for optional "method" parameter in the LinkSpec implementation
> --------------------------------------------------------------------------
>
>                 Key: SHINDIG-1622
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-1622
>             Project: Shindig
>          Issue Type: Bug
>          Components: Java
>    Affects Versions: 2.0.0-RC2, 2.0.2
>            Reporter: Henry Saputra
>            Assignee: Henry Saputra
>              Labels: gadget
>         Attachments: SHINDIG-1622.patch
>
>
> The <Link> representation LinkSpec class missing "method" optional property 
> as defined in the OpenSocial specs: 
> http://opensocial-resources.googlecode.com/svn/spec/2.0/Core-Gadget.xml#LifeCycleEvents

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

        

Reply via email to