ArgumentTransformer for zadd

Doc for zadd: https://redis.io/commands/zadd

Hello, The zadd command can accept endless score member [score member ...]. May I have a default ArgumentTransformer for zadd? I can create a PR for that.

My code:

function convertFilpObjectToArray (obj) {
  var result = []
  for (var key in obj) {
    result.push(obj[key], key)
  }
  return result
}

Redis.Command.setArgumentTransformer('zadd', args => {
  if (args.length > 1) {
    let last = args.pop()
    if (typeof last === 'object' && last !== null) {
      args.concat(convertFilpObjectToArray(last))
    } else args.push(last)
  }
  return args
})

[luin]

I’m aware of this fact. It’s just not straightforward compared with the transformer of hmset for the hash key, which itself is an object.

Anyway, I’d like to accept a PR for that, as long as the transformer keeps the same format with hmset that only takes effect when the argument length is 2 and supports both Map and plain object.

[luin]

It’s a little confused whether score should be the key or the value of an object (at least more confused than mset for a hash key).

Since ioredis flattens arguments, the following form is supported:

redis.zadd('key', [17, 'a'], [18, 'b'], [19, 'c'])

Given that I don’t think the transformer is necessary.