Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040246Openbravo ERPA. Platformpublic2019-02-20 17:022019-03-28 14:56
Reportercaristu 
Assigned Tocaristu 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR19Q2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0040246: 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.
Additional Information
TagsNo tags attached.
Relationships
related to defect 0040047 closed caristu Fix database export to avoid manually copying wstx-asl-3.0.2.jar inside Ant folder 
depends on backport 00402553.0PR19Q1 closed caristu It is not needed to include the woodstox library in the runtime classpath 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2019-02-20 17:02caristuNew Issue
2019-02-20 17:02caristuAssigned To => platform
2019-02-20 17:02caristuModules => Core
2019-02-20 17:02caristuTriggers an Emergency Pack => No
2019-02-20 17:03caristuRelationship addedrelated to 0040047
2019-02-20 17:03caristuAssigned Toplatform => caristu
2019-02-20 17:05caristuProposed Solution updated
2019-02-20 17:07caristuDescription Updatedbug_revision_view_page.php?rev_id=18349#r18349
2019-02-20 17:14caristuReview Assigned To => alostale
2019-02-20 17:28hgbotCheckin
2019-02-20 17:28hgbotNote Added: 0109966
2019-02-20 17:28hgbotStatusnew => resolved
2019-02-20 17:28hgbotResolutionopen => fixed
2019-02-20 17:28hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/804ed224407cc8015ae2675d37ee21ccb4cb02ce [^]
2019-02-20 17:41caristuSummaryDoes 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:49caristuDescription Updatedbug_revision_view_page.php?rev_id=18350#r18350
2019-02-20 17:58caristuDescription Updatedbug_revision_view_page.php?rev_id=18351#r18351
2019-02-20 18:22hgbotCheckin
2019-02-20 18:22hgbotNote Added: 0109975
2019-02-21 19:04caristuIssue cloned0040255
2019-02-21 19:04caristuRelationship addeddepends on 0040255
2019-02-22 09:06alostaleNote Added: 0110016
2019-02-22 09:06alostaleStatusresolved => closed
2019-02-22 09:06alostaleFixed in Version => 3.0PR19Q2
2019-03-28 14:56hudsonbotCheckin
2019-03-28 14:56hudsonbotNote Added: 0110827
2019-03-28 14:56hudsonbotCheckin
2019-03-28 14:56hudsonbotNote Added: 0110829

Notes
(0109966)
hgbot   
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   
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   
2019-02-22 09:06   
reviewed + tested
(0110827)
hudsonbot   
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   
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