Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035828Retail ModulesWeb POSpublic2017-01-05 09:562017-05-03 14:25
Reporterguillermogil 
Assigned Tojorge-garcia 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q4.3Fixed in VersionRR16Q4.3 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035828: DataSynchronization is not handling errors for opened layaways
DescriptionIf you try to pay totally an existing layaway and an exception is raised on OrderLoader there is no Error on Errors While Importing and it is not getting updated.
Only the following Warn is shown:
95b61838 410329 [Import Entry - 3] WARN org.openbravo.mobile.core.process.DataSynchronizationProcess - Record in process org.openbravo.retail.posterminal.OrderLoader duplicated with id: DA68B2C89B19CC48E347930EBCAE75CA Error not saved.
Steps To ReproduceLoad a Layaway with a payment on WebPOS
Close the period
Pay totally the layaway
Record is not saved
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0034853 closed jorge-garcia DataSynchronization is not handling errors for opened layaways 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-04-25 10:16jorge-garciaTypedefect => backport
2017-04-25 10:16jorge-garciaTarget Version => RR16Q4.3
2017-04-25 10:27hgbotCheckin
2017-04-25 10:27hgbotNote Added: 0096150
2017-04-25 10:27hgbotStatusscheduled => resolved
2017-04-25 10:27hgbotResolutionopen => fixed
2017-04-25 10:27hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/386a2fd3627e1fec325b3fff8bbca53b71918437 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4.3/org.openbravo.mobile.core/rev/5332ef7314bf1a5d958537fef12323c4cd31bf0b [^]
2017-04-25 10:36hgbotCheckin
2017-04-25 10:36hgbotNote Added: 0096151
2017-04-25 10:37hgbotCheckin
2017-04-25 10:37hgbotNote Added: 0096152
2017-04-25 10:38marvintmStatusresolved => closed
2017-04-25 10:38marvintmFixed in Version => RR16Q4.3
2017-05-03 14:25hgbotCheckin
2017-05-03 14:25hgbotNote Added: 0096350

Notes
(0096150)
hgbot   
2017-04-25 10:27   
Repository: retail/backports/3.0RR16Q4.3/org.openbravo.mobile.core
Changeset: 5332ef7314bf1a5d958537fef12323c4cd31bf0b
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Apr 25 10:26:45 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4.3/org.openbravo.mobile.core/rev/5332ef7314bf1a5d958537fef12323c4cd31bf0b [^]

Fixed issue 35828: DataSynchronization is not handling errors for opened
layaways

Additional duplicate id check is no longer necessary since the implementation
of import entry processor.

---
M src/org/openbravo/mobile/core/process/DataSynchronizationProcess.java
---
(0096151)
hgbot   
2017-04-25 10:36   
Repository: retail/backports/3.0RR16Q4.3/org.openbravo.retail.posterminal
Changeset: 9772a02924b98136d467a408b3e9b241f7137a2e
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Apr 25 10:26:10 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4.3/org.openbravo.retail.posterminal/rev/9772a02924b98136d467a408b3e9b241f7137a2e [^]

Related to issue 35828: DataSynchronization is not handling errors for opened
layaways

AdditionalCheckForDuplicates is no longer an extended function from
DataSynchronizationProcess.

---
M src/org/openbravo/retail/posterminal/ProcessCashClose.java
---
(0096152)
hgbot   
2017-04-25 10:37   
Repository: retail/backports/3.0RR16Q4.3/pi-mobile
Changeset: a4f5504c3496449974f520e09a654e6b5a2fd89e
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Apr 25 10:21:37 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4.3/pi-mobile/rev/a4f5504c3496449974f520e09a654e6b5a2fd89e [^]

Related to issue 35828: DataSynchronization is not handling errors for opened layaways

Test is no longer necessary due to the changes related with this issue

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sales/DuplicatedTicketAreSavedProperly.java
---
(0096350)
hgbot   
2017-05-03 14:25   
Repository: tools/automation/pi-mobile
Changeset: a4f5504c3496449974f520e09a654e6b5a2fd89e
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Apr 25 10:21:37 2017 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/a4f5504c3496449974f520e09a654e6b5a2fd89e [^]

Related to issue 35828: DataSynchronization is not handling errors for opened layaways

Test is no longer necessary due to the changes related with this issue

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sales/DuplicatedTicketAreSavedProperly.java
---