Issue adding members to team who have a whitespace in their names

[peter.eussen]

Recently i encountered a problem with the "add new members" functionality in the admin section. When i tried adding a member who had a space (and dot) in their name, it refused to add them.

The problem arises because the javascript in question (buddypress/bp-groups/admin/js/admin.js) tries to create a remove element where the ID of the element is based on the name.

To reproduce it:

• create a user who has a dot in his name say (Peter v. Eussen)
• Try to add the user to a team

You should see it show up in the list of users to add, but not in the list of added users after you update the group.

Solution would be simple:

In bp-groups/admin/js/admin.js replace line 3 with the following line:

var remove_id = 'bp-groups-remove-new-member-' + ui.item.value.replace(/[,:\.\s\-]/g,'-');

Maybe more characters need to be added to ensure that the value complies to valid HTML ID characters.

I wonder though.. why is the displayed name used in the new_members[] array, instead of say the "login name", which would seem more unique..

[DJPaul]

I think this has been addressed by r8417 (#5633) but we should check.

[imath]

Replying to DJPaul:

I think this has been addressed by r8417 (#5633) but we should check.

You're right DJPaul, just checked with "Peter v. Eussen" and it works fine for me.

Thanks peter.eussen for your feedback, you can check in 2.1 beta1 it's fixed.

Closing this ticket, feel free to reopen if something is still going wrong for you.