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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0027439
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementminorhave not tried2014-08-23 16:382015-08-20 23:16
ReportershuehnerView Statuspublic 
Assigned ToAtulOpenbravo 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisiond74714cd50ad
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tovmromanos
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0027439: pl-function c_getwithholding has uuid error (maybe harmless)

DescriptionThat function has one variable defined as varchar2(32) but used to stored numeric values.

running pl-function without any witholding data defined does suprisingly work

as
withholding:=0
return withholding

seems to work in both oracle & postgres even with that variable defined as varchar.

But it is clearly wrong datatype and possibly broken when using all pl-body doing more calculations using that variable.

NOTE: Technically regression introduced with 2.50 project when changing pk's from 2.50 -> 3.0 as then that datatype of that variable was also accidently changed.

Steps To ReproduceRead that pl-function body and compare variable definition with usage.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0069650)
shuehner (administrator)
2014-08-23 16:39

Note found using plpgsql-check warning
(0078834)
hgbot (developer)
2015-07-15 18:17

Repository: erp/devel/pi
Changeset: d74714cd50ade5882908c11238d4125c2e7cf21f
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Tue Jul 14 12:46:42 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/d74714cd50ade5882908c11238d4125c2e7cf21f [^]

Fixes Issue 27439:pl-function c_getwithholding has wrong datatype for variable

Withholding variable was wrongly defined as varchar during merge r2.5x into
trunk.

---
M src-db/database/model/functions/C_GETWITHHOLDING.xml
---
(0078835)
vmromanos (manager)
2015-07-15 18:22

Code review OK
(0079494)
hudsonbot (developer)
2015-08-20 23:16

A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/8c91718397a5 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2014-08-23 16:38 shuehner New Issue
2014-08-23 16:38 shuehner Assigned To => dmiguelez
2014-08-23 16:38 shuehner Modules => Core
2014-08-23 16:38 shuehner Triggers an Emergency Pack => No
2014-08-23 16:39 shuehner Note Added: 0069650
2014-08-23 16:39 shuehner Summary pl-function c_getwithholding has uuid error (but harmless) => pl-function c_getwithholding has uuid error (maybe harmless)
2014-09-03 13:51 eduardo_Argal Assigned To dmiguelez => eduardo_Argal
2015-07-14 09:07 AtulOpenbravo Assigned To eduardo_Argal => AtulOpenbravo
2015-07-14 09:07 AtulOpenbravo Status new => scheduled
2015-07-15 18:17 hgbot Checkin
2015-07-15 18:17 hgbot Note Added: 0078834
2015-07-15 18:17 hgbot Status scheduled => resolved
2015-07-15 18:17 hgbot Resolution open => fixed
2015-07-15 18:17 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d74714cd50ade5882908c11238d4125c2e7cf21f [^]
2015-07-15 18:22 vmromanos Review Assigned To => vmromanos
2015-07-15 18:22 vmromanos Note Added: 0078835
2015-07-15 18:22 vmromanos Status resolved => closed
2015-08-20 23:16 hudsonbot Checkin
2015-08-20 23:16 hudsonbot Note Added: 0079494


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker