Fixes issues when doing an 'IN' query

[greatwitenorth]

When doing an 'IN' query it looks like this will incorrectly loop over the values and put them into multiple where clauses. This additional check on the symbol should leave the array untouched and pass it on into the orm for processing.

My PR is a:

🐛 Bug fix

Main update on the:

Framework

[lauriejim]

Hello @greatwitenorth , thank you for this PR.

Not sure to completely understand what you want.
Can you give me the issue you fix (and steps to reproduce).

[greatwitenorth]

Consider the following custom method in a controller file:

api/product/controllers/Product.js

  findById: async (ctx) => {
    const ids = ctx.query.ids.split(',');
    return strapi.services.product.fetchAll({id_in: ids});
  },

and the following added to routes:

    {
      "method": "GET",
      "path": "/product/findById",
      "handler": "Product.findById",
      "config": {
        "policies": []
      }
    },

Currently a call to /product/findById?id=1,2,3 results in a 500 error:

You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ''1' and `id` in '2' and `id` in '3' limit 100' at line 1"

The following SQL is produced which is incorrect:

"select `products`.* from `products` where `id` in '1' and `id` in '2' and `id` in '3' limit 100"

With the proposed change, the system works as intended and returns products with ID 1,2,3.

[lauriejim]

Good ! Thank you for this fix !