#2685 closed defect (bug) (fixed)
xprofile_get_field_data() with checkboxes no longer working correctly
Reported by: | r-a-y | Owned by: | johnjamesjacoby |
---|---|---|---|
Milestone: | 1.2.7 | Priority: | blocker |
Severity: | Version: | 1.2.6 | |
Component: | Extended Profile | Keywords: | |
Cc: |
Description
Noting a problem with xprofile field checkboxes as posted here:
http://buddypress.org/community/groups/how-to-and-troubleshooting/forum/topic/1-2-6-upgrade-xprofile_get_field_data-with-checkboxes-no-longer-working-correctly/#post-76283
There might need to be a 1.2.6.1...
Change History (5)
Note: See
TracTickets for help on using
tickets.
The problem arose with [3270], which applies maybe_unserialize() to the output of get_value_byid(). When xprofile_get_field_data() is called on a checkbox, it gets the value from get_value_byid(). The output of xprofile_get_field_data is then run through some formatting filters like kses. The problem is that while those filters could handle the old serialized string (which was then unserialized in the template tags), they don't know how to handle arrays.
Before reverting the change I want to talk to JJJ (who applied [3270]) to see if there's a good reason, other than preferring to use maybe_unserialize in one place instead of two.