Open mauteri opened 8 years ago
Added pull request that just sets enumerable
to false to prevent Liquid properties from showing up in for...in
loops.
I hit this issue too... would be an easy merge. May I ask why this has not been accepted?
Ok.. so it seems to be fixed in master. Are there any known issues in master?
Yeah, whats the status of this issue?
Project looks pretty much abandoned to me... Last commit was December 5, 2016... :-/
I noticed in a few spots that JavaScript's native
Array
andString
objects are being extended by Liquid withclear
,first
,last
, etc.With Liquid on our site, we ran into some pretty nasty bugs due to this and had to roll code back. Is there a reason this needs to be done as a property rather than creating helpers for this functionality? I'd be happy to provide a pull request to change this bit of code.