jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christophe Lombart (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (JCR-1053) Calling size method of a ManageableArrayList causes NullPointerException
Date Thu, 16 Aug 2007 22:15:30 GMT

     [ 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.


Mime
View raw message