Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0036481
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Retail Modules] Web POSmajoralways2017-07-12 18:512017-07-20 15:00
ReporterkchoperenaView Statuspublic 
Assigned Tomtaal 
PriorityurgentResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revision2cd793c01497
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomigueldejuana
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0036481: [STORE SERVER 3094]: A fully paid layaway has isLayaway = N

DescriptionWhen we create an order using ExternalOrderLoader, if the layaway is fully paid it is never detected.

If we pay fully when the execution achieve the orderLoader, the fullypaidLayaway variable is always false because never satisfy the next condition:

fullypaidLayaway = (jsonorder.getBoolean("isLayaway") || jsonorder.optLong("orderType") == 2)
        && Math.abs(jsonorder.getDouble("payment")) >= Math.abs(jsonorder.getDouble("gross"));


This happens because ExternalOrderLoader is setting property payment to value -1
Steps To Reproduce1.- Create a layaway using EcommerceCreateOrderWS
2.- Create a payment using EcommerceCreatePaymentWS

EXPECTED RESULTS:

The flag isLayaway will be N, because payment is not detected.
Proposed SolutionIt is needed to set the payment property from the received JSON.

Order Loader should be able to avoid update delivered quantity when not generated shipment.

It is needed that the ExternalOrderLoader class, add the previous paid quantity (should get it from DB) in order to detect that the layaway has been completely paid.

Attached patch with suggested solution.

TagsNo tags attached.
Attached Filesdiff file icon patchPosTerminal.diff [^] (3,258 bytes) 2017-07-14 12:08 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0098141)
hgbot (developer)
2017-07-20 09:41

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 2cd793c0149767c5dc67d24741bc8fc2f10782f1
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Jul 20 09:41:27 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2cd793c0149767c5dc67d24741bc8fc2f10782f1 [^]

Fixes issue 36481: [STORE SERVER 3094]: A fully paid layaway has isLayaway = N
Compute the value of the payments property in the ExternalOrderLoader
OrderLoader: set delivered and deliveredquantity in the createshipment logic

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0098153)
hgbot (developer)
2017-07-20 14:59

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 4a38031eaf8572a56b41642df630681c4702c1b0
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Jul 20 14:59:13 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4a38031eaf8572a56b41642df630681c4702c1b0 [^]

Related to issue 0036481: [STORE SERVER 3094]: A fully paid layaway has isLayaway = N

Backed out changeset 2cd793c01497

It is breaking: CancelAndRplcAmountIncreased, CancelAndRplcOfCancelAndRplc, I35955_CancelAndRplcFromCredit, CancelAndRplcOfCancelAndRplc, I35955_CancelAndRplcFromCredit

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0098154)
migueldejuana (developer)
2017-07-20 15:00

It is breaking: CancelAndRplcAmountIncreased, CancelAndRplcOfCancelAndRplc, I35955_CancelAndRplcFromCredit, CancelAndRplcOfCancelAndRplc, I35955_CancelAndRplcFromCredit

- Issue History
Date Modified Username Field Change
2017-07-12 18:51 kchoperena New Issue
2017-07-12 18:51 kchoperena Assigned To => mtaal
2017-07-12 18:51 kchoperena Triggers an Emergency Pack => No
2017-07-14 11:20 jonibc Assigned To mtaal =>
2017-07-14 11:20 jonibc Category StoreServer => Web POS
2017-07-14 11:20 jonibc Description Updated View Revisions
2017-07-14 11:20 jonibc Steps to Reproduce Updated View Revisions
2017-07-14 11:20 jonibc Proposed Solution updated
2017-07-14 11:28 jonibc Proposed Solution updated
2017-07-14 12:08 jonibc File Added: patchPosTerminal.diff
2017-07-17 09:37 jonibc Issue Monitored: jonibc
2017-07-19 18:24 mtaal Assigned To => mtaal
2017-07-20 09:41 hgbot Checkin
2017-07-20 09:41 hgbot Note Added: 0098141
2017-07-20 09:41 hgbot Status new => resolved
2017-07-20 09:41 hgbot Resolution open => fixed
2017-07-20 09:41 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2cd793c0149767c5dc67d24741bc8fc2f10782f1 [^]
2017-07-20 09:42 mtaal Review Assigned To => migueldejuana
2017-07-20 14:59 hgbot Checkin
2017-07-20 14:59 hgbot Note Added: 0098153
2017-07-20 15:00 migueldejuana Note Added: 0098154
2017-07-20 15:00 migueldejuana Status resolved => new
2017-07-20 15:00 migueldejuana Resolution fixed => open


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker