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

Jukka Zitting commented on OAK-563:
-----------------------------------

I wouldn't mind for something like {{getNode(":")}} to throw an 
{{IllegalArgumentException}} or something similar as such an argument is 
invalid regardless of repository content (it is a client error to pass 
something like that as a path). But as argued above, a case like 
{{getNode("\{http://purl.org/dc/elements/1.1/}title";)}} should IMHO result in a 
{{PathNotFoundException}} when the namespace is not registered. The case of 
{{getNode("foo:bar")}}, where "foo" is an unknown prefix, could I guess be 
handled either way.
                
> Rename SessionDelegate...OrThrow methods
> ----------------------------------------
>
>                 Key: OAK-563
>                 URL: https://issues.apache.org/jira/browse/OAK-563
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core
>            Reporter: Michael Dürig
>            Assignee: Michael Dürig
>             Fix For: 0.6
>
>         Attachments: OAK-563.patch
>
>
> As a fallout of OAK-553 there are some inconsistency in method naming in 
> {{SessionDelegate}} now. We should align the names of the ...OrThrow methods. 
> That is, drop the OrThrow suffixes.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to