Fixed ajax variations posted data

This commit is contained in:
Claudio Sanches 2015-07-08 13:55:29 -03:00
parent 38d0d0dcad
commit ffef95f56f
3 changed files with 72 additions and 42 deletions

View File

@ -294,6 +294,34 @@ jQuery( function( $ ) {
}); });
}, },
/**
* Ger variations fields and convert to object
*
* @param {object} fields
*
* @return {object}
*/
get_variations_fields: function( fields ) {
var data = {},
index = 0;
fields.each( function( i, element ) {
$.each( $( ':input', element ).serializeArray(), function( key, input ) {
var name = input.name.replace( /\[.*\]/g, '' );
if ( ! data.hasOwnProperty( name ) ) {
data[ name ] = {};
}
data[ name ][ index ] = input.value;
});
index++;
});
return data;
},
/** /**
* Save variations * Save variations
* *
@ -302,20 +330,21 @@ jQuery( function( $ ) {
save_variations: function() { save_variations: function() {
var button = $( this ), var button = $( this ),
wrapper = $( '#variable_product_options .woocommerce_variations' ), wrapper = $( '#variable_product_options .woocommerce_variations' ),
need_update = $( '.variation-needs-update', wrapper ); need_update = $( '.variation-needs-update', wrapper ),
data = {};
// Save only with products need update. // Save only with products need update.
if ( 0 < need_update.length ) { if ( 0 < need_update.length ) {
wc_meta_boxes_product_variations_ajax.block(); wc_meta_boxes_product_variations_ajax.block();
data = wc_meta_boxes_product_variations_ajax.get_variations_fields( need_update );
data.action = 'woocommerce_save_variations';
data.security = woocommerce_admin_meta_boxes_variations.save_variations_nonce;
data.product_id = wrapper.data( 'product_id' );
$.ajax({ $.ajax({
url: woocommerce_admin_meta_boxes_variations.ajax_url, url: woocommerce_admin_meta_boxes_variations.ajax_url,
data: { data: data,
action: 'woocommerce_save_variations',
security: woocommerce_admin_meta_boxes_variations.save_variations_nonce,
product_id: wrapper.data( 'product_id' ),
data: $( ':input', need_update ).serialize()
},
type: 'POST', type: 'POST',
success: function() { success: function() {
// Allow change page, delete and add new variations // Allow change page, delete and add new variations

File diff suppressed because one or more lines are too long

View File

@ -2469,46 +2469,47 @@ class WC_AJAX {
check_ajax_referer( 'save-variations', 'security' ); check_ajax_referer( 'save-variations', 'security' );
// Check permissions again and make sure we have what we need // Check permissions again and make sure we have what we need
if ( ! current_user_can( 'edit_products' ) || empty( $_POST['data'] ) || empty( $_POST['product_id'] ) ) { if ( ! current_user_can( 'edit_products' ) || empty( $_POST ) || empty( $_POST['product_id'] ) ) {
die( -1 ); die( -1 );
} }
global $wpdb; global $wpdb;
parse_str( $_POST['data'], $data );
$product_id = absint( $_POST['product_id'] ); $product_id = absint( $_POST['product_id'] );
$attributes = (array) maybe_unserialize( get_post_meta( $product_id, '_product_attributes', true ) ); $attributes = (array) maybe_unserialize( get_post_meta( $product_id, '_product_attributes', true ) );
if ( isset( $data['variable_sku'] ) ) { die();
$variable_post_id = $data['variable_post_id'];
$variable_sku = $data['variable_sku'];
$variable_regular_price = $data['variable_regular_price'];
$variable_sale_price = $data['variable_sale_price'];
$upload_image_id = $data['upload_image_id'];
$variable_download_limit = $data['variable_download_limit'];
$variable_download_expiry = $data['variable_download_expiry'];
$variable_shipping_class = $data['variable_shipping_class'];
$variable_tax_class = isset( $data['variable_tax_class'] ) ? $data['variable_tax_class'] : array();
$variable_menu_order = $data['variation_menu_order'];
$variable_sale_price_dates_from = $data['variable_sale_price_dates_from'];
$variable_sale_price_dates_to = $data['variable_sale_price_dates_to'];
$variable_weight = isset( $data['variable_weight'] ) ? $data['variable_weight'] : array(); if ( isset( $_POST['variable_sku'] ) ) {
$variable_length = isset( $data['variable_length'] ) ? $data['variable_length'] : array(); $variable_post_id = $_POST['variable_post_id'];
$variable_width = isset( $data['variable_width'] ) ? $data['variable_width'] : array(); $variable_sku = $_POST['variable_sku'];
$variable_height = isset( $data['variable_height'] ) ? $data['variable_height'] : array(); $variable_regular_price = $_POST['variable_regular_price'];
$variable_enabled = isset( $data['variable_enabled'] ) ? $data['variable_enabled'] : array(); $variable_sale_price = $_POST['variable_sale_price'];
$variable_is_virtual = isset( $data['variable_is_virtual'] ) ? $data['variable_is_virtual'] : array(); $upload_image_id = $_POST['upload_image_id'];
$variable_is_downloadable = isset( $data['variable_is_downloadable'] ) ? $data['variable_is_downloadable'] : array(); $variable_download_limit = $_POST['variable_download_limit'];
$variable_download_expiry = $_POST['variable_download_expiry'];
$variable_shipping_class = $_POST['variable_shipping_class'];
$variable_tax_class = isset( $_POST['variable_tax_class'] ) ? $_POST['variable_tax_class'] : array();
$variable_menu_order = $_POST['variation_menu_order'];
$variable_sale_price_dates_from = $_POST['variable_sale_price_dates_from'];
$variable_sale_price_dates_to = $_POST['variable_sale_price_dates_to'];
$variable_manage_stock = isset( $data['variable_manage_stock'] ) ? $data['variable_manage_stock'] : array(); $variable_weight = isset( $_POST['variable_weight'] ) ? $_POST['variable_weight'] : array();
$variable_stock = isset( $data['variable_stock'] ) ? $data['variable_stock'] : array(); $variable_length = isset( $_POST['variable_length'] ) ? $_POST['variable_length'] : array();
$variable_backorders = isset( $data['variable_backorders'] ) ? $data['variable_backorders'] : array(); $variable_width = isset( $_POST['variable_width'] ) ? $_POST['variable_width'] : array();
$variable_stock_status = isset( $data['variable_stock_status'] ) ? $data['variable_stock_status'] : array(); $variable_height = isset( $_POST['variable_height'] ) ? $_POST['variable_height'] : array();
$variable_enabled = isset( $_POST['variable_enabled'] ) ? $_POST['variable_enabled'] : array();
$variable_is_virtual = isset( $_POST['variable_is_virtual'] ) ? $_POST['variable_is_virtual'] : array();
$variable_is_downloadable = isset( $_POST['variable_is_downloadable'] ) ? $_POST['variable_is_downloadable'] : array();
$variable_description = isset( $data['variable_description'] ) ? $data['variable_description'] : array(); $variable_manage_stock = isset( $_POST['variable_manage_stock'] ) ? $_POST['variable_manage_stock'] : array();
$variable_stock = isset( $_POST['variable_stock'] ) ? $_POST['variable_stock'] : array();
$variable_backorders = isset( $_POST['variable_backorders'] ) ? $_POST['variable_backorders'] : array();
$variable_stock_status = isset( $_POST['variable_stock_status'] ) ? $_POST['variable_stock_status'] : array();
$max_loop = max( array_keys( $data['variable_post_id'] ) ); $variable_description = isset( $_POST['variable_description'] ) ? $_POST['variable_description'] : array();
$max_loop = max( array_keys( $_POST['variable_post_id'] ) );
for ( $i = 0; $i <= $max_loop; $i ++ ) { for ( $i = 0; $i <= $max_loop; $i ++ ) {
@ -2660,8 +2661,8 @@ class WC_AJAX {
update_post_meta( $variation_id, '_download_expiry', wc_clean( $variable_download_expiry[ $i ] ) ); update_post_meta( $variation_id, '_download_expiry', wc_clean( $variable_download_expiry[ $i ] ) );
$files = array(); $files = array();
$file_names = isset( $data['_wc_variation_file_names'][ $variation_id ] ) ? array_map( 'wc_clean', $data['_wc_variation_file_names'][ $variation_id ] ) : array(); $file_names = isset( $_POST['_wc_variation_file_names'][ $variation_id ] ) ? array_map( 'wc_clean', $_POST['_wc_variation_file_names'][ $variation_id ] ) : array();
$file_urls = isset( $data['_wc_variation_file_urls'][ $variation_id ] ) ? array_map( 'wc_clean', $data['_wc_variation_file_urls'][ $variation_id ] ) : array(); $file_urls = isset( $_POST['_wc_variation_file_urls'][ $variation_id ] ) ? array_map( 'wc_clean', $_POST['_wc_variation_file_urls'][ $variation_id ] ) : array();
$file_url_size = sizeof( $file_urls ); $file_url_size = sizeof( $file_urls );
$allowed_file_types = get_allowed_mime_types(); $allowed_file_types = get_allowed_mime_types();
@ -2732,9 +2733,9 @@ class WC_AJAX {
if ( $attribute['is_taxonomy'] ) { if ( $attribute['is_taxonomy'] ) {
// Don't use wc_clean as it destroys sanitized characters // Don't use wc_clean as it destroys sanitized characters
$value = isset( $data[ $attribute_key ][ $i ] ) ? sanitize_title( stripslashes( $data[ $attribute_key ][ $i ] ) ) : ''; $value = isset( $_POST[ $attribute_key ][ $i ] ) ? sanitize_title( stripslashes( $_POST[ $attribute_key ][ $i ] ) ) : '';
} else { } else {
$value = isset( $data[ $attribute_key ][ $i ] ) ? wc_clean( stripslashes( $data[ $attribute_key ][ $i ] ) ) : ''; $value = isset( $_POST[ $attribute_key ][ $i ] ) ? wc_clean( stripslashes( $_POST[ $attribute_key ][ $i ] ) ) : '';
} }
update_post_meta( $variation_id, $attribute_key, $value ); update_post_meta( $variation_id, $attribute_key, $value );
@ -2763,8 +2764,8 @@ class WC_AJAX {
if ( $attribute['is_variation'] ) { if ( $attribute['is_variation'] ) {
// Don't use wc_clean as it destroys sanitized characters // Don't use wc_clean as it destroys sanitized characters
if ( isset( $data[ 'default_attribute_' . sanitize_title( $attribute['name'] ) ] ) ) { if ( isset( $_POST[ 'default_attribute_' . sanitize_title( $attribute['name'] ) ] ) ) {
$value = sanitize_title( trim( stripslashes( $data[ 'default_attribute_' . sanitize_title( $attribute['name'] ) ] ) ) ); $value = sanitize_title( trim( stripslashes( $_POST[ 'default_attribute_' . sanitize_title( $attribute['name'] ) ] ) ) );
} else { } else {
$value = ''; $value = '';
} }