[buddypress-trac] [BuddyPress Trac] #7425: XProfile: use filterable field input name in radio/checkbox field type

buddypress-trac noreply at wordpress.org
Tue Jan 24 15:26:12 UTC 2017


#7425: XProfile: use filterable field input name in radio/checkbox field type
------------------------------+-----------------------------
 Reporter:  Offereins         |      Owner:
     Type:  defect (bug)      |     Status:  new
 Priority:  normal            |  Milestone:  Awaiting Review
Component:  Extended Profile  |    Version:
 Severity:  normal            |   Keywords:  has-patch
------------------------------+-----------------------------
 Perhaps there is a good reason, but currently the `radio` and `checkbox`
 profile fields do not use the filterable
 `bp_get_the_profile_field_input_name()` for their input options `name`
 attribute. Using this template tag requires the `$field` global variable,
 but the docblock of the `edit_field_options_html()` field type method
 requires its usage within the profile template, so that should be covered:

 {{{
 Must be used inside the {@link bp_profile_fields()} template loop.
 }}}

 Since the replacement of the current `$field_id` value is equivalent to
 that of the template tag, no obvious damage will be done. The benefit of
 template parity and filter options with other field types is evident. Are
 there any objections to this fix?

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7425>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list