makeSqlIdPlaceHolders now throws an exception if passed an empty list.

Since it is the purpose of makeSqlIdPlaceHolders() to create an SQL
compatible string that can be used in an 'IN ()' clause, it does not
make a lot of sense to allow an empty list as argument; so it now throws
an exception when given an empty list.