You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@shindig.apache.org by "Paul Lindner (JIRA)" <ji...@apache.org> on 2010/07/02 00:49:49 UTC
[jira] Resolved: (SHINDIG-1354)
org.apache.shindig.gadgets.parse.DefaultHtmlSerializer#serialize drops
trailing comments
[ https://issues.apache.org/jira/browse/SHINDIG-1354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Paul Lindner resolved SHINDIG-1354.
-----------------------------------
Fix Version/s: 2.0.0-RC1
Resolution: Fixed
resolved
> org.apache.shindig.gadgets.parse.DefaultHtmlSerializer#serialize drops trailing comments
> ----------------------------------------------------------------------------------------
>
> Key: SHINDIG-1354
> URL: https://issues.apache.org/jira/browse/SHINDIG-1354
> Project: Shindig
> Issue Type: Bug
> Components: Java
> Reporter: Jan Luehe
> Priority: Trivial
> Fix For: 2.0.0-RC1
>
>
> Summary says it all.
> Suggested diffs:
> Index: java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java
> ===================================================================
> --- java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java (revision 951057)
> +++ java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/DefaultHtmlSerializer.java (working copy)
> @@ -66,7 +66,10 @@
> break;
> }
> case Node.DOCUMENT_NODE: {
> - serialize(((Document)n).getDocumentElement(), output, xmlMode);
> + NodeList children = ((Document)n).getChildNodes();
> + for (int i = 0; i < children.getLength(); i++) {
> + serialize(children.item(i), output, xmlMode);
> + }
> break;
> }
> case Node.ELEMENT_NODE: {
> Diffs will be submitted for code review.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.