Unexpected behavior when using reversed and limit on for..in loop

[pdehaan]

Found in liquidjs@v9.23.0 (via @11ty/eleventy@canary).

I noticed that when using a for..in loop with the reversed and limit attributes, it always seems to apply limit first (so I only get items "one" and "two", regardless of which order I define reversed and limit:2):

{% assign arr = "one,two,three,four,five" | split: "," %}

{% for item in arr reversed limit: 2 %}
- {{ item }}
{% endfor %}

{% comment %}
OUTPUT
- two
- one
{% endcomment %}

In this case, I expected reversed to get applied first so limit:2 would pluck items "five", and "four", respectively.

---

Workaround seems to be to define "reverse" filter in the {% assign %} tag instead of in the for..in loop:

{% assign arr = "one,two,three,four,five" | split: "," | reverse %}

{% for item in arr limit: 2 %}
- {{ item }}
{% endfor %}

{% comment %}
OUTPUT
- five
- four
{% endcomment %}

[harttle]

It seems intentional in shopify/liquid, but I'm considering add an option to enhance the behaviour, and merge this PR as well.

[github-actions]

🎉 This issue has been resolved in version 9.26.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[harttle]

Now can be enabled by setting orderedFilterParameters option to true.