Feel free to add a comment on the ticket after I finish my review. I'll leave you a place-holder. You can explain it better than I can. :)<div><br></div><div>Chip<br><br><div class="gmail_quote">On Mon, Aug 30, 2010 at 9:59 AM, Philip M. Hofer (Frumph) <span dir="ltr"><<a href="mailto:philip@frumph.net">philip@frumph.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div bgcolor="#ffffff">
<div><font size="2" face="Arial">Nevermind I see it, header.php has</font></div>
<div><pre>//The Query
query_posts('posts_per_page=10');</pre><pre><font size="2" face="Arial">Which basically overrides any common loop that occures then does a wp_reset_query(), he should be protecting that loops $wp_query and restarting the old wp_query after that loop of links happens.</font></pre>
<pre><font size="2" face="Arial">$temp_wp_query = $wp_query;</font></pre><pre><code></pre><pre>$wp_query = temp_wp_query;</pre><pre>sort of thing.</pre><pre>*or* not use query_posts and creates a new WP_Query which is the better way of handling that.</pre>
</div>
<blockquote style="border-left:#000000 2px solid;padding-left:5px;padding-right:0px;margin-left:5px;margin-right:0px"><div class="im">
<div style="font:10pt arial">----- Original Message ----- </div>
<div style="font:10pt arial;background:#e4e4e4"><b>From:</b>
<a title="chip@chipbennett.net" href="mailto:chip@chipbennett.net" target="_blank">Chip
Bennett</a> </div>
<div style="font:10pt arial"><b>To:</b> <a title="theme-reviewers@lists.wordpress.org" href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
</div>
<div style="font:10pt arial"><b>Sent:</b> Monday, August 30, 2010 7:50
AM</div>
<div style="font:10pt arial"><b>Subject:</b> [theme-reviewers] Mini-Site
Theme query_posts use</div>
<div><br></div></div><div><div></div><div class="h5">Can someone take a look at the <a href="https://themes.trac.wordpress.org/ticket/862" target="_blank">MiniSite Theme</a> for me?
It's making a ton of query_posts() calls. I'm wondering if they're really
necessary?
<div><br></div>
<div>Chip</div>
</div></div><div class="im"><p>
</p><hr>
<p></p>_______________________________________________<br>theme-reviewers
mailing list<br><a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<p></p></div></blockquote></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>