Re: Still seeking solution to bug in interchange reports

David Christensen <[email protected]> Sun, 4 Mar 2018 11:23:10 -0600
Newsgroups gmane.comp.web.interchange.users
Message-ID <[email protected]>

> On Mar 4, 2018, at 12:17 AM, Andrew Baerg <[email protected]> wrote:
> 
> On 2018-03-02 10:55 AM, Davideth wrote:
>> On 03/02/2018 10:22 AM, Jon Jensen wrote:
>>> On Thu, 1 Mar 2018, Davideth wrote:
>>> 
>>>> There appears to be a serious bug in the interchange reports for accounting purposes possibly more than one bug. I have reached out to the community several times but still have not received an answer.
>>>> 
>>>> When the monthly sales report is brought up, it includes canceled orders in the totals which is completely wrong. You can not count money that was never received or that was refunded in your total sales. This can create a tax nightmare if nothing else.
>>>> 
>>>> This problem appears to exist with all reports across the board and is a serious accounting flaw.
>>> 
>>> David,
>>> 
>>> Thanks for the report.
>>> 
>>> Just speaking for myself, I saw what you reported earlier, but I didn't have anything to add because it doesn't have any relevance for Interchange installations I work with.
>>> 
>>> Interchange is not an accounting system, and all serious ecommerce shops I know that are using it have either heavily customized Interchange reports replicate Interchange ecommerce data into a proper accounting system where they run their reports.
>>> 
>>> It sounds like having the reports include only shipped orders would be helpful for you, and I don't imagine it hurting any existing users. So if you would like to work up a fix and submit a patch or GitHub pull request, I think a fix would likely be accepted.
>>> 
>>> Jon
>>> 
>>> 
>> It was working in older versions, 4.8, 4.9 but is not working now. It may just be in using Strap rather than the standard interface. As for a fix, I do not have either time or funds at the moment. As for other users, I have a hard time seeing any e-commerce site working well without the reports. It may even be that some are using it and just do not realize the existence of the errors. I was not aware of the issue myself until after using the 5.10 version with Strap for several months.
>> 
>> 
> 
> I have submitted a pull request to resolve this - https://github.com/interchange/interchange/pull/109 <https://github.com/interchange/interchange/pull/109>
> 
> From what I can tell, the reports have always included canceled orders, so I don't think this is related to using Strap. The current reports will exclude any orders that have been 'deleted'.
> 
> Cheers!
> 
> Andrew

Hi Andrew,

Thanks for the submission; I’ve reviewed and merged this into HEAD; Davideth, you can either pull the changed files from the Git repo or review and make the same changes this patch made.

Best,

David
--
David Christensen
End Point Corporation
[email protected]
785-727-1171

_______________________________________________
interchange-users mailing list
[email protected]
http://www.icdevgroup.org/mailman/listinfo/interchange-users
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEECKA1l4YEfF4eK4/eN0ojyd+bZbgFAlqcK34ACgkQN0ojyd+b
Zbg4qhAAistxi4cUaIelznKjK/1UwPO3p6syeZybuxs4z+P9SyuHsd4boPxOvn/2
Ssr6E3sWjoHReLFlr4LtP1d/i1dg0hh7hzV9+xosqAC+qySXY7kvMup3uE3/c8sQ
AdHn/ySgrP3y2uQFyVscULAhSRwGV+m6HPEJ1lCbBlzyzgalC9zGOKAvYXsX34bw
MAGT/+NYsUGA3X9wLYtuTOvInguWzl/BjEgRJoaRlLIRjf4bESzlSPvWNDJPRk20
GTodmf/nbWQiP4y3zUQXiEsHUyq8TmpZeC0JPiOSB903o6HvQ6gUv6stQIJVMKW2
fKMgepVscBXbhrcKFop8KC6GJY3w9rT8wPFAm3wIADy8zJUudZy8I2qcpDK4O1lC
l6LVXZ70bLW3R7xfEY4yEWfc7SCVRGLNqoRUVkf3DvJeGbxYQNBoSgPv2VAyl+iV
5GxlmP8PBEYNQJ4FD61kdEF8u+D0bL58zEQNxzk50vfVh4f7GU/Ot7VZGaSPuy7f
OK8bPTcPvx+qAWmx/9RiYivzMGVkA8xtYuF0M5fWzWPE4v74lgxKsVBmWzeBWq1e
czg5o4+R1HbJy2/K6do6vS6HnKkj7kBdZMTY6OASy0w9nKu1BTotLqryexyWQAt4
y/anQeYg00+B8Cwzj0hfVrjhjDkZ1VWpjCd5hvDBnFZycZzeZH4=
=7hdd
-----END PGP SIGNATURE-----