From 3673baa402380be5da64e4898b073e62cf5b872f Mon Sep 17 00:00:00 2001
From: Ben Word <ben@benword.com>
Date: Sun, 15 Jan 2012 13:27:30 -0700
Subject: [PATCH] loop code cleanup

---
 loop-page.php   |  4 ++--
 loop-search.php | 33 ---------------------------------
 loop-single.php |  6 +++---
 loop.php        | 26 +++++++++++++-------------
 4 files changed, 18 insertions(+), 51 deletions(-)
 delete mode 100644 loop-search.php

diff --git a/loop-page.php b/loop-page.php
index 51d76b6..7400eea 100644
--- a/loop-page.php
+++ b/loop-page.php
@@ -4,7 +4,7 @@
     <?php roots_post_inside_before(); ?>
       <h1><?php the_title(); ?></h1>
       <?php the_content(); ?>
-      <?php wp_link_pages(array('before' => '<nav id="page-nav"><p>' . __('Pages:', 'roots'), 'after' => '</p></nav>' )); ?>
+      <?php wp_link_pages(array('before' => '<nav id="page-nav"><p>' . __('Pages:', 'roots'), 'after' => '</p></nav>')); ?>
     <?php roots_post_inside_after(); ?>
   <?php roots_post_after(); ?>
-<?php endwhile; // End the loop ?>
+<?php endwhile; /* End loop */ ?>
diff --git a/loop-search.php b/loop-search.php
deleted file mode 100644
index 0c26b64..0000000
--- a/loop-search.php
+++ /dev/null
@@ -1,33 +0,0 @@
-<?php /* If there are no posts to display, such as an empty archive page */ ?>
-<?php if (!have_posts()) : ?>
-  <p><?php _e('Sorry, no results were found.', 'roots'); ?></p>
-  <?php get_search_form(); ?>
-<?php endif; ?>
-
-<?php /* Start loop */ ?>
-<?php while (have_posts()) : the_post(); ?>
-  <?php roots_post_before(); ?>
-    <article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
-    <?php roots_post_inside_before(); ?>
-      <header>
-        <h2><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a></h2>
-        <?php roots_entry_meta(); ?>
-      </header>
-      <div class="entry-content">
-        <?php the_excerpt(); ?>
-      </div>
-      <footer>
-        <?php $tag = get_the_tags(); if (!$tag) { } else { ?><p><?php the_tags(); ?></p><?php } ?>
-      </footer>
-    <?php roots_post_inside_after(); ?>
-    </article>
-  <?php roots_post_after(); ?>
-<?php endwhile; // End the loop ?>
-
-<?php /* Display navigation to next/previous pages when applicable */ ?>
-<?php if ($wp_query->max_num_pages > 1) : ?>
-  <nav id="post-nav">
-    <div class="post-previous"><?php next_posts_link( __( '&larr; Older posts', 'roots' ) ); ?></div>
-    <div class="post-next"><?php previous_posts_link( __( 'Newer posts &rarr;', 'roots' ) ); ?></div>
-  </nav>
-<?php endif; ?>
diff --git a/loop-single.php b/loop-single.php
index b96b02e..1c527e5 100644
--- a/loop-single.php
+++ b/loop-single.php
@@ -11,11 +11,11 @@
         <?php the_content(); ?>
       </div>
       <footer>
-        <?php wp_link_pages(array('before' => '<nav id="page-nav"><p>' . __('Pages:', 'roots'), 'after' => '</p></nav>' )); ?>
-        <p><?php the_tags(); ?></p>
+        <?php wp_link_pages(array('before' => '<nav id="page-nav"><p>' . __('Pages:', 'roots'), 'after' => '</p></nav>')); ?>
+       <?php $tags = get_the_tags(); if ($tags) { ?><p><?php the_tags(); ?></p><?php } ?>
       </footer>
       <?php comments_template(); ?>
       <?php roots_post_inside_after(); ?>
     </article>
   <?php roots_post_after(); ?>
-<?php endwhile; // End the loop ?>
+<?php endwhile; /* End loop */ ?>
diff --git a/loop.php b/loop.php
index 1466916..422dd82 100644
--- a/loop.php
+++ b/loop.php
@@ -1,8 +1,8 @@
 <?php /* If there are no posts to display, such as an empty archive page */ ?>
-<?php if (!have_posts()) : ?>
+<?php if (!have_posts()) { ?>
   <p><?php _e('Sorry, no results were found.', 'roots'); ?></p>
   <?php get_search_form(); ?>
-<?php endif; ?>
+<?php } ?>
 
 <?php /* Start loop */ ?>
 <?php while (have_posts()) : the_post(); ?>
@@ -14,24 +14,24 @@
         <?php roots_entry_meta(); ?>
       </header>
       <div class="entry-content">
-    <?php if (is_archive() || is_search()) : // Only display excerpts for archives and search ?>
-      <?php the_excerpt(); ?>
-    <?php else : ?>
-      <?php the_content(); ?>
-    <?php endif; ?>
+        <?php if (is_archive() || is_search()) { ?>
+          <?php the_excerpt(); ?>
+        <?php } else { ?>
+          <?php the_content(); ?>
+        <?php } ?>
       </div>
       <footer>
-        <?php $tag = get_the_tags(); if (!$tag) { } else { ?><p><?php the_tags(); ?></p><?php } ?>
+        <?php $tags = get_the_tags(); if ($tags) { ?><p><?php the_tags(); ?></p><?php } ?>
       </footer>
     <?php roots_post_inside_after(); ?>
     </article>
   <?php roots_post_after(); ?>
-<?php endwhile; // End the loop ?>
+<?php endwhile; /* End loop */ ?>
 
 <?php /* Display navigation to next/previous pages when applicable */ ?>
-<?php if ($wp_query->max_num_pages > 1) : ?>
+<?php if ($wp_query->max_num_pages > 1) { ?>
   <nav id="post-nav">
-    <div class="post-previous"><?php next_posts_link( __( '&larr; Older posts', 'roots' ) ); ?></div>
-    <div class="post-next"><?php previous_posts_link( __( 'Newer posts &rarr;', 'roots' ) ); ?></div>
+    <div class="post-previous"><?php next_posts_link(__('&larr; Older posts', 'roots')); ?></div>
+    <div class="post-next"><?php previous_posts_link(__('Newer posts &rarr;', 'roots')); ?></div>
   </nav>
-<?php endif; ?>
+<?php } ?>