search

grails / grails-core

The Grails Web Application Framework
http://grails.org
Apache License 2.0
2.79k stars 950 forks source link

fix #13655 - restore dot notation navigation #13869

Open jdaugherty opened 4 days ago

codeconsole commented 4 days ago

@jdaugherty why were all those public access modifiers there previously? Was it a Java class that got converted to Groovy?

jdaugherty commented 3 days ago

@jdaugherty why were all those public access modifiers there previously? Was it a Java class that got converted to Groovy?

That was my assumption, btw: this is still a draft pull request. I need to make sure the functional tests are passing before I can activate this.

codeconsole commented 3 days ago

@jdaugherty why were all those public access modifiers there previously? Was it a Java class that got converted to Groovy?

That was my assumption, btw: this is still a draft pull request. I need to make sure the functional tests are passing before I can activate this.

if that is the case, is going back to Java an option since this class is used so much?

jdaugherty commented 3 days ago

@jdaugherty why were all those public access modifiers there previously? Was it a Java class that got converted to Groovy?

That was my assumption, btw: this is still a draft pull request. I need to make sure the functional tests are passing before I can activate this.

if that is the case, is going back to Java an option since this class is used so much?

I'll look into it.