You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by ga...@apache.org on 2010/11/17 21:46:43 UTC
svn commit: r1036219 - in
/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite:
RewriteModule.java RewritePath.java
Author: gagan
Date: Wed Nov 17 20:46:43 2010
New Revision: 1036219
URL: http://svn.apache.org/viewvc?rev=1036219&view=rev
Log:
patch by gagan.goku | Issue 3175041: Small enhancements to RewriteModule to make it reusable | http://codereview.appspot.com/3175041/
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java?rev=1036219&r1=1036218&r2=1036219&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java Wed Nov 17 20:46:43 2010
@@ -86,16 +86,17 @@ public class RewriteModule extends Abstr
mapbinder.addBinding(rewritePath).toProvider(list);
}
- void addBindingForRewritePath(String container, RewriteFlow rewriteFlow) {
+ protected void addBindingForRewritePath(String container, RewriteFlow rewriteFlow) {
addBindingForRewritePath(container, rewriteFlow, binder().getProvider(
getKey(container, rewriteFlow)));
}
- Provider<List<ResponseRewriter>> getResponseRewriters(String container, RewriteFlow flow) {
+ protected Provider<List<ResponseRewriter>> getResponseRewriters(String container,
+ RewriteFlow flow) {
return binder().getProvider(getKey(container, flow));
}
- Key<List<ResponseRewriter>> getKey(String container, RewriteFlow flow) {
+ protected Key<List<ResponseRewriter>> getKey(String container, RewriteFlow flow) {
return Key.get(new TypeLiteral<List<ResponseRewriter>>() {},
new RewritePath(container, flow));
}
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java?rev=1036219&r1=1036218&r2=1036219&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java Wed Nov 17 20:46:43 2010
@@ -23,7 +23,7 @@ import java.lang.annotation.Annotation;
/**
* Implementation of ResponseRewriterList annotation interface.
*/
-class RewritePath implements ResponseRewriterList {
+public class RewritePath implements ResponseRewriterList {
protected final String container;
protected final RewriteFlow rewriteFlow;
@@ -63,4 +63,4 @@ class RewritePath implements ResponseRew
public Class<? extends Annotation> annotationType() {
return ResponseRewriterList.class;
}
-}
\ No newline at end of file
+}