[wp-trac] [WordPress Trac] #59846: Reinstate the `wpdb::$use_mysqli` property

WordPress Trac noreply at wordpress.org
Wed Nov 8 20:05:10 UTC 2023


#59846: Reinstate the `wpdb::$use_mysqli` property
----------------------------------------+---------------------
 Reporter:  johnbillion                 |       Owner:  (none)
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  6.4.1
Component:  Database                    |     Version:  6.4
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+---------------------

Comment (by hellofromTonya):

 Replying to [comment:10 joemcgill]:
 > I would also be happy to just reinstate the property, but was thinking
 it might provide more info to developers that they should remove these
 calls from their code, since it's no longer in use, and would better
 signal to future core devs that the property is present for back-compat,
 but no longer in use. That said, I don't feel strongly about it.

 Restoring the property resolves the BC break. While triggering
 `wp_trigger_error()` within the `__get()` / `__set()` / `__unset()` /
 `__isset()` is possible, IMO it's not necessary and would add more code to
 be maintained.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/59846#comment:15>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list