[buddypress-trac] [BuddyPress Trac] #5220: Overhaul implementation of xprofile field types

[buddypress-trac]

#5220: Overhaul implementation of xprofile field types
------------------------------------------+-----------------------
 Reporter:  DJPaul                        |       Owner:  DJPaul
     Type:  enhancement                   |      Status:  assigned
 Priority:  normal                        |   Milestone:  2.0
Component:  XProfile                      |     Version:
 Severity:  normal                        |  Resolution:
 Keywords:  has-patch dev-feedback early  |
------------------------------------------+-----------------------

Comment (by imath):

 Hi Paul,

 I've just tested your patch. Great work. Just found some details, i'm
 attaching in 5220.03.diff.

 1. Most important of the details. We've already discussed about it
 previously: in wp-admin/profile, unlike on the front end all group of
 fields are displayed on the same page, so the input name for
 {{{field_ids}}} must be {{{field_ids[]}}} else, you won't save all profile
 fields. If you have 3 groups of fields, only the fields of the third group
 of fields should be saved, 1 and 2 not..
 2. While editing the choices for a "multiple" value field like checkbox,
 radio.. The click on the [x] is hiding the option, but does not reset the
 value. As a result the option is saved.
 3. Multiple select:  when you choose more than one option, it's not saving
 all the options just one and the clear link is not working. Reason is the
 name of a multiselect must be {{{name[]}}}. So i've just went back on
 current trunk for this one.
 4. Maybe you should use the {{{bp_parse_args()}}} function introduced by
 jjj ;)

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