From: <jbo...@li...> - 2006-03-10 02:49:50
|
Author: mar...@jb... Date: 2006-03-09 21:49:45 -0500 (Thu, 09 Mar 2006) New Revision: 2859 Modified: trunk/labs/jbossrules/drools-core/src/main/java/org/drools/rule/Rule.java Log: -RuleBuilder now correctly builds rules so that declarations are determined on pattern addition Modified: trunk/labs/jbossrules/drools-core/src/main/java/org/drools/rule/Rule.java =================================================================== --- trunk/labs/jbossrules/drools-core/src/main/java/org/drools/rule/Rule.java 2006-03-10 02:08:51 UTC (rev 2858) +++ trunk/labs/jbossrules/drools-core/src/main/java/org/drools/rule/Rule.java 2006-03-10 02:49:45 UTC (rev 2859) @@ -305,17 +305,17 @@ * The <code>Test</code> to add. * @throws InvalidRuleException */ - public void addPattern(ConditionalElement ce) throws InvalidRuleException { + public void addPattern(ConditionalElement ce) { addDeclarations( ce ); this.lhsRoot.addChild( ce ); } - public void addPattern(Column column) throws InvalidRuleException { + public void addPattern(Column column) { addDeclarations( column ); this.lhsRoot.addChild( column ); } - private void addDeclarations(Column column) throws InvalidRuleException { + private void addDeclarations(Column column) { // Check if the column is bound and if so add it as a declaration if ( column.isBound() ) { Declaration declaration = column.getDeclaration(); @@ -332,7 +332,7 @@ } } - private void addDeclarations(ConditionalElement ce) throws InvalidRuleException { + private void addDeclarations(ConditionalElement ce) { for ( Iterator it = ce.getChildren().iterator(); it.hasNext(); ) { Object object = it.next(); if ( object instanceof Column ) { |