Everything counted as 1 site after upgrade to firestats 1.6 [#319710]

Comment	File	Size	Author
#7	firestats.module.txt	6.94 KB	lomz

Comment #1

Gurpartap Singh CreditAttribution: Gurpartap Singh commented 14 October 2008 at 10:51

Status:

Active

» Fixed

Fixed in CVS. Thanks!

Log in or register to post comments

Comment #2

lomz CreditAttribution: lomz commented 14 October 2008 at 19:43

Will you make a new official release or would it be better to promote the 6.x-dev to the project frontpage?

Log in or register to post comments

Comment #3

Gurpartap Singh CreditAttribution: Gurpartap Singh commented 15 October 2008 at 03:53

A new release will follow after some changes :)

Log in or register to post comments

Comment #4

lomz CreditAttribution: lomz commented 15 October 2008 at 05:17

Since it would delay some I would put the -dev on the front page, not everybody checks CSS or All releases...

Log in or register to post comments

Comment #5

Anonymous (not verified) CreditAttribution: Anonymous commented 29 October 2008 at 05:21

Status:

Fixed

» Closed (fixed)

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

Log in or register to post comments

Comment #6

lomz CreditAttribution: lomz commented 6 December 2008 at 10:38

Status:

Closed (fixed)

» Active

Problem reappeared in one of the latest revisions to firestats 1.6.

Log in or register to post comments

Comment #7

lomz CreditAttribution: lomz commented 18 December 2008 at 09:25

Status:

Active

» Needs review

File	Size
firestats.module.txt	6.94 KB

here is my firestats.module, with error fixed.
According to http://firestats.cc/ticket/849#comment:15

Log in or register to post comments

Comment #8

Gurpartap Singh CreditAttribution: Gurpartap Singh commented 16 January 2009 at 04:53

Status:

Needs review

» Fixed

firestats 6.x-1.1 is out.

Log in or register to post comments

Comment #9

Gurpartap Singh CreditAttribution: Gurpartap Singh commented 16 January 2009 at 04:54

Log in or register to post comments

Comment #10

lomz CreditAttribution: lomz commented 16 January 2009 at 05:03

Nice

Log in or register to post comments

Comment #11

lomz CreditAttribution: lomz commented 25 January 2009 at 06:08

Funny thing, still the same error, it showed up when I updated from the version of firestats.module that I added to this issue to the newest release.

Log in or register to post comments

Comment #12

lomz CreditAttribution: lomz commented 25 January 2009 at 06:11

Version:	6.x-1.x-dev	» 6.x-1.1
Status:	Fixed	» Active

I put back my edition of the firestats.module and it works again, so there must be a problem with the latest release.

Log in or register to post comments

Comment #13

Gurpartap Singh CreditAttribution: Gurpartap Singh commented 25 January 2009 at 08:22

Version:	6.x-1.1	» 6.x-1.x-dev
Assigned:	Unassigned	» Gurpartap Singh

Can you manually run the function firestats_add_hit() (in firestats.module) and check what version of FireStats it detects for your setup? We are doing nothing more than the fix mentioned on FireStats.cc except being compatible with FireStats 1.5. Let's get this fixed up!