[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