I do like the improved stats this module gives the site. There are though two features that I believe would be very useful, especially for page counts.

  • Role Filtering - Be able to set what roles that shall not affect the counter, including UID #1
  • Own content - Option not to count up the users own content
CommentFileSizeAuthor
#3 adstats_role_counter.patch668 byteswrunt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Added ignoring counter visits by role. I'm not going to implement a special case for uid #1 since it is an un-Drupal way to handle those.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Title: Filter on roles and owner » Exclude counter visits by roles and owner
wrunt’s picture

wrunt CreditAttribution: wrunt commented
FileSize
adstats_role_counter.patch668 bytes

It looks like this change only disabled access logging per role. The node counter still operates.

wrunt’s picture

wrunt CreditAttribution: wrunt commented

How bizarre, my comment was truncated. The rest of it reads:

Attached is a simple patch to correct this. Also note that user_access() always returns TRUE for uid #1 in Drupal, so it may be worth implementing a special case.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Fixed

Access log and node counter are two separate things. You can't put that in the check for excluding from the access log. There is already the 'exclude visits from counters' permission that should be used. Marking as fixed since this is all implemented.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Version: 6.x-1.4 » 6.x-1.x-dev

I should note that this functionality is in the 6.x-1.x-dev version right now. I have yet to create a new release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

doublejosh’s picture

doublejosh CreditAttribution: doublejosh commented

Super awesome great. Looking forward to this ('exclude visits from counters.')

littlemisssunshine’s picture

littlemisssunshine CreditAttribution: littlemisssunshine commented

I've installed the dev version of this module (6.x-1.x-dev) and I can't seem to find the option to disable hits by the admin account.

Am I missing something?

Thanks.