|
|
| Database: |
Any
|
| Participants: |
Martin Dougiamas, Matt Oquist and Nicolas Connault
|
| Security Level: |
None
|
| Resolved date: |
05/Feb/08
|
| Affected Branches: |
MOODLE_18_STABLE, MOODLE_20_STABLE
|
| Fixed Branches: |
MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE
|
|
Call count_records_sql() with a query that returns 0 results. $rs->fields is set to "false", so reset($rs->fields) produces the warning in the summary.
NB: In 2.0 from CVS, the line number is 382 instead of 376.
|
|
Description
|
Call count_records_sql() with a query that returns 0 results. $rs->fields is set to "false", so reset($rs->fields) produces the warning in the summary.
NB: In 2.0 from CVS, the line number is 382 instead of 376. |
Show » |
|
It looks fine to me.
Nick, can you please check this in to 20, 1.9, 1.8, 1.7 and 1.6 (setting all merged flags as you go of course!)