WordPress.org

1. exit6
   Member
   Posted 4 months ago #

   I know this topic has been addressed elsewhere on the forums, but I can't seem to get it to work. I'm new to php so the syntax is tricky to me. Basically I have a category page where posts overlap categories, and I want to list them in order of importance. That works fine, the problem is I get duplicating posts. Here's my code:
   

   <?php query_posts('category_name=featured');
    if ( have_posts() ) : while ( have_posts() ) : the_post();
    $do_not_duplicate = $post->ID;
    $custom_field_date = get_post_meta($post->ID, 'project-date', true);?>
   
   		<!--do stuff-->
   
   <?php endwhile;
   wp_reset_query();
   ?>		
   
   <?php query_posts('category_name=winners');
    while ( have_posts() ) : the_post();
    $do_not_duplicate = $post->ID;
    $custom_field_date = get_post_meta($post->ID, 'project-date', true);?>
   
   		<!--do stuff-->
   
   <?php endwhile;
   wp_reset_query();
   ?>
   
   <?php query_posts('category_name=portfolio&showposts=50');
    while ( have_posts() ) : the_post();
    $do_not_duplicate = $post->ID;
    $custom_field_date = get_post_meta($post->ID, 'project-date', true);?>
   
   	<!--do stuff-->
   
   <?php endwhile; else: ?>
   
   	<p>Sorry, no posts matched your criteria.</p>
   
   <?php endif; ?>

   What am I missing? Thanks --

2. esmi
   Member
   Posted 4 months ago #

   How/when are you checking against $do_not_duplicate? And how are you carrying the "already displayed posts" data over into the next query?

3. exit6
   Member
   Posted 4 months ago #

   I guess I'm not, i mean you're looking at all the code. Like I said, I'm new to php/wordpress. I'm used to Actionscript, and stuff like "$post->ID;" totally baffles me.

4. esmi
   Member
   Posted 4 months ago #

   $post->ID is a just a way of referencing the id of the current post.

   The <!--do stuff--> is where you need to add your code for displaying the posts. The code inside The Loop in your index.php file should give you some idea of what to use here.

5. exit6
   Member
   Posted 4 months ago #

   Right, here's what's in the <!--do stuff--> area, I left it out because I didn't think it would help solve the problem, since that's just about how the given post is displayed. I'm trying to weed out certain posts before they get displayed. Here's the code:
   

   <a href="<?php the_permalink(); ?>" class="hp-entry-link">
   <div class="category-entry">
       <p><img class="alignnone" title="<?php the_title(); ?>" src="<?php echo $custom_field_image; ?>" alt="<?php the_title();?>"></p>
       <div class="entry-copy">
           <h4><?php the_title(); ?></h4>
           <p class="client"><?php echo $custom_field_client; ?></p>
           <p class="date"><?php echo $custom_field_date; ?></p>
           <p class="deliverable"><?php echo $custom_field_deliverable; ?></p>
           <p class="disciplines"><?php echo $custom_field_disciplines; ?></p>
       </div><!--end entry-copy-->
   </div></a><!--end category-entry-->

   That works just like I want (there are some custom fields in there that I didn't pu in the original post). The problem is duplicate posts sneaking through.

6. esmi
   Member
   Posted 4 months ago #

   There's still no check against $do_not_duplicate, though.

   How about this? Just before <?php query_posts('category_name=featured');, add <?php $displayed = array();?>. That will set up an array to store the ids of any posts that have been displayed. Then instead of setting $do_not_duplicate', you check to see if the post ID is already in the $displayed array.

   if(in_array($post->ID,$displayed)) continue; // already displayed this post so go to top of current loop
   else $displayed[] = $post->ID; // about to display - so add it to the array

   Not tested this, so apologies in advance if there are any syntax errors.

Reply

You must log in to post.