[jboss-svn-commits] JBL Code SVN: r15463 - labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/brms/server/rules.

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

Author: michael.neale at jboss.com
Date: 2007-10-01 00:15:50 -0400 (Mon, 01 Oct 2007)
New Revision: 15463

Modified:
   labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/brms/server/rules/SuggestionCompletionLoader.java
Log:
fix for NPE (accidental regression)

Modified: labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/brms/server/rules/SuggestionCompletionLoader.java
===================================================================
--- labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/brms/server/rules/SuggestionCompletionLoader.java	2007-10-01 02:06:11 UTC (rev 15462)
+++ labs/jbossrules/trunk/drools-compiler/src/main/java/org/drools/brms/server/rules/SuggestionCompletionLoader.java	2007-10-01 04:15:50 UTC (rev 15463)
@@ -306,10 +306,8 @@
             this.builder.addFactType( factType );
 
             final String[] fields = new String[templ.getFields().size()];
-            Arrays.sort(fields);
-            this.builder.addFieldsForType( factType,
-                                           fields );
 
+
             int index = 0;
             for ( final Iterator fieldsIt = templ.getFields().iterator(); fieldsIt.hasNext(); ) {
                 final FieldTemplateDescr fieldDescr = (FieldTemplateDescr) fieldsIt.next();
@@ -325,6 +323,11 @@
                 this.builder.addFieldType( factType + "." + fieldDescr.getName(),
                                            getFieldType( fieldTypeClass ) );
             }
+
+            Arrays.sort(fields);
+            this.builder.addFieldsForType( factType,
+                    fields );
+
         }
     }