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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0040246
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorhave not tried2019-02-20 17:022019-03-28 14:56
ReportercaristuView Statuspublic 
Assigned Tocaristu 
PriorityhighResolutionfixedFixed in Version3.0PR19Q2
StatusclosedFix in branchFixed in SCM revision804ed224407c
ProjectionnoneETAnoneTarget Version
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

0040246: It is not needed to include the woodstox library in the runtime classpath

DescriptionWhen deploying the context, all the libraries inside "src-db/database/lib" are deployed into WEB-INF/lib.

This is because it is needed to have some of them included in the runtime classpath because they are required by some features:

- Installing a module from an obx file using the MMC
- Easy Extensible Attributes[1] module

But the woodstox library (wstx-asl-3.0.2.jar) is not required at runtime by any feature in particular. Note that the StaxXMLEntityConverter[2] can retrieve any other available XML reader for its processing.

[1] http://centralrepository.openbravo.com/openbravo/org.openbravo.forge.ui/ForgeModuleDetail/Easy-Extensible-Attributes [^]
[2] https://code.openbravo.com/erp/devel/pi/file/tip/src/org/openbravo/dal/xml/StaxXMLEntityConverter.java#l252 [^]
Steps To ReproduceIn description
Proposed SolutionAvoid the deployment of the libraries inside src-db/database/lib which are not required at runtime, i.e, do not deploy the wstx-asl-3.0.2.jar file.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0040047 closedcaristu Fix database export to avoid manually copying wstx-asl-3.0.2.jar inside Ant folder 
depends on backport 00402553.0PR19Q1 closedcaristu It is not needed to include the woodstox library in the runtime classpath 

-  Notes
(0109966)
hgbot (developer)
2019-02-20 17:28

Repository: erp/devel/pi
Changeset: 804ed224407cc8015ae2675d37ee21ccb4cb02ce
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Feb 20 17:15:55 2019 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/804ed224407cc8015ae2675d37ee21ccb4cb02ce [^]

fixes issue 40246: The woodstox library is not required at runtime

  The woodstox library which provides an implementation of the Stax XML API is used by DBSM for export/import taks but it is not required at runtime. Therefore, now we are not copying it when deploying the context.

---
M src/build.xml
---
(0109975)
hgbot (developer)
2019-02-20 18:22

Repository: erp/devel/pi
Changeset: 345f70ecc8331ba7b9f453a95c58b4e58f621e34
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Feb 20 18:22:12 2019 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/345f70ecc8331ba7b9f453a95c58b4e58f621e34 [^]

related to issue 40246: update comment

---
M src/org/openbravo/dal/xml/StaxXMLEntityConverter.java
---
(0110016)
alostale (manager)
2019-02-22 09:06

reviewed + tested
(0110827)
hudsonbot (developer)
2019-03-28 14:56

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/b2fbc1588df4 [^]
Maturity status: Test
(0110829)
hudsonbot (developer)
2019-03-28 14:56

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

- Issue History
Date Modified Username Field Change
2019-02-20 17:02 caristu New Issue
2019-02-20 17:02 caristu Assigned To => platform
2019-02-20 17:02 caristu Modules => Core
2019-02-20 17:02 caristu Triggers an Emergency Pack => No
2019-02-20 17:03 caristu Relationship added related to 0040047
2019-02-20 17:03 caristu Assigned To platform => caristu
2019-02-20 17:05 caristu Proposed Solution updated
2019-02-20 17:07 caristu Description Updated View Revisions
2019-02-20 17:14 caristu Review Assigned To => alostale
2019-02-20 17:28 hgbot Checkin
2019-02-20 17:28 hgbot Note Added: 0109966
2019-02-20 17:28 hgbot Status new => resolved
2019-02-20 17:28 hgbot Resolution open => fixed
2019-02-20 17:28 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/804ed224407cc8015ae2675d37ee21ccb4cb02ce [^]
2019-02-20 17:41 caristu Summary Does not need to include the woodstox library in the runtime classpath => It is not needed to include the woodstox library in the runtime classpath
2019-02-20 17:49 caristu Description Updated View Revisions
2019-02-20 17:58 caristu Description Updated View Revisions
2019-02-20 18:22 hgbot Checkin
2019-02-20 18:22 hgbot Note Added: 0109975
2019-02-21 19:04 caristu Issue cloned 0040255
2019-02-21 19:04 caristu Relationship added depends on 0040255
2019-02-22 09:06 alostale Note Added: 0110016
2019-02-22 09:06 alostale Status resolved => closed
2019-02-22 09:06 alostale Fixed in Version => 3.0PR19Q2
2019-03-28 14:56 hudsonbot Checkin
2019-03-28 14:56 hudsonbot Note Added: 0110827
2019-03-28 14:56 hudsonbot Checkin
2019-03-28 14:56 hudsonbot Note Added: 0110829


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker