Fixes #3864. Removed old code from profile fields sorting. Name spaced IDs and classes.

diff --git a/views/default/admin/appearance/profile_fields/list.php b/views/default/admin/appearance/profile_fields/list.php
index f4ff1e98691cb6997d7f9251986562fcdf793ad4..3e287ceac91699e2971028a22d86e6f7ac06db14 100644 (file)
--- a/views/default/admin/appearance/profile_fields/list.php
+++ b/views/default/admin/appearance/profile_fields/list.php
@@ -26,8 +26,7 @@ if ($fieldlist) {
        }
 }
 ?>
-<div id="list">
-       <ul id="sortable_profile_fields">
+<ul id="elgg-profile-fields">
 <?php
 
 $save = elgg_echo('save');
@@ -53,8 +52,4 @@ HTML;
 }
 
 ?>
-       </ul>
-</div>
-<div id="tempList"></div>
-
-<input name="sortableListOrder" type="hidden" id="sortableListOrder" value="<?php echo $fieldlist; ?>" />
\ No newline at end of file
+</ul>
\ No newline at end of file

diff --git a/views/default/js/admin.php b/views/default/js/admin.php
index 253a738879e00bf5f2a7c8be0c37662e0d4530e4..d4dd06822d1f4e253b3038af97fb64aed8355d1b 100644 (file)
--- a/views/default/js/admin.php
+++ b/views/default/js/admin.php
@@ -42,7 +42,7 @@ elgg.admin.init = function () {
        });
 
        // draggable profile field reordering.
-       $('#sortable_profile_fields').sortable({
+       $('#elgg-profile-fields').sortable({
                items: 'li',
                handle: 'span.elgg-state-draggable',
                stop: elgg.admin.moveProfileField
@@ -143,7 +143,7 @@ elgg.admin.editProfileField = function(value, settings) {
  * @return void
  */
 elgg.admin.moveProfileField = function(e, ui) {
-       var orderArr = $('#sortable_profile_fields').sortable('toArray');
+       var orderArr = $('#elgg-profile-fields').sortable('toArray');
        var orderStr = orderArr.join(',');
 
        elgg.action('profile/fields/reorder', {