[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 DJPaul):

 Thanks imath. So far, I've merged in three of your changes:
 [https://github.com/paulgibbs/BuddyPress/commit/bc324f0e0c5ad7afa248c894c3f679ce86e0d748
 bc324f0] and
 [https://github.com/paulgibbs/BuddyPress/commit/b3578f13692575ddabf2796834a2dfeaefba917d
 b3578f1] and
 [https://github.com/paulgibbs/BuddyPress/commit/879b5c5609381d88735d63f659b17f8eeb689319
 879b5c5].

 This patch -- https://gist.github.com/paulgibbs/9476911 -- is that related
 to point 2 above? If so, I don't see how this is a new problem introduced
 this patch, as I haven't touched that JS file. If it's a new bug, I need
 to understand how it was introduced, so I have left it out for now so we
 can test BuddyPress trunk to see if this is an existing bug.

 The multiple select change you spotted is interesting. I made that change
 on purpose to take the special handling of the multiselectbox out of the
 core function, so I've fixed it in a slightly different way; see 879b5c5.

 Thanks again for taking a look.

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