opencart icon indicating copy to clipboard operation
opencart copied to clipboard

[master] Order edit - Order data change/not change at refresh/confirm

Open pemaier opened this issue 1 year ago • 3 comments

What version of OpenCart are you reporting this for? master Describe the bug A clear and concise description of what the bug is What section does it affect? the order data is changed in part (store) it should not change and it not changes in parts (language, currency) it should change.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'setup an additional store'
  2. Prepare the category and product settings
  3. Login to new store and order a product
  4. Go to back end sales
  5. Open the order and change the currency
  6. press refresh and confirm
  7. close the order and reopen it again
  8. see the error

Expected behavior A clear and concise description of what you expected to happen. Correct data should be saved

Screenshots / Screen recordings If applicable, add screenshots or recordings to help explain your problem. Please keep these short and if you can, edit them to point out when/where the problem is. https://www.useloom.com/ is perfect for screen recording with Chrome or with their desktop app order-edit-store-language-curreny-change.webm

Server / Test environment (please complete the following information):

  • Local development? Deployed to a web server? server
  • Operating system linux
  • PHP version 8.2
  • Apache version 2.4
  • Browser(s) tested with [e.g. chrome, safari - if applicable] FF
  • Version [e.g. 22] 116.0

pemaier avatar Aug 30 '23 08:08 pemaier

the issue is people complain when the order totals are changed after saving

danielkerr avatar Jan 09 '24 16:01 danielkerr

If it is not allowed for the shop owner to change Store Language and Currency then a) it should not be possible for editing in the administration area (what I really not like as shop owner because then I have to delete the order and create a new order ... that response we really don't like) b) it should be a configuration setting so the shop owner can decide what he need

I guess it only affects automatic payment/refund systems like PayPal... what we already had in the older OpenCart versions. So variant b) is a good compromise and a clear process for all shop owners.

And btw. which shop owner is changing settings of an order if not the customer ask for it? In each way to change the store and language must be changeable because it is reporting relevant.

pemaier avatar Jan 14 '24 10:01 pemaier

i think i need to redo how api calls are made to create orders.

danielkerr avatar Apr 26 '24 19:04 danielkerr