Default Values for Optionals Or Reject All Nulls

Hello,

I think your library is great but it doesn’t quite work for one Scala use case. When there is a property of type Option<T> I want it to set a default value of none even when FAIL_ON_MISSING_CREATOR properties is true.

I would like to do this conventionally without having to apply any annotations to the code. Could you give me advice on the best way to implement this and I will do the work and send a pull request.

Something in here seems sensible:

 protected Object _findMissing(SettableBeanProperty prop) throws JsonMappingException
    {
        // First: do we have injectable value?
        Object injectableValueId = prop.getInjectableValueId();
        if (injectableValueId != null) {
            return _context.findInjectableValue(prop.getInjectableValueId(),
                    prop, null);
        }
        // Second: required?
        if (prop.isRequired()) {
            throw _context.mappingException("Missing required creator property '%s' (index %d)",
                    prop.getName(), prop.getCreatorIndex());
        }
        if (_context.isEnabled(DeserializationFeature.FAIL_ON_MISSING_CREATOR_PROPERTIES)) {
            throw _context.mappingException("Missing creator property '%s' (index %d); DeserializationFeature.FAIL_ON_MISSING_CREATOR_PROPERTIES enabled",
                    prop.getName(), prop.getCreatorIndex());
        }
        // Third: default value
        JsonDeserializer<Object> deser = prop.getValueDeserializer();
        return deser.getNullValue(_context);
    }

Perhaps a new service “defaultCreatorValueProvider” or similar?

If there is a better way, or even an existing way, please let me know.

Thanks

[NTCoding]

Hi,

I understand that you are a Java developer and perhaps are not familiar with monads and optionals (apologies if that is condescending). However, an Option[T] is explicitly stating that the type is optional - part of the type’s DNA is that there may be no value for it - and hence a missing constructor value means it should default to a value of None.

It is explained in more detail here: https://github.com/FasterXML/jackson-module-scala/issues/203

If an object has two properties - one is optional and one is not - the type system is expressing what is mandatory - the optional values. Is there any way to do this currently - if the value is missing throw an exception unless it is an Option?

In the “other case” you mention, then it would not fail if a value was missing from a non-optional type and the value would default to null. Would you consider a pull request with a new deserialization option: “REJECT_ALL_NULL_VALUES” or are you completely against adding new functionality that helps to support Scala optionals?

[cowtowncoder]

Replaced with #1402, which I think would provide all the pieces needed. Scala module might need something additional (and definitely needs to support this); but from jackson-databind perspective I think this should be sufficient. Planning to get implemented for 2.9.