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] Commented: (JCR-1339) ManageableCollectionUtil doesn't support Maps
Date Thu, 10 Apr 2008 19:58:04 GMT

    [ https://issues.apache.org/jira/browse/JCR-1339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12587767#action_12587767
] 

Christophe Lombart commented on JCR-1339:
-----------------------------------------

I just commited a better support for the Map, there are 2 ways to define a Map attributes
: 

Class A 
{

// For unparametrized Map, it is mandatory to define the elementClassName
@Collection (elementClassName=Element.class) Map map1; 

 // For parametized Map, the elementClassName is not necessay
 @Collection private Map<String, MyClass> map2;


}

Furthermore, The class ManageableMap is used only internally by the OCM framework. 

> ManageableCollectionUtil doesn't support Maps
> ---------------------------------------------
>
>                 Key: JCR-1339
>                 URL: https://issues.apache.org/jira/browse/JCR-1339
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-ocm
>    Affects Versions: 1.4
>            Reporter: Aidan O'Loan
>            Assignee: Christophe Lombart
>             Fix For: 1.5
>
>         Attachments: ManageableCollectionUtil_JCR-1339.txt
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> ManageableCollectionUtil has two getManageableCollection methods, which do not currently
return a ManageableCollection which wraps Maps. 
> ManagedHashMap already exists in the codebase which I assume was created for this purpose,
so both getManageableCollection methods could be modified so that they do something like:
>             if (object instanceof Map){
>                 return new ManagedHashMap((Map)object);
>             }
> An alternative solution might be to modify the JCR mapping to support explicitly defining
the 'ManagedXXX' class.

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