Skip to:

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2685 closed defect (bug) (fixed)

xprofile_get_field_data() with checkboxes no longer working correctly

Reported by: r-a-y's profile r-a-y Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 1.2.7 Priority: blocker
Severity: Version: 1.2.6
Component: Extended Profile Keywords:


Change History (5)

#1 @boonebgorges
14 years ago

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.

#2 @DJPaul
14 years ago

  • Priority changed from major to blocker

#3 @johnjamesjacoby
14 years ago

  • Owner set to johnjamesjacoby
  • Status changed from new to assigned
  • Version set to 1.2.6

#4 @johnjamesjacoby
14 years ago

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

(In [3390]) Fixes #2685 by reverting part of r3270 (branch)

#5 @johnjamesjacoby
14 years ago

(In [3391]) Fixes #2685 by reverting part of r3270 (trunk)

Note: See TracTickets for help on using tickets.