Modify ↓

Opened 18 years ago

Closed 14 years ago

#1382 closed enhancement (fixed)

[patch] Make "Delete Account" function on "My Account" page optional

Reported by: Bryan Owned by: Steffen Hoffmann
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: needinfo user account redirect
Cc: seva_fwd@…, Thijs Triemstra Trac Release: 0.10

Description

I'd find it useful be able to specify an option in trac.ini to prevent users from deleting their own accounts.

Attachments (1)

accountmanager.patch​ (4.8 KB) - added by Álvaro Iradier 16 years ago.
Adds allow_delete_account option, and fixes #3233 and #3401

Download all attachments as: .zip

Change History (8)

comment:1 Changed 16 years ago by anonymous

Cc: seva_fwd@… added; anonymous removed

comment:2 Changed 16 years ago by anonymous

More probably it should be a permission check.

comment:3 Changed 16 years ago by Álvaro Iradier

The attached patch also fixes #3233 and #3401

Changed 16 years ago by Álvaro Iradier

Attachment: accountmanager.patch​ added

Adds allow_delete_account option, and fixes #3233 and #3401

comment:4 Changed 16 years ago by Thijs Triemstra

Cc: Thijs Triemstra added
Summary: Give option remove "Delete Account" function from "My Account" page → Make "Delete Account" function on "My Account" page optional

Renaming ticket to give it a better title.

comment:5 Changed 14 years ago by Steffen Hoffmann

Keywords: user account redirect added
Owner: changed from Matt Good to Steffen Hoffmann
Summary: Make "Delete Account" function on "My Account" page optional → [patch] Make "Delete Account" function on "My Account" page optional

The fix for #3233 suggested here has been added to trunk in changeset [7721]. However there's still no agreement reached, if this really resolved that issue. Maybe the additional change introduced with [9263] makes all the difference?

MessageWrapper has already been worked on as well, see [5478].

comment:6 Changed 14 years ago by Steffen Hoffmann

Keywords: needinfo added
Status: new → assigned

The remaining patch content about making user driven account deletion optional looks good. I'll give it a try and adapt to current trunk code.

Thank you for taking care and providing a patch despite of no maintainer reaction for a long time.

Q.: If I'll implement this in trunk to prepare for 0.11 branch, is a backport to 0.10 still required? After all, Trac is stable at 0.12.1 now, and I suggest upgrading ASAP, since it introduced multi-repository capabilities, full i18n support and more.

comment:7 Changed 14 years ago by Steffen Hoffmann

Resolution: → fixed
Status: assigned → closed

(In [9287]) AccountManagerPlugin: Add option to disable users to delete their own account, closes #1382.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.