codeconsole commented on PR #16019:
URL: https://github.com/apache/grails-core/pull/16019#issuecomment-5065908211
Thanks @matrei, all 3 points addressed:
- 48d352fadf: a bean name can now be declared by multiple `bean(...)`
statements as long as every declaration has its own discriminating condition
(e.g. `.annotate(ConditionalOnProperty, ...)`). Unconditioned duplicates are
still a compile error. Your `UrlMappingsAutoConfiguration` conversion is now in
the branch (5e9e162e99), see the Before/After comment above.
- 19a502c420: argument order flipped to name first:
`bean('grailsUrlConverter', UrlConverter)`, `field('cacheUrls', Boolean)`,
`method('fixedLocale', Locale)`. Type-only forms unchanged; the old order
produces a compile error pointing at the new one.
- 855a28347d: `@GrailsBeans` works on any configuration source class,
including the `Application` class (no imports-file entry needed there). Docs
added to "Grails and Spring → Configuring Additional Beans" so they are no
longer plugin-only.
Also, single-argument `.value(...)` now auto-wraps a bare key:
`.value('grails.web.linkGenerator.useCache')` compiles to
`@Value("${grails.web.linkGenerator.useCache}")` (32ead2bbc4). Strings
containing `${...}`/`#{...}` still pass through verbatim.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
