Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0042788Openbravo ERP02. Master data managementpublic2020-01-10 06:472020-02-06 15:53
AtulOpenbravo 
AtulOpenbravo 
normaltrivialhave not tried
closedfixed 
5
 
3.0PR20Q2 
dmiguelez
Core
No
0042788: Apply new HQL style in ServiceDeliverUtility
1. Avoid use of StringBuilder use normal Strings.
2. HQL formatting is important for improving readability.
3. Avoid use of Property Constant if any
-
Follow new style available here http://wiki.openbravo.com/wiki/HQL_Coding_Conventions [^]
No tags attached.
depends on design defect 0041287 acknowledged platform Tracking issue: Convert HQL to apply new style 
Not all the children of this issue are yet resolved or closed.
Issue History
2020-01-10 06:47AtulOpenbravoNew Issue
2020-01-10 06:47AtulOpenbravoAssigned To => AtulOpenbravo
2020-01-10 06:47AtulOpenbravoModules => Core
2020-01-10 06:47AtulOpenbravoTriggers an Emergency Pack => No
2020-01-10 06:47AtulOpenbravoIssue generated from0042786
2020-01-10 06:48AtulOpenbravoRelationship addeddepends on 0041287
2020-01-10 06:51AtulOpenbravoIssue cloned0042789
2020-01-16 03:53AtulOpenbravoStatusnew => scheduled
2020-02-06 15:50dmiguelezNote Added: 0117514
2020-02-06 15:52hgbotCheckin
2020-02-06 15:52hgbotNote Added: 0117515
2020-02-06 15:52hgbotStatusscheduled => resolved
2020-02-06 15:52hgbotResolutionopen => fixed
2020-02-06 15:52hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3addfeff85dc59fe472c54a608b51cebb04c3f40 [^]
2020-02-06 15:52hgbotCheckin
2020-02-06 15:52hgbotNote Added: 0117516
2020-02-06 15:52hgbotCheckin
2020-02-06 15:52hgbotNote Added: 0117517
2020-02-06 15:53dmiguelezReview Assigned To => dmiguelez
2020-02-06 15:53dmiguelezNote Added: 0117518
2020-02-06 15:53dmiguelezStatusresolved => closed
2020-02-06 15:53dmiguelezFixed in Version => 3.0PR20Q2

Notes
(0117514)
dmiguelez   
2020-02-06 15:50   
Test Plan.

Log in with White Valley Admin
Go to Preference window and create a new record with:
  * Property: Web POS Enable Delivery Modes
  * Values: Y
  * Visibility: All filters empty

Log in WebPOS with terminal VBS- and user Vallblanca
Click on browse and select the first product
Click on the ticket line and then in the 'Show More' option of the right side.
Select Delivery Mode and then select Home Delivery. Set:
  * Delivery Date: Today
  * Delivery Time: 2 hours from now
Click Apply.
Click on Checkout -> Pay Total -> Done

From the menu select Issue Sales Order
Search by Order Date and select today.
Select the previously created line and set 1 unit to prepare.
Click on issue selected.

There is a message stating that the issue has been created successfully.
(0117515)
hgbot   
2020-02-06 15:52   
Repository: erp/devel/pi
Changeset: 3addfeff85dc59fe472c54a608b51cebb04c3f40
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Fri Jan 17 09:26:33 2020 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/3addfeff85dc59fe472c54a608b51cebb04c3f40 [^]

Fixes BUG-0042788: Apply new HQL style in ServiceDeliverUtility

**Avoid StringBuilder, use String to format hql query strings
to improve readability

---
M src/org/openbravo/materialmgmt/ServiceDeliverUtility.java
---
(0117516)
hgbot   
2020-02-06 15:52   
Repository: erp/devel/pi
Changeset: 560842f6ff5215a00f2fd1df8805ec030c640e95
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Fri Jan 17 09:28:59 2020 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/560842f6ff5215a00f2fd1df8805ec030c640e95 [^]

Related To ISSUE-42788: Apply new HQL style in ServiceDeliverUtility

**Apply SonarLint suggestions
**Simplify the call to the query object

---
M src/org/openbravo/materialmgmt/ServiceDeliverUtility.java
---
(0117517)
hgbot   
2020-02-06 15:52   
Repository: erp/devel/pi
Changeset: 38ba3924472f084584c65057a1682a55cc06de95
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Fri Jan 17 20:43:00 2020 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/38ba3924472f084584c65057a1682a55cc06de95 [^]

Related To ISSUE-42788:Apply new HQL style in ServiceDeliverUtility

**Revert SonarLint suggestion to create a constructor for class
ServiceDeliverUtility

---
M src/org/openbravo/materialmgmt/ServiceDeliverUtility.java
---
(0117518)
dmiguelez   
2020-02-06 15:53   
Code Review + Testing Ok