[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