Simplify API wrappers (#839)
* Updated code generation * API generation * Updated test * Updated code generation * API generation * API generation
This commit is contained in:
committed by
delvedor
parent
d080f1bdad
commit
c1eadd6e88
14
api/index.js
14
api/index.js
@ -28,6 +28,7 @@ function ESAPI (opts) {
|
||||
|
||||
const { result } = opts
|
||||
opts.handleError = handleError
|
||||
opts.snakeCaseKeys = snakeCaseKeys
|
||||
|
||||
const apis = {
|
||||
bulk: lazyLoad('bulk', opts),
|
||||
@ -493,6 +494,19 @@ function ESAPI (opts) {
|
||||
if (callback) return callback(err, result)
|
||||
return Promise.reject(err)
|
||||
}
|
||||
|
||||
function snakeCaseKeys (acceptedQuerystring, snakeCase, querystring, warnings) {
|
||||
var target = {}
|
||||
var keys = Object.keys(querystring)
|
||||
for (var i = 0, len = keys.length; i < len; i++) {
|
||||
var key = keys[i]
|
||||
target[snakeCase[key] || key] = querystring[key]
|
||||
if (acceptedQuerystring.indexOf(snakeCase[key] || key) === -1) {
|
||||
warnings.push('Client - Unknown parameter: "' + key + '", sending it as query parameter')
|
||||
}
|
||||
}
|
||||
return target
|
||||
}
|
||||
}
|
||||
|
||||
// It's unlikely that a user needs all of our APIs,
|
||||
|
||||
Reference in New Issue
Block a user