Skip to content
This repository has been archived by the owner on Sep 21, 2022. It is now read-only.

JAXB 2.1 Property Change Listener Injector Plugin #4

Open
glassfishrobot opened this issue Jan 12, 2007 · 10 comments
Open

JAXB 2.1 Property Change Listener Injector Plugin #4

glassfishrobot opened this issue Jan 12, 2007 · 10 comments

Comments

@glassfishrobot
Copy link
Contributor

First of all, thanks for the plug-in.

@@@@

The plug-in does not create bound properties as advertised but constrained
properties.

http://java.sun.com/docs/books/tutorial/javabeans/properties/bound.html

http://java.sun.com/docs/books/tutorial/javabeans/properties/constrained.html

Using Bound properties is much more common than using constrained.

So make the creation of bound default and offer customization to create constrained.

Environment

Operating System: All
Platform: All

Affected Versions

[current]

@glassfishrobot
Copy link
Contributor Author

Reported by sdavids

@glassfishrobot
Copy link
Contributor Author

Was assigned to dochez

@glassfishrobot
Copy link
Contributor Author

sdavids said:
Another customization would be to make the created *ChangeSupport transient.

It is not always neccessary or wanted to serialize all listeners.

@glassfishrobot
Copy link
Contributor Author

sdavids said:
Created an attachment (id=2)
patch is against jaxb2-commons

@glassfishrobot
Copy link
Contributor Author

File: patch.txt
Attached By: sdavids

@glassfishrobot
Copy link
Contributor Author

sdavids said:
Includes parameter to set default property type: simple, bound, constrained, both

Includes customizations:

  • transient
  • property type: simple, bound, constrained, both

Includes documentation (index.html)

@glassfishrobot
Copy link
Contributor Author

sdavids said:
The patch doesn't work with:

xs:annotation
xs:appinfo
jaxb:globalBindings
<xjc:simple />
</jaxb:globalBindings>
</xs:appinfo>
</xs:annotation>

and "constrained" or "both"

I do not know why the generated interface does not have the same signature as
the implementation class.

@glassfishrobot
Copy link
Contributor Author

kohsuke said:
Assigning it to Jerome so that it gets his attention.

@glassfishrobot
Copy link
Contributor Author

lexi said:
Seems like the plugin was not ported.

@glassfishrobot
Copy link
Contributor Author

This issue was imported from java.net JIRA JAXB2_COMMONS-4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant