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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0037941
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorhave not tried2018-02-15 18:432018-02-22 18:19
ReportercaristuView Statuspublic 
Assigned Toshuehner 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisione175a39a03a5
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tocaristu
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0037941: JDK 9: remove [dep-ann] warnings by adding the missing @Deprecated annotation (platform code)

DescriptionWhen building Openbravo with JDK 9, several [dep-ann] warnings appears:

warning: [dep-ann] deprecated item is not annotated with @Deprecated

Some of this warnings can be removed just by using the missing @Deprecated because it still makes sense to keep them as part of the API.
Steps To Reproduce1. Build Openbravo with Java 9
  -> Check warnings
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to feature request 0037083 closedalostale support JDK 9 

-  Notes
(0102455)
hgbot (developer)
2018-02-16 09:15
edited on: 2018-02-16 09:15

Repository: erp/devel/pi
Changeset: e175a39a03a5d7afaae951cfbeaaa182bb20a81f
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Feb 16 09:03:06 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/e175a39a03a5d7afaae951cfbeaaa182bb20a81f [^]

fixes bug 37941: remove [dep-ann] warnings by adding the @Deprecated annotation

  Added missing @Deprecated annotation in:

  - BaseProcessActionHandler.fixRequestMap: as method is still referenced in other modules.
  - HttpSessionWrapper: implementing HttpSession interface, so need to keep the deprecated methods. Removed autogenerated TODO comments also.
  - BaseTest: the whole class is deprecated but still used in other modules (src-test).
  - DalUtil.getId: recently deprecated, for the moment it is safer to add the annotation and keep it.
  - Utility.getTabURL + getTransplatedJasperReport: both used in several modules still.

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/process/BaseProcessActionHandler.java
M src-core/src/org/openbravo/base/HttpSessionWrapper.java
M src-core/src/org/openbravo/database/SessionInfo.java
M src-test/src/org/openbravo/test/base/BaseTest.java
M src/org/openbravo/dal/core/DalUtil.java
M src/org/openbravo/erpCommon/utility/Utility.java
---

(0102456)
caristu (developer)
2018-02-16 09:18

Code reviewed
(0102772)
hudsonbot (developer)
2018-02-22 18:19

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/980a6ad5bbf5 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2018-02-15 18:43 caristu New Issue
2018-02-15 18:43 caristu Assigned To => platform
2018-02-15 18:43 caristu Modules => Core
2018-02-15 18:43 caristu Triggers an Emergency Pack => No
2018-02-15 18:43 caristu Relationship added related to 0037083
2018-02-15 18:45 caristu Issue cloned 0037942
2018-02-15 18:46 caristu Summary JDK: remove [dep-ann] warnings => JDK: remove [dep-ann] warnings y adding the missing @Deprecated annotation
2018-02-15 18:56 caristu Summary JDK: remove [dep-ann] warnings y adding the missing @Deprecated annotation => JDK: remove [dep-ann] warnings by adding the missing @Deprecated annotation
2018-02-16 09:14 caristu Assigned To platform => shuehner
2018-02-16 09:15 hgbot Checkin
2018-02-16 09:15 hgbot Note Added: 0102455
2018-02-16 09:15 hgbot Status new => resolved
2018-02-16 09:15 hgbot Resolution open => fixed
2018-02-16 09:15 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e175a39a03a5d7afaae951cfbeaaa182bb20a81f [^]
2018-02-16 09:15 caristu Note Edited: 0102455 View Revisions
2018-02-16 09:18 caristu Summary JDK: remove [dep-ann] warnings by adding the missing @Deprecated annotation => JDK 9: remove [dep-ann] warnings by adding the missing @Deprecated annotation
2018-02-16 09:18 caristu Review Assigned To => caristu
2018-02-16 09:18 caristu Note Added: 0102456
2018-02-16 09:18 caristu Status resolved => closed
2018-02-16 12:14 shuehner Summary JDK 9: remove [dep-ann] warnings by adding the missing @Deprecated annotation => JDK 9: remove [dep-ann] warnings by adding the missing @Deprecated annotation (platform code)
2018-02-16 12:15 shuehner Issue cloned 0037959
2018-02-22 18:19 hudsonbot Checkin
2018-02-22 18:19 hudsonbot Note Added: 0102772


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker