「Advanced Custom Fields」でピックアップ記事の組み合わせ表示について

Question

いつもお世話になっております
「Advanced Custom Fields」で投稿記事の中から特定の記事を
特定の場所に表示したく
色々と調べており、下記の方法で対応することができました

フィールド名：acf-pickup
サブフィールド名
・チェックボックス選択肢（1 : はい）acf-pickup-on

<?php
	$toplist = array(
		'post_type' => 'post',
		'posts_per_page' => 10,
		'meta_key' => 'acf-pickup_acf-pickup-on',
		'meta_value' => '1',
		'meta_compare' => 'LIKE'
	);
?>
<?php query_posts($toplist); ?>
<?php if (have_posts()) : while(have_posts()) : the_post(); ?>

///ここに繰り返しの内容

<?php endwhile; endif; wp_reset_query(); ?>

ここで、さらに追加機能として
その記事に公開期限をつけたく下記の仕様で試したところ
問題なくいけたのですが
実は色々調べてコードの付け足し付け足しをしているのに不安がありまして

といいますのも以前に別の作業で
同じようにコードの付け足しを繰り返し設置したところ
サーバーに負荷がかかってしまい、
閲覧できなくなったことがありました

今回も同じような事がおこるかもとの不安があります
大変申し訳ないのですが
本当にこのコードで問題ないがご指導いただけないでしょうか

フィールド名：acf-pickup
サブフィールド名
・チェックボックス選択肢（1 : はい）acf-pickup-on
・Date Time Picker（開始日）　acf-pickup-open
・Date Time Picker（終了日）　acf-pickup-close

<?php
	$toplist = array(
		'post_type' => 'post',
		'posts_per_page' => 10,
		'meta_key' => 'acf-pickup_acf-pickup-on',
		'meta_value' => '1',
		'meta_compare' => 'LIKE'
	);
?>
<?php query_posts($toplist); ?>
<?php if (have_posts()) : while(have_posts()) : the_post(); ?>
<?php
	$today = date_i18n("YmdHis");
	$date_start = get_field('acf-pickup')['acf-pickup-open'];
	$date_end = get_field('acf-pickup')['acf-pickup-close'];
?>
<?php if(strtotime($today) >= strtotime($date_start) && strtotime($today) <= strtotime($date_end)) : ?>

///ここに繰り返しの内容

<?php else : endif; ?>
<?php endwhile; endif; wp_reset_query(); ?>

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

Answer 1

query_postsは非推奨となっていますのでWP_Queryなどを使用するのをお勧めします。

Comments

[klc_cre]

早々のコメントありがとうございます
query_postsは非推奨とは知りませんでした
下記のように修正し無事に表示することができました
ありがとうございます

<?php
$args = array(
'post_type' => 'post',
'posts_per_page' => 10,
'meta_key' => 'acf-pickup_acf-pickup-on',
'meta_value' => '1',
'meta_compare' => 'LIKE'
);
$wp_query = new WP_Query($args);
if (have_posts()): while (have_posts()): the_post();
?>
<?php
$today = date_i18n("YmdHis");
$date_start = get_field('acf-pickup')['acf-pickup-open'];
$date_end = get_field('acf-pickup')['acf-pickup-close'];
?>
<?php if(strtotime($today) >= strtotime($date_start) && strtotime($today) <= strtotime($date_end)) : ?>

///ここに繰り返しの内容

<?php else : endif; ?>
<?php endwhile; endif; wp_reset_postdata(); ?>