Based on How to write database independent code:

1) Suggest db_query_range when a LIMIT is specified.
2) You might already do this, but suggest NULL instead of ''.
3) Check for table names in braces - one of mine slipped through with back ticks on it.

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Is this a duplicate now?

douggreen’s picture

douggreen commented

Sorry, this ticket fell off my first page of issues. Thanks for bringing it back to the forefront.

  1. Thanks! I just added this
  2. I think that short of adding a SQL parser to coder, that this is going to be difficult. Exactly what are you suggesting?
  3. This was implemented about a week ago.
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

2. I don't remember exactly. It was mentioned in the post I referenced.

stella’s picture

stella commented
Status: Active » Closed (won't fix)

Drupal 5 version is no longer supported, closing.