Assign does not support piped expressions about liquid-rust HOT 4 CLOSED

Hey, thanks for helping out!

Mmh, that doesn't look like it comes from returning an Array. You can only get that error from lexer::tokenize, which is called in parse. I'll try check out your patch and play around with it. My first guess is that the fixture string at the bottom is lexed incorrectly.

from liquid-rust.

Yea, I wasn't sure what that stacktrace was telling me, but figured the code should give you a good lead.

I assumed it was in lexer too based on the error message (not matching SINGLE_STRING_LITERAL or DOUBLE_STRING_LITERAL -- fails with same message on the quote if you try both ways).

Let me know what you find out!

from liquid-rust.

So, this seems to be a problem with the assign tag that does not allow for piping and accepts only single values or variables as rhs elements. I should get around fixing that soon. However, I would recommend just leaving that particular test case out for now (the one in tests/filters.rs). There's nothing wrong with your solution and the array should work fine :)

Thanks! Looking forward to your PR!

from liquid-rust.

Thanks for the fix. This fixes splits on non-comma, etc which I think is being tracked in #41. So I'll make a PR with a simple test which passes, but will need to recheck when the comma fix is in.

from liquid-rust.