Author: etnu Date: Tue Feb 26 20:22:29 2008 New Revision: 631477 URL: http://svn.apache.org/viewvc?rev=631477&view=rev Log: Rolling back most of the GadgetRenderingServlet changes from Bruno's patch as it has resulted in most gadgets breaking (no substitutions are performed).
Modified: incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java Modified: incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java?rev=631477&r1=631476&r2=631477&view=diff ============================================================================== --- incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java (original) +++ incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java Tue Feb 26 20:22:29 2008 @@ -196,32 +196,27 @@ View view = gadget.getView(options.getView()); // use single character for tab to simulate indentation within source code - String t = " "; + /*String t = " "; String n = "\n"; if (!options.getDebug()) { t = ""; n = ""; - } + }*/ - if (!view.getQuirks()) - markup.append("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01//EN\" \"http://www.w3.org/TR/html4/strict.dtd\">"+n); + if (!view.getQuirks()) { + markup.append("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01//EN\" \"http://www.w3.org/TR/html4/strict.dtd\">"); + } // TODO: This is so wrong. - markup.append( - "<html>"+n+ - t+"<head>"+n+ - t+t+"<style type=\"text/css\">"+n+ - t+t+t+"body,td,div,span,p{font-family:arial,sans-serif;}"+n+ - t+t+t+"a {color:#0000cc;}a:visited {color:#551a8b;}"+n+ - t+t+t+"a:active {color:#ff0000;}"+n+ - t+t+t+"body{margin: 0px;padding: 0px;background-color:white;}"+n+ - t+t+"</style>"+n+ - t+"</head>"+n+ - t+"<body>"+n); + markup.append("<style type=\"text/css\">") + .append("body,td,div,span,p{font-family:arial,sans-serif;}") + .append("a {color:#0000cc;}a:visited {color:#551a8b;}") + .append("a:active {color:#ff0000;}") + .append("body{margin: 0px;padding: 0px;background-color:white;}") + .append("</style>"); StringBuilder externJs = new StringBuilder(); StringBuilder inlineJs = new StringBuilder(); - String externFmt = - t+t+"<script src=\"%s\"></script>"+n; + String externFmt = "<script src=\"%s\"></script>"; String forcedLibs = options.getForcedJsLibs(); for (JsLibrary library : gadget.getJsLibraries()) { @@ -263,7 +258,7 @@ } List<GadgetException> gadgetExceptions = new LinkedList<GadgetException>(); - String content = view.getData(); + String content = gadget.getContentData(options.getView()); if (content == null) { // unknown view gadgetExceptions.add( @@ -286,11 +281,8 @@ } markup.append(content); - markup.append( - t+t+"<script>gadgets.util.runOnLoadHandlers();</script>"+n); - markup.append( - t+"</body>"+n+ - "</html>"); + markup.append("<script>gadgets.util.runOnLoadHandlers();</script>"); + markup.append("</body></html>"); resp.getWriter().print(markup.toString()); }