Author: michiel
Date: 2009-11-20 13:02:37 +0100 (Fri, 20 Nov 2009)
New Revision: 39821

Modified:
   mmbase/trunk/core/src/main/java/org/mmbase/module/core/MMBase.java
Log:
clarified some exception by mentioning in what builder it occored. E.g. a type 
in the 'extends' attribute could be a bit cryptical because it was not clear in 
what builder that happens

Modified: mmbase/trunk/core/src/main/java/org/mmbase/module/core/MMBase.java
===================================================================
--- mmbase/trunk/core/src/main/java/org/mmbase/module/core/MMBase.java  
2009-11-20 11:39:15 UTC (rev 39820)
+++ mmbase/trunk/core/src/main/java/org/mmbase/module/core/MMBase.java  
2009-11-20 12:02:37 UTC (rev 39821)
@@ -1059,10 +1059,11 @@
             return builder;
         }
 
+        BuilderReader parser = null;
         try {
             // register the loading of this builder
+            parser = getBuilderReader(ipath + builderName);
             loading.add(builderName);
-            BuilderReader parser = getBuilderReader(ipath + builderName);
             if (parser == null) {
                 log.warn("Not found " + ipath + builderName);
                 loading.remove(builderName);
@@ -1146,11 +1147,13 @@
             }
         } catch (BuilderConfigurationException bcfe) {
             loading.remove(builderName);
-            log.error(bcfe.getMessage() + " " + bcfe.getMessage());
+            log.error((parser == null ? ipath + builderName : 
parser.getSystemId()) + ": " +
+                      bcfe.getMessage() + " " + bcfe.getMessage());
             return null;
         } catch (Throwable e) { // what kind of exceptions are these?
             loading.remove(builderName);
-            log.error(e.getClass() + " " + e.getMessage(), e);
+            log.error((parser == null ? ipath + builderName : 
parser.getSystemId()) + ": " +
+                      e.getClass() + " " + e.getMessage(), e);
             return null;
         }
         loading.remove(builderName);

_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to