Skip to content
This repository has been archived by the owner on Jun 15, 2022. It is now read-only.

Reports totals calculate still on postmeta values #17

Closed
lukecav opened this issue Dec 27, 2017 · 0 comments
Closed

Reports totals calculate still on postmeta values #17

lukecav opened this issue Dec 27, 2017 · 0 comments
Labels

Comments

@lukecav
Copy link
Member

lukecav commented Dec 27, 2017

In WooCommerce reporting, the totals are still calculated based on postmeta fields. The public function get_order_report_data( $args = array() ) method on the class WC_Admin_Report are not designed yet (I believe) to take these calculations to a separate table, instead of postmeta.

Steps to reproduce:

  1. Create a new order
  2. Set the order to completed so the order is taken into account with reporting
  3. Flush transients to make sure resulting report is not cached.
  4. See the reports totals are not updated with new order

Extra verification steps

  1. Run the CLI backfill command
  2. Flush transients again to kill cached reports
  3. See the reports totals are now updated with the new order totals

Approach

My best guess is that this needs WooCommerce core enhancements, but I have to dive in a little deeper to make sure this is the case.

@lukecav lukecav added the bug label Dec 27, 2017
@lukecav lukecav closed this as completed Dec 27, 2017
@liquidweb liquidweb locked as off-topic and limited conversation to collaborators Dec 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant