Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3513 closed enhancement (fixed)

add field_ ID to xprofile_get_field_data filter

Reported by: sboisvert's profile sboisvert Owned by:
Milestone: 1.5 Priority: normal
Severity: minor Version: 1.5
Component: Extended Profile Keywords:
Cc: sboisvert

Description

There may be a much better way to do what I am trying to do, and if so I apologize for wasting your time if this would be better implemented in another way.
But I find it would be particularly useful to have $field_id passed to the xprofile_get_field_data filter as without it I find it hard to see how one would adequately know how to process the data.
I have included an svn patch that adds $field_id to both apply_filter calls in xprofile_get_field_data function. Note that I am open to changing this to move it up one to the bp_get_profile_field_data filter, but I suspect it is most versatile in xprofile_get_field_data.

For information purposes, our particular use case involves displaying translations of the drop down value selected based on the language the user is browsing the site in.

Attachments (1)

filter_xprofile_get_field_data_fieldID.patch (631 bytes) - added by sboisvert 13 years ago.

Download all attachments as: .zip

Change History (4)

#1 @boonebgorges
13 years ago

  • Milestone changed from Awaiting Review to 1.5

Good catch. You're right that the filter is more or less useless without the field_id passed along as well. In fact, I'll do you one better, and pass along the $user_id parameter as well.

#2 @boonebgorges
13 years ago

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

(In [5032]) Pass additional parameters to xprofile_get_field_data filter. Fixes #3513. Props sboisvert

#3 @sboisvert
13 years ago

Thank you very much!

Note: See TracTickets for help on using tickets.