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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0043573
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementminorhave not tried2020-03-25 15:162020-03-26 10:32
ReportershuehnerView Statuspublic 
Assigned Toinigo_lerga 
PrioritynormalResolutionopenFixed in Version
StatusscheduledFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version3.0PR20Q2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0043573: Apply new HQL style in PriceDifferenceUtil

Description1. Avoid use of StringBuilder use normal Strings.
2. HQL formatting is important for improving readability.
3. Avoid use of Property Constant if any
Steps To Reproduce-
Proposed SolutionFollow new style available here http://wiki.openbravo.com/wiki/HQL_Coding_Conventions [^]
TagsNo tags attached.
Attached Filesdiff file icon 43573_partial.diff [^] (518 bytes) 2020-03-25 15:17 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to design defect 0041287 acknowledgedplatform Tracking issue: Convert HQL to apply new style 

-  Notes
(0118812)
shuehner (administrator)
2020-03-25 15:19

That file did use String already to construct HQL query.
However part of that String was creating using String.concat function instead of just doing it using the typical +=

That caused this to come up as non-constant string in automation.

Apart patch fixes that part.

Looking at HQL formatting probably should be reformatted for better readability also.
(0118830)
inigo_lerga (developer)
2020-03-26 10:32

--Test Plan Mantis HQL--
As the F&B International Group Admin role:
  - Go to the Process Price Difference Adjustment window.
    Select "F&B EspaƱa, S.A" as Organization and Today's date
    as Movement Date.
    In the Product field select "Cerveza Ale 0,5L".
  - Press the Done button.

  #In this moment the modified HQL is updated#

- Issue History
Date Modified Username Field Change
2020-03-25 15:16 shuehner New Issue
2020-03-25 15:16 shuehner Assigned To => Triage Finance
2020-03-25 15:16 shuehner Modules => Core
2020-03-25 15:16 shuehner Triggers an Emergency Pack => No
2020-03-25 15:16 shuehner Relationship added related to 0041287
2020-03-25 15:17 shuehner File Added: 43573_partial.diff
2020-03-25 15:19 shuehner Note Added: 0118812
2020-03-26 09:03 inigo_lerga Assigned To Triage Finance => inigo_lerga
2020-03-26 09:03 inigo_lerga Status new => scheduled
2020-03-26 10:32 inigo_lerga Note Added: 0118830


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker