Openbravo Issue Tracking System - Retail Modules
View Issue Details
0031471Retail ModulesWeb POSpublic2015-11-13 14:202015-12-18 09:26
agete 
guilleaer 
highmajoralways
closedfixed 
5
 
RR16Q1 
marvintm
No
0031471: Issue when paying in WEB POS a layaway that was invoiced in backend. The sales order is invoiced again.
Issue when paying in WEB POS a layaway that was invoiced in backend. The invoice is not paid.
CASE 1:
1. Create a layaway in POS and do not pay anything
2. Invoice the order in the backend
3. Pay the layaway in POS, do not uncheck the Invoice opcion
4. Check in backend:
- the order has been paid
- ERROR: the invoiced initially created is pending to be paid
- ERROR: a new invoice has been created, linked to the order, and paid
The synchronization should check if the order already has an invoice created in order not to duplicate it.

CASE 2.
1. Create a layaway in POS and do not pay anything
2. Invoice the order in the backend
3. Pay the layaway in POS, uncheck the Invoice opcion
4. Check in backend:
- the order has been paid
- ERROR: the invoiced initially created is pending to be paid
The synchronization should check if the order already has an invoice created in order also pay it.
No tags attached.
related to defect 0031462 closed mario_castello [RR15Q4] The status of a layaway does not change after 'Void this layaway' 
related to defect 0033179 closed Sandrahuguet OrderLoader.getInvoicesRelatedToOrder skip this function if known that no invoices can exist 
diff issue_invoices.diff (3,031) 2015-11-21 16:26
https://issues.openbravo.com/file_download.php?file_id=8760&type=bug
Issue History
2015-11-13 14:20ageteNew Issue
2015-11-13 14:20ageteAssigned To => Retail
2015-11-13 14:20ageteResolution time => 1448578800
2015-11-13 14:20ageteTriggers an Emergency Pack => No
2015-11-13 14:35ageteDescription Updatedbug_revision_view_page.php?rev_id=10241#r10241
2015-11-13 14:35ageteSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=10243#r10243
2015-11-13 15:26OrekariaRelationship addedrelated to 0031462
2015-11-13 15:26OrekariaStatusnew => acknowledged
2015-11-17 10:37guilleaerAssigned ToRetail => guilleaer
2015-11-17 12:20guilleaerStatusacknowledged => scheduled
2015-11-21 16:26guilleaerFile Added: issue_invoices.diff
2015-11-21 16:27guilleaerNote Added: 0082041
2015-11-23 12:09hgbotCheckin
2015-11-23 12:09hgbotNote Added: 0082071
2015-11-23 12:09hgbotStatusscheduled => resolved
2015-11-23 12:09hgbotResolutionopen => fixed
2015-11-23 12:09hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d32d2a046a97760af7e67b6f735b9d7705e9277f [^]
2015-12-02 10:45OrekariaReview Assigned To => guilleaer
2015-12-02 10:46OrekariaReview Assigned Toguilleaer => marvintm
2015-12-02 16:27hgbotCheckin
2015-12-02 16:27hgbotNote Added: 0082540
2015-12-04 16:01marvintmStatusresolved => closed
2015-12-04 16:01marvintmFixed in Version => RR16Q1
2015-12-15 20:06hgbotCheckin
2015-12-15 20:06hgbotNote Added: 0082786
2015-12-15 20:08hgbotCheckin
2015-12-15 20:08hgbotNote Added: 0082787
2015-12-18 09:26hgbotCheckin
2015-12-18 09:26hgbotNote Added: 0082863
2016-06-07 18:13shuehnerRelationship addedrelated to 0033179

Notes
(0082041)
guilleaer   
2015-11-21 16:27   
Attached a diff which resolve the basic flow. It have passed try.

A new issue will be crated to fix a more complex flow.
(0082071)
hgbot   
2015-11-23 12:09   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: d32d2a046a97760af7e67b6f735b9d7705e9277f
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Mon Nov 23 12:09:02 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d32d2a046a97760af7e67b6f735b9d7705e9277f [^]

Fixed issue 31471: OrderLoader will check if any invoice is already present before create a new invoice

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0082540)
hgbot   
2015-12-02 16:27   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 09a201bc283ce3d8c3e4dede91186ab24f0a140a
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Wed Dec 02 13:37:03 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/09a201bc283ce3d8c3e4dede91186ab24f0a140a [^]

Related to issue 31471: This flow will work fine also if invoice has been already completed

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0082786)
hgbot   
2015-12-15 20:06   
Repository: erp/pmods/org.openbravo.retail.sampledata
Changeset: fa4d879d036ee092d1a63b85f8654dc3e1dc6f16
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Dec 15 00:15:43 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sampledata/rev/fa4d879d036ee092d1a63b85f8654dc3e1dc6f16 [^]

Related to issue 31471: Change sample dato to cover new test

---
M referencedata/sampledata/The_White_Valley_Group/FIN_FINACC_PAYMENTMETHOD.xml
M referencedata/sampledata/The_White_Valley_Group/FIN_FINANCIAL_ACCOUNT.xml
M referencedata/sampledata/The_White_Valley_Group/OBPOS_APPLICATIONS.xml
M referencedata/sampledata/The_White_Valley_Group/OBPOS_APP_PAYMENT.xml
M referencedata/sampledata/The_White_Valley_Group/OBPOS_APP_PAYMENT_TYPE.xml
M referencedata/sampledata/The_White_Valley_Group/OBPOS_TERMINALTYPE.xml
---
(0082787)
hgbot   
2015-12-15 20:08   
Repository: tools/automation/pi-mobile
Changeset: 7e75ffbf51d2ab3ef95ce0184f1dff8398eefa71
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Dec 15 13:34:49 2015 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/7e75ffbf51d2ab3ef95ce0184f1dff8398eefa71 [^]

Related to issue 31471: Add new test

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I31471_PayLayawayAlreadyInvoiced.java
---
(0082863)
hgbot   
2015-12-18 09:26   
Repository: tools/automation/pi-mobile
Changeset: 2e9bec353826f7fb56853074fead494ab36db552
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Wed Dec 16 16:39:44 2015 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/2e9bec353826f7fb56853074fead494ab36db552 [^]

Related to issue 31471, 31503: add code to test issue 31471

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I31471_PayLayawayAlreadyInvoiced.java
---