Fatal error while processing failed plugin upgrade

[akihiroharai]

Fatal error occurs while processing a failed plugin update.

$ wp plugin update disable-comments
メンテナンスモードを有効化中...
https://downloads.wordpress.org/plugin/disable-comments.2.4.5.zip から更新をダウンロード中...
Using cached file '/var/web/.wp-cli/cache/plugin/disable-comments-2.4.5.zip'...
署名が見つからなかったため、disable-comments.2.4.5.zip の信頼性を確認できませんでした。
更新を展開しています…
最新のバージョンをインストールしています…
プラグインの古いバージョンを削除しています…
Warning: 古いプラグインを削除できませんでした。
プラグインの更新に失敗しました。
メンテナンスモードを無効にします…
+------------------+-------------+-------------+--------+
| name             | old_version | new_version | status |
+------------------+-------------+-------------+--------+
| disable-comments | 2.4.4       | 2.4.5       | Error  |
+------------------+-------------+-------------+--------+
Error: No plugins updated (1 failed).
PHP Fatal error:  Uncaught ArgumentCountError: 2 arguments are required, 1 given in /var/web/wp/wp-admin/includes/class-wp-upgrader.php:1209
Stack trace:
#0 /var/web/wp/wp-admin/includes/class-wp-upgrader.php(1209): sprintf('%s \xE3\x81\xAE\xE4\xB8\x80\xE6\x99\x82\xE3\x83\x90...')
#1 /var/web/wp/wp-includes/class-wp-hook.php(308): WP_Upgrader->delete_temp_backup()
#2 /var/web/wp/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters(Object(WP_Error), Array)
#3 /var/web/wp/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
#4 /var/web/wp/wp-includes/load.php(1252): do_action('shutdown')
#5 [internal function]: shutdown_action_hook()
#6 {main}
  thrown in /var/web/wp/wp-admin/includes/class-wp-upgrader.php on line 1209
Fatal error: Uncaught ArgumentCountError: 2 arguments are required, 1 given in /var/web/wp/wp-admin/includes/class-wp-upgrader.php:1209
Stack trace:
#0 /var/web/wp/wp-admin/includes/class-wp-upgrader.php(1209): sprintf('%s \xE3\x81\xAE\xE4\xB8\x80\xE6\x99\x82\xE3\x83\x90...')
#1 /var/web/wp/wp-includes/class-wp-hook.php(308): WP_Upgrader->delete_temp_backup()
#2 /var/web/wp/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters(Object(WP_Error), Array)
#3 /var/web/wp/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
#4 /var/web/wp/wp-includes/load.php(1252): do_action('shutdown')
#5 [internal function]: shutdown_action_hook()
#6 {main}
  thrown in /var/web/wp/wp-admin/includes/class-wp-upgrader.php on line 1209

Fix incorrect right paren for sprintf().

Trac ticket: https://core.trac.wordpress.org/ticket/59320

[afragen]

This looks correct. PR seems appropriate. Good catch @akihiroharai

[costdev]

Hi @akihiroharai, welcome to Trac!

Nice catch!

This was introduced in [55720] (WordPress 6.3). Updating the Version property on the ticket.

The PR looks good to me. Assigning to myself and adding commit.

[costdev]

In 56550:

Upgrade/Install: Fix broken sprintf() call when deleting a backup.

In WP_Upgrader::delete_temp_backup(), a malformed sprintf() call did not pass the value, triggering a Warning in PHP 7 and a Fatal Error in PHP 8.

This fixes the malformed sprintf() call by correctly passing the value.

Follow-up to [55720].

Props akihiroharai, afragen.
Fixes #59320.

Thanks for the PR! Merged in r56550.

[costdev]

Reopening for backport to the 6.3 branch.

[costdev]

In 56582:

Upgrade/Install: Fix broken sprintf() call when deleting a backup.

In WP_Upgrader::delete_temp_backup(), a malformed sprintf() call did not pass the value, triggering a Warning in PHP 7 and a Fatal Error in PHP 8.

This fixes the malformed sprintf() call by correctly passing the value.

Follow-up to [55720].

Props akihiroharai, afragen.
Merges [56550] to the 6.3 branch.
Fixes #59320.

Backported to the 6.3 branch in r56582.