We need a good function to fetch the statistics available through the API. Thought I'd post this patch for review as it will help several other issues:
#519368: Move the statistics graph from admin/settings/mollom to admin/reports/mollom
#423918: Add a basic statistics block (Molstats)
#487052: Add token integration

If everything looks good, I'm going to commit this tomorrow.

CommentFileSizeAuthor
#4 mollom-statistics-api.patch2.01 KBdave reid
#3 mollom-statistics-api.patch1.9 KBdave reid
#2 mollom-statistics-api.patch1.88 KBdave reid
mollom-statistics-api.patch1.77 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Needs work

A question: Do we want a configurable 'minimum' fetch interval? What should we name the variable?

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review
StatusFileSize
new mollom-statistics-api.patch1.88 KB

Probably best to leave it at being fetched no more often than every hour. Wim Leers pointed out some helpful advice when using the built-in cache as well.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new mollom-statistics-api.patch1.9 KB

Small comment change.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed
StatusFileSize
new mollom-statistics-api.patch2.01 KB

Here's the revision I committed to CVS with improved documentation.

dries’s picture

dries commented

Somewhat relevant discussion for future clean-up: http://drupal.org/node/485974#comment-1877010

Status: Fixed » Closed (fixed)

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