feat: Arrange the DBI::Id() object in SQL order

[eauleaf]

My package uses DBI::Id() to write out SQL via dbQuoteIdentifier(), but success or failure of the SQL statement depends on the order that users input the Id params. In my package, I overwrote the Id() function to create a similar DBI object, but this issue appears to have been a problem for others as well.

Changes correct my issue, which is identical to this closed issue:
r-dbi/odbc#214

Might fix this user's issue:
#426

Closes #383.

[aviator-app]

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.

---

See the real-time status of this PR on the Aviator webapp.

Use the Aviator Chrome Extension to see the status of your PR within GitHub.

[krlmlr]

Thanks!

[hadley]

Seems like a low-risk change to me.

@eauleaf in general it's easier to review PRs if you make the minimum set of changes to the existing package.

[eauleaf]

Thanks for the feedback. Per your comments, revised pull request to minimize changes and added another test.

[krlmlr]

Nice, thanks!

[krlmlr]

@eauleaf: Would you like to help review #429?