(tapestry-5) 03/03: Added a warning log message for entity classes without single id attribute.

[thiagohp]

This is an automated email from the ASF dual-hosted git repository.

thiagohp pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tapestry-5.git

commit 404912792e9740eabea288641862e7a085e3c2af
Author: Volker Lamp <vl...@apache.org>
AuthorDate: Wed Oct 16 22:54:11 2024 +0200

    Added a warning log message for entity classes without single id attribute.
---
 .../hibernate/web/modules/HibernateModule.java     | 39 ++++++++++++----------
 1 file changed, 22 insertions(+), 17 deletions(-)

diff --git 
a/tapestry-hibernate/src/main/java/org/apache/tapestry5/hibernate/web/modules/HibernateModule.java
 
b/tapestry-hibernate/src/main/java/org/apache/tapestry5/hibernate/web/modules/HibernateModule.java
index 8ca7403bc..0d02fff33 100644
--- 
a/tapestry-hibernate/src/main/java/org/apache/tapestry5/hibernate/web/modules/HibernateModule.java
+++ 
b/tapestry-hibernate/src/main/java/org/apache/tapestry5/hibernate/web/modules/HibernateModule.java
@@ -50,6 +50,7 @@ import 
org.apache.tapestry5.services.dashboard.DashboardManager;
 import org.apache.tapestry5.services.dashboard.DashboardTab;
 import org.apache.tapestry5.services.transform.ComponentClassTransformWorker2;
 import org.hibernate.Session;
+import org.slf4j.Logger;
 
 /**
  * Supplements the services defined by {@link 
org.apache.tapestry5.hibernate.modules.HibernateCoreModule} with additional
@@ -93,7 +94,7 @@ public class HibernateModule
             @Symbol(HibernateSymbols.PROVIDE_ENTITY_VALUE_ENCODERS) boolean 
provideEncoders,
             final HibernateSessionSource sessionSource, final Session session,
             final TypeCoercer typeCoercer, final PropertyAccess propertyAccess,
-            final LoggerSource loggerSource)
+            final LoggerSource loggerSource, final Logger logger)
     {
         if (!provideEncoders)
             return;
@@ -102,23 +103,27 @@ public class HibernateModule
         for (EntityType<?> entityType : entities)
         {
             Class<?> entityClass = entityType.getJavaType();
-            if (entityClass != null && entityType.hasSingleIdAttribute())
-            {
-                SingularAttribute<?, ?> id = 
entityType.getId(entityType.getIdType().getJavaType());
-                final String idenfierPropertyName = id.getName();
-                ValueEncoderFactory factory = new ValueEncoderFactory()
+            if (entityClass != null)
+            {   
+                if (entityType.hasSingleIdAttribute())
                 {
-                    @Override
-                    public ValueEncoder create(Class type)
-                    {
-                        return new HibernateEntityValueEncoder(entityClass, 
idenfierPropertyName,
-                                session, propertyAccess, typeCoercer,
-                                loggerSource.getLogger(entityClass));
-                    }
-                };
-
-                configuration.add(entityClass, factory);
-
+                    SingularAttribute<?, ?> id = 
entityType.getId(entityType.getIdType().getJavaType());
+                    final String idenfierPropertyName = id.getName();
+                    ValueEncoderFactory factory = new ValueEncoderFactory()
+                    {    
+                        @Override
+                        public ValueEncoder create(Class type)
+                        {
+                            return new 
HibernateEntityValueEncoder(entityClass, idenfierPropertyName,
+                                    session, propertyAccess, typeCoercer,
+                                    loggerSource.getLogger(entityClass));
+                        }
+                    };
+
+                    configuration.add(entityClass, factory);
+                } else {
+                    logger.warn("Not creating a value encoder for {} as it 
does not have a single id attribute.", entityClass);
+                }
             }
         }
     }