Bug 1039618 - Provide the same way to add extra JAXB classes
Summary: Provide the same way to add extra JAXB classes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BPMS Platform 6
Classification: Retired
Component: Business Central
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: ER6
: 6.0.0
Assignee: Marco Rietveld
QA Contact: Ivo Bek
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-09 15:53 UTC by Ivo Bek
Modified: 2016-09-20 05:05 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-06 20:10:23 UTC
Type: Bug

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Ivo Bek 2013-12-09 15:53:07 UTC 
Description of problem:

There are two different approaches for specifying extra JAXB classes and I'd say they should be same.

The first approach, JaxbSerializationProvider has optional parameter extraJaxbClassList of type 

Set<Class<?>>

The another way is to use for example RemoteJmsRuntimeEngineFactory which has addExtraJaxbClasses method with a parameter of type 

Collection<Class<?>> extraJaxbClasses

Please make the types same. I suggest the Collection because then it's simple to create it with Arrays.asList(array of classes). Requiring Set means that I just need to put the collection into HashSet and I don't see any good reasons to do this additional step.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Comment 1 Marco Rietveld 2013-12-11 13:08:45 UTC 
Fixed: 

6.0.x:  https://github.com/droolsjbpm/droolsjbpm-integration/commit/c3353ca2
master: https://github.com/droolsjbpm/droolsjbpm-integration/commit/f9250652
Comment 2 Ivo Bek 2014-01-06 13:35:47 UTC 
Verified in BPMS 6.0.0.ER7
Note You need to log in before you can comment on or make changes to this bug.