Author: zhoresh
Date: Thu Jan 27 22:50:21 2011
New Revision: 1064344

URL: http://svn.apache.org/viewvc?rev=1064344&view=rev
Log:
Set GADGET context for default | http://codereview.appspot.com/3982052/

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

Modified: 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
URL: 
http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java?rev=1064344&r1=1064343&r2=1064344&view=diff
==============================================================================
--- 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
 (original)
+++ 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
 Thu Jan 27 22:50:21 2011
@@ -62,16 +62,17 @@ public enum RenderingContext {
   }
 
   public static RenderingContext valueOfParam(String param) {
-    // Exception: when no &c= parameter provided, default to GADGET.
-    if (param == null) {
-      return RenderingContext.GADGET;
-    }
-    for (RenderingContext rc : RenderingContext.values()) {
-      String rcParam = rc.getParamValue();
-      if (rcParam != null && rcParam.equals(param)) {
-        return rc;
+    // Exception: when no &c= parameter provided or bad, default to GADGET.
+    RenderingContext c = RenderingContext.GADGET;
+    if (param != null) {
+      for (RenderingContext rc : RenderingContext.values()) {
+        String rcParam = rc.getParamValue();
+        if (rcParam != null && rcParam.equals(param)) {
+          c = rc;
+          break;
+        }
       }
     }
-    return null;
+    return c;
   }
 }


Reply via email to