Closed (fixed)
Project:
Commerce Reporting
Version:
7.x-3.0-alpha4
Component:
User interface
Priority:
Major
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
7 Sep 2012 at 07:48 UTC
Updated:
8 Jun 2014 at 19:58 UTC
Jump to comment: Most recent
Comments
Comment #1
cvangysel commentedThe tax reporting module has completely changed in the current version in the repository; the code that caused this bug doesn't exist anymore.
I'm planning on releasing a beta version soon, I hope it'll fix your problems.
Thanks for the report!
Comment #2
steve.m commentedLooks like this isn't actually fixed.
It still happens in alpha4 and the -dev versions, when visiting the admin/commerce/reports dashboard page. It happens without the tax reporting module enabled, so it's something else.
Comment #3
steve.m commentedIt looks like the SQL statements at lines 46 and 79 can be cleaned up by adding 'GROUP BY currency_code'. It might be necessary to change the reporting logic, however, to handle the case of a store selling in multiple currencies.
Comment #4
steve.m commented(I'm taking the liberty of opening this issue again for review.)
Comment #5
cvangysel commentedRight, no idea why I thought this was related to the tax reporting module. There should indeed be a GROUP BY part in those queries.
Anyhow, I fixed it in the HEAD of the 7.x-3.x branch.
Thanks!