Thread

I want to display recent comments on entries in a single channel on a landing page, but it’s showing all comments (including those from another MSM site)!

Very simple template:

{exp:comment:entries sort="desc" channel="archive_discography" limit="10"}
    <div>
        {name} on <a href="http://{comment_url_title_auto_path}">{title}</a>, {comment_date format="%F %j, %Y at %h:%i%a"}<br> 
    </div>
{/exp:comment:entries}

Any thoughts on why it’s not working?

buggles,

When you say all comments, do you mean your issue if that the limit= parameter is not working.

To just show a preferred site then use the site=“site_shortname” parameter. Also try using the dynamic=“no” parameter

No I mean that comments from all channels, on all sites are showing despite the channel selector parameter being supplied. dynamic=“no” makes no difference.

Hey buggles—

Be on the lookout for an email from me.

Kevin

Hey buggles—

What version and build are you running? If it’s 2.1.3 build 20110411, it appears this is a known bug.

If you’re using 2.1.4 beta, does the code you’re now using in your template look exactly like this (with site-shortname replaced)?

{exp:comment:entries sort="desc" channel="archive_discography" limit="10" dynamic="no" site="site_shortname"}
    <div>
        {name} on <a href="http://{comment_url_title_auto_path}">{title}</a>, {comment_date format="%F %j, %Y at %h:%i%a"}<br> 
    </div>
{/exp:comment:entries}

And you’ve double-checked to make sure the channel name is correct, yes? Let me know where you are on this.

Kevin

I’m on the 2.1.3 build, so it looks like that defect is the problem (was a little concerned with the comment on the bug saying it was a deprecated channel parameter…). The channel name was copied and pasted from the control panel (I’ve been burned by too many typo’s over the years to risk anything else!).

There is a proposed fix in the defect report, would Ellis labs recommend applying that?

buggles,

There is a proposed fix in the defect report, would Ellis labs recommend applying that?

Please feel free to go ahead and apply it and see if it works for you.

Note though that the code in the report has not been wrapped in code blocks so make sure to replace all curly quotes with single quotes as below

if (count($channel_ids) == 1)
{
  $this->EE->db->where('channel_id', $channel_ids['0']);
}
elseif (count($channel_ids) > 1)
{
  $this->EE->db->where_in('channel_id', $channel_ids);
}

That’s worked nicely.

Thanks for the help… get it nailed in 2.1.4 😉

Thanks for the update, buggles. Glad that worked for now!