chuckmac / woocommerce-location-report Goto Github PK

Environment:

• Server info: nginx/1.13.1
  PHP version: 5.6.30-11+deb.sury.org~trusty+3
  PHP post max size: 8 MB
  PHP time limit: 30
  PHP max input vars: 1000
  cURL version: 7.35.0, OpenSSL/1.0.2k
  SUHOSIN installed: –
  MySQL version: 5.6.33
  Max upload size: 2 MB
  Default timezone is UTC:
  fsockopen/cURL: ok
  SoapClient: ok
  DOMDocument: ok
  GZip: ok
  Multibyte string: ok
  Remote post: ok
  Remote get: ok
  Woocommerce: 3.1.0
  Wordpress: 4.6
  WooCommerce Sales By Location Report: 1.3.1

When I buy a subscription / product, I got this message in log file

PHP Strict Standards: Non-static method WC_Location_Report::init() should not be called statically in /var/www/xxx/wp-content/plugins/woocommerce-sales-by-location-report/woocommerce-location-report.php on line 161

I just downloaded and installed the latest from the WP repository, and tested this on a live system. The totals are not anywhere near correct. This is easy to see by comparing a regular WC standard Sales By Date report to a Sales by Country from your plugin using the same date ranges in both reports. Incredibly far off. Not sure why, but you may want to find out and fix that. I don't have time to dig into the queries myself.

What you might do is simply copy the queries directly from the WC core, the latest 2.3.x tree .

Add the functionality to export the report data in csv. This can follow the same convention as the other core WC reports.

Would it be possible to add a checkbox or other solution to allow for order totals to exclude shipping? Maybe a separate one doing the same for tax?