@@ -104,27 +104,27 @@ export function createConfiguration(conf: ConfigurationParameters = {}): Configu
104104 * Merge configuration options into a configuration.
105105 */
106106{ {#useInversify} }
107- export function mergeConfiguration(_config : Configuration, _options ?: Configuration): Configuration {
108- if (_options ) {
109- _config = _options ;
107+ export function mergeConfiguration(conf : Configuration, options ?: Configuration): Configuration {
108+ if (options ) {
109+ conf = options ;
110110 }
111- return _config ;
111+ return conf ;
112112}
113113{ {/useInversify} }
114114{ {^useInversify} }
115- export function mergeConfiguration(_config : Configuration, _options ?: ConfigurationOptions): Configuration {
115+ export function mergeConfiguration(conf : Configuration, options ?: ConfigurationOptions): Configuration {
116116 let allMiddleware: Middleware[] = [];
117- if (_options && _options .middleware) {
118- const middlewareMergeStrategy = _options .middlewareMergeStrategy || " replace" // default to replace behavior
117+ if (options && options .middleware) {
118+ const middlewareMergeStrategy = options .middlewareMergeStrategy || " replace" // default to replace behavior
119119 // call-time middleware provided
120- const calltimeMiddleware: Middleware[] = _options .middleware;
120+ const calltimeMiddleware: Middleware[] = options .middleware;
121121
122122 switch (middlewareMergeStrategy) {
123123 case " append" :
124- allMiddleware = _config .middleware.concat(calltimeMiddleware);
124+ allMiddleware = conf .middleware.concat(calltimeMiddleware);
125125 break ;
126126 case " prepend" :
127- allMiddleware = calltimeMiddleware.concat(_config .middleware)
127+ allMiddleware = calltimeMiddleware.concat(conf .middleware)
128128 break ;
129129 case " replace" :
130130 allMiddleware = calltimeMiddleware
@@ -133,31 +133,31 @@ export function mergeConfiguration(_config: Configuration, _options?: Configurat
133133 throw new Error(`unrecognized middleware merge strategy ' ${middlewareMergeStrategy}' `);
134134 }
135135 }
136- if (_options ) {
137- _config = {
138- baseServer: _options .baseServer || _config .baseServer,
139- httpApi: _options .httpApi || _config .httpApi,
140- authMethods: _options .authMethods || _config .authMethods,
141- middleware: allMiddleware || _config .middleware
136+ if (options ) {
137+ conf = {
138+ baseServer: options .baseServer || conf .baseServer,
139+ httpApi: options .httpApi || conf .httpApi,
140+ authMethods: options .authMethods || conf .authMethods,
141+ middleware: allMiddleware || conf .middleware
142142 } ;
143143 }
144- return _config ;
144+ return conf ;
145145}
146146{ {/useInversify} }
147147
148148/**
149149 * Convert Promise-based configuration options to Observable-based configuration options.
150150 */
151- export function wrapOptions(_options ?: PromiseConfigurationOptions): StandardConfigurationOptions | undefined {
152- if (_options ) {
151+ export function wrapOptions(options ?: PromiseConfigurationOptions): StandardConfigurationOptions | undefined {
152+ if (options ) {
153153 return {
154- baseServer: _options .baseServer,
155- httpApi: _options .httpApi,
156- middleware: _options .middleware?.map(
154+ baseServer: options .baseServer,
155+ httpApi: options .httpApi,
156+ middleware: options .middleware?.map(
157157 m => new PromiseMiddlewareWrapper(m)
158158 ),
159- middlewareMergeStrategy: _options .middlewareMergeStrategy,
160- authMethods: _options .authMethods,
159+ middlewareMergeStrategy: options .middlewareMergeStrategy,
160+ authMethods: options .authMethods,
161161 }
162162 }
163163 return;
0 commit comments