Changeset 10324

Timestamp:

11/03/2015 06:44:28 PM (9 years ago)

Author:

imath

Message:

xProfile Admin UI: make sure profile fields can be reordored and moved from a group of fields to another

In r10071 the fieldset ids has been removed to avoid having duplicate ids into the xProfile admin screen. Unfortunately, to save the state of the draggable fields, we need every xProfile fieldsets to have an id attribute. To have the right behavior back, we are putting back an id attribute to these selectors making sure they are unique by prefixing them with "draggable_".

Fixes #6695

File:

• trunk/src/bp-xprofile/bp-xprofile-admin.php (modified) (2 diffs)

trunk/src/bp-xprofile/bp-xprofile-admin.php

@@ -493 +493 @@
     $field_group_id = $_POST['field_group_id'];
 
-    foreach ( (array) $order['field'] as $position => $field_id ) {
+    foreach ( (array) $order['draggable_field'] as $position => $field_id ) {
         xprofile_update_field_position( (int) $field_id, (int) $position, (int) $field_group_id );
     }
@@ -531 +531 @@
     $field = $admin_field; ?>
 
-    <fieldset class="sortable<?php echo ' ' . $field->type; if ( !empty( $class ) ) echo ' ' . $class; ?>">
+    <fieldset id="draggable_field_<?php echo esc_attr( $field->id ); ?>"class="sortable<?php echo ' ' . $field->type; if ( !empty( $class ) ) echo ' ' . $class; ?>">
         <legend>
             <span>