Openbravo Issue Tracking System - Retail Modules
View Issue Details
0034305Retail ModulesWeb POSpublic2016-10-26 15:592016-11-22 08:40
prakashmurugesan88 
ranjith_qualiantech_com 
urgentmajorhave not tried
closedfixed 
5
 
 
migueldejuana
No
0034305: [SER-QA 1995] Error in the payment date - The system is recording the day -1 of the layaway payment
Error in the payment date - The system is recording the day -1 of the layaway payment
Steps to reproduce:
- Go to the webpos
- Create a layaway with a client and do NOT pay it
- In the menu, research the layaway and pay it partially
- In te menu, retrieve the layaway

Error: The payment date is wrong (day D-1)

Please check the video(PaymentDateErrorVideo.webm) in the attachment.
Please check the attachment
SER-QA
related to defect 0034466 closed ranjith_qualiantech_com Layaway payments not created with time offset 
related to defect 0036304 closed ranjith_qualiantech_com Date is displayed with one day less in receipts 
patch PaymentDateSuggestedFix.patch (959) 2016-10-26 15:59
https://issues.openbravo.com/file_download.php?file_id=9960&type=bug
? PaymentDateErrorVideo.webm (3,598,536) 2016-10-26 16:08
https://issues.openbravo.com/file_download.php?file_id=9961&type=bug
Issue History
2016-10-26 15:59prakashmurugesan88New Issue
2016-10-26 15:59prakashmurugesan88Assigned To => Retail
2016-10-26 15:59prakashmurugesan88File Added: PaymentDateSuggestedFix.patch
2016-10-26 15:59prakashmurugesan88Triggers an Emergency Pack => No
2016-10-26 16:08prakashmurugesan88Steps to Reproduce Updatedbug_revision_view_page.php?rev_id=13491#r13491
2016-10-26 16:08prakashmurugesan88File Added: PaymentDateErrorVideo.webm
2016-10-27 16:26adrianromeroResolution time => 1478732400
2016-10-27 16:26adrianromeroSummaryError in the payment date - The system is recording the day -1 of the layaway payment => [SER-QA 1995] Error in the payment date - The system is recording the day -1 of the layaway payment
2016-10-27 16:26adrianromeroTag Attached: SER-QA
2016-10-27 16:26adrianromeroPrioritynormal => urgent
2016-10-27 16:26adrianromeroSeverityminor => major
2016-11-01 07:17ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-11-01 07:17ranjith_qualiantech_comStatusnew => scheduled
2016-11-02 05:47hgbotCheckin
2016-11-02 05:47hgbotNote Added: 0091046
2016-11-02 05:47hgbotStatusscheduled => resolved
2016-11-02 05:47hgbotResolutionopen => fixed
2016-11-02 05:47hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/7b87a812e31fa2b80282d5ad9ebb206ccd548fff [^]
2016-11-02 05:48hgbotCheckin
2016-11-02 05:48hgbotNote Added: 0091047
2016-11-11 11:28migueldejuanaNote Added: 0091382
2016-11-11 11:28migueldejuanaStatusresolved => new
2016-11-11 11:28migueldejuanaResolutionfixed => open
2016-11-14 08:08ranjith_qualiantech_comStatusnew => scheduled
2016-11-16 05:37hgbotCheckin
2016-11-16 05:37hgbotNote Added: 0091587
2016-11-16 05:37hgbotStatusscheduled => resolved
2016-11-16 05:37hgbotResolutionopen => fixed
2016-11-16 05:37hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/7b87a812e31fa2b80282d5ad9ebb206ccd548fff [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/35e7b1777af4f1a3e1f656fdeb79b2c5a2071032 [^]
2016-11-22 08:36migueldejuanaRelationship addedrelated to 0034466
2016-11-22 08:40migueldejuanaReview Assigned To => migueldejuana
2016-11-22 08:40migueldejuanaNote Added: 0091725
2016-11-22 08:40migueldejuanaStatusresolved => closed
2017-06-22 07:57ranjith_qualiantech_comRelationship addedrelated to 0036304

Notes
(0091046)
hgbot   
2016-11-02 05:47   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 7b87a812e31fa2b80282d5ad9ebb206ccd548fff
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Nov 02 10:16:00 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/7b87a812e31fa2b80282d5ad9ebb206ccd548fff [^]

Fixes issue 34305 : Removed Receipt Payments Date validation for timezone

---
M web/org.openbravo.retail.posterminal/js/components/renderorderline.js
---
(0091047)
hgbot   
2016-11-02 05:48   
Repository: tools/automation/pi-mobile
Changeset: 4d95fad57aae82c5d30731ff9db3742c3059766b
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Nov 02 08:35:02 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/4d95fad57aae82c5d30731ff9db3742c3059766b [^]

Verifies issue 34305 : Added automated test 'I34305_VerifyLayawayPaymentDate'

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I34305_VerifyLayawayPaymentDate.java
---
(0091382)
migueldejuana   
2016-11-11 11:28   
Please, when removing code follow this:

- Check the commit and the issue related to removed lines
- Removing code affects to the functionality, be sure doing this, you probably should change code instead of removing.

Check this case and create a test if possible:

- Login with VBS-1
- Create a layaway with a customer and do NOT pay it
- In the menu, search the layaway and pay it partially
- Move to UTC -10(Honolulu) time
- Login with VBS1001
- Open Layaways search modal and look for above created layaway, it will have todays date
- Open this layaway and check payment date. ERROR! The date of the payment is yesterday
(0091587)
hgbot   
2016-11-16 05:37   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 35e7b1777af4f1a3e1f656fdeb79b2c5a2071032
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Nov 15 16:22:57 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/35e7b1777af4f1a3e1f656fdeb79b2c5a2071032 [^]

Fixes issue 34305 : Removed Receipt Payments Date validation for timezone

* In Paid Receipt, paymentDate converted to date without time based on server timezone

---
M src/org/openbravo/retail/posterminal/PaidReceipts.java
M web/org.openbravo.retail.posterminal/js/components/renderorderline.js
---
(0091725)
migueldejuana   
2016-11-22 08:40   
Tested and reviewed in 5c32424130fa