Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035571Retail ModulesWeb POSpublic2017-03-20 13:172017-08-03 15:46
Reportermalsasua 
Assigned Toranjith_qualiantech_com 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q3 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035571: wrong payment term in sales invoice created from WebPOS using credit
Descriptionwhen a receipt is completed using credit in WEBPOS, the sales invoice is generated automatically.

If the invoice payment term is complex (more than one payment date), the payment details related with sales invoice are wrong.
Steps To Reproduce1. go to business partner window, and select VBS Customer:
  change the payment term to: "50% en el acto, resto a 60 días"
2. in WebPOS (VBS-1) create a Credit Sale
3. check the sales invoice payment plan due date, expected date and see that these are set to the invoice date + 0 days. This is incorrect.
4. expected payments are:
  invoice date + 0 days total amount* 50%
  invoice date + 60 days total amount* 50%
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0033830 closed ranjith_qualiantech_com Payment methods for Credit Sales not working 
related to defect 0036907 closed ranjith_qualiantech_com Wrong sales invoice payment plan is created when paying a ticket using credit 
causes defect 0036841 closed ranjith_qualiantech_com Should not be necessary to have "Maturity Date 2" defined in Payment Terms defined as "Fixed Due date" 
causes defect 0037493 closed ranjith_qualiantech_com Error using credit in return receipts with complex payment terms 
Attached Filespatch Issue 35571 - 3.0RR17Q1 source posterminal.patch (11,961) 2017-08-03 15:46
https://issues.openbravo.com/file_download.php?file_id=10964&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-03-20 13:17malsasuaNew Issue
2017-03-20 13:17malsasuaAssigned To => Retail
2017-03-20 13:17malsasuaTriggers an Emergency Pack => No
2017-03-20 13:17malsasuaReproducibilityhave not tried => always
2017-03-20 13:46malsasuaRelationship addedrelated to 0033830
2017-04-20 15:44ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2017-04-20 15:44ranjith_qualiantech_comStatusnew => scheduled
2017-04-27 10:07hgbotCheckin
2017-04-27 10:07hgbotNote Added: 0096213
2017-04-27 10:07hgbotStatusscheduled => resolved
2017-04-27 10:07hgbotResolutionopen => fixed
2017-04-27 10:07hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c03b70eb5d2a2b7a9a396ae49a36eb434b48c4b [^]
2017-04-27 10:08hgbotCheckin
2017-04-27 10:08hgbotNote Added: 0096214
2017-04-27 11:56guilleaerNote Added: 0096218
2017-04-27 11:56guilleaerStatusresolved => new
2017-04-27 11:56guilleaerResolutionfixed => open
2017-04-27 11:57guilleaerStatusnew => scheduled
2017-05-02 16:04hgbotCheckin
2017-05-02 16:04hgbotNote Added: 0096319
2017-05-02 16:05hgbotCheckin
2017-05-02 16:05hgbotNote Added: 0096320
2017-05-02 16:05hgbotStatusscheduled => resolved
2017-05-02 16:05hgbotResolutionopen => fixed
2017-05-02 16:05hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c03b70eb5d2a2b7a9a396ae49a36eb434b48c4b [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/899b962a7512dece49049dd35b374a20ba88df40 [^]
2017-05-04 14:48ranjith_qualiantech_comNote Added: 0096384
2017-05-04 14:48ranjith_qualiantech_comStatusresolved => new
2017-05-04 14:48ranjith_qualiantech_comResolutionfixed => open
2017-05-04 14:48ranjith_qualiantech_comStatusnew => scheduled
2017-05-05 07:27hgbotCheckin
2017-05-05 07:27hgbotNote Added: 0096406
2017-05-05 07:27hgbotStatusscheduled => resolved
2017-05-05 07:27hgbotResolutionopen => fixed
2017-05-05 07:27hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/899b962a7512dece49049dd35b374a20ba88df40 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6835a0e453ce2af7e1798ef9bb7e3a4c78a2f46b [^]
2017-06-22 12:27guilleaerReview Assigned To => guilleaer
2017-06-22 12:27guilleaerStatusresolved => closed
2017-06-22 12:27guilleaerFixed in Version => RR17Q3
2017-08-03 15:46ranjith_qualiantech_comFile Added: Issue 35571 - 3.0RR17Q1 source posterminal.patch
2017-09-12 17:24maiteRelationship addedcauses 0036841
2017-09-20 18:08ngarciaRelationship addedrelated to 0036907
2018-01-31 17:36dmitry_mezentsevRelationship addedcauses 0037493

Notes
(0096213)
hgbot   
2017-04-27 10:07   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6c03b70eb5d2a2b7a9a396ae49a36eb434b48c4b
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Apr 27 13:34:49 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c03b70eb5d2a2b7a9a396ae49a36eb434b48c4b [^]

Fixes issue 35571 : Split Payment Schedule based on Payment terms lines

* If Payment terms are splited, then invoice payment terms should be splited based on payment terms

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0096214)
hgbot   
2017-04-27 10:08   
Repository: tools/automation/pi-mobile
Changeset: 2ea7ea5a7729c0c47474cb9a2ba97daeacb6f893
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Apr 27 13:34:45 2017 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/2ea7ea5a7729c0c47474cb9a2ba97daeacb6f893 [^]

Verifies issue 35571 : Added automated test 'I35571_VerifyReceiptWithMutiPaymentTerm'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I35571_VerifyReceiptWithMutiPaymentTerm.java
---
(0096218)
guilleaer   
2017-04-27 11:56   
With the example used in the automatic test the result should be:

2 -> paid -> today
1.90 -> Pending -> today (because first schedule says 0 days)
3.90 -> Pending -> Calculated day in the future based on payment terms

instead of

2 -> paid -> today
2.90 -> pending -> today
2.90 -> pending -> future

As you can see amounts are wrong, because they are calculated based on pending amount instead of total amount.

total amount: 7.80
50% -> 3.90

paid: 2
pending: 5.80

Currently (wrong)
50% of pending 5.80 = 2.90
Expected
50% of total amount 7.80 = 3.90
difference to reach 50% of total: 1.90
50% pending = 3.90
(0096319)
hgbot   
2017-05-02 16:04   
Repository: tools/automation/pi-mobile
Changeset: b374a421cd9ea5c31c79785979b4b17539eceea3
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue May 02 19:31:28 2017 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/b374a421cd9ea5c31c79785979b4b17539eceea3 [^]

Verifies issue 35571 : Modified automated test 'I35571_VerifyReceiptWithMutiPaymentTerm'

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I35571_VerifyReceiptWithMutiPaymentTerm.java
---
(0096320)
hgbot   
2017-05-02 16:05   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 899b962a7512dece49049dd35b374a20ba88df40
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue May 02 19:34:00 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/899b962a7512dece49049dd35b374a20ba88df40 [^]

Fixes issue 35571 : Changes in spliting Payment Schedule based on Payment terms

* First schedule amount should be included the payment amount

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0096384)
ranjith_qualiantech_com   
2017-05-04 14:48   
While fetching payment terms lines, it should be ordered with lineno
(0096406)
hgbot   
2017-05-05 07:27   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6835a0e453ce2af7e1798ef9bb7e3a4c78a2f46b
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Fri May 05 10:56:39 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6835a0e453ce2af7e1798ef9bb7e3a4c78a2f46b [^]

Fixes issue 35571 : Added orderBy clause for paymentTermLine in OrderLoader

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---