#6129 closed defect (bug) (fixed)
Multiple Make WordPress Team Selections Not Saved on .org Profile
Reported by: | timohaver | Owned by: | coffee2code |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Profiles | Keywords: | |
Cc: |
Description
This worked until recently. Perhaps until the addition of the Photos team to the list?
In the WordPress.org profile under the Contributions tab, a user may select multiple Make WordPress teams from a scrollable list, but it appears only one selection is saved. It is always the selection closest to the bottom of the list. For example, if I select Meta and Photos, my profile will only show that I contribute hours to Photos.
I replicated this on the current production of macOS 11.6.2 and 12.2, with the current production version of Safari, Firefox, and Brave.
Attachments (2)
Change History (7)
#2
@
3 years ago
- Resolution set to fixed
- Status changed from accepted to closed
Thanks for the report!
Fixed in [18490-dotorg]:
Profiles: Ensure multiple Contributor Teams can be selected.
Props timohaver, Otto42, coffee2code.
Fixes #6129.
#3
@
3 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Hey @coffee2code This appears to have broken being able to select multiple teams, at least upon edit of ones profile.
Example POST payload as a result of this:
&field_24=Yes &field_29=38 &field_30[][]=Meta+Team &profile-group-edit-submit=Save+Changes+ &field_ids=24,29,30 &_wpnonce=12345abcd &_wp_http_referer=/dd32/profile/edit/group/5/
field_30[][]
seems wrong.
This is the cause of the PHP Warnings on WordPress.org:
preg_match() expects parameter 2 to be string, array given Source: POST https://profiles.wordpress.org/dd32/profile/edit/group/5/ Timestamp: Mon, 21 Mar 2022 03:42:16 +0000 (1647834136) File: wp-content/plugins/buddypress/bp-xprofile/classes/class-bp-xprofile-field-type.php
#4
follow-up:
↓ 5
@
3 years ago
- Resolution set to fixed
- Status changed from reopened to closed
18490-dotorg did "fix" the issue as experienced at the time. It was in largely untouched code so undoubtedly something somewhere else (BP?) had changed its behavior which caused the reported issue. Since then it seems whatever the original issue was has been fixed, so the 18490-dotorg change is no longer needed.
Reverted in [18627-dotorg].
#5
in reply to:
↑ 4
@
3 years ago
Replying to coffee2code:
Thanks! I don't doubt it was originally needed, it looks like the Warnings started March 18th, around the BuddyPress 10.2.0 release, so just for the record, that's where the change probably came from.
screenshot of multiple selection from the list