fix: remove redundant condition

eugene-kamenev · eugene-kamenev · commit d5da1901561a · 2025-08-20T18:13:40.000+05:00
diff --git a/plugin-acl/plugin/grails-app/services/grails/plugin/springsecurity/acl/AclService.groovy b/plugin-acl/plugin/grails-app/services/grails/plugin/springsecurity/acl/AclService.groovy
@@ -330,9 +330,7 @@ class AclService implements MutableAclService, WarnErros {
     @Transactional
 	protected <T extends GormEntity<T>> T save(T bean) {
 		if (!bean.save()) {
-            if (log.warnEnabled) {
-                log.warn errorsBeanBeingSaved(messageSource, bean)
-            }
+            log.warn errorsBeanBeingSaved(messageSource, bean)
 		}
 		bean
 	}

Original file line number	Diff line number	Diff line change
`@@ -330,9 +330,7 @@ class AclService implements MutableAclService, WarnErros {`
`330`	`330`	`@Transactional`
`331`	`331`	`protected <T extends GormEntity<T>> T save(T bean) {`
`332`	`332`	`if (!bean.save()) {`
`333`		`- if (log.warnEnabled) {`
`334`		`- log.warn errorsBeanBeingSaved(messageSource, bean)`
`335`		`- }`
	`333`	`+ log.warn errorsBeanBeingSaved(messageSource, bean)`
`336`	`334`	`}`
`337`	`335`	`bean`
`338`	`336`	`}`