Mapping creates instances for null objects about modelmapper HOT 6 CLOSED

Two solutions I can see right now. The first is to set a property condition on your type map:

    modelMapper.createTypeMap(Source.class, Dest.class)
        .setPropertyCondition(Conditions.isNotNull());

This condition will be applied to all of the properties in the TypeMap (for a source and destination type), essentially, ensuring that nothing will be mapped for any source values that are null.

The second option is similar to what you were thinking with a global converter, which we don't support right now (since ModelMapper basically is the global converter). There is a notion of a global Provider though. A provider provides destination instances prior to their values being populated. You could try what you were thinking with a global provider set via: modelMapper.getConfiguration().setProvider.

I still intend to look into the problem of instantiating intermediate objects on the destination when mapping a source value that is null.

from modelmapper.

Thanks for reporting. This is a known issue (issue #6). I'll try to address it in the next release.

I'm curious, is this causing any difficulties for you?

from modelmapper.

Oh, sorry, somehow missed that issue. Yes, it is a blocker for us since we expect null values in our object graph in some cases. Looping through the whole tree seems to defeat the point of being able to map objects automatically!

I've already sub-classed ModelMapper to check for a null root value (which i think may be fixed if this issue is fixed) but I didn't see any easy way to solve the above.

@Override
public <D> D map(Object source, Class<D> destinationType)
{
    if (source == null)
        return null;

    return super.map(source, destinationType);
}

from modelmapper.

No worries. I was waiting to see if this issue would be a priority for anyone. Since it is, I'll make it a priority for myself, hopefully within the next day or two.

from modelmapper.

I was wondering if there was a way to also add a generic root level converter. This was my first stab at solving the problem without any code changes but I thought it might be a helpful feature if it could catch objects at all levels of mapping:

Converter<Object, Object> nullCheck = new AbstractConverter<Object, Object>() 
{
    protected Object convert (Object source)
    {
        if (source != null)
        {
            // return something else
        }

        return null;
    }
};

from modelmapper.

Fixed via 3107b8c

from modelmapper.