重複クエリーを消したい

Question

■やりたいこと
wordpressで重複クエリーを消したい
■問題点
the_content();で取得したものが何故か二回表示されてしまうため、Query Monitorで調べてみたところ重複しているクエリーがあるみたいでそれが原因と考えられます。
この重複クエリーを消す方法がわかりません。

//重複クエリー

SELECT SQL_CALC_FOUND_ROWS wp_posts.ID
FROM wp_posts
INNER JOIN wp_postmeta
ON ( wp_posts.ID = wp_postmeta.post_id )
WHERE 1=1
AND ( wp_postmeta.meta_key = '_bbp_last_active_time' )
AND wp_posts.post_type = 'topic'
AND (wp_posts.post_status = 'publish'
OR wp_posts.post_status = 'closed'
OR wp_posts.post_status = 'private'
OR wp_posts.post_status = 'hidden')
GROUP BY wp_posts.ID
ORDER BY wp_postmeta.meta_value DESC
LIMIT 0, 35

//数 2
	
//呼び出し元 WP_Query->get_posts()2件のコール
	
//コンポーネント プラグイン: bbpress2件のコール	

//潜在的脅威 WP_Query->get_posts()2件のコール

//重複しているクエリー1の呼び出し元　WP_Query->get_posts()

//重複しているクエリー2の呼び出し元
WP_Query->get_posts()
wp-includes/class-wp-query.php:3005
WP_Query->query()
wp-includes/class-wp-query.php:3432
WP_Query->__construct()
wp-includes/class-wp-query.php:3541
bbp_has_topics()
wp-content/plugins/bbpress/includes/topics/template.php:203
load_template('wp-content/plugins/bbpress/templates/default/bbpress/content-archive-topic.php')
wp-includes/template.php:725
bbp_locate_template()
wp-content/plugins/bbpress/includes/core/template-functions.php:105
bbp_get_template_part()
wp-content/plugins/bbpress/includes/core/template-functions.php:43
BBP_Shortcodes->display_topic_index()
wp-content/plugins/bbpress/includes/common/shortcodes.php:300
do_shortcode_tag()
wp-includes/shortcodes.php:343
preg_replace_callback()
wp-includes/shortcodes.php:343
do_shortcode()
wp-includes/shortcodes.php:218
apply_filters('the_content')
wp-includes/plugin.php:206
the_content()
wp-includes/post-template.php:253

■環境
phpmyadmin
■追加情報

Comments

[yambejp]

SQL ServerとphpMyAdminだと環境が全然違いますがMySQLということでよろしいですか？

[sazi]

wp_posts.ID と wp_postmeta.post_idの関係は1：多ですよね？

[sazi]

MySQLのバージョンは何ですか？

[sazi]

重複クエリーと言われているのは、「クエリーの結果に同じ値が返却される」という意味ですか？

[cafe1111]

SQL ServerとphpMyAdminだと環境が全然違いますがMySQLということでよろしいですか？
>>mysqlです。
wp_posts.ID と wp_postmeta.post_idの関係は1：多ですよね？
>>すみません、こちらはよくわかりません。
どこから確認すればわかるのでしょうか。
MySQLのバージョンは何ですか？
>>5.7.26 - MySQLです。
重複クエリーと言われているのは、「クエリーの結果に同じ値が返却される」という意味ですか？
>>質問の答えになっているかわかりませんが
<div class="entry-content clearfix">
　<?php the_content(); ?>
</div>
この表記が
<div class="entry-content clearfix">
　<div id="bbpress-forums"></div>
<div id="bbpress-forums"></div>
<div id="bbpress-forums"></div>
</div>
となってしまいます。
正確には3つ表記されるのですが1番上はサイトには表示されていないので問題ないと思います。

[sazi]

>>wp_posts.ID と wp_postmeta.post_idの関係は1：多ですよね？
>すみません、こちらはよくわかりません。
>どこから確認すればわかるのでしょうか。
テーブルの設計段階で決まっているはずです。それが分からなければ正しいSQLは組めません。

[cafe1111]

すみません、このテーブルを作成したのは自分じゃないのでわからないです。
一応作った方に聞いてみます。

[CHERRY]

SQL からすると、WordPress の bbPress プラグインの話ですね。

この現象が出るのは、bbPress のどの画面を表示する際でしょうか？

また、お使いのテーマは何をお使いですか？

[cafe1111]

WordPress の bbPress プラグインの話です。
この現象が出るのは、bbPress のどの画面を表示する際でしょうか？
>>excellent child/page.phpです
また、お使いのテーマは何をお使いですか？
Excellent Childです。

[cafe1111]

すみません、正確には>excellent child/page.phpのトピックが重複します。

[CHERRY]

状況が把握できていなくて、再現ができないので、こちらで...

お使いのテーマは、 https://wordpress.org/themes/excellent/ の子テーマということでしょうか？

このテーマだとするとテーマは、bbPress をサポートしていると書いているので、そんなに変な動きをするとは思えないのですが...

子テーマは、いろいろカスタマイズされていると思いますので、子テーマではなく　カスタマイズしていない状態の親テーマをテーマに指定してみると　どのような状況になりますか？　

同じように重複しますか？

[cafe1111]

ご返信が遅れてしまい申し訳ありません。
テーマはurlのものです。
親テーマだけではなく他のテーマに切り替えてみても重複してました。
しかしbbpressを更新したら自分がカスタマイズした部分は消えましたが重複しなくなりました。
なので更新しても消えないようにしてから、bbpressをアップデートすれば解決するのかなと考えております。

Answer 1

order byの指定によって、暗黙的にwp_postmeta.meta_valueもgroup byになっていないでしょうか。
order byを外してみてください。

外す事によって重複が無くなった場合で、order byも指定が必要なら、wp_postmeta.meta_valueのMax()やMin()などの集計関数を指定したものに対してのorder byとなるように変更して下さい。

尤も、meta_valueの扱いについてルール（仕様）があるなら、そんなに単純ではないでしょうけど。

Comments

[cafe1111]

すみません、どこから外せば良いのでしょうか？

[sazi]

質問にあるSQLの話です。

[cafe1111]

phpmyadminのSQLに下記のように変えてから実行しました効果なしでした。
SELECT SQL_CALC_FOUND_ROWS wp_posts.ID
FROM wp_posts
INNER JOIN wp_postmeta
ON ( wp_posts.ID = wp_postmeta.post_id )
WHERE 1=1
AND ( wp_postmeta.meta_key = '_bbp_last_active_time' )
AND wp_posts.post_type = 'topic'
AND (wp_posts.post_status = 'publish'
OR wp_posts.post_status = 'closed'
OR wp_posts.post_status = 'private'
OR wp_posts.post_status = 'hidden')
GROUP BY wp_posts.ID
LIMIT 0, 35

[sazi]

上記SQLを直接実行して重複があるのですか？
SQL_CALC_FOUND_ROWS を指定しないとどうなりますか？

[cafe1111]

直接実行しても重複がまだあります。
SQL_CALC_FOUND_ROWSをなしにしても特に変化がありませんでした。
一応写真も追加しておきます。

[sazi]

画像には重複しているIDがありませんが。

[cafe1111]

そうなんですか、、
しかし実際にサイトを見に行くと何も変化がありません。
こちらも写真を載せておきます。

[sazi]

クエリーの結果が重複してるということではなく、同じクエリーが複数回実行されているって事みたいですね。

[sazi]

そっち方面には明るくないので、お役に立てそうにありません。
ただ、order byについては事実ですので、見直しされて下さい。

[cafe1111]

わかりました。
ここまで考えてくださりありがとうございました。
order byについては見直してみます。
ありがとうございました。

Answer 2

重複をなくすという意図がわかりませんが
post_status がORで検索してるのですから、条件が複数マッチする可能性はあります

Comments

[cafe1111]

そうなんですか、、
重複をなくす意図は同じ内容が表示されてしまうからです。
例：
<div class="entry-content clearfix">
<?php the_content(); ?>
</div>
これが
<div class="entry-content clearfix">
<p>aaaaaaa</p>
<p>aaaaaaa</p>
</div>
のようになります。
本来は一回表示でいいのに同じものが二回表示されてしまいます。

[yambejp]

テーブルの構造、データ、リレーションの仕方しだいなので
いま提示されているデータでは回答にもつけたとおり「or検索で複数ヒットしてる？」
くらいしか言いようがないですね。
データが複数あれば同じデータが表示されるのは当たり前です

[sazi]

group byしてるのに重複しているって事だと思います。

[cafe1111]

少し見直してみます。