Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore case in if node breaks array contains #9893

Closed
rserranocivitatis opened this issue Jun 29, 2024 · 2 comments
Closed

Ignore case in if node breaks array contains #9893

rserranocivitatis opened this issue Jun 29, 2024 · 2 comments
Labels
in linear Issue or PR has been created in Linear for internal review Released

Comments

@rserranocivitatis
Copy link

Bug Description

Enabling the ignore case option seems to be breaking the "array contains".

To Reproduce

  1. Create an "if" node

  2. Set value one to any JS array expression like {{ ["SAC-TEAM"] }} , and value two to SAC-TEAM.

  3. The condition at this time is true.
    image

  4. But, enable ignore option and the condition becomes false.
    image

Expected behavior

I would expect to get "true" independently on the "Ignore case" option. And in case the cases didn't match, the option should still make the condition true.

Operating System

Docker

n8n Version

1.47.0

Node.js Version

Docker

Database

PostgreSQL

Execution mode

main (default)

@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Jul 1, 2024
@Joffcom
Copy link
Member

Joffcom commented Jul 1, 2024

Hey @rserranocivitatis,

Thanks for the report, I was able to reproduce this and created NODE-1453 as the internal dev ticket to get this fixed.

@janober
Copy link
Member

janober commented Jul 17, 2024

Fix got released with n8n@1.51.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review Released
Projects
None yet
Development

No branches or pull requests

3 participants