This is an automated email from the ASF dual-hosted git repository. borinquenkid pushed a commit to branch 8.0.x-hibernate7 in repository https://gitbox.apache.org/repos/asf/grails-core.git
commit 4eaf7daf077e61b2ffc07712325e637cf870aca9 Author: Walter Duque de Estrada <[email protected]> AuthorDate: Thu Feb 26 10:44:18 2026 -0600 Dataflow analysis fixes --- .../event/listener/HibernateEventListener.java | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/event/listener/HibernateEventListener.java b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/event/listener/HibernateEventListener.java index 21277ec681..6e1b6ad5e9 100644 --- a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/event/listener/HibernateEventListener.java +++ b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/event/listener/HibernateEventListener.java @@ -43,6 +43,7 @@ import org.springframework.context.ApplicationEvent; * @author Burt Beckwith * @since 2.0 */ +@SuppressWarnings({"PMD.CloseResource", "PMD.DataflowAnomalyAnalysis"}) public class HibernateEventListener extends AbstractHibernateEventListener { protected transient ConcurrentMap<SoftKey<Class<?>>, ClosureEventListener> eventListeners = @@ -151,23 +152,15 @@ public class HibernateEventListener extends AbstractHibernateEventListener { } public boolean onPreInsert(PreInsertEvent event) { - boolean evict = false; ClosureEventListener eventListener = findEventListener(event.getEntity(), event.getPersister().getFactory()); - if (eventListener != null) { - evict = eventListener.onPreInsert(event); - } - return evict; + return eventListener != null && eventListener.onPreInsert(event); } public boolean onPreUpdate(PreUpdateEvent event) { - boolean evict = false; ClosureEventListener eventListener = findEventListener(event.getEntity(), event.getPersister().getFactory()); - if (eventListener != null) { - evict = eventListener.onPreUpdate(event); - } - return evict; + return eventListener != null && eventListener.onPreUpdate(event); } public void onPostUpdate(PostUpdateEvent event) { @@ -179,13 +172,9 @@ public class HibernateEventListener extends AbstractHibernateEventListener { } public boolean onPreDelete(PreDeleteEvent event) { - boolean evict = false; ClosureEventListener eventListener = findEventListener(event.getEntity(), event.getPersister().getFactory()); - if (eventListener != null) { - evict = eventListener.onPreDelete(event); - } - return evict; + return eventListener != null && eventListener.onPreDelete(event); } public void onPostDelete(PostDeleteEvent event) {
