Skip to content
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

Grails config dot-notation warnings in 3.3.6 #285

Open
robertoschwald opened this issue Oct 30, 2021 · 1 comment
Open

Grails config dot-notation warnings in 3.3.6 #285

robertoschwald opened this issue Oct 30, 2021 · 1 comment

Comments

@robertoschwald
Copy link

There are still some statements left which use the deprecated dot notation to access config maps.

https://github.com/bertramdev/asset-pipeline/blob/master/asset-pipeline-grails/src/main/groovy/asset/pipeline/grails/AssetProcessorService.groovy#L104 (conf.url)

https://github.com/bertramdev/asset-pipeline/blob/master/asset-pipeline-grails/grails-app/taglib/asset/pipeline/grails/AssetsTagLib.groovy#L84 (conf.bundle)

This lead to warnings in Grails 5.0.0

Accessing config through dot notation is deprecated, and it will be removed in a future release. Use 'config.getProperty(key, targetClass)' instead.

@davydotcom
Copy link
Contributor

So the second one is grabbing off the getProperty Map so this is ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants