Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033256Retail ModulesWeb POSpublic2016-06-14 13:052016-06-21 13:47
Reporterumartirena 
Assigned Toranjith_qualiantech_com 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033256: Delivered property is not properly updated on certain cases when creating a ticket in Web POS.
DescriptionDelivered property is not properly updated on certain cases when creating a ticket in Web POS. If a normal ticket is created with a product, it is paid and completed (and shipped), delivered property still remains as 'N'. However, when a layaway is created and later on fully paid, it is properly updated.
Steps To ReproduceIn Web POS:
 * Add a product to a ticket.
 * Pay the ticket.
 * Click on Done.
 * In database search that ticket on c_order table.
 * Check that 'isDelivered' column is 'N' even the order has been shipped, when it should be 'Y'.

This same case for a layaway behaves correctly:
 * Create a layaway in Web POS.
 * Add a product.
 * Click on Layaway and synchronize the layaway.
 * Retrieve the layaway from backend.
 * Pay the layaway completely.
 * Click on Done.
 * In database search that layaway on c_order table.
 * Check that 'isDelivered' column is 'Y' in this case, so is correct.
Proposed Solution
Additional Information
TagsSER-QA
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-06-14 13:05umartirenaNew Issue
2016-06-14 13:05umartirenaAssigned To => Retail
2016-06-14 13:05umartirenaOBNetwork customer => Yes
2016-06-14 13:05umartirenaResolution time => 1467064800
2016-06-14 13:05umartirenaTriggers an Emergency Pack => No
2016-06-14 13:05umartirenaTag Attached: SER-QA
2016-06-20 10:00ranjith_qualiantech_comStatusnew => acknowledged
2016-06-20 10:14ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-06-20 10:14ranjith_qualiantech_comStatusacknowledged => scheduled
2016-06-21 05:54hgbotCheckin
2016-06-21 05:54hgbotNote Added: 0087691
2016-06-21 05:54hgbotCheckin
2016-06-21 05:54hgbotNote Added: 0087692
2016-06-21 05:54hgbotStatusscheduled => resolved
2016-06-21 05:54hgbotResolutionopen => fixed
2016-06-21 05:54hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/bc4c29a8b5affb42c54d061bff465955909f4936 [^]
2016-06-21 13:47marvintmReview Assigned To => marvintm
2016-06-21 13:47marvintmStatusresolved => closed

Notes
(0087691)
hgbot   
2016-06-21 05:54   
Repository: tools/automation/pi-mobile
Changeset: 7426867cd21d21dabeeadc0a507fb2bde74c2d83
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 21 09:23:35 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/7426867cd21d21dabeeadc0a507fb2bde74c2d83 [^]

Verifies issue 33256 : Added automated test 'I33256_VerifyOrderIsDelivered'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I33256_VerifyOrderIsDelivered.java
---
(0087692)
hgbot   
2016-06-21 05:54   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: bc4c29a8b5affb42c54d061bff465955909f4936
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 21 06:57:56 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/bc4c29a8b5affb42c54d061bff465955909f4936 [^]

Fixes issue 33256 : Added order delivered status in OrderLoader

- For normal, return, fullypaid layaway receipt, order delivered status has been set as 'Y'. for others it has been set as 'N'
- Removed some duplicated code for isQuotation

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