Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Right or wrong?
IMO this doesn't belong under 'Logs'...
(enable statistics.module to see this happen)
Comment | File | Size | Author |
---|---|---|---|
#17 | 80944-statistics-menu-settings-D7.patch | 9.47 KB | Dave Reid |
#15 | Picture 1.png | 18.16 KB | webchick |
#12 | access_log_settings.patch | 1.22 KB | swentel |
#3 | statistics_settings_0.patch | 1.84 KB | ChrisKennedy |
#2 | accesslogs.png | 7.53 KB | forngren |
Comments
Comment #1
ChrisKennedy CreditAttribution: ChrisKennedy commentedAgreed, patch attached.
Comment #2
forngren CreditAttribution: forngren commentedThis patch breaks the alphabetical order of menu items for me, see attached screenshot. +1 for the concept
Comment #3
ChrisKennedy CreditAttribution: ChrisKennedy commentedAh true, I needed to remove the weight from the menu item, which was causing it to be at the end of lists.
Comment #4
Steven CreditAttribution: Steven commentedI'm not sure I agree. Comment settings are in the comments section too.
Comment #5
ChrisKennedy CreditAttribution: ChrisKennedy commentedPerhaps the comment settings page should be moved too, but it makes more sense in that case because it's a tab rather than a basic menu item. I would prefer to stick to statistics in this issue though.
One of the best usability improvements in 5.0 imo is the reorganization of admin into sections. We should default to using the appropriate categories unless there is a good reason not to. In the case of the statistics settings I don't see a compelling reason to diverge from the standard location.
Comment #6
geodaniel CreditAttribution: geodaniel commentedI'd add a +1 for this. I was just trying to set up statistics, and the settings weren't under 'site configuration' where I had initially looked. It took me a while to spot them under 'logs'.
Comment #7
ica CreditAttribution: ica commentedalso there is a confusion here for 'not brainy' ones like myself :)
you can not see statics on the settings + statics actually called 'logs' logs are synonymous to stats but 'statistics' is more generic naming and suits better -imo no need for confusion -unless i am getting all wrong here and actually statistics and logs are entirely different things :)
also if you install Xstatistics.module as an extention to your statistics (logs) and activate the its block and your statistics not activated it'll give u a link -plase activate your statistics and link to
> /admin/settings/statistics
well thats where the Xstatistics.module developer thought it should be!! but its not there of course, instead its here
> admin/logs/settings
imho - the url dir. pattern should not break (without url redirect fix)
and logs should be under statistics - to give to other stat options more than logs
> admin/settings/statistics/logs
and listed under general settings page
-just some thought maybe there are more important reasons for to keep current way and there is a reason for not to do so as above
thanks
Comment #8
drummI think we should leave the menu rearrangements to the development versions of Drupal, currently 6.x.
Comment #9
PasqualleI think, this will not make it into drupal 6
Comment #10
lilou CreditAttribution: lilou commentedMake sense.
Comment #11
lilou CreditAttribution: lilou commentedComment #12
swentel CreditAttribution: swentel commentedComment #13
dawehnerit makes total sense for me
i had this problem many times to find this setting
i runned the tests of the statistic module:
36 passes, 0 fails, and 0 exceptions
anyone else?
Comment #14
Pasquallecode looks good
Comment #15
webchick"Access log settings" looks out of place with the other items here. See screenshot.
And probably, I would look for this under "Statistics," not "Access log" since I enabled the "Statistics" module?
Comment #16
webchickComment #17
Dave Reid+1 to "Statistics" settings since it is the name of the module. This menu item location has always irked me in D5/6. Attached patch fixes all the references to admin/reports/settings that were missed before and adds a much needed 'administer statistics' permission for the moved admin/settings/statistics.
Comment #19
Dave ReidTesting #17 and it installs and works for me. Setting back to code needs review for re-testing.
Comment #20
Dave ReidComment #21
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!
Comment #22
XanoWhat about moving the statistics out of the admin pages, so site administrators may decide to make them public using permissions? A lot of sites, however not the majority, have made their statistics publicly available.
Comment #23
Dave ReidAn interesting idea, but needs a separate issue. :)
Comment #24
XanoDone: #362834: Move statistics out of the administration pages and add permissions.