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 2009/11/19 20:25:55 UTC

svn commit: r882258 - /incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java

Author: johnh
Date: Thu Nov 19 19:25:55 2009
New Revision: 882258

URL: http://svn.apache.org/viewvc?rev=882258&view=rev
Log:
Retain all attributes on OSTemplate and OSData nodes rather than just "type".


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

Modified: incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java?rev=882258&r1=882257&r2=882258&view=diff
==============================================================================
--- incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java (original)
+++ incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java Thu Nov 19 19:25:55 2009
@@ -111,6 +111,17 @@
     if (scriptType != null) {
       Element replacement = elem.getOwnerDocument().createElement("script");
       replacement.setAttribute("type", scriptType);
+      
+      // Retain the remaining attributes of the node.
+      NamedNodeMap attribs = elem.getAttributes();
+      for (int i = 0; i < attribs.getLength(); ++i) {
+        Attr attr = (Attr)attribs.item(i);
+        if (!"type".equals(attr.getNodeName().equalsIgnoreCase("type"))) {
+          Attr newAttr = replacement.getOwnerDocument().createAttribute(attr.getNodeName());
+          newAttr.setValue(attr.getValue());
+          replacement.setAttributeNode(newAttr);
+        }
+      }
       return replacement;
     }
     return elem;