php echoで出力した値が表示できない（Wordpress）

Question

function.phpに下記のようにセットしています。

function get_event_dates($datestart,$dateend,$long=0) {
		
		if ($long == 1) {
			$date_format = 'Y年n月j日';
    } elseif ($long == 2) {
      $date_format = 'Y年n月';
    } elseif ($long == 3) {
      $date_format = 'Y_n';
    } elseif ($long == 4) {
      $date_format = 'Y年n月j日';
      $dateend =  $datestart;
    } else {
			$date_format = 'Y/n/j';
    }

		$day_format = 'N';
		$year_format = 'Y';
		$date_format_2 = 'j日';
  
    $japanese_weekday_start = " (".japanese_weekday(date($day_format,(strtotime($datestart)))) . ")";
    $japanese_weekday_end = " ～  (".japanese_weekday(date($day_format,(strtotime($dateend)))) . ")";
    if ($long == 2 || $long == 3) {
      $japanese_weekday_start = '';
      $japanese_weekday_end = '';
    }


		if($datestart == $dateend):
				$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start;
		else : 

			if($datestart == $dateend):
				$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start;
			else :
				$startmonth = date('M',(strtotime($datestart)));
				$endmonth = date('M',(strtotime($dateend)));
				if($startmonth == $endmonth) :
					$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start . $japanese_weekday_end;
				else :
					$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start . $japanese_weekday_end;
				endif;
			endif;	
		endif;
return $date_string;

}

single.phpに下記でセットしています。

$start_date = get_post_meta(get_the_ID(), 'info_start_date', true);
$end_date = get_post_meta(get_the_ID(), 'info_end_date', true);
$date_string = get_event_dates($start_date,$end_date,1);

そして以下ようにphp echoで出力したいと思っています。

<?php echo $date_string; ?>

　

投稿画面から
開始日付　：　20180904
終了日付　：　20180905
を入力して投稿しました。

しかし出力される値が、
2018年9月4日 (火) ～ (水)

のように開始日付は取得できるのですが、終了日付が取得されません。

下記のように個々にと出力すると

<?php echo $start_date; ?>
<?php echo $end_date; ?>

→
2018年9月4日 (火)
2018年9月5日 (水)

と表示できるのですが、$date_stringでつなげると表示ができません。

　　

　

出力結果を
→
2018年9月4日 (火) ～ 2018年9月5日 (水)

上記のように表示がしたいのですが、どのようにすればよいでしょうか。

Comments

[m.ts10806]

インデントも揃えてもらいたいのですが、ifの書き方もどちらかに統一してください。

Answer 1

Codex を参照する限り、get_event_dates という関数は存在していませんよ。

Answer 2

下の方の

if($startmonth == $endmonth) :
$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start . $japanese_weekday_end;
else :

の部分で終了の日付( date($date_format,(strtotime($datesend))) )が入るようになってないからじゃないですかね。

$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start . date($date_format,(strtotime($dateend))) .$japanese_weekday_end;

とかにしたらどうでしょう？
同じような処理をするところは同様に書き換えると良さそうな気がします。

だいぶ自分仕様に書き直して検証したので自信なしですがいかがでしょう。

Comments

[melpo]

あ。ダメですね。japanese_weekday(date($day_format,(strtotime($dateend))) の方に入らないんですね。japanese_weekday　というのは別の場所にある自作の関数でしょうか？wordpressにはない感じですが。

[melpo]

～ が必要なのはenddateを出したいときだけなので、返り値の方に持ってきて

$date_string = date($date_format,(strtotime($datestart))) . $japanese_weekday_start . "～".date($date_format,(strtotime($dateend))) .$japanese_weekday_end;

とかにしちゃうかですかね。

余計なお世話ですが、写し間違いか、同じif文が二回あったり、$startmonth == $endmonth のときと、そのelseの結果が同じだったりしてるのでそのあたり整頓すると問題もスッキリするかなと思います。

Answer 3

自分がそれと似た関数を書いたことがあるので、
エスパーして答えるとそれは
get_event_datesという関数の仕様
略せるところを略して読みやすくする関数

開始日と年が同じなら年を省略
開始日と月が同じなら月を省略
開始日と週が同じなら日を省略
というような処理をしているのでしょう

同じ週の場合に日を略して曜日だけ残すというのは
あまりわかりやすい仕様でないように思いますが
多分そういう仕様です

何曜日なのかが重要ということなのでしょう

省略したくないなら使う意味がないので
省略したくないなら普通に出力してやればいいです

---

追記の関数見て

このget_event_datesって関数、
なんか意図した仕様通りに書かれてないっぽいよね

再びエスパーしちゃうと、
日付のフォーマットを選択して出力できるようにしたいという要望と
略せるところを略したいという要望が両立できなくて立ち往生しちゃってるって感じに見えるよ

それで、略せるところを略したいという要望を諦めるということになったのかな？

幸いにもフォーマットの文字列を指定して、みたいな感じになってないから
関数の引数を変えたりすることなく、略せるところを略したいという要望との両立は可能とは思うけどね