Include suggestions on what to customize when migrating to own FormatMappers

Author: marko-bekhta

extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/customized/BuiltinFormatMapperBehaviour.java

@@ -59,8 +59,8 @@ private static String message(String puName, String type, List<String> causes) {
                         + " and @PersistenceUnitExtension"
                         + (PersistenceUnitUtil.isDefaultPersistenceUnit(puName) ? "" : "(\"%1$s\")")
                         + " to address your database serialization/deserialization needs."
-                        + "\nThe reasons why the check was triggered are: \n\t-"
-                        + String.join("\n\t-", causes)
+                        + "\nThe reasons why the check was triggered are: \n\t- "
+                        + String.join("\n\t- ", causes)
                         + "\nSee the migration guide for more details and how to proceed.",
                 puName, type);
     }

extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/customized/JsonFormatterCustomizationCheck.java

@@ -75,7 +75,8 @@ public List<String> apply(ArcContainer container) {
                     }
                     // ObjectMapper was potentially customized
                     causes.add("Detected '" + handle.getBean().getBeanClass().getName() + "' bean registered. "
-                            + "It may have customized the ObjectMapper in a way not compatible with the default one.");
+                            + "It may have customized the ObjectMapper in a way not compatible with the default one. "
+                            + "Review the customizer and apply customizations you need for the database serialization/deserialization of JSON fields to your custom mapper.");
                 }
             }
 
@@ -87,15 +88,23 @@ public List<String> apply(ArcContainer container) {
                     "quarkus.jackson.write-durations-as-timestamps", "true",
                     "quarkus.jackson.accept-case-insensitive-enums", "false",
                     "quarkus.jackson.timezone", "UTC");
+            Map<String, String> actionsToTake = Map.of(
+                    "quarkus.jackson.write-dates-as-timestamps",
+                    "disable the corresponding serialization feature, i.e. 'mapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)'",
+                    "quarkus.jackson.write-durations-as-timestamps",
+                    "disable the corresponding serialization feature, i.e. 'mapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)'",
+                    "quarkus.jackson.accept-case-insensitive-enums",
+                    "enable the corresponding mapper feature, i.e. 'mapper.enable(MapperFeature.ACCEPT_CASE_INSENSITIVE_ENUMS)'",
+                    "quarkus.jackson.timezone", "configure the mapper timezone, i.e. 'mapper.setTimeZone(yourTimezone)'");
             for (String propertyName : ConfigProvider.getConfig().getPropertyNames()) {
                 if (propertyName.startsWith("quarkus.jackson.") && !acceptableConfigs.contains(propertyName)) {
                     String okValue = expectedDefaults.get(propertyName);
                     if (okValue != null && !okValue
                             .equalsIgnoreCase(ConfigProvider.getConfig().getConfigValue(propertyName).getRawValue())) {
                         causes.add("Detected '" + propertyName + "' property set to '"
                                 + ConfigProvider.getConfig().getConfigValue(propertyName).getRawValue() + "'. "
-                                + "For ObjectMapper being compatible with the clean, default one the expected value is: '"
-                                + okValue + "'.");
+                                + "For your custom ObjectMapper to be compatible with this configuration: "
+                                + actionsToTake.get(propertyName) + ".");
                     }
                 }
             }
@@ -130,7 +139,8 @@ public List<String> apply(ArcContainer container) {
 
                 // JSON-B was potentially customized
                 causes.add("Detected '" + handle.getBean().getBeanClass().getName() + "' bean registered. "
-                        + "It may have customized the Jsonb in a way not compatible with the default one.");
+                        + "It may have customized the Jsonb in a way not compatible with the default one. "
+                        + "Review the customizer and apply customizations you need for the database serialization/deserialization of JSON fields to your custom Jsonb.");
             }
 
             // JSON-B does not have the config properties so nothing else to check..