PHP5.2→PHP7.3でのrestore_error_handlerの挙動の変化について

Question

お世話になります。

PHP5.2からPHP7.3へのバージョンアップを命じられ取り組んでいるのですがどうしてもわからないことがあります。

データベースへの接続用クラス内で、

set_error_handler(array($this,'errorHandler'));

とエラーハンドラーを設定し、データベースへの接続用クラス使用後に

$DB = new データべースへの接続用クラス
restore_error_handler(array($DB,'errorHandler'));

上記のようにエラーハンドラーを解除したかった(？)ような記述がありました。
ただ、restore_error_handlerのリファレンスを読むと、引数を入れられない関数なのではないかと思います。

実際、PHP7.3では引数が多いとエラーが出ています。
PHP5.2ではエラーが出ていなかったのでスルーされてきたようです。
単純に引数を消してしまおうと思いましたが、もしかしたら最初の作成者のなにかしらの意図があっての書き方かとも思い躊躇しています。

①PHP5.2ではエラーが出ず、PHP7.3ではエラーがでる理由。
②単純にrestore_error_handler()と直した場合、最初の製作者の意図どおりの動きと同じになるのか。

以上2点アドバイスいただけますと幸いです。

Answer 1

ちょっと調べた感じだとPHP5の時代から引数無しの関数みたいですから、最初から使い方が間違えていたのではないか？と思います。

（それを書いた人、現場により何か察せられるのではないかなぁ、とは思いますが外部の人間には全く分かりません。）

他に情報が無いのでなんとも言えませんが、

set_error_handler(array($this,'errorHandler'));

と

restore_error_handler(array($DB,'errorHandler'));

の間に起ったエラーに関してerrorHandlerに当たるメソッドで処理しようとしていた、という事かと思うので、内容で判断するしか無いと思います。

php7についてあまり触ってないので1に関してはまぁ、php7の方が色々厳しくなったんだろうな、くらいです。
（php5では、func_get_argsが使える前提だし色々余計な引数付けてても別にエラーは出なかったはずです。）

Comments

[asus2]

ご回答ありがとうございました。
自分でもリファレンスを隅々までチェックしてみましたが引数が必要だった時代はありませんでした。最初から間違いがあったのですね。
もう作成者も関わっていた方もいないシステムなので内容吟味して対応したいと思います。