Sales report 2.0

[southshorepizza]

@carlvt88 $36.32 included a $3.00 tip. But that isn't an issue as this happens with or without the tip being involved.

I guess what I'm not following is in order confirmation it looks as though includes/classes/order.php that saves the order and generates the emailed receipt saves everything to table_order_totals But the SR does not use that instead it uses table_orders as shown in admin/includes/classes/sales_report.php. It looks at each ordered product and then recalculates the subtotals. However it appears to be missing something.

[aalthouse]

Thanks for this module; it has helped me with my reporting needs quite a bit and saved me from running SQL against the db. Here's a bug report and a request:

v1.5 original (no upgrade)
When exporting the report as a CSV, it appears to drop the "Goods Tax" column header. (I just need to add one space to the right in the header row to bring it in-line as a workaround.) I tested it on Chrome and IE and it seems to be consistent. I searched this thread to see if anyone else has reported it and didn't find anything, so sorry if this is a known issue.

Also it would be nice to have a way to omit specific order statuses, such as Cancelled from being returned, similar to the way you can choose to omit a payment method.

Regards,

Aaron

[southshorepizza]

I don't get any canceled orders returned. For some reason all deleted or orders with a $0.00 total are skipped.

Also this report does not match the emailed receipt on a consistent basis. Usually the report is off $15-20 for every 30 orders.

[DivaVocals]

Anyone having issues with the "Last Year" option displaying THIS YEAR (2014) and NOT 2013??

[lhungil]

Anyone having issues with the "Last Year" option displaying THIS YEAR (2014) and NOT 2013??

I can confirm, I'm seeing "THIS YEAR (2014)" with Sales Report version 3.1.0 under Zen Cart 1.5.1.

[DivaVocals]

I can confirm, I'm seeing "THIS YEAR (2014)" with Sales Report version 3.1.0 under Zen Cart 1.5.1.

Well I'm stumped on a fix.. (tried a few things.. none worked) **sigh**

[carlvt88]

Found the error - needs day and date in the mktime command:
stats_sales_report.php line 532:

sprintf(SEARCH_DATE_LAST_YEAR, date("Y", mktime(0,0,0,1,1,date("y")-1)));

[DivaVocals]

Found the error - needs day and date in the mktime command:
stats_sales_report.php line 532:

sprintf(SEARCH_DATE_LAST_YEAR, date("Y", mktime(0,0,0,1,1,date("y")-1)));

I found this code in line 457 not 532.. Made the change and it made no difference..

              <td class="smallText" id="td_last_year"><?php echo zen_draw_radio_field('date_preset', 'last_year', false) . sprintf(SEARCH_DATE_LAST_YEAR, date("Y", mktime(0,0,0,1,1,date("y")-1))); ?></td>

[carlvt88]

@aalthouse - I can see where the tax column is messed up. In the function output_csv, if there is any tax, the variable $display_tax is set to true. Now, when the columns are printed out an extra header is included:

            if ($display_tax) echo TABLE_HEADING_TAX . CSV_SEPARATOR;

but two extra data are written:

              if ($display_tax) {
                echo $o_data['goods_tax'] . CSV_SEPARATOR;
                echo $o_data['order_recorded_tax'] . CSV_SEPARATOR;
              }

Just need to add both elements to the header:

            if ($display_tax) {
                echo TABLE_HEADING_TAX . CSV_SEPARATOR;
                echo TABLE_HEADING_ORDER_RECORDED_TAX . CSV_SEPARATOR;
            }

Slightly different for the type of report detail (timeframe, order, or product), but the same concept holds for all.

[carlvt88]

While I was looking at it, I found another problem in the csv data - if you're looking at 'timeframe' level of detail, the taxes aren't written to the csv file. In sales_report.php line 992, add 'echo' to these lines:

            if ($display_tax) {
              echo $timeframe['total']['goods_tax'] . CSV_SEPARATOR;
              echo $timeframe['total']['order_recorded_tax'] . CSV_SEPARATOR;
            }