Opened 12 years ago
Closed 12 years ago
#5425 closed defect (bug) (fixed)
wp-admin/profile not saving all files since r7970
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 2.0 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Core | Keywords: | has-patch commit |
| Cc: |
Description
since r7970, the wp-admin/profile is not working the right way.
I think it's due to the direct use of wp_parse_id_list() on $_POST['field_ids'].
As the wp-admin profile displays all groups of field on one page (which is not the case on front end), $_POST['field_ids'] is an array containing each group list of comma separated field ids.
For instance :
group base has 2 fields > id 1 and id 2
another group of fields have 1 field > id 3
Then the $_POST['field_ids'] is not 1,2,3 but array( 0 => "1,2", 1 => "3" ). So we first need to merge the values in a unique comma separated list before using wp_parse_id_list()
The patch is a way to come back from ("revenir en arrière par rapport à") r7970
5425.02.diff seems better, it avoids a foreach.