Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0048096
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] External Data Integrationminorhave not tried2021-11-17 17:172022-03-18 11:35
ReportershuehnerView Statuspublic 
Assigned ToAugustoMauch 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Regression introduced in release
Summary

0048096: DirectPropertyMapping.executeQueryForReferencedRecord/getHqlQueryForReferencedRecord is not using bind-params

DescriptionThe getHqlQueryForReferencedRecord method is constructing dynamic HQL without using bind params.

This concrete case looks like it could be rewritten using OBCriteria instead.

Steps To Reproduce-
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0048035 closedmeriem_azaf Apply HQL style guide to service.externaldata.integration module 

-  Notes
(0134233)
hgbot (developer)
2022-01-11 17:57

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/merge_requests/9 [^]
(0135840)
hgbot (developer)
2022-03-18 11:35

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration [^]
Changeset: 29cc000f93ecc895d2cc89f4dfc539f87070120f
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 18-03-2022 11:34:31
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/commit/29cc000f93ecc895d2cc89f4dfc539f87070120f [^]

Fixes ISSUE-48096: Uses OBCriteria instead of injecting entity and property name

We can use OBCriteria to build the query instead of injecting the entity and property names

---
M src/org/openbravo/service/external/integration/mapping/DirectPropertyMapping.java
---
(0135842)
hgbot (developer)
2022-03-18 11:35

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/merge_requests/9 [^]

- Issue History
Date Modified Username Field Change
2021-11-17 17:17 shuehner New Issue
2021-11-17 17:17 shuehner Assigned To => platform
2021-11-17 17:17 shuehner Relationship added related to 0048035
2022-01-11 17:54 AugustoMauch Assigned To platform => AugustoMauch
2022-01-11 17:55 AugustoMauch Status new => scheduled
2022-01-11 17:57 hgbot Note Added: 0134233
2022-03-18 11:35 hgbot Resolution open => fixed
2022-03-18 11:35 hgbot Status scheduled => closed
2022-03-18 11:35 hgbot Note Added: 0135840
2022-03-18 11:35 hgbot Note Added: 0135842


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker