Skip to:

Opened 6 years ago

Closed 6 years ago

#7686 closed enhancement (fixed)

XProfile: Add method parameter(s) to delete field hooks

Reported by: offereins's profile Offereins Owned by: offereins's profile offereins
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Extended Profile Keywords: has-patch commit
Cc: lmoffereins@…


See #7600. The added hooks in BP_XProfile_Field::delete() are missing the $delete_data parameter which provides context to the type of field deletion. This is the suggested new ticket for including said parameter.

Attachments (1)

7686.01.patch (1.5 KB) - added by Offereins 6 years ago.

Download all attachments as: .zip

Change History (3)

6 years ago

#1 @johnjamesjacoby
6 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.0
  • Type changed from defect (bug) to enhancement

👍 Looks good to my eyes

#2 @offereins
6 years ago

  • Owner set to offereins
  • Resolution set to fixed
  • Status changed from new to closed

In 11846:

XProfile: add context parameter to field deletion actions

Passes the $delete_data parameter to the 'xprofile_field_before_delete'
and 'xprofile_field_after_delete' hooks, to provide context for whether
data is deleted along with the field.

Fixes #7686.

Note: See TracTickets for help on using tickets.