[
https://issues.apache.org/jira/browse/JCR-1678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12614717#action_12614717
]
angela commented on JCR-1678:
-----------------------------
i applied the patch and created a test wrapping LockException. this revealed
one drawback of the patch: the order of the map-values is not defined,
resulting in the first matching exception being taken. in my case it was
RepositoryException where i expected LockException. i would therefore suggest
to use a LinkedHashMap can move the entry for RepositoryException (the very
base) to the end.
> NullPointerException in constructor of JcrDavException
> ------------------------------------------------------
>
> Key: JCR-1678
> URL: https://issues.apache.org/jira/browse/JCR-1678
> Project: Jackrabbit
> Issue Type: Bug
> Components: jackrabbit-webdav
> Affects Versions: 1.4
> Reporter: Thomas Weber
> Assignee: angela
> Attachments: patch
>
>
> within DavSessionProviderImpl.attachSession() a
> org.apache.jackrabbit.rmi.client.RemoteRepositoryException is thrown, catched
> as RepositoryException and passed into the
> ctor of JcrDavException
> the lookup for the class in the static HashMap codeMap for the class fails
> and a NPE is thrown
> suggested fix:
> The static lookup by the javax.jcr.*Exception classed must be fixed to
> include derived exception classes.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
