LogFormatter convertParams fix for passing nested array as params

[0x9be00ff1]

add support to doctrine json_array type

[patrickbrouwers]

Can you back this up with unit tests, please :)

[0x9be00ff1]

I'll add the tests.
Is it proper to covert params using laravel e helper? it looks little unfriendly when using a FileLogger.

[patrickbrouwers]

Can you post file logger example, I didn't notice anything strange when I tested it

[0x9be00ff1]

SELECT * FROM table WHERE column = "{"key1":"value1","key2":{"key21":"value22"}}

is it what you want? :)

[patrickbrouwers]

hm perhaps the arrays should be escaped then.

[0x9be00ff1]

These sqls are just for logging, unescaped strings looks much better, shall we just leave it alone?
or we need something like PDO::quote().

[patrickbrouwers]

Better safe than sorry

} else {
    $param = e($param);
}

return (string) $param;

[patrickbrouwers]

Can you make the change I suggested?

[patrickbrouwers]

👍

[0x9be00ff1]

@patrickbrouwers Can we tag a new release with this pr?

[patrickbrouwers]

Sorry, completely forgot about this. Yes, will do.