Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0038131Openbravo ERPA. Platformpublic2018-03-14 11:102018-03-15 15:32
Reporterjarmendariz 
Assigned Tojarmendariz 
PrioritynormalSeveritytrivialReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR18Q2 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0038131: Remove deprecated message for AccessibleObject.isAccesible
DescriptionStarting from JDK9, AccesibleObject.isAccesible() is marked as deprecated. The documentation [1] suggests that it should be replaced by canAccess(Object), however this method is available only from JDK9.


[1] https://docs.oracle.com/javase/9/docs/api/java/lang/reflect/AccessibleObject.html#isAccessible-- [^]
Steps To ReproduceCompile project with Java 9, open the project with Eclipse and see the resulting warnings.
Proposed SolutionMark with @SupressWarnings("deprecation") the lines with the deprecated code.
When the minimum Java version supported raises to 9, then it should be replaced with canAccess(Object).
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0037083 closed alostale support JDK 9 
related to design defect 0038132 acknowledged Triage Platform Base Replace deprecated isAccesible() with canAccess() introduced in JDK9 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-03-14 11:10jarmendarizNew Issue
2018-03-14 11:10jarmendarizAssigned To => platform
2018-03-14 11:10jarmendarizOBNetwork customer => No
2018-03-14 11:10jarmendarizModules => Core
2018-03-14 11:10jarmendarizTriggers an Emergency Pack => No
2018-03-14 11:14jarmendarizRelationship addedrelated to 0037083
2018-03-14 11:19jarmendarizRelationship addedrelated to 0038132
2018-03-14 11:34jarmendarizAssigned Toplatform => jarmendariz
2018-03-14 11:34jarmendarizStatusnew => scheduled
2018-03-14 17:06hgbotCheckin
2018-03-14 17:06hgbotNote Added: 0103246
2018-03-14 17:06hgbotStatusscheduled => resolved
2018-03-14 17:06hgbotResolutionopen => fixed
2018-03-14 17:06hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/bf94b5ddb7433eda47159094d702dff2493dd9a9 [^]
2018-03-14 18:31jarmendarizReview Assigned To => caristu
2018-03-14 23:43hudsonbotCheckin
2018-03-14 23:43hudsonbotNote Added: 0103259
2018-03-15 08:57hgbotCheckin
2018-03-15 08:57hgbotNote Added: 0103267
2018-03-15 09:11caristuNote Added: 0103268
2018-03-15 09:11caristuStatusresolved => closed
2018-03-15 09:11caristuFixed in Version => 3.0PR18Q2
2018-03-15 09:13caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=16850#r16850
2018-03-15 15:32hudsonbotCheckin
2018-03-15 15:33hudsonbotNote Added: 0103280

Notes
(0103246)
hgbot   
2018-03-14 17:06   
Repository: erp/devel/pi
Changeset: bf94b5ddb7433eda47159094d702dff2493dd9a9
Author: Javier Armendáriz <javier.armendariz <at> openbravo.com>
Date: Wed Mar 14 16:52:23 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/bf94b5ddb7433eda47159094d702dff2493dd9a9 [^]

Fixed bug 38131: Removed deprecation warning of isAccesible

Added @SupressWarning("all") to temporarily remove the deprecation warning of isAccessible
method until it can be replaced with the new canAccess(). Using the "all" value
because using @SuppressWarning("deprecation") shows an "unnecesary suppress warning"
warning in JDK8 and lower.

---
M modules/org.openbravo.base.weld/src-test/org/openbravo/base/weld/test/ParameterCdiTestRule.java
M modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/DisplayLogicAtServerLevelTest.java
M src-test/src/org/openbravo/test/base/HiddenObjectHelper.java
---
(0103259)
hudsonbot   
2018-03-14 23:43   
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/91dc222dee6a [^]
Maturity status: Test
(0103267)
hgbot   
2018-03-15 08:57   
Repository: erp/devel/pi
Changeset: c6d53eea0b79de58d67e2f588a70398c50975035
Author: Javier Armendáriz <javier.armendariz <at> openbravo.com>
Date: Thu Mar 15 08:54:37 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c6d53eea0b79de58d67e2f588a70398c50975035 [^]

Related to bug 38131: Removed deprecation warning of isAccesible

Moved code in order to be able to put @SupressWarnings for a single line
instead of the whole method.

---
M modules/org.openbravo.base.weld/src-test/org/openbravo/base/weld/test/ParameterCdiTestRule.java
---
(0103268)
caristu   
2018-03-15 09:11   
Verified
(0103280)
hudsonbot   
2018-03-15 15:32   
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/c6d6e97c6d07 [^]
Maturity status: Test