`wp theme mod set` prints its failure message as `Success` and exits 0

オープン 初心者向け
  • 何か問題がありますか？
#570 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

@dhrupo がすでに取り組んでいます。

2026年11月6日 から。

  • #571 @dhrupo による — オープン

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
82/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
php, wordpress
領域
cli

調査の方向性

issueに記載されている、不一致の値が検出され失敗メッセージが出力される箇所として Theme_Mod_Command::set() を読みます。近隣のコマンドがエラーをどのように報告し、終了ステータスを設定するかを確認し、説明されているフィルター pre_set_theme_mod_background_color で再現します。コマンドが Error: Could not update theme mod background_color. を出力し、コード1で終了すれば完了です。

索引モデルが issue の本文から書いたものです。

説明

bug command:theme-mod-set status:unconfirmed

When wp theme mod set cannot save the value, it prints the failure through WP_CLI::success():

$ wp theme mod set background_color 123456
Success: Could not update theme mod background_color.
$ echo $?
0

To reproduce, add a pre_set_theme_mod_background_color filter that returns a different value (for example from a mu-plugin). Theme_Mod_Command::set() reads the mod back, sees it does not match, and calls WP_CLI::success( "Could not update theme mod {$mod}." ). Scripts that check the exit code treat the failure as a success.

Expected: Error: Could not update theme mod background_color. with exit code 1.

主要言語
Gherkin
スター
90
フォーク
87
平均マージ
2日 7時間
マージ済み PR（30日）
3

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

wp-cli/extension-command のほかの issue

wp-cli/extension-command の issue をすべて見る

似ている issue

CLI の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。