NE-ON: multiline string support

[ViliamKopecky]

I tried to make a pull-request, but got lost in the Tokenizer insides.

I propose using the python multiline string syntax.

From NE-ON:

"""
this
is
"multiline"
'string'
"""

To PHP:

"this\nis\n\"multiline\"\n'string'"

[enumag]

Are you sure such string should be in config and not database or a template?

[vojtech-dobes]

@enumag I think that @ViliamKopecky would welcome it in Ristretto. Basically NEON is just data format, does not have to be necessarily for configuration (which should not contain such long strings).

[enumag]

In case of Ristretto, yes it might be a good thing. :-)

[ViliamKopecky]

@vojtech-dobes exactly ;)

[Majkl578]

It would be good for translations as well.

[dg]

In Neon is missing support for multiline strings, because I don't know how handle indentation in this:

item:
    inner: 
        "multi
        line
        string"

[enumag]

@dg I'd do it like this:

item:
    inner: 
        "multi
        line
        string"

= "multi\nline\nstring"

item:
    inner: 
        "multi
         line
         string"

= "multi\n line\n string"

[Majkl578]

Maybe something like HEREDOC in PHP? One line starts multiline content:

foo:
    bar: """
        abc
        def    
"""

But doesn't look good. :/

[enumag]

@Majkl578 What about this

foo:
    bar:
        """
        abc
        def    
        """

[Majkl578]

It's still confusing since indentation level normally means "array"... But probably best option.

[ViliamKopecky]

I think the indetation should be ignored (value should be as is, without removing indentation) inside the string. Its the value, what if you would want to have the indentation inside the string.

foo:
    bar:
        test:
            more:
"""
multi
line
string
"""
        blah: anything
    something:
        - 'i think'
        - 'this could work'

It's not nice, but multiline strings aren't nice anywhere :) so... And I presume it won't be widely used feature.

[vojtech-dobes]

In response to @dg's example - I think it should strip the indentation.

item:
    inner:
        multi
        line
        string

Should return:

multi
line
string

But

item:
    inner:
        multi
          line
         string

Should return:

multi
  line
 string

If I am right, then just text without - at beginning or text without : after key part does not mean anything in NEON now.

[JanTvrdik]

👎 all of the proposed syntaxes are ugly

[fprochazka]

What about:

item:
    inner: "multi
        line
        string"

should return

multi
        line
        string

and

item:
    inner: 
        """
        multi
        line
        string
        """

should return

multi
line
string

[Elijen]

What's wrong with:

item:
    inner: "multi\nline\nstring"