[wp-trac] [WordPress Trac] #41944: Add %u support to wpdb->prepare

WordPress Trac noreply at wordpress.org
Thu Sep 21 14:42:11 UTC 2017


#41944: Add %u support to wpdb->prepare
--------------------------+------------------------------
 Reporter:  charlestonsw  |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Database      |     Version:  4.8.2
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------

Comment (by charlestonsw):

 Not sure how many other use %u.  WP add ons (plugins, themes, etc.) is a
 big codebase to assume nobody else uses that format.   Remember, not all
 WP related code lives in the WP plugin/theme repo so you'll never really
 know what else was broken.

 wpdb->prepare %u support was dropped because it wasn't in the format
 notes. Dropping that support broke things.    It was not a change that
 made WP more secure but did make it less functional for some users; at
 least 5,000 installed Power add on users in my case.

 Not a lot of code references to "up the tally" for %u but how to we value
 the change?  By customers affected or by line-counts of code?

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


More information about the wp-trac mailing list