Make WordPress Core

Changes between Version 2 and Version 3 of Ticket #52506, comment 5


Ignore:
Timestamp:
01/26/2022 08:30:39 PM (16 months ago)
Author:
craigfrancis
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #52506, comment 5

    v2 v3  
    11As requested by Tonya (@hellofromtonya)...
    22
    3 I've split [https://github.com/WordPress/wordpress-develop/pull/2192 my patch] so we can discuss Identifiers (table/field names) separately to the `IN()` operator.
     3I've split [https://github.com/WordPress/wordpress-develop/pull/2192 my patch] so we can discuss escaping Identifiers (table/field names). The other part is covered by [https://core.trac.wordpress.org/ticket/54042 Ticket 54042] to support the `IN()` operator.
    44
    55There should be no conflicts with printf(), as [https://en.wikipedia.org/wiki/Printf_format_string#Type_field printf() Types] have historically used both `%d` and `%i` for signed integers (it's `scanf()` that treats them differently). [https://www.php.net/manual/en/function.printf.php PHP printf()] only uses `%d`, with `%i` resulting in the fatal error `Unknown format specifier "i"`, so we can use `%i` for Identifiers.