Selaa lähdekoodia

Merge pull request #4309 from RaananW/config-order

Viewer configuration-loading order changed:
David Catuhe 7 vuotta sitten
vanhempi
commit
98edd74172
1 muutettua tiedostoa jossa 3 lisäystä ja 4 poistoa
  1. 3 4
      Viewer/src/configuration/loader.ts

+ 3 - 4
Viewer/src/configuration/loader.ts

@@ -36,8 +36,6 @@ export class ConfigurationLoader {
 
         let extendedConfiguration = getConfigurationType(loadedConfig.extends || "");
 
-        loadedConfig = deepmerge(extendedConfiguration, loadedConfig);
-
         if (loadedConfig.configuration) {
 
             let mapperType = "json";
@@ -72,13 +70,14 @@ export class ConfigurationLoader {
                 }
             }).then((data: any) => {
                 let mapper = mapperManager.getMapper(mapperType);
-                let parsed = mapper.map(data);
-                let merged = deepmerge(loadedConfig, parsed);
+                let parsed = deepmerge(mapper.map(data), loadedConfig);
+                let merged = deepmerge(extendedConfiguration, parsed);
                 processConfigurationCompatibility(merged);
                 if (callback) callback(merged);
                 return merged;
             });
         } else {
+            loadedConfig = deepmerge(extendedConfiguration, loadedConfig);
             processConfigurationCompatibility(loadedConfig);
             if (callback) callback(loadedConfig);
             return Promise.resolve(loadedConfig);