You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by jo...@apache.org on 2011/01/27 23:19:42 UTC

svn commit: r1064336 - /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java

Author: johnh
Date: Thu Jan 27 22:19:42 2011
New Revision: 1064336

URL: http://svn.apache.org/viewvc?rev=1064336&view=rev
Log:
Create cache key out of RenderingContext's implied .toString() for better null-safety.



Modified:
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java?rev=1064336&r1=1064335&r2=1064336&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/features/FeatureRegistry.java Thu Jan 27 22:19:42 2011
@@ -486,7 +486,7 @@ public class FeatureRegistry {
     List<String> neededList = Lists.newArrayList(needed);
     Collections.sort(neededList);
     return new StringBuilder().append(StringUtils.join(neededList, ":"))
-        .append("|").append(ctx.getRenderingContext().name())
+        .append("|").append(ctx.getRenderingContext())
         .append("|").append(ctx.getContainer())
         .append("|").append(unsupported != null)
         .toString();