Ah, yeah. comments_popup_link() is a bit of a unique function; what sort of filters do you have in mind through which to run its output?<div><br></div><div>Problems with dynamic sidebars inside of the_content()?</div><div>
<br></div><div>1) Sidebar management via Dashboard -> Appearance -> Widgets would quickly become unwieldy and untenable</div><div>2) Arbitrary code execution inside of the_content(), based on the type of Widgets used</div>
<div>3) Overkill. If you *really* wanted to output Widgets inside of the_content(), why not just write a custom shortcode for <a href="http://codex.wordpress.org/Function_Reference/the_widget">the_widget()</a>?</div><div>
<br></div><div>Chip</div><div><br><div class="gmail_quote">On Wed, Jun 29, 2011 at 8:55 AM, Sayontan Sinha <span dir="ltr"><<a href="mailto:sayontan@gmail.com">sayontan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I think some of the comment link functions such as comments_popup_link and comments_number directly print output. <br><br>What kind of potential problems do you see with dynamic sidebars inside the_content? It is not something a theme includes by default, but it is something explicitly invoked by a post author to put in specific widgets in the content.<div>
<div></div><div class="h5"><br>
<br><div class="gmail_quote">On Wed, Jun 29, 2011 at 6:41 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can you give an example of such functions? Almost every core WordPress function either has a get_*() analog, or else incorporates apply_filters().<div><br></div><div>A dynamic sidebar *inside* of the_content()? That sounds somewhat fraught with potential problems...</div>
<div><br></div><div><font color="#888888">Chip</font><div><div></div><div><br><br><div class="gmail_quote">On Wed, Jun 29, 2011 at 8:36 AM, Sayontan Sinha <span dir="ltr"><<a href="mailto:sayontan@gmail.com" target="_blank">sayontan@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There are some specific uses I can think of:<br><ol><li>Some functions have no "echo" parameter and always print the output. If you wanted to run a filter over the output of such a function, this seems like an easy way to do so.</li>
<li>Ad hoc insertions of widgets/widget areas. Let's say you have defined a sidebar that you want to be able to insert within your posts. You could do this by defining a shortcode, which prints out dynamic_sidebar in an output buffer.<br>
</li></ol><div><div></div><div><br><div class="gmail_quote">On Wed, Jun 29, 2011 at 5:18 AM, Ryan Hellyer <span dir="ltr"><<a href="mailto:ryan@pixopoint.com" target="_blank">ryan@pixopoint.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There are many reasons to use output buffering (ob_start etc), but<br>
they would rarely be used within a theme. I've seen themes which use<br>
buffering as a way to dynamically minify the markup, but that's a<br>
horrid way to do it - much better incorporated within a caching plugin<br>
IMO.<br>
<br>
There may be a practical use for output buffer in a theme, but I'm<br>
darned if I can think of a good example. I wouldn't want to see it<br>
banned outright, as someone may come up with a cunning use for it that<br>
none of us have thought of.<br>
<br>
Perhaps the theme check plugin could check for output buffering<br>
functions and leave a request for the theme developer to provide a<br>
reason for using it? That way they'll know it's non-standard and<br>
provide a reason (good or bad) for the theme reviewers to read. Could<br>
potentially save a little to and fro'ing in the future. Just an idea.<br>
_______________________________________________<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>
</blockquote></div><br><br clear="all"><br></div></div>-- <br>Sayontan Sinha<br><a href="http://mynethome.net" target="_blank">http://mynethome.net</a> | <a href="http://mynethome.net/blog" target="_blank">http://mynethome.net/blog</a><br>
<font color="#888888">
--<br>Beating Australia in Cricket is like killing a celebrity. The death gets more coverage than the crime.<br><br>
</font><br>_______________________________________________<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>
<br></blockquote></div><br></div></div></div>
<br>_______________________________________________<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>
<br></blockquote></div><br><br clear="all"><br>-- <br>Sayontan Sinha<br><a href="http://mynethome.net" target="_blank">http://mynethome.net</a> | <a href="http://mynethome.net/blog" target="_blank">http://mynethome.net/blog</a><br>
--<br>Beating Australia in Cricket is like killing a celebrity. The death gets more coverage than the crime.<br><br>
</div></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>