Closed (fixed)
Project:
Drupal core
Version:
4.7.0-beta1
Component:
statistics.module
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
13 Dec 2005 at 16:49 UTC
Updated:
24 Jan 2006 at 21:47 UTC
under each node, we render a number of views or that node. IF a user does not have "access statistics" we should not show this.
Patch fixes that. its a small one-liner.
| Comment | File | Size | Author |
|---|---|---|---|
| stqts_lost_perm_check.diff | 954 bytes | Bèr Kessels |
Comments
Comment #1
rkerr commentedSeems to make sense... :)
Comment #2
jvandyk commentedYes, this is good. Current behavior is a bug and this patch fixes it.
But I'd rather see a new permission 'access view counts' or some such so that I can turn on view counts without having to give full statistics access to that role.
Comment #3
Bèr Kessels commentedThinking about it a bit more.
* any $link[] can now be filtered. thus this patch is not at all needed to get the same result: just a theme function that strips out the count 'link' should do.
* any themer can add $count data . so *not* showing this at all by default makes sense too.
I guess its al about what we want as default behaviour.
My vote goes to:
Apply this patch. Then anyone who wants to show anyone the node-view count can do so in a theme function.
But Im perfectly fine if people decide to:
Won't fix this patch and let me+others, who do not want to show this to users, 'hack' it out in a rather simple theme function.
But I am not for a separate permission for this. its not really a permission. And if we go for an option/permission, it should live in the "admin > settings > content-types" and be a per-node option, imnsho.
Comment #4
Bèr Kessels commentedforgot: that last option might be the best. but i'm not coding it :p
Though imho it should just be as simple as a nodapi thing that adds a $node->viewcount. or even $node->viewcount->daily $node->viewcount->alltime. That would be soooo usefull :)
Comment #5
Bèr Kessels commentedwe now have a permission "view post access counter". this fixes this issue.
Comment #6
raema commented