[jboss-cvs] JBossAS SVN: r67166 - in projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee: support and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Fri Nov 16 05:25:29 EST 2007


Author: wolfc
Date: 2007-11-16 05:25:28 -0500 (Fri, 16 Nov 2007)
New Revision: 67166

Modified:
   projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/jboss/JBossServiceReferencesMetaData.java
   projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/support/JavaEEMetaDataUtil.java
Log:
JBAS-4506: Fixed regression on MetaDataRepositoryUnitTestCase.testAnnotationMetaData

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/jboss/JBossServiceReferencesMetaData.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/jboss/JBossServiceReferencesMetaData.java	2007-11-16 08:56:21 UTC (rev 67165)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/jboss/JBossServiceReferencesMetaData.java	2007-11-16 10:25:28 UTC (rev 67166)
@@ -60,6 +60,8 @@
       if (override == null && overriden == null)
          return null;
 
+      // TODO: if overriden is empty, it's silly to do a merge
+      
       JBossServiceReferencesMetaData merged = new JBossServiceReferencesMetaData();
       JavaEEMetaDataUtil.merge(merged, overriden, override, "service-ref", overridenFile, overrideFile, false);
       return merged;

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/support/JavaEEMetaDataUtil.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/support/JavaEEMetaDataUtil.java	2007-11-16 08:56:21 UTC (rev 67165)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/javaee/support/JavaEEMetaDataUtil.java	2007-11-16 10:25:28 UTC (rev 67166)
@@ -76,6 +76,11 @@
       // No override
       if (overriden == null || overriden.isEmpty())
       {
+         // There are no overrides and no overriden
+         // TODO: it has been really silly to call upon merge, but allas
+         if (mapped == null)
+            return merged;
+         
          if (mapped.isEmpty() == false && mustOverride)
             throw new IllegalStateException(overridenFile + " has no " + context + "s but " + overrideFile + " has " + mapped.keySet());
          if(mapped != merged)




More information about the jboss-cvs-commits mailing list