Skip to content

Commit ff4930b

Browse files
authored
Merge pull request #3593 from oysteinsigholt/fix-domNodeMerge
Fixes #3589 - Remove domNode from deepExtend
2 parents 668827b + 7287c74 commit ff4930b

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/core/index.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,9 @@ module.exports = function SwaggerUI(opts) {
6767

6868
let queryConfig = parseSearch()
6969

70+
const domNode = opts.domNode
71+
delete opts.domNode
72+
7073
const constructorConfig = deepExtend({}, defaults, opts, queryConfig)
7174

7275
const storeConfigs = deepExtend({}, constructorConfig.store, {
@@ -110,8 +113,8 @@ module.exports = function SwaggerUI(opts) {
110113
let mergedConfig = deepExtend({}, localConfig, constructorConfig, fetchedConfig || {}, queryConfig)
111114

112115
// deep extend mangles domNode, we need to set it manually
113-
if(opts.domNode) {
114-
mergedConfig.domNode = opts.domNode
116+
if(domNode) {
117+
mergedConfig.domNode = domNode
115118
}
116119

117120
store.setConfigs(mergedConfig)

0 commit comments

Comments
 (0)