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

Kevin Brown commented on SHINDIG-152:
-------------------------------------

Note that this patch doesn't add proper Guice integration for the social data 
servlets. I'm going to leave this up to Cassie or other people working on the 
OS java bits for now, since there's still a lot  of work going on in those 
parts and I don't want to step on any toes. I've done the bare minimum to make 
it work the same as it did before.

> Replace CrossServletState and other hacky junk with Guice
> ---------------------------------------------------------
>
>                 Key: SHINDIG-152
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-152
>             Project: Shindig
>          Issue Type: Improvement
>          Components: Gadgets Server - Java
>            Reporter: Kevin Brown
>            Assignee: Kevin Brown
>         Attachments: guice-2.patch.txt
>
>
> http://code.google.com/p/google-guice
> The attached patch will get rid of several remaining bits of clunkiness that 
> weren't properly addressed in the last refactoring patch by introducing 
> Guice, an open source dependency injection framework.
> This patch mostly consists of removing code that is no longer necessary, 
> moving some code around to facilitate DI, and adding the guice wiring. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to