Bug: Array Indexes about liquid-rust HOT 4 CLOSED

I've started investigating this issue. Not done yet, but working on it.

Here's what I've found atm:

• https://github.com/Shopify/liquid/wiki/Liquid-for-Designers this doc says they support complex indexing subexpressions like a[0].b["c"]
• Ruby's \w is not like in rust, so ".xxx" is parsed as Identifier(".xxx") while it shouldn't.

I think I can fix this eventually.

from liquid-rust.

That'd be a big help!

I've not been able to set aside time to try to step through the regex based parser. I'd actually love to switch it to nom or some other parser though it does lose the comparison to shopify's but as you pointed it, it can be misleading.

from liquid-rust.

I think I could look into it in future as well, but after fixing this bug.
I'd (personally) prefer using https://github.com/Marwes/combine instead of nom.

from liquid-rust.

I've created #138 to track changing the parser.

I've not explored the parser options, so I'll be deferring to whoever does the work. It would be helpful if you posted to #138 what your recommendation is and why so that if someone comes along and goes "you did X, why not Y" we have a reason.

from liquid-rust.