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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0040521
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] External Integration Infrastructureminorhave not tried2019-04-04 13:242019-04-05 15:06
ReportercaristuView Statuspublic 
Assigned Tocaristu 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisiona66bee8a4806
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned ToAugustoMauch
Regression introduced in release
Summary

0040521: ConditionalPropertyMapping and AfterFlushHook instances are not being injected

DescriptionAfter the Weld upgrade (0040193), ConditionalPropertyMapping and AfterFlushHook instances are not being injected.

Seems that the behavior when injecting classes that implement generic interfaces has been redefined. See[1].

[1] https://docs.jboss.org/cdi/spec/2.0-PFD/cdi-spec.html#assignable_parameters [^]
Steps To Reproduce1) Install the https://code.openbravo.com/erp/pmods/org.openbravo.externaldata.integration.examples [^] test module
2) Execute the ExportPropertyMappingTest.testUnsatisfiedConditionalMapping test. Note that the test fails because the related expected ConditionalPropertyMapping instance is not executed.

Note that also test of CountryImportWithExtensionsTest class are failing for the same reason.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
caused by feature request 0040193 closedcaristu Openbravo ERP upgrade Weld to version 3.1.0.Final 

-  Notes
(0110982)
hgbot (developer)
2019-04-04 13:32

Repository: erp/pmods/org.openbravo.service.external.integration
Changeset: a66bee8a48068e8a0b9a497fbf18d2e8f3677d22
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu Apr 04 13:30:38 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/a66bee8a48068e8a0b9a497fbf18d2e8f3677d22 [^]

fixes bug 40521: ConditionalPropertyMapping and AfterFlushHook are not injected

  The behavior when injecting classes that implement generic interfaces is different in CDI 2.0. (See[1]). For this reason we need to use wildcards for defining the injection points for such classes.

---
M src/org/openbravo/service/external/integration/importer/SynchronizableBusinessObjectImporter.java
M src/org/openbravo/service/external/integration/mapping/PropertyMapper.java
---
(0111007)
AugustoMauch (manager)
2019-04-05 15:06

Code reviewed and verified

- Issue History
Date Modified Username Field Change
2019-04-04 13:24 caristu New Issue
2019-04-04 13:24 caristu Assigned To => platform
2019-04-04 13:24 caristu Relationship added related to 0040193
2019-04-04 13:24 caristu Description Updated View Revisions
2019-04-04 13:25 caristu Steps to Reproduce Updated View Revisions
2019-04-04 13:27 caristu Summary ConditionalPropertyMapping instances are not being injected => ConditionalPropertyMapping and AfterFlushHook instances are not being injected
2019-04-04 13:27 caristu Description Updated View Revisions
2019-04-04 13:29 caristu Steps to Reproduce Updated View Revisions
2019-04-04 13:32 hgbot Checkin
2019-04-04 13:32 hgbot Note Added: 0110982
2019-04-04 13:32 hgbot Status new => resolved
2019-04-04 13:33 hgbot Resolution open => fixed
2019-04-04 13:33 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/a66bee8a48068e8a0b9a497fbf18d2e8f3677d22 [^]
2019-04-04 13:33 caristu Assigned To platform => caristu
2019-04-04 13:33 caristu Relationship deleted related to 0040193
2019-04-04 13:33 caristu Relationship added caused by 0040193
2019-04-04 13:34 caristu Review Assigned To => AugustoMauch
2019-04-05 15:06 AugustoMauch Note Added: 0111007
2019-04-05 15:06 AugustoMauch Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker