Remove redundant substitute index when constructing bind values

We end up re-ordering them either way when we construct the Arel AST (in order
to deal with rewhere, etc), so we shouldn't bother giving it a number in the
first place beforehand.
I think the index will be always 1
shouldn't we have `connection.substitute_at(column,  idx)` instead ?