Correctly encode array for native param binding

[sander-hash]

Fix incorrect array encoding for native parameter binding

Problem

When passing an Array type to selectWithParams() or writeWithParams(), the previous implementation serialized the array using json_encode():

  if (is_array($value)) {
      return json_encode($value);
  }

This produced JSON encoding — double-quoted strings and JSON syntax:

["foo","bar","baz"]

ClickHouse's native HTTP parameter protocol expects array literals with single-quoted strings:

['foo','bar','baz']

The double-quoted output caused a query error when executing any native param query with a Array(String) parameter.

Fix

Replaced json_encode with a recursive approach that delegates each element back through convertParamValue(), with a special case for string elements which are wrapped in single
quotes:

  if (is_array($value)) {
      $arrayValues = [];
      foreach ($value as $val) {
          if (is_string($val)) {
              $arrayValues[] = sprintf("'%s'", $val);
              continue;
          }
          $arrayValues[] = $this->convertParamValue($val);
      }
      return sprintf('[%s]', implode(',', $arrayValues));
  }

This correctly handles:

• Array(String) → ['foo','bar']
• Array(UInt32) → [1,2,3]
• Array(Array(...)) — nested arrays via recursion

[isublimity]

Thanks a lot for the fix — really useful catch! 🎉 The json_encode() vs ClickHouse single-quote format mismatch was a real problem.

Merged to master. I'll add a follow-up commit to escape single quotes and backslashes inside array strings (e.g. ["it's"]), since sprintf("'%s'", $val) doesn't handle those cases yet. Plus additional tests covering these edge cases.