silverstripe-scienceninjas/sapphire — comment on commit 458c3afca31519a2a0daaeedea65b05c1c43c5a5
File: model/DataList.php line 235
By sminnee:

Drop the implode and just call where() multiple times. It has the same effect and it keeps the SQL generation out of this method.

Also, I'd suggest calling where() on the underlying DataQuery, so that we have the option of deprecating / avoiding the use of DataList::where() in the future. DataList::where() is a bit of a hack, and we might, for example, have it throw a warning if you run your site in strict mode.