Title says all.
See bugreport on Firestats.cc
http://firestats.cc/ticket/849

CommentFileSizeAuthor
#7 firestats.module.txt6.94 KBlomz

Comments

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Active » Fixed

Fixed in CVS. Thanks!

lomz’s picture

lomz commented

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

Gurpartap Singh’s picture

Gurpartap Singh commented

A new release will follow after some changes :)

lomz’s picture

lomz commented

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

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

lomz’s picture

lomz commented
Status: Closed (fixed) » Active

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

lomz’s picture

lomz commented
Status: Active » Needs review
StatusFileSize
new firestats.module.txt6.94 KB

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

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Needs review » Fixed

firestats 6.x-1.1 is out.

Gurpartap Singh’s picture

Gurpartap Singh commented
lomz’s picture

lomz commented

Nice

lomz’s picture

lomz commented

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.

lomz’s picture

lomz commented
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.

Gurpartap Singh’s picture

Gurpartap Singh commented
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!

lomz’s picture

lomz commented

How do I do that?