Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0040832Openbravo ERP05. Production managementpublic2019-04-10 11:412019-06-05 15:54
shuehner 
nonofrancisco 
normalminorhave not tried
closedfixed 
5
 
3.0PR19Q2.13.0PR19Q2.1 
Sandrahuguet
Core
No
0040832: SequenceProductCreate.getLineNum is not using bind-parameters
This code is not using bind-params:

    String hql = " SELECT COALESCE(MAX(l.lineNo),0)+10 AS DefaultValue FROM ManufacturingOperationProduct l WHERE l.mASequence.id= '"
        + SequenceId + "'";
    Query<Long> q = OBDal.getInstance().getSession().createQuery(hql, Long.class);
-
No tags attached.
blocks defect 00405743.0PR19Q3 closed nonofrancisco SequenceProductCreate.getLineNum is not using bind-parameters 
Issue History
2019-05-09 23:25nonofranciscoTypedefect => backport
2019-05-09 23:25nonofranciscoTarget Version => 3.0PR19Q2
2019-05-10 08:12SandrahuguetTarget Version3.0PR19Q2 => 3.0PR19Q2.1
2019-06-05 15:53hgbotCheckin
2019-06-05 15:53hgbotNote Added: 0112426
2019-06-05 15:53hgbotStatusscheduled => resolved
2019-06-05 15:53hgbotResolutionopen => fixed
2019-06-05 15:53hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR19Q2.1/rev/785a223ac67f4cb4144410f9185f243831b4510c [^]
2019-06-05 15:54SandrahuguetReview Assigned To => Sandrahuguet
2019-06-05 15:54SandrahuguetNote Added: 0112428
2019-06-05 15:54SandrahuguetStatusresolved => closed
2019-06-05 15:54SandrahuguetFixed in Version => 3.0PR19Q2.1

Notes
(0112426)
hgbot   
2019-06-05 15:53   
Repository: erp/backports/3.0PR19Q2.1
Changeset: 785a223ac67f4cb4144410f9185f243831b4510c
Author: Nono Carballo <nonofce <at> gmail.com>
Date: Wed May 01 22:53:24 2019 -0400
URL: http://code.openbravo.com/erp/backports/3.0PR19Q2.1/rev/785a223ac67f4cb4144410f9185f243831b4510c [^]

Fixes issue 40832: Uses bind parameter.

Instead of string concatenation, bind parameter is used to build the query.

---
M src/org/openbravo/erpCommon/ad_actionButton/SequenceProductCreate.java
---
(0112428)
Sandrahuguet   
2019-06-05 15:54   
code review + testing ok