github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#9071] Fix for DefaultConfiguration#derive()

Browse Source 
`DefaultConfiguration#derive()` could lead to surprising effects because
the derived `Configuration` would be linked to a `MetaProvider` which
still referenced the original `Configuration`.
This commit is contained in:
Knut Wannheden 2019-08-22 14:24:10 +02:00
parent 66a45669c4
commit 5830b1b0d5
1 changed files with 18 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
jOOQ/src/main/java/org/jooq/impl/DefaultConfiguration.java
View File

@ -186,27 +186,27 @@ public class DefaultConfiguration implements Configuration {
*
* @param configuration The pre-existing configuration.
*/
DefaultConfiguration(Configuration configuration) {
DefaultConfiguration(DefaultConfiguration configuration) {
this(
configuration.connectionProvider(),
configuration.metaProvider(),
configuration.executorProvider(),
configuration.transactionProvider(),
configuration.recordMapperProvider(),
configuration.recordUnmapperProvider(),
configuration.recordListenerProviders(),
configuration.executeListenerProviders(),
configuration.visitListenerProviders(),
configuration.transactionListenerProviders(),
configuration.diagnosticsListenerProviders(),
configuration.unwrapperProvider(),
configuration.converterProvider(),
configuration.connectionProvider,
configuration.metaProvider,
configuration.executorProvider,
configuration.transactionProvider,
configuration.recordMapperProvider,
configuration.recordUnmapperProvider,
configuration.recordListenerProviders,
configuration.executeListenerProviders,
configuration.visitListenerProviders,
configuration.transactionListenerProviders,
configuration.diagnosticsListenerProviders,
configuration.unwrapperProvider,
configuration.converterProvider,
configuration.clock(),
configuration.clock,
configuration.dialect(),
configuration.settings(),
configuration.data()
configuration.dialect,
configuration.settings,
configuration.data
);
}