Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#3888 closed enhancement (fixed)

Data will be lost prompt might be useful before deleting a Profile Field

Reported by: jimgroom Owned by: johnjamesjacoby
Priority: normal Milestone: 1.6
Component: Extended Profile Version: 1.5.2
Severity: normal Keywords: needs-patch
Cc: jimgroom@…

Description

I just wanted to suggest that you prompt users that when they delete a profile field under BuddyPress--->Profile Fields that they are made aware that their data will be deleted, or at least tell you somewhere. Fact is, one might think that is a cosmetic deletion or removal rather than a whoelsale deletion of the data from the database.

Best, keep up the awesome.

Attachments (1)

3888-1.patch​ (4.8 KB ) - added by DJPaul 15 years ago.

Download all attachments as: .zip

Change History (6)

#1 @johnjamesjacoby
15 years ago

  • Keywords needs-patch added; rtl-feedback removed
  • Milestone Awaiting Review → 1.6
  • Owner set to johnjamesjacoby
  • Status new → assigned

@DJPaul
15 years ago

#2 @DJPaul
15 years ago

So I spent a little time looking at this. Patch attached. It's not complete -- the field_id in the data table is going to need to be updated if that field no longer exists (when a new field is created). And we're probably going to need to store old profile field names when they are deleted, so this can be achieved.

#3 @johnjamesjacoby
15 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [5598]) When deleting a field, do not delete the field data by default. Fixes #3888. In memory of jimgroom's lost data.

#4 @johnjamesjacoby
15 years ago

Variant of Paul's patch committed. No reason to shift the field ID's around as there isn't a UI to merge/replace old ID's with new ones. Similar to WordPress pages/nav menus, I'm fine with deleting the field itself but the children shouldn't get deleted along with it.

#5 @DJPaul
15 years ago

So this change is going to orphan all field data when the field is deleted? Why? Creating a new field with the same name doesn't "restore" the data.

Note: See TracTickets for help on using tickets.