-
Notifications
You must be signed in to change notification settings - Fork 536
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Change wrong info about controller default scope #885
Conversation
This reverts commit 9b3884d.
There used to be a default configuration to set the default scope for controller as
|
@puneetbehl Yes, the missing configuration is also an issue: grails/grails-forge#263. But changing the default scope in configuration does not make it the default scope of the framework. The default scope of controllers in Grails is prototype unless changed in config. The docs says different things in different places: https://docs.grails.org/6.1.1/guide/single.html#_scoped_controllers, https://docs.grails.org/6.1.1/ref/Controllers/scope.html. I also think that the default controller scope should eventually be changed, as proposed by grails/grails-core#13332. |
The default scope in the framework is configured to |
Yes, I agree! But for Grails 6.1.x the documentation is wrong and this PR addresses that. |
The documentation will only be released with the next patch or minor release where we will change the default to
We should address it in this PR. |
I think we need to separate the issues in grails-forge and grails-core. The bugfix in grails-forge should be applied to both 6.0.x and 6.1.x where the default controller scope is still prototype. We should not change default controller scope in grails-core 6.1.x or earlier as that would change behavior in a patch release. After the behavior has been updated in grails-core 6.2.x the configuration can be removed in grails-forge 6.2.x as it will then be superfluous. However, this PR is regarding behavior in grails-core 6.1.x where the default controller scope is still prototype and the documentation says it is singleton. |
This reverts commit 0f1e201.
No description provided.