Openbravo Issue Tracking System - Retail Modules |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0047386 | Retail Modules | Send By Email | public | 2021-07-13 14:35 | 2021-07-22 18:48 |
|
| Reporter | andrea_lopez | |
| Assigned To | rqueralta | |
| Priority | urgent | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | | |
| Merge Request Status | |
| Review Assigned To | |
| OBNetwork customer | |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0047386: "Send E-mail" pop-up shows up after failure to finish payment. |
| Description | When using the 'Send email from web POS' module, create a ticket with a partial payment and try to finish it, the popup "Send email" is shown even though the ticket has not been completed and it should not been shown. |
| Steps To Reproduce | Install the Send email from web POS module (org.openbravo.retail.sendemail)
Configure an smtp server to be able to send emails (on the Client window)
Configure a customer to receive invoices via email, and configure the Touchpoint Type to send emails.
Create a ticket in Web POS, select a customer, add a product, and make a partial payment.
Click: Done and then Open.
An error popup will appear stating that is not possible to continue, click "Ok".
It should return to the ticket to either complete the payment or Delete the receipt, but instead shows the the "Send E- mail" popup. |
| Proposed Solution | The popup visibility is managed with a OBPOS_PreOrderSave hook in file:
org.openbravo.retail.sendemail/web/org.openbravo.retail.sendemail/source/obsmail_hook_preordersave.js
A simple solution is to add a check to var: shouldDialogBeDisplayed, to see if the order has a pending amount to pay: currentOrder.getPending(), it is already being done but only for layway (var paidLayaway). |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files |  0047386_20Q1_BUT.diff (1,085) 2021-07-14 14:19
https://issues.openbravo.com/file_download.php?file_id=16009&type=bug |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2021-07-13 14:35 | andrea_lopez | New Issue | |
| 2021-07-13 14:35 | andrea_lopez | Assigned To | => Retail |
| 2021-07-13 14:35 | andrea_lopez | Resolution time | => 1627941600 |
| 2021-07-13 14:35 | andrea_lopez | Triggers an Emergency Pack | => No |
| 2021-07-14 14:19 | andrea_lopez | File Added: 0047386_20Q1_BUT.diff | |
| 2021-07-14 14:20 | andrea_lopez | Note Added: 0130474 | |
| 2021-07-19 22:51 | rqueralta | Assigned To | Retail => rqueralta |
| 2021-07-19 22:51 | rqueralta | Status | new => scheduled |
| 2021-07-21 19:56 | hgbot | Note Added: 0130675 | |
| 2021-07-22 18:48 | hgbot | Resolution | open => fixed |
| 2021-07-22 18:48 | hgbot | Status | scheduled => closed |
| 2021-07-22 18:48 | hgbot | Note Added: 0130706 | |
| 2021-07-22 18:48 | hgbot | Note Added: 0130707 | |