Change BeanDeserializerFactory.addBeanProps to check creator properties

[zhenlin]

To address #4920, changed BeanDeserializerFactory.addBeanProps to check creator properties when a ValueInstantiator is available.

[cowtowncoder]

Ok, fix makes sense, I think, and looks good overall. Thank you!

Just two things... first, although I can merge it for 2.19 as submitted, it might make more sense to create PR against 2.18 branch instead, so it could be included in 2.18.3 patch.

Second: before I merge pr (regardless of target branch), I'd need CLA (just needs to be done once before the first pr merged). It's from here:

https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

and is usually easiest to do by printing, filling & signing, scanning/photo, emailing to cla at fasterxml dot com.

Once this is done I can proceed with merging.

Thank you again for submitting this, looking forward to merging.

[zhenlin]

Thanks. I have opened #4984 to backport this to 2.18. Please let me know if there is anything else I can help with to get this fix released.

[cowtowncoder]

@zhenlin See my note on CLA, that's the main thing.

On branch: going forward one pr is fine, starting with the oldest target: then we just merge forward.

[zhenlin]

Sure. I sent the signed CLA earlier.

[cowtowncoder]

@zhenlin wow, that was quick! Did not realize, saw it now.

[cowtowncoder]

Closing in favor of #4984 (will merge forward)