質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.23%

親子階層の情報を取得する関数が、記事のないアーカイブページでphpエラーになる

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 129

aaasss

score 8

WordPress5.3のテンプレートarchive.phpでphpエラーが発生しています。
どのように修正すればよいか、ご教授いただければ幸いです。

発生しているエラー文

Notice: Trying to get property 'ancestors' of non-object in /xxx/xxx/wp-content/themes/xxx/functions.php on line 136

function.phpの該当部分

//固定ページの親子階層ごとに条件分岐
function page_is_ancestor_of($slug){
    global $post;
    $page = get_page_by_path($slug); // 親か判別したい固定ページスラッグからページ情報を取得
    $result = false;
    if(isset($page)){
        foreach ((array)$post->ancestors as $ancestor) {
            if($ancestor == $page->ID){ $result = true; }
        }
    }
    return $result;
}


↑のコード内の、下記の部分が 136行目です。

foreach ((array)$post->ancestors as $ancestor) {

archive.phpのエラーを吐いている部分

<?php if(is_page('about') || page_is_ancestor_of('about')): ?>

   固定ページのaboutか、aboutの子階層で表示するコンテンツ

<?php endif; ?>

エラーの発生条件

該当記事が1つもないカテゴリーの、アーカイブページを開く。
(カテゴリーに該当する記事を1つ作成すればエラーは消える)

エラーの原因?

表示する記事がないので、function.phpに書いたページ情報の取得が、うまく働いていないのだと思いますが…。
記事がない場合は何もしなくてよいのですが、phpをどのように書けばよいのでしょうか?

3/29追記:補足情報

「archive.phpのエラーを吐いている部分」について、補足です。
当初はarchive.phpでのみエラーが発生していたので、簡略化して書いてしまいました。エラーが発生しているのは、正確にはarchive.phpで読み込んでいるサイドバー用のsidebar.phpです。そして、このsidebar.phpは全ての固定ページと、投稿アーカイブページで共用しています。

なので、実現したいsidebar.phpの動きは以下になります。
・固定ページで読み込まれているときは親子階層により条件分岐をする
・投稿アーカイブページで読み込まれているときは親子階層の判別は不要

現状、固定ページは問題なく動いており、また投稿アーカイブページで読み込まれているときも、投稿記事が1つでもあればエラーは出ていません。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

これでどうでしょう。

    if(isset($page) && $post->ancestors){
        foreach ((array)$post->ancestors as $ancestor) {
            if($ancestor == $page->ID){ $result = true; }
        }
    }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/03/29 10:57

    コメントありがとうございます。
    いただいたコードを適用してみたところ、エラー文は同じで、エラー箇所が1行前の if(isset($page) && $post->ancestors){ に変わりました。

    そこで、こちらで以下のように書き換えてみたところ、archive.phpではエラーが出なくなりました。
    if(isset($page) && ($post) && $post->ancestors){


    ただし、今度は固定ページの子階層において、以下の条件分岐の箇所が一切表示されなくなってしまいました。表示されないだけで、エラーは出ていません。また、最上の親階層では問題がありません。

    <?php if(is_page('about') || page_is_ancestor_of('about')): ?>
       固定ページのaboutか、aboutの子階層で表示するコンテンツ
    <?php endif; ?>

    問題の箇所がアーカイブページではなく、固定ページ(page.php)に移ってしまったので、補足情報として、エラー部分が正確にはsidebar.phpである旨を質問文に追記いたしました。
    情報が足りず申し訳ありません。

    //現在のfunction.php
    //固定ページの親子階層ごとに条件分岐
    function page_is_ancestor_of($slug){
    global $post;
    $page = get_page_by_path($slug); // 親か判別したい固定ページスラッグからページ情報を取得
    $result = false;
    if(isset($page) && ($post) && $post->ancestors){ //投稿はよいが、固定ページはだめ
    foreach ((array)$post->ancestors as $ancestor) {
    if($ancestor == $page->ID){ $result = true; }
    }
    }
    }

    キャンセル

  • 2020/03/29 11:14

    if(isset($page) && !empty($post->ancestors))
    とか。

    キャンセル

  • 2020/03/30 10:33

    ありがとうございます! 以下のコードで思い通りの挙動になりました。大変助かりました。

    //固定ページの親子階層ごとに条件分岐
    function page_is_ancestor_of($slug){
    global $post;
    $page = get_page_by_path($slug);
    $result = false;
    if(isset($page) && !empty($post->ancestors)) {
    foreach ((array)$post->ancestors as $ancestor) {
    if($ancestor == $page->ID){ $result = true; }
    }
    }
    return $result;
    }

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.23%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる