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

Carsten Ziegeler commented on SLING-2502:
-----------------------------------------

Thanks for the patch, Alex!

Looks basically good with one exception :) I guess if the property is just a 
registered prefix followed by a colon, this is an invalid name as well. So the 
prefix searching is only necessary if there is a colon and if the key has 
characters after the colon.
                
> Incorrect escaping of property names in JcrModifiablePropertyMap on save
> ------------------------------------------------------------------------
>
>                 Key: SLING-2502
>                 URL: https://issues.apache.org/jira/browse/SLING-2502
>             Project: Sling
>          Issue Type: Bug
>          Components: JCR
>    Affects Versions: JCR Resource 2.1.0
>            Reporter: Alex Parvulescu
>         Attachments: SLING-2502-v2.patch, SLING-2502.patch, 
> org.apache.sling.jcr.resource.patch
>
>
> This follows SLING-2425 where there was a change in the way JCR properties 
> are escaped before being persited.
> The regression happens for properties that have ":" in the name.
> For example: jcr:title after the #save() call will turn into jcr%3Atitle so 
> the node will end up with a brand new property, instead of having the old 
> jcr:title updated.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to