Skip to content

Commit 790008a

Browse files
authored
fix: Remove redundant default controller scope from app config (#342)
As singleton is the default scope for controllers since Grails 6.2 we can remove this setting from the application config. Related apache/grails-core#13332
1 parent a8d5aea commit 790008a

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

grails-forge-core/src/main/java/org/grails/forge/feature/web/GrailsWeb.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ public boolean supports(ApplicationType applicationType) {
5252
@Override
5353
public void apply(GeneratorContext generatorContext) {
5454
final Map<String, Object> config = generatorContext.getConfiguration();
55-
config.put("grails.controllers.defaultScope", "singleton");
5655
config.put("grails.views.default.codec", "html");
5756
generatorContext.addBuildPlugin(GradlePlugin.builder().id("org.grails.grails-web").build());
5857
}

grails-forge-core/src/test/groovy/org/grails/forge/feature/web/GrailsWebSpec.groovy

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,7 @@ class GrailsWebSpec extends ApplicationContextSpec implements CommandOutputFixtu
2525
final String applicationYaml = output["grails-app/conf/application.yml"]
2626
def config = new YamlSlurper().parseText(applicationYaml)
2727

28-
2928
then:
30-
config.grails.controllers.defaultScope == 'singleton'
3129
config.grails.views.default.codec == 'html'
3230
}
3331
}

0 commit comments

Comments
 (0)