Merge pull request #405 from Codeinwp/fix/meta-reorder

fix: add fields re-ordering for JSON payload

Author: selul

inc/admin.php

@@ -276,6 +276,7 @@ function ppom_admin_save_form_meta() {
 	}
 
 	$_REQUEST['ppom'] = is_array( $_REQUEST['ppom'] ) ? $_REQUEST['ppom'] : json_decode( wp_unslash( $_REQUEST['ppom'] ), true );
+
 	global $wpdb;
 
 	extract( $_REQUEST );
@@ -453,6 +454,7 @@ function ppom_admin_update_form_meta() {
 		wp_send_json( $resp );
 	}
 	$_REQUEST['ppom'] = is_array( $_REQUEST['ppom'] ) ? $_REQUEST['ppom'] : json_decode( wp_unslash( $_REQUEST['ppom'] ), true );
+	
 	global $wpdb;
 
 	$ppom_meta    = isset( $_REQUEST['ppom_meta'] ) ? $_REQUEST['ppom_meta'] : $_REQUEST['ppom'];

js/admin/ppom-admin.js

@@ -128,8 +128,14 @@ jQuery(function($) {
         jQuery(".ppom-meta-save-notice").html('<img src="' + ppom_vars.loader + '">').show();
 
         $('.ppom-unsave-data').remove();
-        var data = $(this).serializeJSON();
+        const data = $(this).serializeJSON();
+        
+        const fieldsOrder = Array(...document.querySelectorAll('.ui-sortable-handle[id^="ppom_sort_id_"]'))
+            .map( node => node.id.replace('ppom_sort_id_', '') ); // ['2', '3']
+        data.ppom = fieldsOrder.map( fieldId => data.ppom[fieldId] );
+        
         data.ppom = JSON.stringify(data.ppom);
+
         // Send the JSON data via POST request
         $.ajax({
             url: ajaxurl,
@@ -329,7 +335,7 @@ jQuery(function($) {
             placeholder = '-';
         }
 
-        var html = '<tr class="row_no_' + id + '" id="ppom_sort_id_' + id + '">';
+        var html = '<tr class="row_no_' + id + ' ui-sortable-handle" id="ppom_sort_id_' + id + '">';
         html += '<td class="ppom-sortable-handle"><i class="fa fa-arrows" aria-hidden="true"></i></td>';
         html += '<td class="ppom-check-one-field ppom-checkboxe-style">';
         html += '<label>';