Match MySQL mixed-type result and comparison semantics for IF(), CASE, and IFNULL()
メンテナーはふだん 1 日以内に返信
@JanJakes がすでに取り組んでいます。
2026年10月6日 から。
評価
この issue はまだ評価されていません。
説明
Summary
Implement MySQL-compatible mixed-type result conversion and comparison semantics for IF(), CASE, and IFNULL().
This is a pre-existing compatibility gap, separate from the IF() condition-evaluation bug in #517. Keep #518 focused on that bug. Build this follow-up on the comparison work in #508.
Reproduction and rationale
SELECT IF(1, 2, '3') = '2';
MySQL returns 1. The raw SQLite translation returns 0:
SELECT CASE WHEN 1 THEN 2 ELSE '3' END = '2';
SQLite preserves the selected branch's integer storage class. MySQL applies result-type rules across the branches. Related mixed-type conversion gaps affect CASE and IFNULL().
An unconditional text cast can fix this example but break numeric comparisons. The solution must account for both result-type rules and the surrounding comparison semantics.
Required changes
- Determine MySQL result-type and comparison rules for mixed numeric, string, and NULL values in
IF(),CASE, andIFNULL(). - Extend the translator and comparison handling from #508 to reproduce those rules consistently.
- Preserve lazy evaluation of unselected branches and fallback expressions.
- Add regression tests against MySQL reference results for string comparisons, numeric comparisons, NULLs, nested expressions, and column-derived values.
- Avoid an unconditional text cast as the general solution.
Affected areas
packages/mysql-on-sqlite/src/sqlite/class-wp-mysql-on-sqlite.php:translate_runtime_function_call(), expression translation, and comparison handling.packages/mysql-on-sqlite/tests/WP_MySQL_On_SQLite_Tests.php: regression coverage forIF(),CASE, andIFNULL().
Acceptance criteria
SELECT IF(1, 2, '3') = '2'returns1, matching MySQL.- Mixed-type string and numeric comparison cases for all three constructs match MySQL reference results.
- NULL handling and lazy evaluation remain correct.
- Existing IF() condition-evaluation and comparison tests continue to pass.
Context
Requested by @JanJakes.
- Originating PR: https://github.com/WordPress/sqlite-database-integration/pull/518
- Review discussion: https://github.com/WordPress/sqlite-database-integration/pull/518#discussion_r4194619968
- Related comparison work: https://github.com/WordPress/sqlite-database-integration/pull/508
- 主要言語
- PHP
- スター
- 362
- フォーク
- 67
- 平均マージ
- 1日 19時間
- マージ済み PR（30日）
- 5
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
WordPress/sqlite-database-integration のほかの issue
-
MySQL-on-SQLite does not translate DATE arithmetic with INTERVAL HOUR対応中かも @samuelmbabhazi が 18 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
WordPress/sqlite-database-integration#454 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
WordPress/sqlite-database-integration#516 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
WordPress/sqlite-database-integration#515 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
WordPress/sqlite-database-integration#513 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
WordPress/sqlite-database-integration#504 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
WordPress/sqlite-database-integration の issue をすべて見る
似ている issue
-
📚 Documentation: Placeholder link `link-to-realtime-docs` in Flutter SDK changelog対応中かも @ShyneChikwapulo が今日担当しました。 オープンapi / realtime product / auth product / messaging product / vcs
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
appwrite/appwrite#14272 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
Boavizta/boaviztapi#580 · コメント 1 件 ·
-
Add Prestashopオープンrequest
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
endoflife-date/endoflife.date#11303 ·
メンテナーはふだん 1 日以内に返信
-
0. to triage enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信