Ticket #1382 (closed enhancement: fixed)

Opened 6 years ago

Last modified 3 years ago

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

Reported by: Bryan Assigned to: hasienda
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: needinfo user account redirect
Cc: seva_fwd@yahoo.com, thijs 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

accountmanager.patch (4.8 kB) - added by airadier on 04/30/09 11:21:57.
Adds allow_delete_account option, and fixes #3233 and #3401

Change History

07/18/08 05:58:58 changed by anonymous

  • cc set to seva_fwd@yahoo.com.

07/28/08 07:28:11 changed by anonymous

More probably it should be a permission check.

04/30/09 11:21:16 changed by airadier

The attached patch also fixes #3233 and #3401

04/30/09 11:21:57 changed by airadier

  • attachment accountmanager.patch added.

Adds allow_delete_account option, and fixes #3233 and #3401

06/26/09 14:26:18 changed by thijs

  • cc changed from seva_fwd@yahoo.com to seva_fwd@yahoo.com, thijs.
  • summary changed from Give option remove "Delete Account" function from "My Account" page to Make "Delete Account" function on "My Account" page optional.

Renaming ticket to give it a better title.

10/11/10 01:14:43 changed by hasienda

  • keywords set to user account redirect.
  • owner changed from mgood to hasienda.
  • summary changed from Make "Delete Account" function on "My Account" page optional to [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].

10/11/10 01:20:32 changed by hasienda

  • status changed from new to assigned.
  • keywords changed from user account redirect to needinfo user account redirect.

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.

10/12/10 22:16:55 changed by hasienda

  • status changed from assigned to closed.
  • resolution set to fixed.

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

Add/Change #1382 ([patch] Make "Delete Account" function on "My Account" page optional)




Change Properties
Action