Pagination mit Transient-Cache funktioniert nicht zusammen

Hallo Wordpress-Freunde,

ich habe eine überlange Liste, die ich in mehrere Seiten splitten möchte. Dazu benutze ich folgenden Code:

$items_per_page = 30;
$page = isset( $_GET['cpage'] ) ? abs( (int) $_GET['cpage'] ) : 1;
$offset = ( $page * $items_per_page ) - $items_per_page;   
            
// $results = get_transient('test'.$page);
// if ($results === false) {   


$query = "SELECT ID, post_title, post_name FROM wp_posts WHERE wp_posts.post_type = 'person' AND wp_posts.post_status = 'publish'";
$total_query = "SELECT COUNT(1) FROM (${query}) AS combined_table";
$total = $wpdb->get_var( $total_query );


$results = $wpdb->get_results( $query.' ORDER BY ID DESC LIMIT '. $offset.', '. $items_per_page, OBJECT );
        
//set_transient('test'.$page, $results, 3600 * 24);
//}       


global $post;


foreach( $results as $post )
{
the_title( );
}
    
echo paginate_links( array(
'base' => add_query_arg( 'cpage', '%#%' ),
'format' => '',
'prev_text' => __('«'),
'next_text' => __('»'),
'total' => ceil($total / $items_per_page),
'current' => $page
));

Dieser Code funktioniert auch einwandfrei, die Seitenpaginierung am Ende der Seiten wird korrekt angezeigt.

Wenn ich jedoch die Auskommentierung für den Transient-Cache entferne, um den SQL-Code zu cachen, verschwindet die Seitenpaginierung. Die Transienten werden aber korrekt erstellt, d.h. die Variable $page wird auch wie gewünscht übergeben. Also die Namen lauten dann je nach Seite test1, test2, test3 usw. Das habe ich mit dem Plugin "Transienten-Manager" überprüft.

Woran kann es dann liegen, dass die Seitenpaginierung nicht angezeigt wird?

Vielen Dank für eure Hilfe im voraus :-)

Andreas

Hallo mensmaximus,

sorry, dass ich mich erst jetzt melde, aber ich hatte auf eine Email-Benachrichtigung gewartet, aber wohl die Benachrichtigungsfunktion nicht aktiviert ;-)

Vielen Dank für deine Antwort, du hast (mal wieder) meinen Tag gerettet :-D

Ich habe die Definition von $query, $total_query und $total vor die if-Schleife gesetzt, und jetzt klappt es!
Jetzt erklärt sich auch die Fehlermeldung, die QueryMonitor mir bzgl. der unbekannten Variable $total ausgegeben hat. Ich bin da partout nicht drüber gefallen.

Aber bei der Gelegenheit muss ich dich gleich noch mal wegen der Performance des von mir verwendeten SQL-Befehls fragen, denn den Inhalt von $query hatte ich der besseren Übersicht wegen vereinfacht.

Der originale SELECT-Befehl lautet folgendermaßen:

SELECT ID, post_title, post_name, TIMESTAMPDIFF(YEAR, m1.meta_value, CURDATE()) as alter_jahr, TIMESTAMPDIFF(YEAR, m1.meta_value, m2.meta_value) as alter_jahr2
FROM wp_posts
INNER JOIN wp_postmeta m1
ON ( wp_posts.ID = m1.post_id )
INNER JOIN wp_postmeta m2
ON ( wp_posts.ID = m2.post_id )
WHERE wp_posts.post_type = 'person'
AND wp_posts.post_status = 'publish'
AND m1.meta_key = 'geburtsdatum'
AND m2.meta_key = 'todesdatum'   
HAVING alter_jahr = '$alter' OR alter_jahr2 = '$alter'";

Mit dieser query filtere ich über benutzerdefinierte Felder, welche ja in der wp_postmeta liegen, alle Personen aus der Datenbank, die ein bestimmtes Alter haben. Dabei muss ich darauf Rücksicht nehmen, ob die Person noch lebt oder verstorben ist. Die Variable $alter lese ich aus der URL aus.

Trotz der korrekt gesetzten Transienten wird mir über Query Monitor aber immer noch angezeigt, dass es sich um einen "langsamen Query" handelt. Ich hatte den SELECT-Befehl vorher nicht mit dem Operator "OR" ausgeführt, sondern UNION benutzt, aber da war die Performance noch schlechter.

Leider lässt sich der aufwendige Code aufgrund der Struktur in der wp_meta-Tabelle, wo es ja nur ein Feld meta_key und meta_value gibt, nicht anders bewerkstelligen, oder?

Hast du einen Tipp, wie ich die Abfrage trotzdem schneller gestalten könnte?

Schon mal herzlichen Dank im voraus für deine Meinung!

Andreas

Danke nochmals :-)

Ja, du hast recht, es ist wohl das sinnvollste, bei verstorbenen Personen das Todesdatum gleich mit in die Tabelle zu schreiben, dann braucht man das einfach nur abzurufen.

Aber wo genau müsste ich denn die Variable $total eingefügen, damit kein Error erzeugt wird, wenn es schon einen Transienten gibt?

Und wie könnte man alternativ auf $total verzichten? Ich hatte aus dem paginate_links mal spaßeshalber die Zeile für total rausgeschmissen, dann verschwindet die Pagination aber wieder. Da bräuchte ich noch mal einen Denkanstoß ;-)

Also kurz zum Hintergrund, hatte ich eben vergessen: habe eine Datenbank mit ca. 10.000 Personen-Einträgen und filtere für jedes Alter (z.B. 50) mit o.g. Code die entsprechenden Personen raus, sodass ich eine Liste bekomme mit Personen, die z.B. aktuell 50 Jahre sind oder aber im Alter von 50 Jahren verstorben sind. Das Alter für die Verstorbenen kann ich natürlich fest in der Tabelle speichern, für die Lebenden aber nicht, da das Alter zum aktuellen Zeitpunkt ja erst berechnet wird. So muss ich wenigstens für die lebenden Personen die SQL-Abfrage machen.
Der Transienten-Cache hat schon etwas an Geschwindigkeit gebracht, daher würde ich das eigentlich gern verwenden. Es sei denn, es gibt eine andere Möglichkeit der Performance-Verbesserung...

Aaaaaa, alles klar. Aber wie lange braucht denn so ein Cronjob für ca. 10.000 Einträge über Nacht, funktioniert das?

Wäre es nicht noch eine Steigerung der Nutzerfreundlichkeit, wenn ich das Alter direkt als Taxonomy Term abspeichere? So könnte ich die Abfrage noch einfacher und vor allem mit besserer Performance gestalten.
Meinst du, es würde gehen, dass man so einen Taxonomy Term per Cronjob aktualisieren kann?

Perfekt, so mache ich das :-D

Vielen Dank für die tolle Hilfe und ein schönes Wochenende!