[jboss-svn-commits] JBL Code SVN: r19723 - labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder.

[jboss-svn-commits at lists.jboss.org]

Author: mark.proctor at jboss.com
Date: 2008-04-25 21:32:20 -0400 (Fri, 25 Apr 2008)
New Revision: 19723

Modified:
   labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/PackageBuildContext.java
Log:
JBRULES-1581 Invalid dialect name causes NullPointerException in BRMS and Eclipse plug-in rule editor

Modified: labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/PackageBuildContext.java
===================================================================
--- labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/PackageBuildContext.java	2008-04-26 00:23:08 UTC (rev 19722)
+++ labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/PackageBuildContext.java	2008-04-26 01:32:20 UTC (rev 19723)
@@ -21,9 +21,12 @@
 import java.util.List;
 import java.util.Map;
 
+import org.drools.compiler.DescrBuildError;
 import org.drools.compiler.Dialect;
 import org.drools.compiler.DialectRegistry;
+import org.drools.compiler.DroolsError;
 import org.drools.compiler.PackageBuilderConfiguration;
+import org.drools.compiler.RuleBuildError;
 import org.drools.lang.descr.BaseDescr;
 import org.drools.rule.Dialectable;
 import org.drools.rule.Package;
@@ -91,7 +94,12 @@
         this.errors = new ArrayList();
 
         this.dialectRegistry = dialectRegistry;
+        
         this.dialect = (component != null && component.getDialect() != null) ? this.dialectRegistry.getDialect( component.getDialect() ) : defaultDialect;
+        
+        if ( dialect == null && (component != null && component.getDialect() != null) ) {
+            this.errors.add( new DescrBuildError(null,parentDescr, component, "Unable to load Dialect '" + component.getDialect() + "'") );
+        }
     }
     
     public BaseDescr getParentDescr() {