Move the statistics graph from admin/settings/mollom to admin/reports/mollom
Dave Reid - July 14, 2009 - 22:07
| Project: | Mollom |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Dave Reid |
| Status: | closed |
Description
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.
#1
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.
#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
#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
Will file issues for new features on this page. Marking as fixed.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.