Download & Extend

Move the statistics graph from admin/settings/mollom to admin/reports/mollom

Project:Mollom
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Dave Reid
Status:closed (fixed)

Issue Summary

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.

Comments

#1

Status:active» needs review

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.

AttachmentSizeStatusTest resultOperations
519368-mollom-reports-D6.patch6.23 KBIgnored: Check issue status.NoneNone

#2

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.

#3

Assigned to:Anonymous» Dave Reid

#4

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.

#5

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

#6

Status:needs review» fixed

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

#7

Status:fixed» closed (fixed)

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

nobody click here