Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035827Retail ModulesWeb POSpublic2017-01-05 09:562017-05-12 16:25
Reporterguillermogil 
Assigned Tojorge-garcia 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q1.1Fixed in VersionRR17Q1.1 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customerOBPS
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035827: 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 => RR17Q1.1
2017-04-26 11:42hgbotCheckin
2017-04-26 11:42hgbotNote Added: 0096192
2017-04-26 11:42hgbotStatusscheduled => resolved
2017-04-26 11:42hgbotResolutionopen => fixed
2017-04-26 11:42hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/386a2fd3627e1fec325b3fff8bbca53b71918437 [^] => http://code.openbravo.com/retail/backports/3.0RR17Q1.1/org.openbravo.mobile.core/rev/026f3d0c709291d146c8b6feca3db3f778bf28e8 [^]
2017-04-26 11:51hgbotCheckin
2017-04-26 11:51hgbotNote Added: 0096193
2017-04-26 18:19guilleaerReview Assigned Tomarvintm => guilleaer
2017-04-26 18:19guilleaerStatusresolved => closed
2017-04-26 18:19guilleaerFixed in Version => RR17Q1.1
2017-05-04 09:25hgbotCheckin
2017-05-04 09:25hgbotNote Added: 0096367
2017-05-12 16:25hgbotCheckin
2017-05-12 16:25hgbotNote Added: 0096572

Notes
(0096192)
hgbot   
2017-04-26 11:42   
Repository: retail/backports/3.0RR17Q1.1/org.openbravo.mobile.core
Changeset: 026f3d0c709291d146c8b6feca3db3f778bf28e8
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 26 11:41:25 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR17Q1.1/org.openbravo.mobile.core/rev/026f3d0c709291d146c8b6feca3db3f778bf28e8 [^]

Fixed issue 35827: 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
---
(0096193)
hgbot   
2017-04-26 11:51   
Repository: retail/backports/3.0RR17Q1.1/org.openbravo.retail.posterminal
Changeset: 749082662ce205f76755fa6f7660b0d8168c9ea3
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 26 11:40:37 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR17Q1.1/org.openbravo.retail.posterminal/rev/749082662ce205f76755fa6f7660b0d8168c9ea3 [^]

Related to issue 35827: 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
---
(0096367)
hgbot   
2017-05-04 09:25   
Repository: retail/backports/3.0RR17Q1.1/pi-mobile
Changeset: 67a60a718cfd16d3adbbb889f12bca4687d9eac8
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue May 02 13:54:06 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR17Q1.1/pi-mobile/rev/67a60a718cfd16d3adbbb889f12bca4687d9eac8 [^]

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

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

---
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sales/DuplicatedTicketAreSavedProperly.java
---
(0096572)
hgbot   
2017-05-12 16:25   
Repository: tools/automation/pi-mobile
Changeset: 67a60a718cfd16d3adbbb889f12bca4687d9eac8
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue May 02 13:54:06 2017 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/67a60a718cfd16d3adbbb889f12bca4687d9eac8 [^]

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

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

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