Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 320836 - Hessian Serializer Factory extension require class to implement internal class
Summary: Hessian Serializer Factory extension require class to implement internal class
Status: RESOLVED FIXED
Alias: None
Product: Riena
Classification: RT
Component: communication (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P2 minor (vote)
Target Milestone: ---   Edit
Assignee: Stefan Liebig CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-25 04:38 EDT by Patrick He CLA
Modified: 2011-08-23 06:29 EDT (History)
0 users

See Also:
Stefan.Liebig: documentation-

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

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick He CLA 2010-07-25 04:38:16 EDT 
Build Identifier: 

The extension point org.eclipse.riena.communication.factory.hessian.serializerFactories requires the plugin to extend an internal class org.eclipse.riena.internal.communication.factory.hessian.AbstractRienaSerializerFactory. So there will be an warning from Eclipse when creating this extension.

It works if this warning is ignored.

Reproducible: Always
Comment 1 Elias Volanakis CLA 2010-07-26 17:20:28 EDT 
+1 

In think this class should be API.
Comment 2 Stefan Liebig CLA 2011-08-10 09:58:27 EDT 
+1

Setting priority to P2
Comment 3 Stefan Liebig CLA 2011-08-23 06:29:46 EDT 
This has been fixed! AbstractRienaSerializerFactory is now API.

However, while working on this issue we realized a few "shortcomings" and thus refactored:
- AbstractRienaSerializerFactory is now in a different package and has a different interface (no more magic number for defining an order)
- the extension point now allows to specify the ordering of contributed de/serializer factories (instead of the contributed serializer factory)
- the serializer factories are now able to specify the types which the are replacing from hessians internal mapping