search

gravitystorm / openstreetmap-website

The Rails application powering OpenStreetMap
http://www.openstreetmap.org/
GNU General Public License v2.0
2 stars 1 forks source link

refactor IN sql statements #273

Closed gravitystorm closed 1 year ago

gravitystorm commented 1 year ago

Prefer .where(:foo => %w[abc cde]) vs .where("foo IN ('abc', 'cde')")

I think there's two places where they still exist. https://github.com/openstreetmap/openstreetmap-website/commit/7f556f94f1d479346cbefb5cbeec0a6a0b7b8547 was a while ago.

gravitystorm commented 1 year ago

https://github.com/openstreetmap/openstreetmap-website/pull/4136