Closed (fixed)
Project:
Drupal core
Version:
4.7.4
Component:
database system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
27 Dec 2006 at 19:44 UTC
Updated:
15 Jan 2007 at 18:02 UTC
The comment for db_query_range() in database.mysqli.inc and database.mysql.inc says
* Note that if you need to know how many results were returned, you should do
* a SELECT COUNT(*) on the temporary table afterwards. db_num_rows() and
* db_affected_rows() do not give consistent result across different database
* types in this case.This is untrue. It looks like someone copied the docs for db_query_temporary() and forgot to delete this paragraph, as db_query_range() does not use temporary tables; for MySQL it simply adds a LIMIT clause. This patch removes this comment.
| Comment | File | Size | Author |
|---|---|---|---|
| notempjv.patch | 1.75 KB | jvandyk |
Comments
Comment #1
dries commentedGood catch. Committed to CVS HEAD. Thanks. Needs to be backported, I think.
Comment #2
killes@www.drop.org commentedapplied.
Comment #3
(not verified) commented