Notice: Trying to get property of non-objectについて

Question

Notice: Trying to get property of non-object in C:\Users\xxxxxx\Desktop\xxxxxx\wordpress\wp-content\plugins\ps-auto-sitemap\ps_auto_sitemap.php on line 61

wordpressのプラグインで上記のエラーがでるようになってしまいました

以下がコードが記述されているファイルですが、どのように記述し直すとエラーが解決しますでしょうか？

文字数が多すぎたためこちらに記述ができず、宅ふぁいる便にアップロードしました

よろしくお願いします

https://free.filesend.to/filedn_infoindex?rp=98a8f4179fff7dd2d833e725ae6be4do

Answer 1

PS Auto Sitemap の エラー対処

Comments

[ryuujinn]

Kosuke_Shibuyaさん、コメントありがとうございました

ご指摘のエラー対処法試してみましたが改善されませんでした

Answer 2

このNoticeは存在しないプロパティを参照しているときに発生するものです。

if ( $this->option['post_id'] && $post->ID == $this->option['post_id'] ) {
				if ( isset( $_GET['category'] ) && $category = get_category( (int)$_GET['category'] ) ) {
					$sitemap_content = $this->make_category_sitemap( $category );
				} else {
					$cache_dir = $this->check_cache_dir();
					if ( $cache_dir && file_exists( $cache_dir . '/site_map_cache.html' ) && $this->option['use_cache'] ) {
						$sitemap_content = file_get_contents( $cache_dir . '/site_map_cache.html' );
					} else {
						$sitemap_content = $this->create_sitemap_content();
					}
				}
			$content = preg_replace( '/(<p><!-- SITEMAP CONTENT REPLACE POINT --><\/p>|<!-- SITEMAP CONTENT REPLACE POINT -->)/', $sitemap_content, $content, 1 );
}

ここの一番上のif文で発生しているようなので、$this->optionもしくは$post->IDのどちらかがプロパティの参照エラーになっています。
エラーが発生しているようであればvar_dumpを使って出力してみると良いかもしれません。

Comments

[ryuujinn]

コメントありがとうございます
具体的には$this->optionもしくは$post->IDを
どのようにvar_dumpを記述すればよろしいのでしょうか？

[deadcode]

if文のひとつ上の行に
var_dump($this->option['post_id']);
var_dump($post->ID);
exit;
上記のようにし、エラーが出るページで実行してみては如何でしょうか。
もしかしたらどちらかがNULLで返ってくる（Noticeとして）可能性があります。
その場合、NULLとして返ってきた変数を追う必要があるかもしれません。

[ryuujinn]

具体的なコメントありがとうございます
早速上記のコードを追加したんですが
以下のエラーが出ました、ご指摘ととおり、NULLが出てきたのですが
対処はどのようにしたらよろしいでしょうか？

string(3) "633"
Notice: Trying to get property of non-object in C:\Users\xxxxxxx\Desktop\xxxxxxxx\wordpress\wp-content\plugins\ps-auto-sitemap\ps_auto_sitemap.php on line 62
NULL

[deadcode]

var_dump($this->option['post_id']);
var_dump($post->ID);
exit;

上記の通りの場合、$post->IDというプロパティが存在しないということです。
このファイルだけで追うのは難しいのですが、$post->IDは480行目の
print_sitemap_prepare_css
というメソッドでも使われているので同様のエラーが出る可能性があります。
$postだけでdumpしてみてはどうでしょうか。

実際この情報でできるのはここまでかと思います。
$postのプロパティがどこで設定されているかがわかれば解決できるかと思います。

[ryuujinn]

deadcodeさん、コメントありがとうございます
$postだけでdumpとは具体的には以下のように記述し直せばいいのでしょうか？

if文のひとつ上の行に
var_dump($post->ID);
exit;

ただ、これだと今回は以下のエラーが出ました

Notice: Trying to get property of non-object in C:\Users\xxxx\Desktop\xxxxx\wordpress\wp-content\plugins\ps-auto-sitemap\ps_auto_sitemap.php on line 61
NULL

＞実際この情報でできるのはここまでかと思います。
＞$postのプロパティがどこで設定されているかがわかれば解決できるかと思います。

こちらでご提示できる情報が他にもあればご提示したいのですが、
現状はこれ以上何をご提示すればいいのか、ご親切対応頂いているのにすいません

[deadcode]

PHPの知見がないとこの問題を解決するには少し難しいかもしれません。
$post内のプロパティがないと言われているので、$post自体にまずプロパティがあるのか確認をする。
var_dump($post);
ということです。
$post自体が存在しているか確認してください。
$postがNULLではない場合はIDというプロパティに代入している部分があるので、そこが抜けているのではないかという予測です。

$postがNULLであれば$postにObjectが入っていないということなので、$postに入れるはずのObjectをどこかで入れる事が必要になります。

[ryuujinn]

deadcodeさん、コメントありがとうございます

大変親切にご説明を頂けてるのはわかるのですが

あいにく、PHPの知見がないので、どのようにコードを記述したらいいのかわかりません

折角ご親切に問題部分をご指摘、切り分けて頂いているのに申し訳ありません