Create Logger, replace existing logging

src/main/java/io/beanmapper/config/OverrideField.java

@@ -2,7 +2,7 @@
 
 import java.util.function.Supplier;
 
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperPerformanceLogger;
 
 public class OverrideField<T> {
 
@@ -34,7 +34,7 @@ public T get() {
             return null;
         }
         if (this.value == null) {
-            this.value = BeanMapperLogger.logTimed("Retrieving nested configuration field.", this.supplier);
+            this.value = BeanMapperPerformanceLogger.runTimedTask("Retrieving nested configuration field.", this.supplier);
         }
         return value;
     }

src/main/java/io/beanmapper/config/StrictMappingProperties.java

@@ -2,7 +2,7 @@
 
 import io.beanmapper.core.unproxy.BeanUnproxy;
 import io.beanmapper.core.unproxy.SkippingBeanUnproxy;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperPerformanceLogger;
 
 public class StrictMappingProperties {
 
@@ -80,7 +80,7 @@ public void setApplyStrictMappingConvention(boolean applyStrictMappingConvention
     }
 
     public BeanPair createBeanPair(Class<?> sourceClass, Class<?> targetClass) {
-        BeanPair beanPair = BeanMapperLogger.logTimed("Creating BeanPair, and unproxying source %s-class and target %s-class.".formatted(sourceClass.getCanonicalName(), targetClass.getCanonicalName()), () -> {
+        BeanPair beanPair = BeanMapperPerformanceLogger.runTimedTask("Creating BeanPair, and unproxying source %s-class and target %s-class.".formatted(sourceClass.getCanonicalName(), targetClass.getCanonicalName()), () -> {
             Class<?> unproxiedSource = beanUnproxy.unproxy(sourceClass);
             Class<?> unproxiedTarget = beanUnproxy.unproxy(targetClass);
             return new BeanPair(unproxiedSource, unproxiedTarget);

src/main/java/io/beanmapper/core/BeanMatch.java

@@ -8,8 +8,13 @@
 import io.beanmapper.config.BeanPair;
 import io.beanmapper.exceptions.BeanNoSuchPropertyException;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 public class BeanMatch {
 
+    private static final Logger log = LoggerFactory.getLogger(BeanMatch.class);
+
     private final BeanPair beanPair;
 
     private final Map<String, BeanProperty> sourceNodes;
@@ -106,6 +111,7 @@ private void checkForMandatoryUnmatchedNodes(String side, Class<?> containingCla
         for (Map.Entry<String, BeanProperty> entry : nodes.entrySet()) {
             BeanProperty currentField = entry.getValue();
             if (currentField.isUnmatched()) {
+                log.error("{} {} has no match for property {}", side, containingClass.isAnonymousClass(), entry.getKey());
                 throw new BeanNoSuchPropertyException(side + " " + containingClass.getCanonicalName() + " has no match for property " + entry.getKey());
             }
         }

src/main/java/io/beanmapper/core/BeanMatchStore.java

@@ -34,7 +34,7 @@
 import io.beanmapper.exceptions.BeanMissingPathException;
 import io.beanmapper.exceptions.BeanNoSuchPropertyException;
 import io.beanmapper.exceptions.FieldShadowingException;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
 import io.beanmapper.utils.Trinary;
 
 public class BeanMatchStore {
@@ -284,7 +284,7 @@ private BeanPropertyWrapper dealWithBeanProperty(BeanPropertyMatchupDirection ma
                                 .determineNodesForPath());
             } catch (BeanNoSuchPropertyException err) {
 
-                    BeanMapperLogger.log("""
+                    BeanMapperTraceLogger.log("""
                             BeanNoSuchPropertyException thrown by BeanMatchStore#dealWithBeanProperty(BeanPropertyMatchupDirection, Map<String, BeanProperty>, Class, PropertyAccessor), for {}.
                             {}""", wrapper.getName(), err.getMessage());
 

src/main/java/io/beanmapper/core/BeanPropertyCreator.java

@@ -6,8 +6,13 @@
 import io.beanmapper.core.inspector.PropertyAccessors;
 import io.beanmapper.exceptions.BeanNoSuchPropertyException;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 public class BeanPropertyCreator {
 
+    private static final Logger log = LoggerFactory.getLogger(BeanPropertyCreator.class);
+
     private final BeanPropertyMatchupDirection matchupDirection;
 
     private final Class<?> baseClass;
@@ -68,6 +73,7 @@ private void traversePath(Stack<BeanProperty> beanProperties) {
         for (String node : route.getRoute()) {
             final PropertyAccessor property = PropertyAccessors.findProperty(currentBaseClass, node);
             if (property == null) {
+                log.error("Property '{}' does not exist in: {}", node, currentBaseClass.getSimpleName());
                 throw new BeanNoSuchPropertyException("Property '" + node + "' does not exist in: " + currentBaseClass.getSimpleName());
             }
             beanProperties.push(new BeanProperty(

src/main/java/io/beanmapper/core/BeanPropertyMatch.java

@@ -10,10 +10,15 @@
 import io.beanmapper.exceptions.BeanMappingException;
 import io.beanmapper.exceptions.BeanNoLogicSecuredCheckSetException;
 import io.beanmapper.exceptions.BeanNoRoleSecuredCheckSetException;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class BeanPropertyMatch {
 
+    private static final Logger log = LoggerFactory.getLogger(BeanPropertyMatch.class);
+
     private final BeanMatch beanMatch;
     private final Object source;
     private final Object target;
@@ -77,7 +82,7 @@
             if (enforcedSecuredProperties) {
                 throw new BeanNoLogicSecuredCheckSetException(message);
             }
-            BeanMapperLogger.warn(message);
+            BeanMapperTraceLogger.log(message);
             return true;
         }
         return logicSecuredCheck.isAllowed(source, target);
@@ -92,7 +97,7 @@
             if (enforcedSecuredProperties) {
                 throw new BeanNoRoleSecuredCheckSetException(message);
             }
-            BeanMapperLogger.warn(message);
+            log.warn(message);
         }
     }
 

src/main/java/io/beanmapper/core/BeanStrictMappingRequirementsException.java

@@ -3,10 +3,14 @@
 import java.util.Collections;
 import java.util.List;
 
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class BeanStrictMappingRequirementsException extends RuntimeException {
 
+    private static final Logger log = LoggerFactory.getLogger(BeanStrictMappingRequirementsException.class);
     private final List<BeanMatchValidationMessage> validationMessages;
 
     public BeanStrictMappingRequirementsException(BeanMatchValidationMessage validationMessage) {
@@ -23,7 +27,7 @@ private void logErrors(List<BeanMatchValidationMessage> validationMessages) {
             if (validationMessage.isLogged()) {
                 continue;
             }
-            BeanMapperLogger.error("""
+            log.error("""
                             Missing matching properties for source [{}] {} > target [{}] {} for fields:
                             """,
                     validationMessage.getSourceClass().getCanonicalName(),
@@ -32,7 +36,7 @@ private void logErrors(List<BeanMatchValidationMessage> validationMessages) {
                     (validationMessage.isTargetStrict() ? "*" : ""));
 
             for (BeanProperty field : validationMessage.getFields()) {
-                BeanMapperLogger.error("""
+                log.error("""
                                 > {}.{}
                                 """,
                         validationMessage.getStrictClass().getSimpleName(),

src/main/java/io/beanmapper/core/collections/AbstractCollectionHandler.java

@@ -5,7 +5,7 @@
 import io.beanmapper.config.CollectionFlusher;
 import io.beanmapper.core.constructor.DefaultBeanInitializer;
 import io.beanmapper.exceptions.BeanCollectionUnassignableTargetCollectionTypeException;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperPerformanceLogger;
 import io.beanmapper.utils.Classes;
 
 public abstract class AbstractCollectionHandler<C> implements CollectionHandler<C> {
@@ -33,7 +33,7 @@ public Object mapItem(
             BeanMapper beanMapper,
             Class<?> collectionElementClass,
             Object source) {
-        return BeanMapperLogger.logTimed("Recursively calling BeanMapper#map(Object), to map collection elements of type %s, to %s."
+        return BeanMapperPerformanceLogger.runTimedTask("Recursively calling BeanMapper#map(Object), to map collection elements of type %s, to %s."
                 .formatted(source != null
                         ? source.getClass().getCanonicalName()
                         : "null", collectionElementClass.getCanonicalName()),

src/main/java/io/beanmapper/core/constructor/DefaultBeanInitializer.java

@@ -12,17 +12,22 @@
 import io.beanmapper.BeanMapper;
 import io.beanmapper.config.BeanMapperBuilder;
 import io.beanmapper.strategy.ConstructorArguments;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
 import io.beanmapper.utils.DefaultValues;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 public class DefaultBeanInitializer implements BeanInitializer {
 
+    private static final Logger log = LoggerFactory.getLogger(DefaultBeanInitializer.class);
+
     /**
      * {@inheritDoc}
      */
     @Override
     public <T> T instantiate(Class<T> beanClass, ConstructorArguments arguments) {
-        BeanMapperLogger.log("Creating a new instance of type %s, using reflection.".formatted(beanClass));
+        BeanMapperTraceLogger.log("Creating a new instance of type %s, using reflection.".formatted(beanClass));
         try {
             if (arguments == null) {
                 return beanClass.getConstructor().newInstance();
@@ -31,7 +36,7 @@
             var constructorParameterTypes = Arrays.stream(constructor.getParameters()).map(Parameter::getParameterizedType).toArray(Type[]::new);
             return beanClass.getConstructor(arguments.getTypes()).newInstance(mapParameterizedArguments(constructorParameterTypes, arguments.getValues()));
         } catch (NoSuchMethodException | InstantiationException | IllegalAccessException | InvocationTargetException e) {
-            BeanMapperLogger.error("Could not instantiate bean of class %s. Returning the default value associated with the given type. %s".formatted(beanClass.getName(),
+            log.error("Could not instantiate bean of class %s. Returning the default value associated with the given type. %s".formatted(beanClass.getName(),
                     e.getMessage()));
             return DefaultValues.defaultValueFor(beanClass);
         }

src/main/java/io/beanmapper/core/converter/collections/CollectionConverter.java

@@ -4,7 +4,7 @@
 import io.beanmapper.core.BeanPropertyMatch;
 import io.beanmapper.core.collections.CollectionHandler;
 import io.beanmapper.core.converter.BeanConverter;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperPerformanceLogger;
 
 public class CollectionConverter implements BeanConverter {
 
@@ -25,7 +25,7 @@ public <R, U> U convert(
             return targetClass.cast(source);
         }
 
-        return BeanMapperLogger.logTimed("Calling BeanMapper#map(Object) recursively, to convert object of type %s, to type %s."
+        return BeanMapperPerformanceLogger.runTimedTask("Calling BeanMapper#map(Object) recursively, to convert object of type %s, to type %s."
                 .formatted(source != null
                         ? source.getClass().getCanonicalName()
                         : "null", targetClass.getCanonicalName()),

src/main/java/io/beanmapper/core/converter/impl/ObjectToOptionalConverter.java

@@ -10,6 +10,9 @@
 import io.beanmapper.core.converter.BeanConverter;
 import io.beanmapper.exceptions.BeanNoSuchPropertyException;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * This converter facilitates the conversion of an object to an Optional wrapping another object. This converter does
  * not support the conversion of complex datastructures, such as Collections, to Optionals. If that functionality is
@@ -18,6 +21,8 @@
  */
 public class ObjectToOptionalConverter implements BeanConverter {
 
+    private static final Logger log = LoggerFactory.getLogger(ObjectToOptionalConverter.class);
+
     @Override
     public <S, T> T convert(BeanMapper beanMapper, S source, Class<T> targetClass, BeanPropertyMatch beanPropertyMatch) {
         if (source == null) {
@@ -28,6 +33,7 @@
         try {
             targetField = beanPropertyMatch.getTarget().getClass().getDeclaredField(beanPropertyMatch.getTargetFieldName());
         } catch (NoSuchFieldException e) {
+            log.error(e.getMessage());
             throw new BeanNoSuchPropertyException(e.getMessage());
         }
 

src/main/java/io/beanmapper/core/converter/impl/OptionalToObjectConverter.java

@@ -12,7 +12,7 @@
 import io.beanmapper.BeanMapper;
 import io.beanmapper.core.BeanPropertyMatch;
 import io.beanmapper.core.converter.BeanConverter;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
 import io.beanmapper.utils.Classes;
 
 /**
@@ -32,7 +32,7 @@ public <S, T> T convert(BeanMapper beanMapper, S source, Class<T> targetClass, B
 
         if (targetClass.equals(Optional.class)) {
             // Not always possible to get the actual source class, so just report the name of the field. Debug log will show the call stack.
-            BeanMapperLogger.log("Converting Optional to Optional. Perhaps the target does not need to be an Optional?\nSource-field: {}\nTarget: {}.{}",
+            BeanMapperTraceLogger.log("Converting Optional to Optional. Perhaps the target does not need to be an Optional?\nSource-field: {}\nTarget: {}.{}",
                     beanPropertyMatch.getSourceFieldName(),
                     beanPropertyMatch.getTarget().getClass(),
                     beanPropertyMatch.getTargetFieldName());

src/main/java/io/beanmapper/core/unproxy/SkippingBeanUnproxy.java

@@ -6,7 +6,7 @@
 import java.util.HashSet;
 import java.util.Set;
 
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
 
 /**
  * Unproxy that allows you to configure classes to skip.
@@ -33,7 +33,7 @@ public SkippingBeanUnproxy(BeanUnproxy delegate) {
      */
     @Override
     public Class<?> unproxy(Class<?> beanClass) {
-        BeanMapperLogger.log("Unproxying class %s.".formatted(beanClass != null ? beanClass.getCanonicalName() : "null"));
+        BeanMapperTraceLogger.log("Unproxying class %s.".formatted(beanClass != null ? beanClass.getCanonicalName() : "null"));
         if (isSkippedProxyClass(beanClass)) {
             return beanClass;
         }

src/main/java/io/beanmapper/exceptions/BeanNoLogicSecuredCheckSetException.java

@@ -1,11 +1,8 @@
 package io.beanmapper.exceptions;
 
-import io.beanmapper.utils.BeanMapperLogger;
-
 public class BeanNoLogicSecuredCheckSetException extends IllegalArgumentException {
 
     public BeanNoLogicSecuredCheckSetException(String message) {
         super(message);
-        BeanMapperLogger.error(message);
     }
 }

src/main/java/io/beanmapper/exceptions/BeanNoRoleSecuredCheckSetException.java

@@ -1,11 +1,8 @@
 package io.beanmapper.exceptions;
 
-import io.beanmapper.utils.BeanMapperLogger;
-
 public class BeanNoRoleSecuredCheckSetException extends IllegalArgumentException {
 
     public BeanNoRoleSecuredCheckSetException(String message) {
         super(message);
-        BeanMapperLogger.error(message);
     }
 }

src/main/java/io/beanmapper/exceptions/BeanNoSuchPropertyException.java

@@ -3,16 +3,12 @@
  */
 package io.beanmapper.exceptions;
 
-import io.beanmapper.utils.BeanMapperLogger;
-
 /**
  * Exception thrown when a property could not be found.
  */
 public class BeanNoSuchPropertyException extends IllegalArgumentException {
 
     public BeanNoSuchPropertyException(String message) {
         super(message);
-        BeanMapperLogger.error(message);
     }
-
 }

src/main/java/io/beanmapper/strategy/AbstractMapStrategy.java

@@ -12,7 +12,7 @@
 import io.beanmapper.core.converter.BeanConverter;
 import io.beanmapper.exceptions.BeanConversionException;
 import io.beanmapper.exceptions.BeanPropertyNoMatchException;
-import io.beanmapper.utils.BeanMapperLogger;
+import io.beanmapper.utils.BeanMapperTraceLogger;
 import io.beanmapper.utils.Records;
 
 public abstract class AbstractMapStrategy implements MapStrategy {
@@ -107,7 +107,7 @@ private void dealWithMappableNestedClass(BeanPropertyMatch beanPropertyMatch) {
         Object encapsulatedSource = beanPropertyMatch.getSourceObject();
         Object target;
         if (encapsulatedSource != null) {
-            BeanMapperLogger.log("    {");
+            BeanMapperTraceLogger.log("    {");
             BeanMapper localBeanMapper = getBeanMapper()
                     .wrap()
                     .setParent(beanPropertyMatch.getTarget())
@@ -118,7 +118,7 @@ private void dealWithMappableNestedClass(BeanPropertyMatch beanPropertyMatch) {
                 target = localBeanMapper.map(encapsulatedSource, beanPropertyMatch.getTargetObject());
             }
             beanPropertyMatch.writeObject(target);
-            BeanMapperLogger.log("    }");
+            BeanMapperTraceLogger.log("    }");
         }
     }
 
@@ -135,7 +135,7 @@ public Object convert(Object value, Class<?> targetClass, BeanPropertyMatch bean
         BeanConverter converter = getConverterOptional(valueClass, targetClass);
 
         if (converter != null) {
-            BeanMapperLogger.log("{}{}{}", INDENT, converter.getClass().getSimpleName(), ARROW);
+            BeanMapperTraceLogger.log("{}{}{}", INDENT, converter.getClass().getSimpleName(), ARROW);
             BeanMapper wrappedBeanMapper = beanMapper
                     .wrap()
                     .setParent(beanPropertyMatch.getTarget())
@@ -207,9 +207,9 @@ private void processProperty(BeanPropertyMatch beanPropertyMatch) {
             return;
         }
         if (beanPropertyMatch.isMappable()) {
-            BeanMapperLogger.log("{}{}", beanPropertyMatch.sourceToString(), ARROW);
+            BeanMapperTraceLogger.log("{}{}", beanPropertyMatch.sourceToString(), ARROW);
             copySourceToTarget(beanPropertyMatch);
-            BeanMapperLogger.log("{}{}", INDENT, beanPropertyMatch.targetToString());
+            BeanMapperTraceLogger.log("{}{}", INDENT, beanPropertyMatch.targetToString());
         }
     }