[buddypress-trac] [BuddyPress Trac] #6520: xprofile_check_is_required_field should also check for capability when in admin

buddypress-trac noreply at wordpress.org
Mon Jun 22 14:40:26 UTC 2015


#6520: xprofile_check_is_required_field should also check for capability when in
admin
----------------------------------+--------------------
 Reporter:  thomaslhotta          |       Owner:
     Type:  defect (bug)          |      Status:  new
 Priority:  normal                |   Milestone:  2.4
Component:  Component - XProfile  |     Version:  2.3.2
 Severity:  normal                |  Resolution:
 Keywords:                        |
----------------------------------+--------------------
Changes (by boonebgorges):

 * milestone:  Awaiting Review => 2.4


Comment:

 Introduced in [5165]. See #3593. My intepretation is that the `is_admin()`
 check was put there to hide the "Required" label in the admin, but that
 the `xprofile_check_is_required_field()` function is the wrong place to
 put the `is_admin()` check - that hiding should happen when the markup is
 built.

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


More information about the buddypress-trac mailing list