You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Christophe Lombart (JIRA)" <ji...@apache.org> on 2007/08/17 00:15:30 UTC
[jira] Resolved: (JCR-1053) Calling size method of a
ManageableArrayList causes NullPointerException
[ https://issues.apache.org/jira/browse/JCR-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christophe Lombart resolved JCR-1053.
-------------------------------------
Resolution: Fixed
Fixed. Can you check if it is ok for you ?
Right now, if there is no node matching to the collection element, the collection is null. We can imagine to return an empty collection. I think returning a null value or an empty collection is almost the same in this case. Please, add your comment if you are not agree. Thanks
> Calling size method of a ManageableArrayList causes NullPointerException
> ------------------------------------------------------------------------
>
> Key: JCR-1053
> URL: https://issues.apache.org/jira/browse/JCR-1053
> Project: Jackrabbit
> Issue Type: Bug
> Components: jcr-mapping
> Affects Versions: 1.3
> Reporter: André Bierwolf
> Assignee: Christophe Lombart
> Attachments: NTCollectionConverterImpl.diff
>
>
> When using the NTCollectionConverterImpl with proxy="true" a call on the size () method of a ManageableArrayList causes a NullPointerException if there is no underlying List. LazyCollectionLoader doLoad returns null because there is are no children.
> The ManageableArrayList is created because the isNull method of the NTCollectionConverterImpl class always returns false.
> According to the comment line this is done because the getCollectionNodes always returns a list.
> But after the fix for JCR-882 this is not correct anymore.
> The attached fix corrects this.
> The only question remaining is how to differ between an empty list and a null-value for the field containing the list.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.