🐛 Fix bug in Zendesk Trigger node with wrong conditions

[bartvollebregt]

This is my first PR for n8n so if I'm doing something wrong, please let me know. Never too late to learn!

The conditions for the Zendesk trigger node are populated in the resultAll and resultAny variables.
While populating these variables, the const aux: IDataObject = {}; variable from the outer scope is used to create entries for the resultAll and resultAny variables.

However, the aux variable from the outer scope is pushed to the resultAll and resultAny variables by reference. Therefore, at the second index in the loop, this for example aux.field = conditionAll.field; will also update the value of the previously pushed item.

The result is resultAll and resultAny variable where the entry's are all the same.

Fixed by moving the aux variable inside the loop scope.

[CLAassistant]

All committers have signed the CLA.

[janober]

Thanks a lot for fixing this @bartvollebregt! Got merged.

[janober]

Got released with n8n@0.116.0