NGINXのerror.logにて、PHPのcount()関数で引数にCountableでない変数を設定していたため、エラーが発生していました。自分のテーマではなくwordpressのファイルで発生していましたが、どうにか対処することができましたよ。
事象
NGINXのerror.logに下記のようなエラーログがたくさん出力されてました。このエラーは当ブログの古い記事ページへのアクセス時に発生していました。
2018/08/01 19:02:23 [error] 14031#14031: *12154 FastCGI sent in stderr: "PHP message: PHP Warning: count(): Parameter must be an array or an object that implements Countable in 【wordpressインストールディレクトリ】/wp-includes/post-template.php on line 284" while reading response header from upstream, client: 【IPアドレス】, server: 【サーバー名】, request: "GET 【アクセス先のスラッグ(例:/hello-world/)】 HTTP/1.1", upstream: "fastcgi://unix:/var/run/php-fpm/php-fpm.sock:", host: 【ホスト名】
このブログは昔はFC2ブログで書いていて、その後wordpressに引っ越しして、記事をwordpressにインポートしています。wordpressに移行してから新しく投稿した記事ではこのエラーはあまり見当たらなかったです(少しは発生していた)。
ブログの表示自体には特に問題はなかったので、放置していても良かったのですが、エラーログが多すぎでどうしても気になります。
原因
PHP7.2からcount関数の引数にNULLやカウントできない型の変数を設定した場合に、上記のようなエラーが発生するようになったそうです。PHP7.1までは発生しないみたいです。
また、当ブログで上記エラーが発生したときの各ソフトウェアのバージョンですが、PHP 7.2.8、nginx 1.15.2、wordpress 4.9.7になります。
エラーログに表示されているように、wordpressのファイル「/wp-includes/post-template.php」の284行目でエラー発生しています。
function get_the_content( $more_link_text = null, $strip_teaser = false ) {
global $page, $more, $preview, $pages, $multipage;
~
略
~
if ( $page > count( $pages ) ) // if the requested page doesn't exist ←284行目
ここに登場する$pagesというグローバル変数ですが、下記のソースコメントより、一つの記事を複数ページに分けたとき(「続きを読む」的なやつ)の記事内容の配列みたいですね。
$pages Array of all pages in post/page. Each array element contains part of the content separated by the <!--nextpage--> tag.
でも、私は今まで、一つの記事を「続きを読む」で複数ページに分けたことないんですよね。。。謎です。
対策
古い記事でこのエラーが発生していることをヒントにして解析していきました。昔はアイキャッチ画像を設定していなかったり、抜粋(description)を書いていなかったり、記事にh2見出しを設定していなかったりしていました。
ですが、このエラーが発生している記事に共通していることは、head部のmetaタグで抜粋(description)を設定していないことでした。アイキャッチ画像やh2見出しの有無は関係ないようです。
当ブログのテーマファイルでは、抜粋を取得するための関数として、get_the_excerpt()を使っていたのですが、試しに呼ばないようにしたら、本事象のエラーログが出力されなくなりました。get_the_excerptの中まではソースを読んでないですが、そこが原因なのでしょうね。
ということで、対策としては下記のどちらかかなと思います。
- 抜粋(description)をちゃんと書く
- 抜粋(description)が無い記事ではget_the_excerpt()関数を使わないようにする
本来は抜粋を書いたほうがSEO的な意味でもいいと思うのですが、昔の記事を一つずつリライトしていくのはとても面倒なので、テーマファイルでget_the_excerpt()を使っていることころを修正しました。has_excerpt()で抜粋の有無を判定するようにしましたよ。
if(is_single()) {
//抜粋を設定している場合
if(has_excerpt()){
//get_the_excerptで抜粋を取得する
$description = get_the_excerpt();
//抜粋を設定していない場合
} else {
//先頭80文字を抜粋とする
$description = mb_substr( strip_tags( $post->post_content ), 0, 80 );
}
}
?>
<meta id="description" name="description" itemprop="description" content="<?php echo $description; ?>">
おわりに
wordpressのPHPのファイルで発生しているエラーなので、本当はwordpressのバージョンアップで直して欲しいと思いますが、とりあえずテーマファイルの修正で対応してみました。気が向いた時に過去記事のリライトもしていこうかと思います。