[jboss-svn-commits] JBL Code SVN: r14158 - in labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect: mvel and 1 other directory.

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

Author: mark.proctor at jboss.com
Date: 2007-08-11 11:27:39 -0400 (Sat, 11 Aug 2007)
New Revision: 14158

Modified:
   labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/java/JavaDialect.java
   labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java
Log:
-provide better error messages if it can't analyse the code.

Modified: labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/java/JavaDialect.java
===================================================================
--- labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/java/JavaDialect.java	2007-08-11 15:26:31 UTC (rev 14157)
+++ labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/java/JavaDialect.java	2007-08-11 15:27:39 UTC (rev 14158)
@@ -216,8 +216,8 @@
         } catch ( final Exception e ) {
             context.getErrors().add( new RuleError( context.getRule(),
                                                     descr,
-                                                    null,
-                                                    "Unable to determine the used declarations" ) );
+                                                    e,
+                                                    "Unable to determine the used declarations.\n" + e) );
         }
         return result;
     }
@@ -232,8 +232,8 @@
         } catch ( final Exception e ) {
             context.getErrors().add( new RuleError( context.getRule(),
                                                     descr,
-                                                    null,
-                                                    "Unable to determine the used declarations" ) );
+                                                    e,
+                                                    "Unable to determine the used declarations.\n" + e) );
         }
         return result;
     }

Modified: labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java
===================================================================
--- labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java	2007-08-11 15:26:31 UTC (rev 14157)
+++ labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/rule/builder/dialect/mvel/MVELDialect.java	2007-08-11 15:27:39 UTC (rev 14158)
@@ -342,7 +342,7 @@
             context.getErrors().add( new RuleError( context.getRule(),
                                                     descr,
                                                     e,
-                                                    "Unable to determine the used declarations\n" + e.getMessage()) );
+                                                    "Unable to determine the used declarations.\n" + e.getMessage()) );
         }
         return result;
     }