wordpressのパンくずリストが2行になってしまったので1行にしたいです。

Question

前提・実現したいこと

wordpressのパンくずリストが2行になってしまったので1行にしたいです。

wordpressでTHE THORというテーマでEvents Managerを使用し、ページを作成したところ、エラーが出たので、パンくずリストに使われているphpファイルを編集してみましたが、エラーは消えたものの、パンくずリストが2行になってしまったので、1行になるように解決していただけませんでしょうか。

どうぞよろしくお願い致します。

発生している問題・エラーメッセージ

最初にページに出たエラーは以下です。

エラー修正後、パンくずが二行になってしまいました。

該当のソースコード

エラー修正前のPHPです。

	function current_crumb_tag( $current_permalink, $current_text = '', $args = array(), $current_class = 'breadcrumb__item breadcrumb__item-current' ) {
		$args = wp_parse_args( $args );
		$args = (object) $args;
		$current_class      = $current_class ? ' class="' . esc_attr( $current_class ) . '"' : '';
		$start_anchor_tag   = $current_permalink ? '<a href="' . $current_permalink . '">' : '';
		$end_anchor_tag     = $current_permalink ? '</a>' : '';
		$current_before     = '<li' . $current_class . '>' . $start_anchor_tag . '';
		$current_crumb_tag  = $current_text;
		$current_after      = '' . $end_anchor_tag . '</li>';
		if ( get_query_var( 'paged' ) ) {
			if ( is_paged() || is_category() || is_day() || is_month() || is_year() || is_search() || is_tag() || is_author() ) {
				$current_after = ' (ページ' . get_query_var( 'paged' ) . ')' . $current_after;
			}

		} elseif ( ( is_page() || is_single() ) && get_query_var( 'page' ) ) {
			$current_after = ' (ページ' . get_query_var( 'page' ) . ')' . $current_after;
		}

		return $current_before . $current_crumb_tag . $current_after;
	}

試したこと

エラーが出たPHPをif (!function_exists( 'current_crumb_tag' ) ) {}で囲いましたが、パンくずが2行になってしまいました。

	if (!function_exists( 'current_crumb_tag' ) ) {
		function current_crumb_tag( $current_permalink, $current_text = '', $args = array(), $current_class = 'breadcrumb__item breadcrumb__item-current' ) {
		$args = wp_parse_args( $args );
		$args = (object) $args;
		$current_class      = $current_class ? ' class="' . esc_attr( $current_class ) . '"' : '';
		$start_anchor_tag   = $current_permalink ? '<a href="' . $current_permalink . '">' : '';
		$end_anchor_tag     = $current_permalink ? '</a>' : '';
		$current_before     = '<li' . $current_class . '>' . $start_anchor_tag . '';
		$current_crumb_tag  = $current_text;
		$current_after      = '' . $end_anchor_tag . '</li>';
		if ( get_query_var( 'paged' ) ) {
			if ( is_paged() || is_category() || is_day() || is_month() || is_year() || is_search() || is_tag() || is_author() ) {
				$current_after = ' (ページ' . get_query_var( 'paged' ) . ')' . $current_after;
			}

		} elseif ( ( is_page() || is_single() ) && get_query_var( 'page' ) ) {
			$current_after = ' (ページ' . get_query_var( 'page' ) . ')' . $current_after;
		}

		return $current_before . $current_crumb_tag . $current_after;
	}}

Comments

[gogoweb_ikeda]

実際にパンくずリストを表示する部分のコードはどうなっていますか？
エラーメッセージと修正後の挙動からbreadcrumb.phpを2回読み込んでいるような気がします。

[koalachan]

ご連絡ありがとうございます。はやり仰っていただいたように同じコードが二回でてきてしまっています。これはbreadcrumb.phpの修正だけでは解決できないのでしょうか？お手数をおかけしますが、どうぞよろしくお願い致します。

<div class="l-headerBottom">
<div class="wider">
<div class="breadcrumb">
<ul class="breadcrumb__list container">
<li class="breadcrumb__item icon-home">
<a href="https://zioness.jp">HOME</a></li>
<li class="breadcrumb__item">
<a href="https://zioness.jp/%e3%82%a4%e3%83%99%e3%83%b3%e3%83%88/">イベント</a></li>
<li class="breadcrumb__item breadcrumb__item-current">
<a href="https://zioness.jp/events/test/">TEST</a></li></ul></div> </div> </div>

<div class="wider">
<div class="breadcrumb">
<ul class="breadcrumb__list container">
<li class="breadcrumb__item icon-home">
<a href="https://zioness.jp">HOME</a></li>
<li class="breadcrumb__item">
<a href="https://zioness.jp/%e3%82%a4%e3%83%99%e3%83%b3%e3%83%88/">イベント</a></li>
<li class="breadcrumb__item breadcrumb__item-current"><a href="https://zioness.jp/events/test/">TEST</a></li></ul></div> </div>

[gogoweb_ikeda]

テーマが有料のようなので中身は検証できませんが、breadcrumb.php内部にcurrent_crumb_tag関数とcurrent_crumb_tagを実行している箇所があるのでしょうか？
このパンくずリストはページ内に自動で挿入されるのですか？
パンくずリストを挿入が手動の場合間違って2回実行してませんか？
エラーがでたページのページテンプレート自体は修正していませんか？

[koalachan]

ご連絡ありがとうございます。パンくずリストは自動生成されたもので、こちらで挿入したり、テンプレートを触ったりはしていません。PHP修正後にこのようになってしまいました。

Answer 1

親テーマのsingle.phpを子テーマにコピー
子テーマにコピーしたsingle.phpのファイル名をsingle-event.phpに変更

そして100行目付近のパンくずに関するphpの記述をコメントアウトか削除

//single-event.php
中略

  <div class="wider">
    <?php fit_breadcrumb();?>←これを消す
  </div>


 <!--l-wrapper-->
中略

たしかevent managerは、カスタム投稿タイプ「event」を自動的に作成していたような気がするので、
カスタム投稿タイプ用のsingleページを設けてあげたらいけそうな気がします。

eventじゃなかったら申し訳ありません。

自分はこの方法でthorのカスタム投稿タイプのパンくずが二重になる問題を解消しました。

Comments

[koalachan]

返信が遅くなり申し訳ありません。
bokuoirokiさんとまったく同じ現象だったようです。
解決していただいたので、ベストアンサーとさせていただきます。
本当にありがとうございます。

[bokupiroki]

解決してよかったです。
自分も最近使い始めましたが、よくわからないテーマですね

Answer 2

PHP修正後にこのようになってしまいました。

ということは、function_existsで囲んだ修正しかしていないということでしょうか。

しかしそれだけでなる症状ではなさそうで、修正点を失念されているようなので何から探っていけばいいのか難しいです。

なので力技で…

まず症状としてはこういうものだと思うので

function current_crumb_tag() {
    return 'パンくずリスト';
}
for ( $i=0; $i<2; $i++ ) {
    echo current_crumb_tag(); // 二回出力される
}

力技すぎますが応急処置としてはこうでいかがでしょうか。

function current_crumb_tag() {
    if(defined('CURRENT_CRUMB_TAG')) return; // 追加
    define('CURRENT_CRUMB_TAG',true);        // 追加
    return 'パンくずリスト';
}
for ( $i=0; $i<2; $i++) {
    echo current_crumb_tag(); // 一回だけ出力される
}

つまりご質問に沿っていえば、current_crumb_tag()の中の一行目と二行目に追加してください。

if (!function_exists( 'current_crumb_tag' ) ) {
function current_crumb_tag( $current_permalink, $current_text = '', $args = array(), $current_class = 'breadcrumb__item breadcrumb__item-current' ) {
    if(defined('CURRENT_CRUMB_TAG')) return; // 追加
    define('CURRENT_CRUMB_TAG',true);        // 追加
    /*
     * 割愛
    */
    return $current_before . $current_crumb_tag . $current_after;
}}

Comments

[koalachan]

ご連絡ありがとうございます。
いただいたコードを追加したのですが、まだ2行のままです。ただ、私のphp修正だけでは起こる現象ではないということがわかっただけでも進歩なので、本当にありがたいです。ほかに触ってしまった箇所はないかもう一度ページを細かく見てみます。

[mossbarger]

なるほどすみません。実行側が２回呼ばれているわけですね。そうしましたらお手数おかけして申し訳ございませんが、current_crumb_tag()を実行しているテンプレートファイルをコピペしていただけませんでしょうか。

[mossbarger]

エディタに指定ディレクトリの一括文字列検索機能があると思いますので、テーマファイルをまるごと指定して、current_crumb_tagで検索していただければ該当の実行部分がすべて表示されると思います。その実行がループの中にあるか、または１回の読み込みの過程で２つ書かれているかだと思うので、どちらかわかりませんが見てみたいです。