With version 1.0 beta2 (and beta1) I get this error in nodes:

user warning: Statistic "ip_address" does not exist. in /var/www/vhost/pinomillershow.org/home/html/sites/all/modules/user_stats/user_stats.module on line 167.
user warning: Statistic "ip_address" does not exist. in /var/www/vhost/pinomillershow.org/home/html/sites/all/modules/user_stats/user_stats.module on line 167.
user warning: Statistic "ip_address" does not exist. in /var/www/vhost/pinomillershow.org/home/html/sites/all/modules/user_stats/user_stats.module on line 167.
...

Only users without permission to see IP get this error.

Since there's one error line for each node comment, and "ip_address" is shown in the author_pane I've thougth that's an author_pane integration problem.

Furthermore, I've tried to change line 18 in "user_stats.author-pane.inc" setting $ip = ""
and everything went well (obviously excluding the ip ).

I hope this help. ;)

bye

CommentFileSizeAuthor
#6 ipaddress.patch612 bytesIngumsky

Comments

liam mcdermott’s picture

liam mcdermott commented

Good catch, it's a problem with the logic in user_stats_get_stats(). I've committed a fix, but can't do another release for bugs that aren't super-critical. :)

The -dev release will automatically update within 24 hours, then you can switch to it.

Ingumsky’s picture

Ingumsky commented

Thank you Liam!

Any patches to fix it? ;)

liam mcdermott’s picture

liam mcdermott commented

Any patches to fix it? ;)

I didn't bother creating a patch as most users wouldn't know what to do with it. The -dev release has updated now, so I advise anyone with this issue to just use that. :)

liam mcdermott’s picture

liam mcdermott commented
Status: Active » Fixed
Ingumsky’s picture

Ingumsky commented

Thanks mate!

Ingumsky’s picture

Ingumsky commented
Status: Fixed » Reviewed & tested by the community
StatusFileSize
new ipaddress.patch612 bytes
Ingumsky’s picture

Ingumsky commented

Liam McDermott
I've made the patch-file based on your work for those lazy fellas (like myself ;) who don't want to switch between "stable" and "unstable" module branches every time new minor bug was found -)

liam mcdermott’s picture

liam mcdermott commented
Status: Reviewed & tested by the community » Fixed

Ingumsky: nice one, thanks. :)

Status: Fixed » Closed (fixed)

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

aharown07’s picture

aharown07 commented

Thanks for patch. Working well for me