Modify

Opened 8 years ago

Closed 4 years ago

#1382 closed enhancement (fixed)

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

Reported by: Bryan Owned by: hasienda
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: needinfo user account redirect
Cc: seva_fwd@…, 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 (1)

accountmanager.patch (4.8 KB) - added by airadier 6 years ago.
Adds allow_delete_account option, and fixes #3233 and #3401

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by anonymous

  • Cc seva_fwd@… added; anonymous removed

comment:2 Changed 6 years ago by anonymous

More probably it should be a permission check.

comment:3 Changed 6 years ago by airadier

The attached patch also fixes #3233 and #3401

Changed 6 years ago by airadier

Adds allow_delete_account option, and fixes #3233 and #3401

comment:4 Changed 5 years ago by thijs

  • Cc thijs added
  • 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.

comment:5 Changed 4 years ago by hasienda

  • Keywords user account redirect added
  • 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].

comment:6 Changed 4 years ago by hasienda

  • Keywords needinfo added
  • Status changed from new to 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 4 years ago by hasienda

  • Resolution set to fixed
  • Status changed from assigned to closed

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

Add Comment

Modify Ticket

Action
as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.