I think you should use thecsv processor instead.

This feature should go in4.4 branch instead ofmaster, same for the corresponding docs PR please 🙏

Agree with@nicolas-grekas , you can already use thecsv processor:https://3v4l.org/iGdpS

@OskarStark base branch updated ;)

@Orkin can you please give us some more insights, why you cannot/wont use thecsv processor?

Thank you

i realized there can be different behavior :)https://3v4l.org/gOCPR

that's escaping support out-of-the box 😅 but perhaps that works counter-wise for@Orkin

for thecsv processor itself, from the example above; im not sure why PHP doesnt unescaped\",\"e

So for the default parameter values "" and \" have the same meaning. (https://php.net/str_getcsv)

Perhaps SF should fix it?

@ro0NL same as in#32007? Worth a PR on the csv process? Please submit a PR if you think so.
But anyway, we're missing feedback from@Orkin before we can make guesses.

not sure if#32007 is related, as it fixes the encoding ofbool(true) into"1". AFAIK Serializer doesnt decode"1" back intobool(true). Nor does PHP:https://3v4l.org/YckLo

For DI it'sbool:key:N:csv:ENV IMHO.

If we decide to unescape the escape char, both DI + Serializer should do that yes. (Unless the escape char is":https://3v4l.org/lgrDY)

https://bugs.php.net/bug.php?id=55413 is the bug report.

Hi,@OskarStark@nicolas-grekas first of all because I didn't know the csv processor. For my usecase it's for memcached server list retrieve via environment variable so for my need it's easier to split by something.
I choose a comma, but perhaps in the future it can be aspace or a pipe|. It can be usefull to separate csv processor to a real flat array string and to get the availability to specify the separator.

I don't know as much env var processor so I don't have any ideas if it's possible and if not what kind of format it can be used. But in the future it can be a improvement.

And last thing, I don't want my string to be escaped because it can be url or something that need to stay in the same format

In this case it makes sense to me to not use the csv processor and got with the new explode one. Also from a DX perspective, having csv:MEMCACHED_HOSTS... looks weird

Id prefersplit then 👍

Sounds good to me, but I think it’s more common to use explode kn the php context, also because of split is an old PHP 5.3 deprecated method

https://www.php.net/manual/en/function.split.php

But definitely better then using csv here 👍🏻

that's the thing, explode sounds PHP specific whereas in SF DI config it should be more language agnostic IMHO. Agreecsv might look weird.

split by something. I choose a comma , but perhaps in the future it can be a space or a pipe |

i think having bothsplit:ENV as well assplit:|:CSV seems sensible 👍

It didn’t know and find thecsv processor because it seems to me weird for my use case.
I’m agree withsplit because it’s a more common word but we are in a PHP world so most of the users of SF are PHP developers and when a PHP developer want to transform a string into an array the first function in his mind isexplode that’s why it seems better to me.
We can imagine someone typing something like that in googlehow to explose an array in symfony yaml config

maybecsv is weird because of#25627 (comment), currently it accepts a single row, so it's somewhatsplit_escaped:ENV actually

i tend to believe people associate "CSV" with a collection of rows

explode vs.split: the latter is shorter.

Let’s go with split then

Everything is up to date ;)

Please update the PR title

Not sure we need yet another processor.csv works for most cases (even if not easily discoverable), andjson works for more complex cases.

Here, I'm sure some will want to have another separator than,, and others will want to be able to escape the separator.

So, I'm 👎 for this addition.

Closing as explained, thanks for proposing.