thirtybees icon indicating copy to clipboard operation
thirtybees copied to clipboard

pdf invoice info/addresses are dynamically modified

Open kintaro1981 opened this issue 6 years ago • 6 comments

thirty bees 1.0.4 (upgraded following the blog article)

I changed the invoice address of the customer AFTER the order was created and the PDF invoice displays the updated address.

I think that it needs to display the OLD address. I think that invoice needs to be "STATIC", or at least with the possibility to be manually modified but not dynamically modified with updated customer info.

kintaro1981 avatar Apr 05 '18 11:04 kintaro1981

True. I am always surprised when old invoices change. What would you suggest? Lock an address once there are one or more orders with that address?

firstred avatar Apr 10 '18 16:04 firstred

Magento does this by capturing the invoice and shipping address in a different table the can be updated but is done by the admins. I think this is a good solution and am toying with doing it here too.

slvrspiral avatar Apr 14 '18 18:04 slvrspiral

A pretty failsafe solution would be to store the created PDF. More in general, all documents sent to the customer.

This should also meet legal archiving requirements. Changing such documents after the fact is a no-go, even when done by admins.

Traumflug avatar Apr 14 '18 19:04 Traumflug

That may work for your purposes, but my company had different requirements.

slvrspiral avatar Apr 14 '18 22:04 slvrspiral

Some has changed and we are grabbing it from the wrong spot now. in the _order_invoice table the addresses are taken at the time of the order and stored in html format for the invoices. That is where we should be grabbing them from.

Dh42 avatar Apr 22 '18 19:04 Dh42

is this fixed?

kintaro1981 avatar Mar 08 '22 18:03 kintaro1981