[wp-trac] [WordPress Trac] #41956: Errno in $wpdb
WordPress Trac
noreply at wordpress.org
Mon Oct 2 09:24:35 UTC 2017
#41956: Errno in $wpdb
-------------------------------------+------------------------------
Reporter: LeonN1960 | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Database | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-refresh | Focuses:
-------------------------------------+------------------------------
Comment (by munyagu):
Thank you for reply, @LeonN1960.
Yes, I thought about it.
{{{$last_ error}}} field is considered to be directly referenced, because
it is public.
So, I thought it was more convenient to output the error code in the same
usage as before by stuffing into {{{$last_error}}} field.
Also, wp-db.php has a conditional branch {{{if ($this->last_error) {}}} I
thought that the change based on {{{$last_error}}} field is more
convenient for users.
However, if originally, {{{$last_errno}}} field should be added (and do
not change last_error) as you say, and it seems that keep the impact on
users to a minimum.
I will try it.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/41956#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list