web-dev-qa-db-ja.com

Ajax Dynamic Archivesが正しい結果を表示しない

1カ月だけを選択して100件の投稿がサイトに表示されているとすると3件しか表示されず、カテゴリと月を選択すると適切な結果が表示される。

これがアーカイブのテンプレートです

<div id="archive-browser">
<div>
<h4>Month</h4>

<select id="month-choice">
<option val="no-choice"> &mdash; </option>
<?php wp_get_archives(array('type'    => 'monthly', 'format'  => 'option')); ?>
</select>
</div>

<div>
<h4>Category</h4>

<?php wp_dropdown_categories('show_option_none= -- ');?> 
</div>
</div>

<div id="archive-wrapper">
<div id="archive-pot"></div>
</div>

これがアーカイブのゲッターです

<?php

/*
    Template Name: Archives Getter
*/

$year = trim($_POST['digwp_y']);
$month = trim($_POST['digwp_m']);
$cat = trim($_POST['digwp_c']);
$querystring = "year=$year&monthnum=$month&cat=$cat&posts_per_page=-1";
query_posts($querystring); 

?>

<?php if (($year == '') && ($month == '') && ($cat == '-1')) { ?>

<table id="archives-table"><tr><td style='text-align: center; font-size: 15px; padding: 5px;'>Please choose from above.</td></tr></table>

<?php } else { ?>

<table id="archives-table">
    <?php    
        if (have_posts()) : while (have_posts()) : the_post(); ?>
            <tr>
                <td><img src="<?php echo get_post_meta($post->ID, 'PostThumb', true); ?>" alt="" style="width: 35px;" /></td>
                <td><a href='<?php the_permalink(); ?>'><?php the_title(); ?></a></td>
                <td><?php comments_popup_link(' ', '1 Comment', '% Comments'); ?></td>
                <td><?php the_date('m/j/Y'); ?></td>
            </tr>
    <?php 
        endwhile; else:

            echo "<tr><td style='text-align: center; font-size: 15px; padding: 5px;'>Nothing found.</td></tr>";

        endif; 
    ?>
</table>

そして最後にjqueryコード

jQuery(function($) {
$("#archive-wrapper").height($("#archive-pot").height());
$("#archive-browser select").change(function() {
$("#archive-pot")
    .empty()
    .html("<div style='text-align: center; padding: 30px;'>loading...</div>");

var dateArray = $("#month-choice").val().split("/");
var y = dateArray[3];
var m = dateArray[4];
var c = $("#cat").val();

$.ajax({
    url: "/archive-getter/",
    dataType: "html",
    type: "POST",
    data: ({
        "digwp_y": y,
        "digwp_m" : m,
        "digwp_c" : c
    }),
    success: function(data) {
        $("#archive-pot").html(data);
        $("#archive-wrapper").animate({
            height: $("#archives-table tr").length * 50
        });
    }
});
});
});
postsjqueryajaxhtml
1
chrismccoy

これは私が最後にうまくいったものです。

functions.phpパート

<?php

function scripts_enqueue() {
if(is_page('archives')) {
    wp_enqueue_script('ajax_dropdown', get_stylesheet_directory_uri() . '/js/loadposts.js',array('jquery'));
    wp_localize_script( 'ajax_dropdown', 'myajax', array('custom_nonce' => wp_create_nonce('nonce-ajax-dropdown'), 'ajaxurl' => admin_url( 'admin-ajax.php' ) ) );
}
}

add_action( 'wp_enqueue_scripts', 'scripts_enqueue' );

function wp_ajax_load_posts(){

if(!wp_verify_nonce( $_GET['_wpnonce'], 'nonce-ajax-dropdown'))
    die( 'Go away!' );

$args = array(
    'year' => trim($_GET['year']),
    'monthnum' => trim($_GET['month']),
    'posts_per_page' => -1,
    'orderby' => 'date',
    'cat' => trim($_GET['cat'] != "-1") ? trim($_GET['cat']) : 0,
);

$ajaxsort = new WP_Query($args);
?>
<table id="archives-table">
    <?php if ($ajaxsort->have_posts()) : while ($ajaxsort->have_posts()) : $ajaxsort->the_post();?>
            <tr>
                <td><a href='<?php the_permalink(); ?>'><?php the_title(); ?></a></td>
                <td><?php the_time('m/j/Y'); ?></td>
        <td><?php comments_popup_link('0 Comments', '1 Comment', '% Comments'); ?></td>
        <td><?php the_category();?>
            </tr>
    <?php 
        endwhile; else:
            echo "<tr><td style='text-align: center; font-size: 15px; padding: 5px;'>Nothing found.</td></tr>";
        endif; 
    ?>
</table>
<?php
    exit;
}

add_action('wp_ajax_load_posts', 'wp_ajax_load_posts');
add_action('wp_ajax_nopriv_load_posts', 'wp_ajax_load_posts');

jquery part

jQuery(document).ready(function($) {
$("#archive-browser select").change(function() {
    $(".message").hide();
    $("#archive-content").empty().html("<div style='text-align: center; padding: 30px;'><img src='http://i.imgur.com/TA3o5.gif' /></div>");
    var date = $('#month-choice option:selected').val();
    var dateArray = $("#month-choice").val().split("/");
    var year = dateArray[3];
    var month = dateArray[4];
    var cat = $('#cat').val();
    $.ajax({
        url: myajax.ajaxurl,
        type: 'GET',
        data: {
            action: 'load_posts',
            _wpnonce: myajax.custom_nonce,
            cat: cat,
            month: month,
            year: year,
        },
        success: function(data) {
            if (date == 'no-choice' && cat == "-1") {
                $("#archive-content").empty().html('<table class="message" id="archives-table"><tr><td style="text-align: center; font-size: 15px; padding: 5px;">Please choose from above.</td></tr></table>');
            } else {
                $("#archive-content").empty().html(data);
            }
        }
    });
    return false;
});
});
1
chrismccoy

私のサーバーにはあなたのデータがありませんが、あなたが私を危険にさらすいくつかのことをしているので、このようなものをデバッグするのは難しいでしょう。

First、あなたはquery_postsを使用しています。 query_postsを使用しないでください。

ページ上でメインクエリを置き換える _にこれを使用すると、ページ読み込み時間を増やす _、最悪の場合、シナリオ{必要な作業量が2倍以上になる）以上になる。使いやすいとはいえ、この関数はまた後から混乱や問題を起こしやすいになります。詳細については、下記の注意事項を参照してください。

http://codex.wordpress.org/Function_Reference/query_posts （私の強調点）

Second、クエリを実行したいかどうかを確認する前に、クエリを実行しています。つまり、この行の前に：

<?php if (($year == '') && ($month == '') && ($cat == '-1')) { ?>

なぜクエリを実行してから、クエリを実行するための要件をすべて満たしていないかをユーザーに伝えたいかどうかを確認するのはなぜですか。

Third、これ...

if (($year == '') && ($month == '') && ($cat == '-1')) {

...は奇妙な状態です。 $catが空の場合はfalseです。おそらく、あなたのフォームはその値を-1に設定します。

第4、あなたはユーザー提供のデータを100％信頼しているようです。 POSTはGETより安全ではありません（おそらく1％以上）。どこかに投稿されていない検証がない限り、悪意のあるユーザーはそのクエリを通して危険なデータをプッシュする可能性があります。

Fifth、あなたは AJAX API を使用していません、そしてこれはまさにそれが意図していることの一種です。

0
s_ha_dum