[jboss-svn-commits] JBL Code SVN: r31163 - labs/jbossrules/branches/guvnor_expressionEditor2_baunax_esteban/drools-compiler/src/main/java/org/drools/guvnor/client/modeldriven/brl.

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

Author: baunax
Date: 2010-01-20 15:22:21 -0500 (Wed, 20 Jan 2010)
New Revision: 31163

Modified:
   labs/jbossrules/branches/guvnor_expressionEditor2_baunax_esteban/drools-compiler/src/main/java/org/drools/guvnor/client/modeldriven/brl/RuleModel.java
Log:
not returning null from getBoundFacts

Modified: labs/jbossrules/branches/guvnor_expressionEditor2_baunax_esteban/drools-compiler/src/main/java/org/drools/guvnor/client/modeldriven/brl/RuleModel.java
===================================================================
--- labs/jbossrules/branches/guvnor_expressionEditor2_baunax_esteban/drools-compiler/src/main/java/org/drools/guvnor/client/modeldriven/brl/RuleModel.java	2010-01-20 20:21:14 UTC (rev 31162)
+++ labs/jbossrules/branches/guvnor_expressionEditor2_baunax_esteban/drools-compiler/src/main/java/org/drools/guvnor/client/modeldriven/brl/RuleModel.java	2010-01-20 20:22:21 UTC (rev 31163)
@@ -1,6 +1,7 @@
 package org.drools.guvnor.client.modeldriven.brl;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 public class RuleModel
@@ -104,7 +105,7 @@
      */
     public List<String> getBoundFacts() {
         if ( this.lhs == null ) {
-            return null;
+            return Collections.emptyList();
         }
         final List<String> list = new ArrayList<String>();
         for ( int i = 0; i < this.lhs.length; i++ ) {
@@ -113,10 +114,7 @@
                 if ( p.boundName != null ) {
                     list.add( p.boundName );
                 }
-                List<String> fieldBindings = getListFieldBinding( p );
-                if ( fieldBindings != null ) {
-                    list.addAll( fieldBindings );
-                }
+                list.addAll( getListFieldBinding( p ) );
             }
         }
         return list;