This is how it is supposed to work
http://java.sun.com/javaee/5/docs/api/javax/xml/bind/annotation/XmlElements.html

There supposed to be only one bar element wrapping the XmlElements.

The XmlElementWrapper should have maxOccurs=1 according to the javadoc examples.

Yes, maxOccurs=1 for XmlElementWrapper according to the spec.

I am looking into this.

I see two ways to workaround this.

In both cases the Module class should be bound to choice with @JBossXmlModelGroup.

First is to create a ModuleHolder that would be bound to the module element. And then add JavaXmlTypeAdapter that would take ModuleHandler and return the module it holds.

Second is to introduce an XB annotation equivalent to TermBinding.setSkip(trueOrFalse) and set it on the module element.

"scott.stark@jboss.org" wrote:
So how would an application.xml module element have to be handled? It should be something like this, but the schema for the wrapping module element has an unbounded max:

XmlAccessorType(XmlAccessType.FIELD)
@XmlType(name = "Foo")
public class Foo {
 @XmlElement(required = true)
 public List<Foo.Bar> bar;

 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "")
 public static class Bar {
 @XmlElements({
 @XmlElement(name="A", type=Integer.class),
 @XmlElement(name="B", type=Float.class)
 })
 public List items;
 }
}

This should be closer to what we would like to have

@XmlRootElement
 public static class Application
 {
 private List<Module> modules;

 public void setModules(List<Module> modules)
 {
 this.modules = modules;
 }

 @XmlElement(name="module", type=ModuleHolder.class)
 public List<Module> getModules()
 {
 return modules;
 }
 }

 public static class ModuleAdapter extends XmlAdapter<ModuleHolder, Module>
 {
 @Override
 public ModuleHolder marshal(Module arg0) throws Exception
 {
 // TODO Auto-generated method stub
 return null;
 }

 @Override
 public Module unmarshal(ModuleHolder arg0) throws Exception
 {
 return arg0.getModule();
 }
 }

 @XmlJavaTypeAdapter(value = ModuleAdapter.class)
 public static class ModuleHolder
 {
 private Module module;

 public void setModule(Module module)
 {
 this.module = module;
 }

 @XmlTransient
 public Module getModule()
 {
 return module;
 }

 public void setConnector(ConnectorModule module)
 {
 this.module = module;
 }

 public ConnectorModule getConnector()
 {
 return (ConnectorModule) module;
 }


 public void setEjb(EjbModule module)
 {
 this.module = module;
 }

 public EjbModule getEjb()
 {
 return (EjbModule) module;
 }

 public void setJava(JavaModule module)
 {
 this.module = module;
 }

 public JavaModule getJava()
 {
 return (JavaModule) module;
 }
 }

 public static abstract class Module
 {
 private String value;

 public void setValue(String value)
 {
 this.value = value;
 }

 @XmlValue
 public String getValue()
 {
 return value;
 }
 }

 public static class ConnectorModule extends Module
 {
 }

 public static class EjbModule extends Module
 {
 }

 public static class JavaModule extends Module
 {
 }

Actually the last approach is wrong. The adapter on ModuleHolder will make Module dictate the schema structure. That's pointless.

Ah, I see you dont want a nested maxOccurs="unbounded".

So is this what you want?

@XmlAccessorType(XmlAccessType.FIELD)
@XmlType(name = "application")
public class Foo {
 @XmlElement(required = true)
 public List<Module> module;

 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "")
 public static class Module {
 @XmlElements({
 @XmlElement(name="connector", type=Connector.class),
 @XmlElement(name="java", type=Java.class)
 })
 public Component component;

 @XmlElement(name="alt-dd")
 public String altDD;
 }

 public static interface Component {}
 public static class Connector implements Component
 {
 }

 public static class Java implements Component
 {
 }
}

XmlAccessorType(XmlAccessType.FIELD)
@XmlType(name = "application")
public class Foo {
 @XmlElement(required = true)
 public List<Module> module;

 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "")
 public static class Module {
 @XmlElement(name="java")
 public Java java;

 @XmlElement(name="connector")
 public Connector connector;

 @XmlElement(name="alt-dd")
 public String altDD;
 }

 public static class Connector
 {
 }

 public static class Java
 {
 }
}

Yes, that would work. I was trying to avoid Module as a wrapping class since this is what I think Scott wanted to have at the end.