[buddypress-trac] [BuddyPress Trac] #5633: Group Administration - Add new members metabox & changing opinion

[buddypress-trac]

#5633: Group Administration - Add new members metabox & changing opinion
--------------------------+-----------------------
 Reporter:  imath         |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  2.1
Component:  Groups        |    Version:  2.0
 Severity:  normal        |   Keywords:  has-patch
--------------------------+-----------------------
 While on a particular Group Administration, it's possible to use the "Add
 new members" metabox to add a user to the current group using the
 autocomplete feature.

 If i select a member, a new {{{li}}} is created in the {{{ul#bp-groups-
 new-members-list}}} and an hidden text field containing the user id is
 created. This hidden field ( {{{new_members[]}}} ) is the one used to
 actually get the members to add to the group.

 The problem is when before updating the group i decide to change my mind
 about one of the users i want to add. If i click on the {{{x}}} link, it
 removes the {{{li}}} but not the corresponding hidden field
 {{{new_members[]}}}. As a result this member will "wrongly" be added when
 updating the group.

 I suggest the attached patch to make sure it's not the case.

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