Eigene Auslesung macht Endlosschleife

hoersie85

Moin zusammen,

ich habe unter: https://rikerandom.de/nerdy/how-to/rezensionsindex-part-4/
eine Anleitung gefunden für einen Index mit dem man beispielsweise Rezensionen zu Büchern nach Buchautoren sortiert anzeigen lassen können soll. Dazu habe ich ein Seitentemplate erstellt.

Ich habe die Funktion soweit hinbekommen, dass der Index angezeigt wird und zu den Buchstaben auch mittels Anker gesprungen wird. Nun sollen alle Artikel ausgelesen werden, die als benutzerdefinierte Felder einen Buchautor haben. (Aktuell zu Testzwecken 2 Artikel) Die Auslesung funktioniert. Es werden beide Artikel bei Ihren Buchstaben angezeigt.

Jetzt kommen wir zu dem was nicht funktioniert. Wenn Ihr auf http://blog.hoerske.de/rezensionsindex/
schaut, dann seht Ihr, dass der Loop für jeden Buchstaben des Alphabets eine neue Reihe an Buchstaben auf macht. Also läuft der Loop irgendwie unkontrolliert.

Hier zu nächst mein Code für den Index:

PHP

<?php the_content(); endwhile; ?>
<h3><a href="#a">A</a> 
<a href="#b">B</a> 
<a href="#c">C</a> 
<a href="#d">D</a>
<a href="#e">E</a> 
<a href="#f">F</a>
<a href="#g">G</a>
<a href="#h">H</a>
<a href="#i">I</a> 
<a href="#j">J</a>
<a href="#k">K</a>
<a href="#l">L</a>
<a href="#m">M</a>
<a href="#n">N</a>
<a href="#o">O</a>
<a href="#p">P</a>
<a href="#q">Q</a>
<a href="#r">R</a> 
<a href="#s">S</a> 
<a href="#t">T</a>
<a href="#u">U</a>
<a href="#v">V</a>
<a href="#w">W</a>
<a href="#x">X</a> 
<a href="#y">Y</a> 
<a href="#z">Z</a></h3>
<p></p>

Alles anzeigen

Dann kommt die Auslesung mit dem ersten Buchstaben A:

PHP

<?php $my_query = new WP_Query( array( ‚meta_key‘ => ‚buchautor‘,
‚orderby‘ => ‚meta_value‘,
‚order‘ => ‚ASC‘
) );
while ( $my_query->have_posts() ) : $my_query->the_post();
$autor = get_post_meta($post->ID, buchautor, true);
$this_letter = strtoupper(substr($autor,0,1)); ?>
                
<h3 id="a">A</h3>
<ul><?php if ($this_letter == A) { ?>
<li><?php $key="buchautor"; echo get_post_meta($post->ID, $key, true); ?>:
<?php $key="buchtitel"; echo get_post_meta($post->ID, $key, true); ?> -
Artikel: <a href="<?php echo esc_url( get_permalink() ); ?>">
<?php the_title(); ?></a></li>
<?php } ?></ul>

Alles anzeigen

Jeder weitere Buchstabe sieht so aus:

PHP

<h3 id="b">B</h3>
<ul><?php if ($this_letter == B) { ?>
<li><?php $key="buchautor"; echo get_post_meta($post->ID, $key, true); ?>:
<?php $key="buchtitel"; echo get_post_meta($post->ID, $key, true); ?> - 
Artikel: <a href="<?php echo esc_url( get_permalink() ); ?>">
<?php the_title(); ?></a></li>
<?php } ?></ul>

Der Schluss des Skriptes endet mit dem Buchstaben Z:

PHP

<h3 id="z">Z</h3>
<ul><?php if ($this_letter == Z) { ?>
<li><?php $key="buchautor"; echo get_post_meta($post->ID, $key, true); ?>:
<?php $key="buchtitel"; echo get_post_meta($post->ID, $key, true); ?> - 
Artikel: <a href="<?php echo esc_url( get_permalink() ); ?>">
<?php the_title(); ?></a></li>


<?php } endwhile; ?>
        </div>
        <!-- .entry-content -->

Alles anzeigen

Habt Ihr eine Ahnung wo der Fehler liegen könnte? Ich sehe den Wald vor lauter Bäumen nicht.
Ich danke euch...
Timo

Werbepost

Hallo!

Wenn du gerade an deiner Website arbeitest oder dein aktuelles Hosting überdenkst: Wir betreiben mit NetzLiving eine Hosting-Plattform, die speziell auf Performance, Sicherheit und einfache Verwaltung ausgelegt ist.

✔️ Schnelle Ladezeiten (optimiert für WordPress & Co.)
✔️ Deutsche Server & DSGVO-konform
✔️ Persönlicher Support (kein 0815-Ticket-System)

Mehr erfahren

Azrael_0815

PHP

<?php $my_query = new WP_Query( array( ‚meta_key‘ => ‚buchautor‘,

sollte IMHO sein

PHP

<?php $my_query = new WP_Query(
array(
                        'meta_key' => 'buchautor',
            
                    ),

und

Code

$autor = get_post_meta($post->ID, buchautor, true);

sollte

Code

$autor = get_post_meta($post->ID, 'buchautor', true);

sein. Hier passiert keine Endlosschleife, da Ding bricht wegen Syntax-Error ab :-)

hoersie85

Danke für die Hinweis. Ändere ich dir "Anführungsstriche" bekomme ich eine weiße Seite...

Azrael_0815

Dann hast Du noch mehr Fehler drin oder es falsch eingebaut :-) PHP kennt kein oben/unten bei Anführungsstrichen.

b3317133

Neben den bereits genannten falschen "single quotes" und vorausgesetzt, der restliche Code funktioniert, sollte es pro Buchstabe wohl eher so in der Art aussehen:

PHP

<?php if ($this_letter == 'A') { ?>
<h3 id="a">A</h3>
<ul>
<li><?php $key="buchautor"; echo get_post_meta($post->ID, $key, true); ?>:
<?php $key="buchtitel"; echo get_post_meta($post->ID, $key, true); ?> -
Artikel: <a href="<?php echo esc_url( get_permalink() ); ?>">
<?php the_title(); ?></a></li>
</ul>
<?php } ?>

Wenn der entspr. Buchstabe in der Loop nicht vorkommt, wird dann allerdings auch keine "Ansprungstelle" erstellt.

Weiterhin fehlen bei [FONT=Courier New]get_post_meta($post->ID, [COLOR=#ff0000]'[/COLOR]buchautor[COLOR=#ff0000]'[/COLOR], true);[/FONT] und bei den [FONT=Courier New]if ($this_letter == [COLOR=#ff0000]'[/COLOR]A[COLOR=#ff0000]'[/COLOR])[/FONT] Abfragen die Anführungszeichen.

hoersie85

Ich habe das jetzt mal komplett so übernommen, wie ihr es geschrieben habt:

PHP

<?php $my_query = new WP_Query(
            array (
            'meta_key‘ => 'buchautor‘,
            'orderby‘ => 'meta_value‘,
            'order‘ => 'ASC‘
            ) );
                while ( $my_query->have_posts() ) : $my_query->the_post();
                $autor = get_post_meta($post->ID, 'buchautor', true);
                $this_letter = strtoupper(substr($autor,0,1)); ?>
            
            <?php if ($this_letter == A) { ?>   
            <h3 id="a">A</h3>
            <ul>
            <li><?php $key="buchautor"; echo get_post_meta($post->ID, $key, true); ?>: <?php $key="buchtitel"; echo get_post_meta($post->ID, $key, true); ?> - Artikel: <a href="<?php echo esc_url( get_permalink() ); ?>"><?php the_title(); ?></a></li>
            </ul>
<?php } ?>

Alles anzeigen

Nun bekomme ich eine wieder eine weiße Seite, also gar nichts mehr angezeigt...

mensmaximus

Bis auf die Tatsache, dass Du einige Syntaxfehler hast und viel zu viel redundanten Code in einer viel zu großen Abfrage, macht er genau das was Du programmiert hast. Wenn schon, dann müssten Deine H3 Überschriften, also die Einzelbuchstaben, hinter die IF Abfrage. Die IF Abfrage muss Strings vergleichen, also gehören die Buchstaben in Anführungszeichen. Zudem wiederholt man nicht für jeden Buchstaben die IF Abfrage und Ausgabe. Und einiges ergibt gar keinen Sinn (woher kommt $post, was ist das für ein post_type, wieso steht ein Titel in einer Metainfo usw.).

Lösungsansatz:

Code

$alphabet = range( 'A', 'Z' );
foreach ( $alphabet as $letter ){
   
   $args = array(
       'post_type'  => 'post', // oder wie auch immer der post type der bücher heißt
       'meta_key'   => 'buchautor',
       'orderby'    => 'meta_value',
       'order'      => 'ASC',
       'meta_query' => array(
           array(
               'key'     => 'buchautor',
               'value'   => '^' .  $letter,
               'compare' => 'REGEXP',
           ),
       ),
   );
   
   $my_query = new WP_Query( $args );
   
   if ( $my_query->have_posts() ) {
       while ( $my_query->have_posts() ) {
           the_post();
           
           echo '<h3 id="' .$letter. '">' .$letter. '</h3>'
           echo '<ul>';
           echo '<li>';
           echo // Deine Ausgabe
           echo '</li>';
           echo '</ul>';


       }
       wp_reset_postdata();
   }


}

Alles anzeigen

hoersie85

Danke an Alle für die bisherigen Antworten. Irgendwie bekomme ich es noch nicht so richtig hin.

Zu euren Fragen: Ich habe mich an den oben genannten Link und die Anleitung darin gehalten. Wenn es einen anderen Weg gibt, dies umzusetzen dann gerne.

Der Buchtitel steht in den benutzerdefinierten Feldern, weil ich meine Artikel nicht umschreiben möchte und nicht alle Rezensionen im Titel Autor und Buchtitel in der gleichen Art und Weise enthalten. Daher wollte ich es gerne über die benutzerdefinierten Felder lösen.

Die Anführungsstriche und auch die <h3> hinter den if, haben nur zu einer weißen Seite geführt.

@mensmaximu wie und wo müsste ich deinen Lösungsteil einfügen. Ich habe es jetzt in einigen Varianten probiert, komme aber über eine weiße Seite nicht hinaus.

Hier mal das komplette Seitentemplate entfernt um den nicht funktionierenden Auslesebereich aber noch mit dem Index. Als Vorlage habe ich für das Seitentemplate, dass originale vom Theme Merlin Pro genommen:

PHP

<?php /* Template Name: Rezensionsindex */ ?>
<?php
get_header(); ?>


    <section id="primary" class="content-area">
        <main id="main" class="site-main" role="main">
        
            <?php if ( function_exists( 'themezee_breadcrumbs' ) ) themezee_breadcrumbs(); ?>
            
            <?php while (have_posts()) : the_post(); ?>


            <article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
        
        <header class="entry-header">
            
            <?php the_title( '<h1 class="page-title">', '</h1>' ); ?>
            
        </header><!-- .entry-header -->
<p></p>
        <div class="entry-content clearfix">
            <?php the_content(); endwhile; ?>
            <h3><a href="#a">A</a> <a href="#b">B</a> <a href="#c">C</a> <a href="#d">D</a> <a href="#e">E</a> <a href="#f">F</a> <a href="#g">G</a> <a href="#h">H</a> <a href="#i">I</a> <a href="#j">J</a> <a href="#k">K</a> <a href="#l">L</a> <a href="#m">M</a> <a href="#n">N</a> <a href="#o">O</a> <a href="#p">P</a> <a href="#q">Q</a> <a href="#r">R</a> <a href="#s">S</a> <a href="#t">T</a> <a href="#u">U</a> <a href="#v">V</a> <a href="#w">W</a> <a href="#x">X</a> <a href="#y">Y</a> <a href="#z">Z</a></h3>
        </div>
        <!-- .entry-content -->


    </article>       
        </main><!-- #main -->
    </section><!-- #primary -->
    
    <?php get_sidebar(); ?>


<?php get_footer(); ?>

Alles anzeigen

hoersie85

So für euch zur Rückmeldung und falls wer anderes ebenfalls so eine Herrausforderung hat, nach langem probieren und hin und her:

Ich habe ausgehend von der ursprünglichen Konfiguration ganz oben nur die if Abfrage über die h3 Überschrift gesetzt und darin den Buchstaben in ' ' . Jetzt funktioniert alles. Die restlichen Anführungsstriche habe ich belassen. Sobald ich diese auch auf ' ' geändert hatte, war nur eine weiße Seite zu sehen. Ich habe keine Ahnung warum es so funktioniert... Danke euch allen für die Hilfe...

Willkommen! Melden Sie sich an oder registrieren Sie sich.

Eigene Auslesung macht Endlosschleife

Jetzt mitmachen!