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

Posted to dev@shindig.apache.org by jo...@apache.org on 2008/02/02 01:58:30 UTC

svn commit: r617726 - /incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetSpecParser.java

Author: johnh
Date: Fri Feb  1 16:58:26 2008
New Revision: 617726

URL: http://svn.apache.org/viewvc?rev=617726&view=rev
Log:
Fall back to the default view (if available) when a "custom" view is specified which doesn't
exist in the spec.


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

Modified: incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetSpecParser.java
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetSpecParser.java?rev=617726&r1=617725&r2=617726&view=diff
==============================================================================
--- incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetSpecParser.java (original)
+++ incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetSpecParser.java Fri Feb  1 16:58:26 2008
@@ -556,10 +556,14 @@
 
       StringBuilder content = contentData.get(view);
       if (content == null) {
+        content = contentData.get(DEFAULT_VIEW);
+      }
+
+      if (content == null) {
         return "";
-      } else {
-        return content.toString();
       }
+
+      return content.toString();
     }
 
     // TODO: Synchronizing this seems unnecessary...a parse job should never