日付が1970年１月１日になる

Question

Wordpressで記事を更新したときに日付を表示させているのですが、
1970年や6800年というわけのわからない日付になってしまいます。
おそらくここが原因だと思うのですが、どのように改善すればいいのでしょうか？

該当のソースコード

	$date1 = date("Y年m月d日", strtotime(get_post_meta($post->ID, 'wpcf-date1', true)));
	$date2 = date("Y年m月d日", strtotime(get_post_meta($post->ID, 'wpcf-date2', true)));

	if($date2 != ''){
		$detail .= '<div class="detail">日時：'.$date1.' ～ '.$date2.'</div>';
	}else{
		$detail .= '<div class="detail">日時：'.$date1.'</div>';
	}

Comments

[CHERRY]

PHP的な内容は、回答がついている通りですが、カスタムフィールド wpcf-date1 と wpcf-date2 には、どのような値が入っていますか？

[kei344]

回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。

Answer 1

1970年１月１日

date()の第2引数がNULLだからでは。

echo date("Ymd",null);
//19700101

あと、date()の第2引数は左記マニュアルの通り、タイムスタンプでないといけません。
strtotime()を通してタイムスタンプに変換しましょう。

echo date("Y年m月d日","20190204");
//1970年08月22日　→これは単に1970.1.1から20190204秒経過した日付が出ている
echo date("Y年m月d日",strtotime("20190204"));
//2019年02月04日

※そもそもdate()の第二引数はnullはともかく他は数値型でないとエラーになります。

Comments

[megune]

おっしゃる通り値がうまく取れてきていないようでした。
ありがとうございました。

[m.ts10806]

実際にdateなどでフォーマットかける前に値を確認してみるといいですよ。
空ではないからと想定通りの情報が渡ってきている保証はないですし。もう少しifを通す条件を厳しくしてみてもいいかもしれません

[papinianus]

ベストアンサーを選定するうえで、nullであると確認しましたか？
get_post_metaは第三引数$singleをtrueにした場合、見つからないときのもどりは''(空文字)ですよね？
https://wpdocs.osdn.jp/%E9%96%A2%E6%95%B0%E3%83%AA%E3%83%95%E3%82%A1%E3%83%AC%E3%83%B3%E3%82%B9/get_post_meta#.E6.88.BB.E3.82.8A.E5.80.A4

[m.ts10806]

取れた値で見るのも大事ですが「うまく取れてない」原因探る方が先ですね。
取れないことがあっていいのか、どうかの仕様も含めて

Answer 2

PHPの日時データは1970/1/1 0:0:0からの経過秒数で表します
なので与えたデータが0だったということでしょう。
get_post_meta()の使い方がおかしいのでしょう

Answer 3

$date1_1 = get_post_meta($post->ID, 'wpcf-date1', true);
$date1 = date("Y年m月d日", $date1_1);
$date2_2 = get_post_meta($post->ID, 'wpcf-date2', true);
$date2 = date("Y年m月d日", $date2_2);

じゃないでしょうか。

Comments

[m.ts10806]

それだと質問者さんのコードを冗長に書き直しただけはないでしょうか。
get_post_meta()はPOSTされた文字列が返ってくる関数のようなのでdate()にかけたら結局同じです。

[pontaq]

すみません、何かでカスタムフィールドにいきなりいれることができないとみたことがあったような気がして。
そもそもwpcf-date1,2がnullなんですね。失礼しました

[m.ts10806]

nullかどうかは分からないですけどね（0かもしれないし・・）
私の回答はWordPressかどうかは関係のない内容なので、あわせ技は必要かもしれませんが、この質問内容からは分からないというところですね・・

[kei344]

To: mts10806さん
＞ get_post_meta()はPOSTされた文字列が返ってくる関数
ここは違います。postは記事のようなレコード単位のことで、postに関連付けられたmeta（文字列以外も可）を取得する物です。（なのでmetaに何を入れたか、および複数同じキーで登録しているかなどによって返り値も変わります）

【get_post_meta – WordPress私的マニュアル】
https://elearn.jp/wpman/function/get_post_meta.html

[m.ts10806]

ありがとうございます。
ざっとマニュアル読んだだけで安直な理解をしてしまっていました。

[pontaq]

おそらくですが、
1970年→Start
6800年→End(経過時間)
をとってしまっているんじゃないかと思うので、wpcf-date1,2のとってくる値が間違っていそうですね。