[wp-trac] [WordPress Trac] #54841: wpdb::has_cap utf8mb4_520 detection does not work properly

WordPress Trac noreply at wordpress.org
Tue Oct 4 15:39:43 UTC 2022


#54841: wpdb::has_cap utf8mb4_520 detection does not work properly
---------------------------+-----------------------------
 Reporter:  jamieburchell  |       Owner:  SergeyBiryukov
     Type:  defect (bug)   |      Status:  closed
 Priority:  normal         |   Milestone:  6.1
Component:  Database       |     Version:  5.8.3
 Severity:  normal         |  Resolution:  fixed
 Keywords:  has-patch      |     Focuses:
---------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"54384" 54384]:
 {{{
 #!CommitTicketReference repository="" revision="54384"
 Database: Correct MariaDB version check in `wpdb::has_cap()`.

 MariaDB version is reported differently between PHP versions:
 * PHP 8.0.16 or later: `10.6.8-MariaDB`
 * PHP 8.0.15 or earlier: `5.5.5-10.6.8-MariaDB`

 The latter includes PHP 7.4.x and PHP 5.6.x as well, where the version is
 also reported with the `5.5.5-` prefix.

 This commit makes an adjustment to `wpdb::has_cap()` to check for the
 correct MariaDB version.

 This resolves an issue where the `utf8mb4_unicode_520_ci` collation, which
 is available in MariaDB since version 10.2, was previously not detected
 correctly.

 References:
 * [https://github.com/php/php-src/issues/7972 php-src: #7972: MariaDB
 version prefix 5.5.5- is not stripped]
 * [https://github.com/php/php-src/pull/7963 php-src: PR #7963 Fix GH-7932:
 MariaDB version prefix not always stripped]
 *
 [https://mariadb.com/docs/reference/mdb/collations/utf8mb4_unicode_520_ci/
 MariaDB Documentation: utf8mb4_unicode_520_ci]

 Follow-up to [37523], [53919].

 Props jamieburchell, SergeyBiryukov.
 Fixes #54841.
 }}}

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


More information about the wp-trac mailing list