No other way because of problems in legacy model
On Jan 31, 2014 10:27 PM, "Tomaz Cerar" <notifications@github.com> wrote:

> In
> connector/src/test/java/org/jboss/as/connector/subsystems/datasources/DatasourcesSubsystemTestCase.java:
>
> > @@ -132,10 +131,20 @@ private void testTransformer(String subsystemXml, ModelTestControllerVersion con
> >
> >          // Add legacy subsystems
> >          builder.createLegacyKernelServicesBuilder(null, controllerVersion, modelVersion)
> > -                  .addMavenResourceURL("org.jboss.as:jboss-as-connector:" + controllerVersion.getMavenGavVersion())
> > -                  .setExtensionClassName("org.jboss.as.connector.subsystems.datasources.DataSourcesExtension")
> > -                  .configureReverseControllerCheck(AdditionalInitialization.MANAGEMENT, null)
> > -                  .excludeFromParent(SingleClassFilter.createFilter(ConnectorLogger.class));
> > +                .addMavenResourceURL("org.jboss.as:jboss-as-connector:" + controllerVersion.getMavenGavVersion())
> > +                .setExtensionClassName("org.jboss.as.connector.subsystems.datasources.DataSourcesExtension")
> > +                .excludeFromParent(SingleClassFilter.createFilter(ConnectorLogger.class))
> > +                .configureReverseControllerCheck(AdditionalInitialization.MANAGEMENT, new ModelFixer() {
> > +                    @Override
> > +                    public ModelNode fixModel(ModelNode modelNode) {
> > +                        //Replace the value used in the xml
> > +                        modelNode.get(Constants.XA_DATASOURCE).get("complexXaDs_Pool").remove(Constants.JTA.getName());
> > +                        //modelNode.get(Constants.DATA_SOURCE).get("complexDs_Pool").remove(Constants.ENABLED.getName());
> > +                        //modelNode.get(Constants.XA_DATASOURCE).get("complexXaDs_Pool").remove(Constants.ENABLED.getName());
> > +                        return modelNode;
>
> Would this make it non-forward compatible?
> Or is this case were there is no other way because of problems in legacy
> model?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wildfly/wildfly/pull/5823/files#r9358089>
> .
>