PHPで全てのエラーを catch したい（WordPressにお詳しい方へ）

Question

前提

WordPress特有の仕様についての質問になります。
お詳しい方宜しくお願い致します。

実現したいこと

２つのテーブルを 「厳密に」 LOCK TABLESし、ROLLBACKしたいです。

「厳密に」 とは 「一方のテーブルだけが更新されることを絶対になくしたい（２つのテーブルを絶対に一緒に更新したい）」 という意味です。

一般的に$wpdb->last_errorの場合に自ら throw～catch し、ROLLBACKさせると思います。
今回はPHPのNoticeエラーの場合でも throw～catch し、ROLLBACKさせたいのです。

発生している問題

該当のソースコードのように、Noticeエラーの場合でもそれを throw～catch させようしてset_error_handlerを書きました。

しかしこうすると次の errorMessage のように、なぜかwp_optionsやwp_commentsのLOCK TABLESを要求されます。

（どこにもwp_optionsやwp_commentsを更新するような処理は書いていないのに、です。）

[13-Mar-2022 14:14:33 UTC] WordPress データベースエラー: Table 'wp_options' was not locked with LOCK TABLES for query SELECT option_value FROM wp_options WHERE option_name = '_site_transient_update_plugins' LIMIT 1 made by require('wp-blog-header.php'), require_once('wp-includes/template-loader.php'), include('/themes/apple/index.php'), get_footer, locate_template, load_template, require_once('/themes/apple/footer.php'), wp_footer, do_action('wp_footer'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_render, do_action_ref_array('admin_bar_menu'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_updates_menu, wp_get_update_data, get_site_transient, get_site_option, get_network_option, get_option, QM_DB->query

[13-Mar-2022 14:14:33 UTC] WordPress データベースエラー: Table 'wp_options' was not locked with LOCK TABLES for query SELECT option_value FROM wp_options WHERE option_name = '_site_transient_update_themes' LIMIT 1 made by require('wp-blog-header.php'), require_once('wp-includes/template-loader.php'), include('/themes/apple/index.php'), get_footer, locate_template, load_template, require_once('/themes/apple/footer.php'), wp_footer, do_action('wp_footer'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_render, do_action_ref_array('admin_bar_menu'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_updates_menu, wp_get_update_data, get_site_transient, get_site_option, get_network_option, get_option, QM_DB->query

[13-Mar-2022 14:14:33 UTC] WordPress データベースエラー: Table 'wp_options' was not locked with LOCK TABLES for query SELECT option_value FROM wp_options WHERE option_name = '_site_transient_update_core' LIMIT 1 made by require('wp-blog-header.php'), require_once('wp-includes/template-loader.php'), include('/themes/apple/index.php'), get_footer, locate_template, load_template, require_once('/themes/apple/footer.php'), wp_footer, do_action('wp_footer'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_render, do_action_ref_array('admin_bar_menu'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_updates_menu, wp_get_update_data, wp_get_translation_updates, get_site_transient, get_site_option, get_network_option, get_option, QM_DB->query

[13-Mar-2022 14:14:33 UTC] WordPress データベースエラー: Table 'wp_comments' was not locked with LOCK TABLES for query 
		SELECT comment_approved, COUNT( * ) AS total
		FROM wp_comments
		
		GROUP BY comment_approved
	 made by require('wp-blog-header.php'), require_once('wp-includes/template-loader.php'), include('/themes/apple/index.php'), get_footer, locate_template, load_template, require_once('/themes/apple/footer.php'), wp_footer, do_action('wp_footer'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_render, do_action_ref_array('admin_bar_menu'), WP_Hook->do_action, WP_Hook->apply_filters, wp_admin_bar_comments_menu, wp_count_comments, get_comment_count, QM_DB->query

該当のソースコード

ロック対象の２つのテーブルです。text aやtag aという値が入っています。

問題のソースコードは後述のupdate()で生じます。

CREATE TABLE my_texts (
	 ID    BIGINT(20)   UNSIGNED NOT NULL AUTO_INCREMENT
	,text  VARCHAR(10)  NOT NULL 
	,PRIMARY KEY  (ID)
);
INSERT INTO my_texts (text) VALUES ('text a');

CREATE TABLE my_tags (
	 ID    BIGINT(20)   UNSIGNED NOT NULL AUTO_INCREMENT
	,tag   VARCHAR(10)  NOT NULL 
	,PRIMARY KEY  (ID)
);
INSERT INTO my_tags (tag) VALUES ('tag a');

以下がupdate()です。

上記のtext aやtag aを、text bやtag bへ更新しようとしますが、11行目のtag_dataのキーがidでなくxxであるために、59行目で「Notice: Undefined index」を生じさせています。

このエラーを throw～catch させるべく、set_error_handlerを書いたのですが、すると上記の errorMessage が表示されてしまうのが問題です。

// PHPのNoticeエラーの場合でも throw～catch 
function exceptions_error_handler($severity, $message, $filename, $lineno) {
	throw new ErrorException($message, 0, $severity, $filename, $lineno);
}

// 更新データ
$post_data = [
	'id'       => 1,
	'text'     => 'text b',
	'tag_data' => [
		'xx'  => 1, // 「Notice: Undefined index」
		'tag' => 'tag b'
	]
];

// 更新を実行
$posted = update( $post_data );
var_dump( $posted );

// 更新の関数
function update( $post_data ){
	global $wpdb;
	
	// PHPのNoticeエラーの場合でも throw～catch 
	set_error_handler('exceptions_error_handler');
	
	$result = ['status'=>'error'];
	try {
		$is_error = null;
		
		// テーブルのロック
		$wpdb->query('SET autocommit=0');
		$sql = 'LOCK TABLES my_texts WRITE, my_tags WRITE';
		if ( $wpdb->query($sql) === false ) {
			$is_error = true;
			$mes = 'テーブルのロックで失敗';
			error_log( $mes . ' $sql = ' . json_encode($sql) );
			throw new Exception( $mes );
			
		} else {
			
			// my_texts の投稿
			$sql = 'UPDATE my_texts SET text = %s WHERE ID IN ( %d )';
			$prepares = [ $post_data['text'], $post_data['id'] ];
			$rows = $wpdb->get_results( $wpdb->prepare( $sql, ...$prepares ) );
			if ( $wpdb->last_error ) {
				$is_error = true;
				$mes = 'my_texts の get_results で失敗';
				error_log( $mes . ' $sql = ' . json_encode($sql) );
				throw new Exception( $mes );
				
			} else {
				
				// タグ情報
				$tag_data = $post_data['tag_data'];
				
				// my_tags の投稿
				$sql = 'UPDATE my_tags SET tag = %s WHERE ID IN ( %d )';
				$prepares = [ $tag_data['tag'], $tag_data['id'] ]; // 「Notice: Undefined index」
				$rows = $wpdb->get_results( $wpdb->prepare( $sql, ...$prepares ) );
				if( $wpdb->last_error ){
					$is_error = true;
					$mes = 'my_tags の get_results で失敗';
					error_log( $mes . ' $sql = ' . json_encode($sql) );
					throw new Exception( $mes );
		
				}else{	
					$result = ['status'=>'ok'];
				}
				
			}
		}

		// ロック解除
		if( $is_error ){
			$wpdb->query('ROLLBACK');
			$wpdb->query('UNLOCK TABLES'); 
			error_log( 'ROLLBACK しました' );
			
		}else{
			$wpdb->query('COMMIT');
			$wpdb->query('UNLOCK TABLES'); 
			error_log( 'COMMIT しました' );
		}

	} catch (Exception $e) {
		$result = $e->getMessage();
		
	} finally {
		restore_error_handler();
	}
	
	return $result; 
}

試したこと

errorMessage のいう事を素直に聞いてwp_optionsやwp_commentsのLOCK TABLESを次のように実行すると、errorMessage はなくなりました。

$sql = 'LOCK TABLES my_texts WRITE, my_tags WRITE, wp_options WRITE, wp_comments WRITE';

しかし、そもそもwp_optionsやwp_commentsのLOCK TABLESがなぜ必要なのかわからないので、これであっているのか？とても不安な心境です。

そこで、２つのテーブルを 「厳密に」 LOCK TABLESするような方法として、適切な処理を求めて質問させて頂きました。

PHPのNoticeエラーの場合でもROLLBACKさせるにあたっては、どのような処理が良いのでしょうか？

長い質問で申し訳ございませんが、どなたかWordPressにお詳しい方がいらっしゃいましたら何卒、宜しくお願い致します。

理由不明だが解決した方法（2022/03/14 23:09）

上記「// ロック解除」の処理を下記のようにしたところ、 「厳密に」 LOCK TABLESもでき、errorMessage もなくなりました。しかしなぜ解決したのか？がわからずやはり不安な心境です。正しい処理なのか、他に問題はないのか、大いに疑問です。

原因がお分かりになる方いらっしゃいましたらご教示頂けませんでしょうか。

■ 上記「// ロック解除」の処理↓

		// ロック解除
		if( $is_error ){
			$wpdb->query('ROLLBACK');
			$wpdb->query('UNLOCK TABLES'); 
			error_log( 'ROLLBACK しました' );
			
		}else{
			$wpdb->query('COMMIT');
			$wpdb->query('UNLOCK TABLES'); 
			error_log( 'COMMIT しました' );
		}

	} catch (Exception $e) {
		$result = $e->getMessage();
		
	} finally {
		restore_error_handler();
	}
	
	return $result; 
}

■ 理由不明だが解決した方法↓

		// ロック解除
		$wpdb->query('COMMIT');
		error_log( 'COMMIT しました' );
		
	} catch (Exception $e) {
		$result = $e->getMessage();		
		$wpdb->query('ROLLBACK');
		error_log( 'ROLLBACK しました' );
			
	} finally {
		restore_error_handler();
		$wpdb->query('UNLOCK TABLES'); 
	}
			
	return $result; 
}

Answer 1

エラーメッセージに出てるtransientはメインクエリの
投稿検索結果のキャッシュの処理に係るものと思われます

推測ですがSTART TRANSACTION句がないために
メインクエリのSQLまでロールバックしようとしてしまっているのではないでしょうか

ちゃんと追いかけてはいないですが、WordPressは元々１プロセス中のSQLを
まとめてtransactionとしてrollbackできるようにしてるんじゃないかと思いますので
SAVEPOINTを指定して入れ子のtransactionを開始し
そこにROLLBACK TOするようにしてみると解消するかもしれません

Comments

[gorimaz]

また詳しくありがとうございます。

> 推測ですがSTART TRANSACTION句がないために
試しに 'SET autocommit=0' の替わりに 'START TRANSACTION' にしてみましたが、全く同じ errorMessage でした。

> SAVEPOINTを指定して入れ子のtransactionを開始し
> そこにROLLBACK TOする
こちらは使い方を勉強中で、まだ試せていません。

ただその前に、質問に追記した「理由不明だが解決した方法（2022/03/14 23:09）」で解決しまして、しかしなぜ解決したのか不明です。

もしこの解決した方法についてお心当たりございましたらまたお知恵を拝借できませんでしょうか。

[KazuhiroHatano]

LOCK TABLESでTRANSACTION開始した場合は一度閉じて開始するんですね、
では入れ子になってるTRANSACTIONのROLLBACKが外側まで戻ってるというのはなさそうですね

コードを見るに、修正前は例外を投げられるとTRANSACTIONが開いたままになってしまうかと思います

自前のSQLでLOCK TABLESすることで閉じたWordPressによるTRANSACTIONが
LOCKしていたテーブルと自前のSQLのLOCKしたテーブルが一致していないために
本来WordPressによるTRANSACTIONがCOMMITするはずのところでエラーになってる、という感じでしょうか

[gorimaz]

> 修正前は例外を投げられるとTRANSACTIONが開いたままになってしまう

あぁ！なるほど！
set_error_handler を書いたことによって、NOTICEエラーのときに処理が catch に飛んでしまい、だから

if( $is_error ){
}else{
}

の if の中も else の中も通らず、それらの中の 'UNLOCK TABLES' は実行されず、TRANSACTIONが開いたままになってしまう。ですか。

さすがです。拙いコードの解読をして頂きありがとうございます。

そしてこの場合に、

> 本来WordPressによるTRANSACTIONがCOMMITするはずのところ

と混ざってしまう。わけですね。

いや、ご慧眼に感服致します。
どうもありがとうございました。

Answer 2

エラーハンドラをセットし

set_error_handler('myErrorHandler');
function myErrorHandler($errno, $errstr, $errfile, $errline){
  throw new exception();
  return true;
}

tryでcatchしてください

try{
}catch(Exception $e){
  //ロールバック？
  die($e->getMessage());
}catch(PDOException $e){
  die($e->getMessage());
}

Comments

[gorimaz]

そのエラーハンドラを書いて問題が生じると質問に書いたつもりなのですが、どんな違いを意図していらっしゃいますか？それともお目に留まらなかっただけでしょうか？