[wp-trac] [WordPress Trac] #26887: Make wpdb::base_prefix explicitly public

WordPress Trac noreply at wordpress.org
Wed Jan 29 00:48:33 UTC 2014


#26887: Make wpdb::base_prefix explicitly public
---------------------------+------------------------------
 Reporter:  DavidAnderson  |       Owner:
     Type:  enhancement    |      Status:  new
 Priority:  normal         |   Milestone:  Awaiting Review
Component:  Database       |     Version:
 Severity:  normal         |  Resolution:
 Keywords:                 |     Focuses:
---------------------------+------------------------------

Comment (by DavidAnderson):

 Hi,

 I switched from isset() to !empty(), because I believed that explicitly
 listing it with var would cause isset() to always be true. This belief
 turns out to be false!

 I see no problem with using public explicitly - I was just copying the
 style of the existing class.

 New patch attached.

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


More information about the wp-trac mailing list