WordPressで、「WP_Query」を「オリジナルのSQL」へと書き換えたい

Question

###実現したいこと
「WP_Query」を「オリジナルのSQL」へと書き換えたいです。

具体的にいいますと、こちらの「$event_query」に、後の「オリジナルのSQL」を入れたいです。

if($event_query->have_posts()): while($event_query->have_posts()): $event_query->the_post();
  the_title();
endwhile; endif;

###オリジナルのSQL
カスタムフィールド「cf_number」に応じて記事を取得するオリジナルのSQLがこちらで、問題ないです。

function my_posts_join( $join ){
  global $wpdb;

  if( ! is_admin() ) {
    $join .= "INNER JOIN ". $wpdb->postmeta ." ON ". $wpdb->posts .".ID = ". $wpdb->postmeta .".post_id ";

    $join .= "INNER JOIN ( ";
    $join .= "  SELECT ". $wpdb->postmeta .".meta_value as cf_number, max(". $wpdb->posts .".post_date) as max_date ";
    $join .= "      FROM  ". $wpdb->posts;
    $join .= "      INNER JOIN ". $wpdb->postmeta ." ON ". $wpdb->posts .".id = ". $wpdb->postmeta .".post_id  ";
    $join .= "      WHERE ". $wpdb->postmeta .".meta_key = 'cf_number' ";
    $join .= "      GROUP BY ". $wpdb->postmeta .".meta_value ";
    $join .= "    ) AS j1 ";
    $join .= "  ON ". $wpdb->postmeta .".meta_value = j1.cf_number AND ". $wpdb->posts .".post_date = j1.max_date ";
  }

  return $join;
}

add_filter('posts_join', 'my_posts_join' );

###試したこと
上の「オリジナルのSQL」を「$event_query」に入れるために、「posts_request」を使ってみました。しかし下記の書き方ではできませんでした。

間違っているところがどこか、分かりますでしょうか？

<?php
// 「オリジナルのSQL」に書き変える
add_filter('posts_request', function ($query) {
  global $wpdb;
  $query = "
	SELECT wp_posts.ID
	FROM wp_posts
	INNER JOIN wp_postmeta ON wp_posts.ID = wp_postmeta.post_id
	INNER JOIN ( 
		SELECT wp_postmeta.meta_value as cf_number, max(wp_posts.post_date) as max_date
		FROM   wp_posts
		INNER JOIN wp_postmeta ON wp_posts.ID = wp_postmeta.post_id 
		WHERE wp_postmeta.meta_key = 'cf_number'
		GROUP BY wp_postmeta.meta_value
	  ) AS j2
	  ON wp_postmeta.meta_value =j2.cf_number AND wp_posts.post_date = j2.max_date
  ";
  return $query;
});

// 「オリジナルのSQL」で記事を取得する
$event_query = new WP_Query([
  'posts_per_page' => 20,
]);
if($event_query->have_posts()): while($event_query->have_posts()): $event_query->the_post();
  the_title();
endwhile; endif;
?>

###追記
なぜ「my_posts_join」だけでメインループを使うという方法ではなくて、わざわざ「$event_query」を使うのか。といいますと、記事を「表示ボタン」をクリックしたときに指定のdiv内に表示したいためです。

Comments

[KazuhiroHatano]

何がどうダメだったのか、お願いします
少なくともまるっとSQLを上書きしちゃってるから
posts_per_pageが効かなくなってるだろうし
フィルタに条件がないので後のget_postsとかに影響してしまうだろうし
post_statusの条件がないのでゴミ箱の中のものも拾うだろうし
$wpdb->postsとかでテーブルの名前を取得していないので
プレフィクスがwp_じゃなかったら動かないだろうし
たくさん問題は見えます

[dany]

そこまでオリジナルSQLへのこだわりが強ければ、WP_Queryを使うメリットがないのでは？

<?php

$event_posts = $wpdb->get_results($wpdb->prepare("SELECT * FROM $wpdb->posts INNER JOI..."));

$event_posts = array_map('get_post', $event_posts);

foreach ($event_posts as $p) {
echo get_the_title($p->ID);
} ?>