Details
Improvement
Major
Description
Rather than having to do:
xstream.alias("com.blah.mypackage.MyClass", "MyClass");
xstream.alias("com.blah.mypackage.YourClass", "YourClass");
xstream.alias("com.blah.mypackage.SheepClass", "SheepClass");
I'd like to be able to do:
xstream.aliasPackage("com.blah.mypackage");
Attachments
Issue Links
| This issue is duplicated by: | ||||
| XSTR-533 | An alias for a whole package (and sub packages) |
|
|
|
Activity
Hide
Grégory Joseph (old account)
added a comment -
The previous patch isn't really satisfying me, because :
- it doesn't actually map a package name to a "shortcut" name, but to empty string (ie, com.blah.bleh.SomeClass can not be aliased to "foo.SomeClass" but only to SomeClass)
- it's based on startsWith and substring methods, which makes it a bit hairy - and possibly not very safe - for the sake of aliasing com.blah.bleh.bluh.SomeOtherClass to "bluh.SomeOtherClass"
I'm planning on submitting another patch that would
- allow actual aliasing of a package to a "prefix", but only A package and not it's subpackages
- be based on full package name of a Class instead of just using startsWith on Class.getName()
... I'll do that .. eventually.. when my pc resurects.
Please tell me what you think, anyway 
Show
Grégory Joseph (old account)
added a comment - The previous patch isn't really satisfying me, because :
- it doesn't actually map a package name to a "shortcut" name, but to empty string (ie, com.blah.bleh.SomeClass can not be aliased to "foo.SomeClass" but only to SomeClass)
- it's based on startsWith and substring methods, which makes it a bit hairy - and possibly not very safe - for the sake of aliasing com.blah.bleh.bluh.SomeOtherClass to "bluh.SomeOtherClass"
- allow actual aliasing of a package to a "prefix", but only A package and not it's subpackages
- be based on full package name of a Class instead of just using startsWith on Class.getName()
Here's a very basic patch that allows this.
I've factored out two methods in DefaultClassMapper and made the loadClass method of DefaultClassMapper protected; this PackageAliasClassMapper simply extends DefaultClassMapper.
Please review and tell me what you think.