[ 
https://issues.apache.org/jira/browse/CAMEL-7064?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846587#comment-13846587
 ] 

Claus Ibsen commented on CAMEL-7064:
------------------------------------

The patch looks good. Only one checkstyle error

/Users/davsclaus/workspace/camel/components/camel-jcr/src/test/java/org/apache/camel/component/jcr/JcrProducerSubNodeTest.java:80:
 method call child at indentation level 16 not at correct indentation, 20

Details here how to build with checkstyle
http://camel.apache.org/building.html

> JcrProducer ignores existing nodes and creates invalid node hierarchies
> -----------------------------------------------------------------------
>
>                 Key: CAMEL-7064
>                 URL: https://issues.apache.org/jira/browse/CAMEL-7064
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-jcr
>            Reporter: Gregor Zurowski
>         Attachments: camel-jcr-existing-nodes.patch
>
>
> The CamelJcrInsert operation of the JcrProducer does not handle existing 
> nodes properly: If some of the nodes already exist within the path provided 
> by the CamelJcrNodeName property, they will be skipped, resulting in the 
> creation of an incorrect node hierarchy. Therefore, JcrProducer only works 
> correctly with newly created node hierarchies.
> E.g.: For the requested path "/exists1/exists2/new-node", JcrProducer will 
> incorrectly create "/new-node" (i.e. directly on root level), given nodes 
> "exists1" and "exists2" do already exist in the repository.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to