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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0030949
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformtrivialhave not tried2015-09-28 17:202015-11-23 21:16
ReporterinigosanchezView Statuspublic 
Assigned Toinigosanchez 
PriorityhighResolutionfixedFixed in Version3.0PR16Q1
StatusclosedFix in branchFixed in SCM revision2bed8a827ab9
ProjectionnoneETAnoneTarget Version3.0PR16Q1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toalostale
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0030949: Variable shadowing in ConnectionProviderContextListener.

DescriptionVariable shadowing in ConnectionProviderContextListener. There are two warnings in src-core/src/org/openbravo/base/ConnectionProviderContextListener.java . They are newly introduced in pool project.

'reloadPool' and 'destroyPool' methods have a local variable pool. Besides, there is a static class field with same name too.
Steps To ReproduceGet all warnings in src-core from IDE.
Proposed SolutionIt is proposed to rename the parameter of the methods.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks feature request 0031477 newshuehner Tracking issue to reduce the huge number of eclipse warnings in pi 

-  Notes
(0080862)
hgbot (developer)
2015-10-07 16:06

Repository: erp/devel/pi
Changeset: 2bed8a827ab9924be2b6c3b62abdf578e690064b
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Wed Oct 07 16:04:13 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/2bed8a827ab9924be2b6c3b62abdf578e690064b [^]

Fixes issue 30949: Variable shadowing in ConnectionProviderContextListener

The problem was that ConnectionProviderContextListener.java had static class field with same name than parameters in two methods.

Now it has been resolved by renaming the vars in the functions.

---
M src-core/src/org/openbravo/base/ConnectionProviderContextListener.java
---
(0081188)
alostale (manager)
2015-10-22 09:22

code reviewed, tested continues working as it did
(0082114)
hudsonbot (developer)
2015-11-23 21: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/7b56bebaaa88 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2015-09-28 17:20 inigosanchez New Issue
2015-09-28 17:20 inigosanchez Assigned To => inigosanchez
2015-09-28 17:20 inigosanchez Modules => Core
2015-09-28 17:20 inigosanchez Triggers an Emergency Pack => No
2015-09-30 08:16 alostale Status new => acknowledged
2015-09-30 08:16 alostale Priority low => high
2015-09-30 08:16 alostale Target Version => 3.0PR16Q1
2015-10-07 16:00 inigosanchez Issue Monitored: alostale
2015-10-07 16:05 inigosanchez Review Assigned To => alostale
2015-10-07 16:06 hgbot Checkin
2015-10-07 16:06 hgbot Note Added: 0080862
2015-10-07 16:06 hgbot Status acknowledged => resolved
2015-10-07 16:06 hgbot Resolution open => fixed
2015-10-07 16:06 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/2bed8a827ab9924be2b6c3b62abdf578e690064b [^]
2015-10-22 09:22 alostale Note Added: 0081188
2015-10-22 09:22 alostale Status resolved => closed
2015-10-22 09:22 alostale Fixed in Version => 3.0PR16Q1
2015-11-14 13:01 shuehner Relationship added blocks 0031477
2015-11-23 21:16 hudsonbot Checkin
2015-11-23 21:16 hudsonbot Note Added: 0082114


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker