Barry Hughes

Results 204 comments of Barry Hughes
trafficstars

Thanks for flagging this, @KevinBrent. I'm not sure this will actually lead to any concrete problems because `order_` is effectively ignored (and an order value of `0` will then be...

OK, that sounds fair—and, if you wish to submit a PR that introduces greater consistency, we'll be happy to review. From our perspective, though, since I don't think there is...

*Some notes on performance...* ✍🏽 Switching from `$wpdb->update()` to `wp_update_post()` also solves this, because that takes care of invalidating various caches, etc, but with a large catalog the performance impact...

Thanks for the question, @bazza. > Switching from $wpdb->update() to wp_update_post() also solves this I should start by acknowledging that, by itself, this is a very naive approach (because, if...

@bazza let me know if any other questions/concerns on this one.

Thanks for reporting this problem, @dnwjn. We can't replicate, however, and I wanted to confirm a few things: - Can you replicate when only WooCommerce and a default theme are...

Merged (via https://github.com/woocommerce/woocommerce/pull/34472 ... same set of commits, but we added a changelog entry).

Thanks for flagging this, @iamdharmesh. If you have time to submit a PR that would be great, otherwise we'll get to it in an upcoming dev cycle :+1:

Thanks for the snippet, @strarsis. ```php $special_rate = new WC_Shipping_Rate( 'flat_rate_special', __( 'Special flat rate', 'custom-plugin' ), strval($special_flatrate_price), array( $special_flatrate_tax ), #

✍🏽 So we don't forget ... once done we can probably update the way the `ListTable` determines order counts by status and improve efficiency there (I'm happy to follow-up with...