Ajax Archivi dinamici non mostra risultati corretti

Question

Ajax Archivi dinamici non mostra risultati corretti

24 giu 2013, 13:22:37

Visualizzazioni: 1.01K

Voti: 1

Sono perplesso sul perché non mostra i risultati corretti: quando seleziono solo il mese e ci sono 100 post sul sito, ne compaiono solo 3, mentre se seleziono sia la categoria che il mese compaiono quelli corretti.

Ecco il template degli archivi

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

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

<div>
<h4>Categoria</h4>

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

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

Ecco il codice per ottenere gli archivi

<?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;'>Seleziona dai filtri sopra.</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="Miniatura post" style="width: 35px;" /></td>
                <td><a href='<?php the_permalink(); ?>'><?php the_title(); ?></a></td>
                <td><?php comments_popup_link(' ', '1 Commento', '% Commenti'); ?></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;'>Nessun risultato trovato.</td></tr>";

        endif; 
    ?>
</table>

E infine il codice 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;'>caricamento...</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
        });
    }
});
});
});

chrismccoy

309

post jquery ajax html

Tutte le risposte alla domanda

Sarà difficile eseguire il debug di qualcosa del genere poiché non ho i tuoi dati sul mio server, ma stai facendo diverse cose che mi sembrano rischiose.

Primo, stai utilizzando query_posts. Non usare query_posts.

È importante notare che utilizzare questa funzione per sostituire la query principale su una pagina può aumentare i tempi di caricamento, negli scenari peggiori raddoppiando o più la quantità di lavoro necessaria. Sebbene sia facile da usare, la funzione è anche soggetta a confusione e problemi in seguito. Vedi la nota più sotto per i dettagli sulle avvertenze.

http://codex.wordpress.org/Function_Reference/query_posts (enfasi mia)

Secondo, stai eseguendo la tua query prima di verificare se vuoi effettivamente eseguire una query. Cioè, prima di questa riga:

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

Perché eseguire una query e poi controllare se vuoi dire all'utente che non ha compilato tutti i requisiti necessari per l'esecuzione della query?

Terzo, questa condizione...

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

... è strana. È false se $cat è vuoto. Presumibilmente, il tuo modulo imposta quel valore a -1, ma questo mi porta a...

Quarto, sembra che tu stia dando totale fiducia ai dati forniti dall'utente. POST non è più sicuro (forse solo l'1% in più) di GET. A meno che non ci sia una validazione da qualche parte non mostrata, un utente malintenzionato potrebbe potenzialmente inserire dati non sicuri attraverso quella query.

Quinto, non stai utilizzando l'API AJAX e questo è esattamente il tipo di cosa per cui è stata creata.

s_ha_dum

65.5K

24 giu 2013 17:59:08

chrismccoy 309 · Accepted Answer · 2013-06-26T23:55:49Z

questo è ciò con cui ho finito e funziona perfettamente.

parte di 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( 'Vai via!' );

$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 Commenti', '1 Commento', '% Commenti'); ?></td>
        <td><?php the_category();?>
            </tr>
    <?php 
        endwhile; else:
            echo "<tr><td style='text-align: center; font-size: 15px; padding: 5px;'>Nessun risultato trovato.</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');

parte jQuery

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' alt='Caricamento in corso' title='Caricamento in corso' /></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;">Seleziona un'opzione dal menu sopra.</td></tr></table>');
            } else {
                $("#archive-content").empty().html(data);
            }
        }
    });
    return false;
});
});