Bug in Sales By Item CSV Export

OpenEMR Development
1

tmccormi wrote on Wednesday, March 11, 2015:

Rod,
in 4.2.1-dev the sales by item report you wrote will only export the Header line, no details. Got a quick fix?

2

sunsetsystems wrote on Wednesday, March 11, 2015:

Tony, not sure. I haven’t touched it since 2010, but Kevin did a bunch of updates for the IPPF update to 4.x so you might try their version. If that works we should port it over.

Rod
http://www.sunsetsystems.com/

3

yehster wrote on Wednesday, March 11, 2015:

These are the changes I made in 4.1 for IPPF and that report.

But the reality is that I simply ported the changes Rod made in the 3.x IPPF version to 4.1 mechanically, without truly understanding the changes.

4

visolveemr wrote on Thursday, March 12, 2015:

Hello Tony,

To export the reports in Sales by item report, you can make the following changes in regards with OpenEMR-4.2.0

Add ‘}’ in line 333
Add ‘if($_POST[‘form_refresh’]){’ after “if (! $_POST[‘form_csvexport’]) {” in line 475

Thanks
OpenEMR Customization/Support Team,
ViSolve Inc
services@visolve.com | Phone: 408-850-2243
Demo’s @ ViSolve Demo Library

5

tmccormi wrote on Thursday, March 12, 2015:

Thanks Guys!

6

tmccormi wrote on Friday, March 27, 2015:

This fix has been committed.

commit 82c1b0c0b87a3f34a7368123b4cf1b8d950157f7
Author: Tony McCormick tony@mi-squared.com
Date: Fri Mar 27 16:24:18 2015 -0700

Fix CSV Export - Visolve solution

M interface/reports/sales_by_item.php