Improvements to load @service classes

puneetbehl · puneetbehl · commit cf45fd0104c3 · 2020-12-17T09:05:50.000+05:30
Update the check to resolve service name from the annotation to either null or empty.
diff --git a/grails-datastore-gorm/src/main/groovy/org/grails/datastore/gorm/bootstrap/AbstractDatastoreInitializer.groovy b/grails-datastore-gorm/src/main/groovy/org/grails/datastore/gorm/bootstrap/AbstractDatastoreInitializer.groovy
@@ -311,7 +311,7 @@ abstract class AbstractDatastoreInitializer implements ResourceLoaderAware{
                 final Class<?> serviceClass = loadServiceClass(clazz)
                 final grails.gorm.services.Service ann = clazz.getAnnotation(grails.gorm.services.Service)
                 String serviceName = ann?.name()
-                if (serviceName == null) {
+                if (!serviceName) {
                     serviceName = Introspector.decapitalize(serviceClass.simpleName)
                 }
                 if (secondaryDatastore) {

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,7 @@ abstract class AbstractDatastoreInitializer implements ResourceLoaderAware{`
`311`	`311`	`final Class<?> serviceClass = loadServiceClass(clazz)`
`312`	`312`	`final grails.gorm.services.Service ann = clazz.getAnnotation(grails.gorm.services.Service)`
`313`	`313`	`String serviceName = ann?.name()`
`314`		`- if (serviceName == null) {`
	`314`	`+ if (!serviceName) {`
`315`	`315`	`serviceName = Introspector.decapitalize(serviceClass.simpleName)`
`316`	`316`	`}`
`317`	`317`	`if (secondaryDatastore) {`