I'm not sure exactly why it wasn't working, but it also was very nasty from using db-specific code. The .install file was also polluted with some db-specific functions. (which are now phased out using an upgrade path)

This patch removes all the junk, and as a side effect, the code actually works now :) yay!

CommentFileSizeAuthor
uc_reports.get_sales.patch5.5 KBcha0s

Comments

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Closed (duplicate)

Marking this as duplicate since I've included this patch into the one at #344890-9: Timezone causes off-by-one error in reports (use Drupal time instead of server's). There was a slight change I made to the use format_date() as explained in that post.