The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I've wrote patch to exclude root user from statistics with option on settings page. Please, check strings twice, because my English is bad.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | patch2.diff | 6.33 KB | dmitriy.trt |
| exclude_root_user.diff | 1.37 KB | dmitriy.trt |
Comments
Comment #1
dmitriy.trt commentedHere is new patch with more modifications: added statistics about pages and hosts to block:
Everything can be disabled.
Also deleted description from some settings, and deleted call to drupal_bootstrap in visitors_exit().
New patch includes previous.
Sorry for bad English, again. Please, check strings.
Comment #2
gashev commentedexclude_root_user.diff:
if ($user->uid == '1' && variable_get('visitors_exclude_root_user', 1)
Root user always with uid = 1? I don't know.
patch2.diff:
I don't want to add more functionality to visitors_exit function. Visitors_exit function - very fast, only log current user info. No more.
Comment #3
gashev commentedvisitors_exclude_administer_user feature added to next release.
Different from visitors_exclude_root_user: checking user_access('administer').
Comment #4
gashev commentedComment #5
gashev commentedVisitors 6.x-0.12, 2009-04-25
------------------------
- Changed:
* feature request #439254: Exclude administer users from statistics.
Comment #6
dmitriy.trt commentedRoot user always has uid==1. See http://api.drupal.org/api/function/user_access/6
Comment #7
gashev commentedCheck plz visitors_exclude_administer_user (6.x-0.13) (is the same of visitors_exclude_root_user).
Comment #8
dmitriy.trt commentedpatch2 doesn't add more code to visitors_exit. All additional code is for block which displays statistics and for settings page.
Sorry for reply after 3 months, but I still need features added by patch2. Of course, I'm using modified version of your module, but it would be great to see this features in upstream.
Comment #9
gashev commented