Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0037941 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2018-02-15 18:43 | 2018-02-22 18:19 | |||
Reporter | caristu | View Status | public | |||||
Assigned To | shuehner | |||||||
Priority | high | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | e175a39a03a5 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | caristu | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0037941: JDK 9: remove [dep-ann] warnings by adding the missing @Deprecated annotation (platform code) | |||||||
Description | When 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 Reproduce | 1. Build Openbravo with Java 9 -> Check warnings | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
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 |