Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034017Retail ModulesWeb POSpublic2016-09-19 14:252016-10-21 09:14
Reporteradrianromero 
Assigned Toranjith_qualiantech_com 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q1 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034017: [SER QA 1730] Function getLastDocumentNumber relies on the separator '/' to process Document numbers
DescriptionIn the function getLastDocumentNumber the SQL regular expression relies on having a separator in the document number of orders to calculate the latest document number.
This is not correct because since issue 32857 the application can be configured to not to have a separator in the document numbers generated by the POS
Steps To ReproduceThe lines affected are:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/b61897bd8ae0/src/org/openbravo/retail/posterminal/POSUtils.java#l342 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/b61897bd8ae0/src/org/openbravo/retail/posterminal/POSUtils.java#l346 [^]

Proposed SolutionModify the regular expression to work properly independently the document numbers has been generated with separator or not.
Additional Information
TagsSER-QA
Relationships
related to feature request 0032857 closed adrianromero The document no separator should be customized 
Attached Filespatch issue 34017 source posterminal 7054.patch (6,932) 2016-10-18 08:36
https://issues.openbravo.com/file_download.php?file_id=9923&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-09-19 14:25adrianromeroNew Issue
2016-09-19 14:25adrianromeroAssigned To => Retail
2016-09-19 14:25adrianromeroResolution time => 1475445600
2016-09-19 14:25adrianromeroTriggers an Emergency Pack => No
2016-09-19 14:25adrianromeroTag Attached: SER-QA
2016-09-19 14:25adrianromeroRelationship addedrelated to 0032857
2016-10-17 14:02ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-10-17 14:02ranjith_qualiantech_comStatusnew => scheduled
2016-10-18 08:36ranjith_qualiantech_comFile Added: issue 34017 source posterminal 7054.patch
2016-10-18 13:01marvintmResolution time1475445600 => 1477346400
2016-10-19 08:10hgbotCheckin
2016-10-19 08:10hgbotNote Added: 0090685
2016-10-19 08:10hgbotStatusscheduled => resolved
2016-10-19 08:10hgbotResolutionopen => fixed
2016-10-19 08:10hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c3ce0736516785ac74007e2c5142f792d8c7a19 [^]
2016-10-21 09:14guilleaerReview Assigned To => guilleaer
2016-10-21 09:14guilleaerStatusresolved => closed
2016-10-21 09:14guilleaerFixed in Version => RR17Q1

Notes
(0090685)
hgbot   
2016-10-19 08:10   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6c3ce0736516785ac74007e2c5142f792d8c7a19
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Oct 18 11:07:55 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c3ce0736516785ac74007e2c5142f792d8c7a19 [^]

Fixes issue 34017 : Modified Document number regex to get max document number based on document seperator

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