search

apertus-open-source-cinema / opencart_customer_country_report

Opencart 2.X report about number of orders and totals per country
GNU General Public License v3.0
1 stars 1 forks source link

logical error #1

Open andybehre opened 8 years ago

andybehre commented 8 years ago

hello,

your report has logical error. you select country from customer addresses instead of shipping addresses from orders.

think of the following example: one customer hast done 100 orders and got different addresses, one in germany, one in france and one in italy. your report will show 100 orders from germany, 100 orders from france and 100 orders from italy. the customer has always choosen the german address as shipping address.

So right report should only show 100 orders from germany (--> shipping address country)...

regards

andreas

se6astian-pi commented 8 years ago

thanks for the report! Good catch, I will look into it!

se6astian-pi commented 8 years ago

https://github.com/apertus-open-source-cinema/opencart_customer_country_report/commit/4a00eae57a850063fc688f2c341e20b44c006158 now uses order payment address as source. Please verify the issue is fixed this way.

andybehre commented 8 years ago

hello again. think now it's great!