community-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Luciano Resende <luckbr1...@gmail.com>
Subject Re: OCM - Bug fix - @Collection
Date Wed, 16 Sep 2015 19:01:00 GMT
You might want to redirect your question to Apache Jackrabbit mailing list

http://jackrabbit.apache.org/jcr/mailing-lists.html

On Wed, Sep 16, 2015 at 6:07 AM, Ricardo Ulate <ricardo.ulate@valtech.de>
wrote:

>
> Hi,
>
> I'm sure I'm not using the right channel, but this page
> http://www.apache.org/dev/contrib-email-tips.html
> Was too confusing, I don't want to invest 1h time to help out with a
> simple fix.
>
>
>
> When using the @Collection tag on lists (where the element is a node WITH
> an ID):
>
> The class DefaultCollectionConverterImpl (in version 2.0.0 of the OCM)
> around lines at line 230:
>
>
>         while (collectionIterator.hasNext()) {
>
>             Object item = collectionIterator.next();
>
>             String elementJcrName = null;
>
>
>
>             if (elementClassDescriptor.hasUUIdField()){
>
>             elementJcrName = collectionDescriptor.getJcrElementName();
>
>             elementJcrName = (elementJcrName == null)?
> COLLECTION_ELEMENT_NAME : elementJcrName;
>
>
>
> Should be:
>
>         while (collectionIterator.hasNext()) {
>
>             Object item = collectionIterator.next();
>
>             String elementJcrName = null;
>
>
>
>             if (elementClassDescriptor.hasUUIdField()){
>
> if (elementClassDescriptor.hasIdField()) {
>
>                  String idFieldName =
> elementClassDescriptor.getIdFieldDescriptor().getFieldName();
>
>                 elementJcrName = ReflectionUtils.getNestedProperty(item,
> idFieldName).toString();
>
> }
>
> else {
>
>             elementJcrName = collectionDescriptor.getJcrElementName();
>
> }
>
>             elementJcrName = (elementJcrName == null)?
> COLLECTION_ELEMENT_NAME : elementJcrName;
>
>
> Cheers,
>
> Ricardo
>



-- 
Luciano Resende
http://people.apache.org/~lresende
http://twitter.com/lresende1975
http://lresende.blogspot.com/

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message