jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jukka Zitting (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JCR-1328) Session.itemExists implementation wrong
Date Fri, 18 Jan 2008 14:28:58 GMT

    [ https://issues.apache.org/jira/browse/JCR-1328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12560402#action_12560402

Jukka Zitting commented on JCR-1328:

Good point. I think Session.itemExists() is about the only method in JCR that explicitly mentions
how malformed paths should be handled.

On a more general note, I'd actually prefer to modify all methods that take JCR paths to treat
the referenced item as not found instead of throwing an exception if given a malformed path.
Only modification methods like addNode() should IMHO throw exceptions on malformed paths.
But thats a topic for another issue.

> Session.itemExists implementation wrong
> ---------------------------------------
>                 Key: JCR-1328
>                 URL: https://issues.apache.org/jira/browse/JCR-1328
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-core
>    Affects Versions: 1.4
>            Reporter: Felix Meschberger
> IMHO the implementation of the Session.itemExists(String) method is wrong when called
with a malformed path such as "/a/b/c/*" (note the trailing star). According to the spec,
the method must return "false" for a malformed path like this.
> In reality, the method throws a RepositoryException which is allowed to be thrown by
the spec "if an error occurrs" (whatever that means). But catching this exception means, we
cannot handle it: Is it a connection issue or a general repository problem ? If so, I cannot
do anything about it. It is really a path problem, I can do something about it. But how do
I know (without rebuilding internals) ?
> See also SLING-152 for  more info.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message