Bug #35181
Allow disabling of 'Clear logs' function
| Status: | Resolved | Start date: | 2012-03-23 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assignee: | Reinhard Führicht | % Done: | 100% |
|
| Category: | Backend Module | |||
| Target version: | v1.4 | |||
| Votes: | 1 (View) |
Description
The 'clear logs' function is nuclear - it really deletes (not just marks records as deleted) ALL form data in the database. It's not restricted to the current website or page, and all users that have access to this module can use it.
One customer of mine just destroyed 6 months worth of data because of this. I propose:
- allow system-wide disabling of this option
- allow group-level disabling of this option
- include a clear warning about the consequentes of this action.
Thanks!
Associated revisions
New TS option to allow disabling/enabling the "Clear logs" option in backend module for different users (resolves #35181)
New TS option to allow disabling/enabling the "Clear logs" option in backend module for different users (resolves #35181)
History
Updated by Bart Veldhuizen about 1 year ago
I forgot to mention that I think the system wide disable should be default, to prevent accidental loss of data.
Updated by Bart Veldhuizen 12 months ago
Hi guys,
any thoughts on this one? It's been reported 2 months ago but no reply yet.
Thanks!
Bart
Updated by Reinhard Führicht 12 months ago
- Status changed from New to Accepted
- Assignee set to Reinhard Führicht
- Target version set to v1.4
Sorry, I forgot to answer here.
I already had a look at the code, but changes to the backend related code aren't easy. I thought about rewriting the backend module completely for the 2.0 release.
But I think at least the "group-level disabling" should be possible to include in the next release.
Updated by Bart Veldhuizen 12 months ago
Excellent, thanks!
Updated by Reinhard Führicht 9 months ago
- File 35181.patch added
The attached patch adds this new TS setting:
tx_formhandler_mod1.config {
enableClearLogs = 1
}
Default: 0
Result: Only admins and users with enableClearLogs set to 1 can clear the logs.
Furthermore, I changed the output a bit. Since there is only one log table, a simple message output is enough.
Updated by Reinhard Führicht 9 months ago
- Status changed from Accepted to Resolved
- % Done changed from 0 to 100
Applied in changeset r65720.