I suggest adding this to "You can. . ." in help > statistics:

  • Enable users to see how many times a node has been accessed. administer > access control
    (Under "statistics module," tick "view post access counter" for each role desired.)

This option to show or hide the count has moved from where it was in 4.6, so we need to help administrators find it.

CommentFileSizeAuthor
#8 58238_statistics.module_r1.241_patch_11.8 KBAjK
#7 58238_statistics.module_r1.225.2.1_patch_21.73 KBAjK
#4 58238_statistics.module_r1.225.2.1_patch1.73 KBAjK

Comments

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Version: 4.7.0-rc1 » 4.7.0
Category: task » bug

Torn whether to mark as feature for CVS, or a bug for 4.7.... it could be seen as a documentation bug, I guess.

Suggestions for this text, anyone? I will roll a patch.

Anonymous’s picture

Anonymous (not verified) commented

>Suggestions for this text, anyone? I will roll a patch.

I suggest the text from my original message:

  • Enable users to see how many times a node has been accessed. administer > access control
    (Under "statistics module," tick "view post access counter" for each role desired.)

Best wishes.

flevour’s picture

flevour commented

+1 for adding the proposed text. Stuff hidden under "access content" is hard to spot after module installation. A pointer from the help documentation should help administrators find their way.

AjK’s picture

AjK commented
Status: Active » Needs review
StatusFileSize
new 58238_statistics.module_r1.225.2.1_patch1.73 KB

So here's a patch for this for review

flevour’s picture

flevour commented
Status: Needs review » Reviewed & tested by the community

Once it has been agreed to add the text, the patch is ready to be committed.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

In documentation, use 'post' not 'node'.

AjK’s picture

AjK commented
StatusFileSize
new 58238_statistics.module_r1.225.2.1_patch_21.73 KB

Corrected version, use "post" rather than "node"

AjK’s picture

AjK commented
StatusFileSize
new 58238_statistics.module_r1.241_patch_11.8 KB

The above patch is for DRUPAL-4-7 as per this issue's quoted "Version".

Attached to this comment is a patch for HEAD

AjK’s picture

AjK commented
Status: Needs work » Needs review
Paul Natsuo Kishimoto’s picture

Paul Natsuo Kishimoto commented

@o: should we send this to 6.x-dev as well?

killes@www.drop.org’s picture

killes@www.drop.org commented
Version: 4.7.0 » 6.x-dev

I don't want to break translations in 4.7.x and 5.x has a stringfreeze too.

bdragon’s picture

bdragon commented

Autopatch Results for 58238_statistics.module_r1.241_patch_1:
patching file modules/statistics/statistics.module
Hunk #1 FAILED at 37.
1 out of 1 hunk FAILED -- saving rejects to file modules/statistics/statistics.module.rej

bdragon’s picture

bdragon commented
Status: Needs review » Needs work
keith.smith’s picture

keith.smith commented
Component: statistics.module » documentation

Moving to documentation component.

keith.smith’s picture

keith.smith commented
Status: Needs work » Closed (duplicate)

This is the earlier issue, but a current patch is at http://drupal.org/node/175127. Does anyone object if I mark this duplicate and continue there? I believe that all the points raised here have been addressed in there.