Needs work
Project:
Disqus
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
28 Sep 2009 at 22:31 UTC
Updated:
13 Oct 2021 at 21:18 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
interestingaftermath commentedI have 1/2 way answered my question.
That will display the comments and reactions. Now any suggestions on how to hide that unless there are comments or reactions?
Comment #2
robloachYay! You can probably change how 0 comments show up in the Disqus administration section. Out of scope for this module, unfortunately. Would have to be on the Disqus end.
Comment #4
basvredelingYou can use:
print theme('disqus_comments_num', $node->disqus['domain'], $node->disqus['url']);in Drupal 6Comment #5
ryan.ryan commentedHow about a 7.x solution?
Comment #6
ryan.ryan commentedComment #7
Grayside commentedThese seems like it should be a checkbox on the node settings, or possibly something integrated with managing field display.
Comment #8
Grayside commentedThere's some minimal work to be done here, as the current codebase does not provide the necessary API functionality to easily hack this. I want it more like an on-demand Views' field, which seems to take some more work. Attached patch does a basic refactor of existing code to at least isolate the behavior outside of Views.
Comment #9
Grayside commentedReroll
Comment #10
slashrsm commentedCould we use the same function also in hoo_node_view where we add comment count to teasers?
Comment #11
Grayside commentedI took a quick look at this, and I think you can. I haven't tested this patch at all, but I think it just might work.
Note that the existing behavior in hook_node_view() adds the disqus information as "links" to the existing links structure. This patch replaces it as a singular link added to that structure.
Comment #12
slashrsm commentedIt looks that this brakes teaser link. It simply disappears.
Comment #13
imanoop commentedYou can get the comment_count from node_load($nid);
Comment #14
damienmckennaThis could use some minor refactoring to include the new Panels content_type plugin.