[jboss-svn-commits] JBL Code SVN: r19733 - labs/jbossrules/branches/4.0.x/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-26 12:24:32 -0400 (Sat, 26 Apr 2008)
New Revision: 19733

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

Modified: labs/jbossrules/branches/4.0.x/drools-compiler/src/main/java/org/drools/rule/builder/RuleBuildContext.java
===================================================================
--- labs/jbossrules/branches/4.0.x/drools-compiler/src/main/java/org/drools/rule/builder/RuleBuildContext.java	2008-04-26 14:27:35 UTC (rev 19732)
+++ labs/jbossrules/branches/4.0.x/drools-compiler/src/main/java/org/drools/rule/builder/RuleBuildContext.java	2008-04-26 16:24:32 UTC (rev 19733)
@@ -29,6 +29,7 @@
 import org.drools.compiler.DialectConfiguration;
 import org.drools.compiler.DialectRegistry;
 import org.drools.compiler.PackageBuilderConfiguration;
+import org.drools.compiler.RuleError;
 import org.drools.lang.descr.AttributeDescr;
 import org.drools.lang.descr.QueryDescr;
 import org.drools.lang.descr.RuleDescr;
@@ -124,8 +125,8 @@
         this.dialectRegistry = dialectRegistry;
         this.dialect = (this.rule.getDialect() != null) ? this.dialectRegistry.getDialect( this.rule.getDialect() ) : defaultDialect;
         
-        if ( dialect == null && (component != null && component.getDialect() != null) ) {
-            this.errors.add( new DescrBuildError(null,parentDescr, component, "Unable to load Dialect '" + component.getDialect() + "'") );
+        if ( dialect == null && this.rule.getDialect()  != null ) {
+            this.errors.add( new RuleError(this.rule,ruleDescr, null, "Unable to load Dialect '" +this.rule.getDialect() + "'") );
         }         
 
         if ( dialect != null ) {