[buddypress-trac] [BuddyPress Trac] #5630: Enable BP_XProfile_Field_Type classes to register display filter

buddypress-trac noreply at wordpress.org
Thu Jul 17 00:54:11 UTC 2014


#5630: Enable BP_XProfile_Field_Type classes to register display filter
-----------------------------------+---------------------------
 Reporter:  boonebgorges           |       Owner:  boonebgorges
     Type:  defect (bug)           |      Status:  reopened
 Priority:  normal                 |   Milestone:  2.1
Component:  XProfile               |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+---------------------------

Comment (by boonebgorges):

 Good catch about `esc_html()`. I hadn't thought about that.

 Changing the priority seems fine to me. I think there's still value in
 having it be less than 10, so that third-party filters at the default
 priority will receive an already-formatted string. 8 seems fine (as long
 as we know that it'll run after the `esc_html` call which is also at 8 -
 though it seems like this is a race condition)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5630#comment:9>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list