Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BO Order Page -> Payment adjustment #22977

Closed
matik4 opened this issue Jan 25, 2021 · 1 comment
Closed

BO Order Page -> Payment adjustment #22977

matik4 opened this issue Jan 25, 2021 · 1 comment
Labels
No change required Resolution: issue closed because expected as is

Comments

@matik4
Copy link

matik4 commented Jan 25, 2021

Describe the bug

Hello,

Just wanted to highlight a possible bug (as it looks intentional). On the order page in back office, if you edit the order in a way that the total paid changes - let's say I will apply free shipping - you will not be able to balance the total by adding a negative amount in payment details.

You might probably suggest refunding by using one of the payment modules, but in a case with card payments, merchants should not charge customer's card until the goods are shipped, therefore the charge is 'authorised' and not 'captured'. This allows to simply 'capture' the new adjusted total (since we only 'authorised' the amount, there's no refund).

Expected behavior

Allow negative value to be input as in Prestashop 1.6 for example.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Orders -> View -> Apply any discount to existing order to change the total -> Add 'Payment' with negative amount to balance the total

Screenshots

https://www.dropbox.com/sh/sepn0799nprj3ts/AADILK1YpNai7vPUFT6ednNka?dl=0

Additional information

  • PrestaShop version: 1.7.7.0
  • PHP version: 7.3

Kind Regards,
Mateusz

@hibatallahAouadni
Copy link
Contributor

Hello @matik4

You can't add negative value to the payment method. To balance an overdue by the clients, it is recommended to make a credit slip (See the original issue).

So, I close it cause it's not a bug.

Thanks!

@hibatallahAouadni hibatallahAouadni added the No change required Resolution: issue closed because expected as is label Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No change required Resolution: issue closed because expected as is
Projects
None yet
Development

No branches or pull requests

2 participants