[buddypress-trac] [BuddyPress Trac] #5647: xProfileFields - deletion of drop down select box values not possible

[buddypress-trac]

#5647: xProfileFields - deletion of drop down select box values not possible
-----------------------------------+------------------------------
 Reporter:  GreenFeetz             |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  XProfile               |     Version:  2.0
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+------------------------------

Comment (by DJPaul):

 Some feedback on the patch.

 * `-1` appears to be a magic number. What is -1? Why don't we want to
 render this bit of the template for -1? It also appears to be not in the
 1.9.2 version; did I change the logic in the 2.0 version that made this
 extra check necessary?
 * For `id="delete-<?php echo esc_attr( $options[$i]->id ); ?>"`, it's
 neater to concatenate `delete-` with the variable inside the `esc_attr()`
 call (similar how to you built the `href` value), e.g. `id="<?php echo
 esc_attr( 'delete-' . $options[$i]->id ); ?>"`

 While `[x]` is a lousy, untranslatable way of indicating a delete button
 (which is the fault of an old version of BP), I don't think we need to
 adjust that for this ticket, but we should make another ticket to go
 through and replace ticks/crosses with genericons or dashicons or
 something similar.

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