Wordpress query_posts adding two “post-entry” divs, one with “post-thumbnail” and one without

Question

Hi my name is Antony and this is my first question. I'm currently making a worpress theme. And in the loop I want to have two "post-entry" divs one that's small that goes next to my post thumb, and one for without the thumb.

I know that I need to do a

<!-- Start the Loop. -->
<?php query_posts();
if (have_posts()) : while (have_posts()) : the_post(); ?>

<!--Begin Post-->
<div class="post">
<?php if ( has_post_thumbnail()) { ?>

<div class="post-thumb">
    <?php the_post_thumbnail(); ?>
</div>
<div class="post-entry-1">

<? } else { ?>

<!--Begin Post Entry-->
<div class="post-entry-2">
<?php } ?>
<h2><a href="<?php the_permalink() ?>" rel="bookmark" title="Permanent Link to <?php the_title_attribute(); ?>"><?php the_title(); ?></a></h2>
<div class="post-meta"><?php the_tags('Tags: ', ', ', '<br />'); ?> Posted <?php the_time('F jS, Y') ?> <?php _e('in'); ?> <?php the_category(', ') ?> <?php _e('by'); ?> <?php the_author(); ?> <?php edit_post_link('Edit', ' &#124; ', ''); ?>  <?php comments_popup_link('No Comments &#187;', '1 Comment &#187;', '% Comments &#187;'); ?></div>
<?php the_excerpt(); ?>
</div>
<!--End Post Entry-->

</div>

</div>
<!--End Post-->

<div class="clear"></div>
<?php endwhile; ?>

<?php if (function_exists('wp_pagenavi')) { ?><?php wp_pagenavi(); ?>
<?php } else { ?>
<div class="left"><?php next_posts_link('&laquo; Older Entries') ?></div>
<div class="right"><?php previous_posts_link('Newer Entries &raquo;') ?></div>
<?php } ?>

<?php else : ?>
<h3>Not Found</h3>
<p>Sorry, but nothing matched your search criteria. Please try again with some different keywords.</p>

<?php endif; wp_reset_query(); ?>
<!-- REALLY stop The Loop. -->

Any help will be greatly appreciated, as I have been stuck on this for a few days now.

Answer 1

Something like this?..

<?php if ( have_posts() ) : while ( have_posts() ) : the_post(); ?>

    <div id="post-<?php the_ID(); ?>">

        <?php if ( has_post_thumbnail()) { ?>

          <div class="post-thumb">
            <?php the_post_thumbnail(); ?>
          </div>
          <div class="post-entry-1">
              <?php the_excerpt(); ?>
          </div>

        <? } else { ?>

          <div class="post-entry-2">
              <?php the_excerpt(); ?>
          </div>

        <?php } ?>

    </div>

    <?php endwhile; ?>

<?php else: ?>

    <p class="no-data">
        <?php _e( 'Apologies, but no results were found.' ); ?>
    </p>

<?php endif; ?>

Answer 2

One problem you may be having is that you have included an extra </div>

<!--Begin Post Entry-->
<div class="post-entry-2">
//your code
</div>
<!--End Post Entry-->

</div><!--End Post-->

</div><!--this appears to be extra-->

Try fixing that and see what happens. Your loop seems to be fine, just that posts are being displayed outside of your wrapper.

The error you are receiving is pointing to the fact that you haven't supplied any arguments to query_posts. Why are you using query_posts if you aren't supplying any arguments. This function is only used when you intend to modify the default WP query.