The serializeJSON function was causing a long delay when preparing the JSON if there are many input fields within the variations (even worse if 3rd party plugins add their own extra fields there too).
Created wc_add_help_tip() function to generate the icons
Replaced all .help_tip elements and [?]
We'll keep the old image and styles in order to maintain backwards
compatibility
Closes#9381
WooCommerce prior to WC 2.4 saved the product type before any variations were saved because
WC_Meta_Box_Product_Data::save_variations() was called by WC_Meta_Box_Product_Data::save().
However, in WC 2.4 the variations are saved independently of other data about the containing
variable product, including product type. Because the product type hasn't been saved yet,
extensions that need to save their own variation level meta data can't know when saving
variations if the product is of the type they want to act on. They also can't check `$_POST`
to find out when saving variations, because 'product-type' isn't passed to that as it's
variable level meta data, not variation level meta data.
This patch passes the product type along with the variation level meta data when saving variations.
It then uses that to save the product type if the variable product has not yet been saved (and
therefore the product type has never been stored, which means calling get_product() would instantiate
a 'simple' product, as that is the default product type). This can lead to fatal errors if callbacks
expect the product type to be variable and attempt to call methods that only exist on those product
types, like variable_product_sync().
It will also update the product type if it was previously saved but has since changed. This prevents
fatal errors like that mentioned above but caused by switching from one product type, like a simple
product, to another, like a variable product.
To allow extensions which add their own variation bulk actions to pass
data back to WooCommerce core's handler so that only one ajax call
is necessary for updating the data.
Previously, a duplicate ajax call would be required as trigger() does
not capture the callbacks return value.
Peter Fabian
vedanshujain
Kathy Daring
Christopher Allford
Salva Machí
Rodrigo Primo
Gerhard
Mike Jolley
Caleb Burks
claudiulodro
Claudio Sanches
Umangvaghela
Ninos Ego
Sabbir Ahmed
roykho
Claudio Sanches
Florian Ludwig
Brent Shepherd