As I was watching http://www.ustream.tv/channel/lullabot-tv live, Angie was trying to demonstrate the admin/reports pages and thought that the Mollom statistics reports should have been there. I agree it doesn't make much sense to have the statitics graph in the 'settings' page and we should move it to admin/reports/mollom.

CommentFileSizeAuthor
#1 519368-mollom-reports-D6.patch6.23 KBdave reid

Comments

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new6.23 KB

Patch also adds a new access callback called _mollom_access() that will return TRUE if the module has been configured (entered access keys) and an optional permission string through user_access(). This helps lockdown such paths as mollom/comment when the module hasn't been configured yet.

dave reid’s picture

What could be nice for us to add to this is more debugging info say maybe like the last 10 mollom watchdog errors or warnings. And add a text-export function to this so people can attach their debugging reports to issues here or in Zendesk.

dave reid’s picture

Assigned: Unassigned » dave reid
dave reid’s picture

Basic reports page committed to CVS. Keeping this issue open to track when API statistics are added, a basic report of the latest watchdog errors, and a text file export ability.

dave reid’s picture

List of things I want to add:
- Most recent 'Mollom' watchdog errors or warnings (added, but needs tests)
- List of the current servers used and test connection to each one
- API statistics
- Text file export capability

dave reid’s picture

Status: Needs review » Fixed

Will file issues for new features on this page. Marking as fixed.

Status: Fixed » Closed (fixed)

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