Closed (won't fix)
Project:
Drupal core
Version:
7.x-dev
Component:
statistics.module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
2 Apr 2005 at 17:48 UTC
Updated:
25 May 2009 at 18:36 UTC
Jump to comment: Most recent file
Comments
Comment #1
bdragon commentedadmin/reports/pages is still missing help text. Couldn't locate the other one.
Comment #2
dave reidAdding novice tag. Anyone want to fix this? :)
Comment #3
reaneyk commentedI'll start working on it. The link in the description seems to be broken though...
Comment #4
dave reidYeah it is. :/ Should be fairly easy to take a look at statistics_menu() and compare to the current pages covered by statistics_help() and investigate which pages should be added. Thanks for taking this up!
Comment #5
reaneyk commentedOK I've added documentation for each of the items in statistics_menu(). I wasn't sure how elaborate they needed to be but I'd be happy to update them if need be.
KC
Comment #7
dave reidreaneyk: Ok nice work so far. Could you please run diff from the root of your Drupal directory? Otherwise the testing bot won't be able to apply it for testing.
Comment #8
reaneyk commentedDave Reid: Thanks for pointing me in the right direction. Here's an updated patch.
KC
Comment #9
spuky commentedfor the
admin/reports/pages
you are stating 3 days in the helptext but this can be changed in the settings of the statistics module...
Comment #10
reaneyk commentedOk, I've changed the text to be more generic.
Comment #11
spuky commentedreupload for the bot... seems he didn't catch it
Comment #12
spuky commenteddoes the bot only test if status is needs review ???
Comment #13
dries commentedTo me, some of these help text feel redundant. We've been removing similarly redundant help texts elsewhere in Drupal. I'm tempted to mark this "won't fix" ...
Comment #14
xanoComment #15
xanoWe don't need help texts for the sake of help texts. The UI already shows far too much text to the user. Everything this patch adds should be explained by the page title or possibly other elements on the page.
Comment #16
yoroy commentedreaneyk, spuky: thanks for your work on this. don't take it personal! :-)
This particular issue was just very old and the current mission is to *remove* as much help text as possible.