Some more code clean up

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/BeanDescription.java

@@ -52,6 +52,11 @@ protected BeanDescription(JavaType type) {
 
     public Class<?> getBeanClass() { return _type.getRawClass(); }
 
+    /**
+     * @since 2.15
+     */
+    public boolean isRecordType() { return _type.isRecordType(); }
+
     /**
      * @since 2.9
      */

src/main/java/com/fasterxml/jackson/databind/deser/BasicDeserializerFactory.java

@@ -24,7 +24,6 @@
 import com.fasterxml.jackson.databind.exc.InvalidDefinitionException;
 import com.fasterxml.jackson.databind.ext.OptionalHandlerFactory;
 import com.fasterxml.jackson.databind.introspect.*;
-import com.fasterxml.jackson.databind.jdk14.JDK14Util;
 import com.fasterxml.jackson.databind.jsontype.NamedType;
 import com.fasterxml.jackson.databind.jsontype.TypeDeserializer;
 import com.fasterxml.jackson.databind.jsontype.TypeResolverBuilder;
@@ -540,9 +539,9 @@ protected void _addImplicitConstructorCreators(DeserializationContext ctxt,
                 JacksonInject.Value injectable = intr.findInjectableValue(param);
                 final PropertyName name = (propDef == null) ? null : propDef.getFullName();
 
-                if (propDef != null
-                        // NOTE: Record canonical constructor will have implicitly named propDef
-                        && (propDef.isExplicitlyNamed() || beanDesc.getType().isRecordType())) {
+                if ((propDef != null)
+                        // [databind#3724]: Record canonical constructor will have implicitly named propDef
+                        && (propDef.isExplicitlyNamed() || beanDesc.isRecordType())) {
                     ++explicitNameCount;
                     properties[i] = constructCreatorProperty(ctxt, beanDesc, name, i, param, injectable);
                     continue;

src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializer.java

@@ -466,12 +466,11 @@ protected Object _deserializeUsingPropertyBased(final JsonParser p, final Deseri
             }
             // regular property? needs buffering
             SettableBeanProperty prop = _beanProperties.find(propName);
-            // Special handling because Records' ignored creator props weren't removed (to help in creating
-            // constructor-backed PropertyCreator) so they ended up in _beanProperties, unlike POJO (whose ignored
+            // [databind#3724]: Special handling because Records' ignored creator props
+            // weren't removed (to help in creating constructor-backed PropertyCreator)
+            // so they ended up in _beanProperties, unlike POJO (whose ignored
             // props are removed)
-            boolean isClassWithoutMutator = _beanType.isRecordType();
-
-            if (prop != null && !isClassWithoutMutator) {
+            if ((prop != null) && !_beanType.isRecordType()) {
                 try {
                     buffer.bufferProperty(prop, _deserializeWithErrorWrapping(p, ctxt, prop));
                 } catch (UnresolvedForwardReference reference) {

src/main/java/com/fasterxml/jackson/databind/introspect/POJOPropertiesCollector.java

@@ -220,7 +220,14 @@ public MapperConfig<?> getConfig() {
     public JavaType getType() {
         return _type;
     }
-    
+
+    /**
+     * @since 2.15
+     */
+    public boolean isRecordType() {
+        return _type.isRecordType();
+    }
+
     public AnnotatedClass getClassDef() {
         return _classDef;
     }
@@ -629,7 +636,7 @@ protected void _addCreators(Map<String, POJOPropertyBuilder> props)
                 }
             }
         }
-        if (_classDef.getType().isRecordType()) {
+        if (isRecordType()) {
             List<String> recordComponentNames = new ArrayList<String>();
             AnnotatedConstructor canonicalCtor = JDK14Util.findRecordConstructor(
                     _classDef, _annotationIntrospector, _config, recordComponentNames);
@@ -947,9 +954,9 @@ protected void _removeUnwantedProperties(Map<String, POJOPropertyBuilder> props)
             }
             // Otherwise, check ignorals
             if (prop.anyIgnorals()) {
-                // Special handling for Records, as they do not have mutators so relying on constructors with (mostly)
-                // implicitly-named parameters...
-                if (_classDef.getType().isRecordType()) {
+                // Special handling for Records, as they do not have mutators so relying on constructors
+                // with (mostly)  implicitly-named parameters...
+                if (isRecordType()) {
                     // ...so can only remove ignored field and/or accessors, not constructor parameters that are needed
                     // for instantiation...
                     prop.removeIgnored();