Don't count unpublished posts
chrislb - March 24, 2009 - 07:41
| Project: | Tagadelic |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Description
Tagadelic currently includes unpublished posts which makes clouds appear bigger to users then their actual content is.
In line 209 change
$result = db_query_range('SELECT COUNT(*) AS count, d.tid, d.name, d.vid FROM {term_data} d INNER JOIN {term_node} n ON d.tid = n.tid WHERE d.vid IN ('. substr(str_repeat('%d,', count($vids)), 0, -1) .') GROUP BY d.tid, d.name, d.vid ORDER BY count DESC', $vids, 0, $size);
to
$result = db_query_range('SELECT COUNT(*) AS count, d.tid, d.name, d.vid FROM {term_data} d INNER JOIN {term_node} n ON d.tid = n.tid JOIN {node} o USING (nid) WHERE d.vid IN ('. substr(str_repeat('%d,', count($vids)), 0, -1) .') AND o.status = 1 GROUP BY d.tid, d.name, d.vid ORDER BY count DESC', $vids, 0, $size);
#1
this works for 6.x-1.2 too, the only difference is that it's in line 228
#2
subscribe
#3
I applied this modification successfully to Tagadelic 6.x-1.2 and rolled it into a proper patch, might also work for 5 version.
Frank
#4
I've tested this on a couple of sites, and it is working beautifully! I suspect it should probably be against head or the dev version, but I'm not sure if the maintainers mind either way, so leaving as "needs review".
--Susan
#5
Fixing the accidental RTBC above (see the non-matching comment).
#6
please remove the commented lines from the patch.
#7
work against dev please
#8
Will do. Just one question: Which CVS branch shall I use?
Tagadelic 6.x-1.x-dev is taken from CVS branch DRUPAL-6--1 (http://drupal.org/node/227507) but HEAD seems to be more current.
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/tagadelic/?...
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/tagadelic/?...
tia
Frank
#9
@Frank: if git suits you, you can fire against http://github.com/berkes/tagadelic
Else, you choose: 6.x-1.xdev, or 6.x-1.2 will both work. I will make sure a patch commits. And since I do that in git, merging and fiddling is rather easy :)
So: whatever suits you. My remark above was not for the patch, but for bugtracking. :)
#10
@Bèr
Thanks for the hint. There has been a lot of excitement around git lately but I'm busy enough to understand even one version control system sufficiently so I will stay with CVS for a while ;-)
I rolled the patch again against DRUPAL-6--1 from CVS. And while I was at it, I also ran the module through Coder and amended some minor glitches.
hth
Frank
#11
Subscribing, greetings, Martijn
#12
I've tried the modified SQL query part with 5.x-1.1 and seems to work without problems.