![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#27693 closed defect (bug) (fixed)
Incorrect call to `mysqli_fetch_field()`
| Reported by: | Denis-de-Bernardy | Owned by: | pento |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Database | Version: | 3.9 |
| Severity: | blocker | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
We call it with two parameters, whereas it only expects a single one:
http://us2.php.net/manual/en/mysqli-result.fetch-field.php
This breaks $wpdb->col_info among other things.
Attachments (1)
Change History (8)
#3
follow-up:
↓ 6
@
12 years ago
Quick aside, here: is there any good reason to silence errors in those calls the way things are, instead of ensuring that it won't spit out a warning?
#6
in reply to: ↑ 3
@
12 years ago
Replying to Denis-de-Bernardy:
Quick aside, here: is there any good reason to silence errors in those calls the way things are, instead of ensuring that it won't spit out a warning?
Good question! This has always existed, so I'm going to go with "no-one has been brave enough to change it".
Note:
See TracTickets
for help on using tickets.
Nice catch. :-)
As we're retrieving fields sequentially, we don't really need the field offset, anyway.