added SET_PROPERTY_CREATOR_AS_DEFAULT MapperFeature

[lpandzic]

Fixes #1631.
The tests for this are in the parameter names module.

[cowtowncoder]

I'm sorry but I don't think this makes sense. This would enable auto-detection of all constructors, without annotation.

[lpandzic]

Only for those for which creator data is created. This solves the issue of not providing @JsonCreator when using parameter names module and since it's disabled by default, there's no bc issue.

[cowtowncoder]

Need javadoc, including both explanation and since 2.9 indicator.

[lpandzic]

added javadoc

[cowtowncoder]

Ah. Ok. So only gets called if "creator-ness" is first decided. Makes sense.

I think it'd be good to have a test case in jackson-databind itself if at all possible, since detection of cross-component breakages is bit more primitive at this point (Travis does have dependencies but it is easy to get detached, and seems to occasionally get false positives -- basically meaning some breakages are not immediately noticed).

[lpandzic]

I agree that there should be a test, but I'm not sure what use case to cover without ParameterNamesModule. Constructor with JsonProperty parameters and without JsonCreator won't work.
Maybe a unit test?

[cowtowncoder]

@lpandzic Right, a unit test could work. There are couple of tests that implement mock AnnotationIntrospector to imitate parameter names module (and Scala module's name handling functionality). I'll merge this and see if I can create a test.