Updated Branches:
  refs/heads/master 23854b2d5 -> 2414eb613

CAMEL-6940: xml tokenizer not handling innner self-closed elements


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/2414eb61
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/2414eb61
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/2414eb61

Branch: refs/heads/master
Commit: 2414eb613c816468f5d89b12493d3cbffda0877d
Parents: 23854b2
Author: Akitoshi Yoshida <[email protected]>
Authored: Tue Nov 12 10:50:18 2013 +0100
Committer: Akitoshi Yoshida <[email protected]>
Committed: Tue Nov 12 11:02:46 2013 +0100

----------------------------------------------------------------------
 .../apache/camel/support/TokenXMLExpressionIterator.java  |  2 +-
 .../camel/language/tokenizer/TokenizeLanguageTest.java    | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/2414eb61/camel-core/src/main/java/org/apache/camel/support/TokenXMLExpressionIterator.java
----------------------------------------------------------------------
diff --git 
a/camel-core/src/main/java/org/apache/camel/support/TokenXMLExpressionIterator.java
 
b/camel-core/src/main/java/org/apache/camel/support/TokenXMLExpressionIterator.java
index 938b1d6..3048c64 100644
--- 
a/camel-core/src/main/java/org/apache/camel/support/TokenXMLExpressionIterator.java
+++ 
b/camel-core/src/main/java/org/apache/camel/support/TokenXMLExpressionIterator.java
@@ -47,7 +47,7 @@ import org.apache.camel.util.ObjectHelper;
 public class TokenXMLExpressionIterator extends ExpressionAdapter {
     private static final Pattern NAMESPACE_PATTERN = 
Pattern.compile("xmlns(:\\w+|)\\s*=\\s*('[^']+'|\"[^\"]+\")");
     private static final String SCAN_TOKEN_NS_PREFIX_REGEX = 
"([^:<>]{1,15}?:|)";
-    private static final String SCAN_BLOCK_TOKEN_REGEX_TEMPLATE = 
"<{0}(\\s+[^/]*)?/>|<{0}(\\s+[^>]*)?>(?:(?!(</{0}\\s*>)).)*</{0}\\s*>";
+    private static final String SCAN_BLOCK_TOKEN_REGEX_TEMPLATE = 
"<{0}(\\s+[^/^>]*)?/>|<{0}(\\s+[^>]*)?>(?:(?!(</{0}\\s*>)).)*</{0}\\s*>";
     private static final String SCAN_PARENT_TOKEN_REGEX_TEMPLATE = 
"<{0}(\\s+[^>]*\\s*)?>";
     
     protected final String tagToken;

http://git-wip-us.apache.org/repos/asf/camel/blob/2414eb61/camel-core/src/test/java/org/apache/camel/language/tokenizer/TokenizeLanguageTest.java
----------------------------------------------------------------------
diff --git 
a/camel-core/src/test/java/org/apache/camel/language/tokenizer/TokenizeLanguageTest.java
 
b/camel-core/src/test/java/org/apache/camel/language/tokenizer/TokenizeLanguageTest.java
index bfc3bfa..1860436 100644
--- 
a/camel-core/src/test/java/org/apache/camel/language/tokenizer/TokenizeLanguageTest.java
+++ 
b/camel-core/src/test/java/org/apache/camel/language/tokenizer/TokenizeLanguageTest.java
@@ -64,6 +64,16 @@ public class TokenizeLanguageTest extends ContextTestSupport 
{
         assertMockEndpointsSatisfied();
     }
 
+    public void testSendMixedClosingTagInsideMessageToTokenize() throws 
Exception {
+        getMockEndpoint("mock:result").expectedBodiesReceived(
+            "<child name='child1'><grandchild name='grandchild1'/> <grandchild 
name='grandchild2'/></child>", "<child name='child2'><grandchild 
name='grandchild1'></grandchild><grandchild 
name='grandchild2'></grandchild></child>");
+
+        template.sendBody("direct:start",
+            "<parent><child name='child1'><grandchild name='grandchild1'/> 
<grandchild name='grandchild2'/></child><child name='child2'><grandchild 
name='grandchild1'></grandchild><grandchild 
name='grandchild2'></grandchild></child></parent>");
+
+        assertMockEndpointsSatisfied();
+    }
+
     public void testSendNamespacedChildMessageToTokenize() throws Exception {
         getMockEndpoint("mock:result").expectedBodiesReceived(
             "<c:child xmlns:c='urn:c' some_attr='a' 
anotherAttr='a'></c:child>", "<c:child xmlns:c='urn:c' some_attr='b' 
anotherAttr='b' />");

Reply via email to