[jira] [Commented] (OAK-11516) org.apache.jackrabbit.oak.jcr.observation.QueuingHandler uses paths where names are expected

["Manfred Baedke (Jira)" <ji...@apache.org>]

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

Manfred Baedke commented on OAK-11516:
--------------------------------------

Looks like there is no reason to require these strings to be JCR names, see PR.

> org.apache.jackrabbit.oak.jcr.observation.QueuingHandler uses paths where names are expected
> --------------------------------------------------------------------------------------------
>
>                 Key: OAK-11516
>                 URL: https://issues.apache.org/jira/browse/OAK-11516
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: jcr
>            Reporter: Manfred Baedke
>            Assignee: Manfred Baedke
>            Priority: Major
>
> https://github.com/apache/jackrabbit-oak/blob/8512acc1c564df8d7e467f37ea39e01035b410e9/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/QueueingHandler.java#L162
> https://github.com/apache/jackrabbit-oak/blob/8512acc1c564df8d7e467f37ea39e01035b410e9/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/QueueingHandler.java#L216
> Found while working on OAK-11486, which will patch the method LocalNameMapper#getJcrName(String oakName). So far, this method used to return just oakName if there are no local namespace remappings. It even does that if oakName is not a name at all. This will (likely) change with the patch for OAK-11486.
> I'm not sure if the names in question are supposed to be JCR names or just strings, but they will be be passed to LocalNameMapper#getJcrName(String oakName) during event consumption.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)