#27693 closed defect (bug) (fixed)
Incorrect call to `mysqli_fetch_field()`
Reported by: | Denis-de-Bernardy | Owned by: | pento |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | blocker | Version: | 3.9 |
Component: | Database | Keywords: | has-patch commit |
Focuses: | Cc: |
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)
#2
@
11 years ago
- Keywords has-patch commit added
- Owner set to pento
- Status changed from new to accepted
#3
follow-up:
↓ 6
@
11 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
@
11 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.