[teiid-commits] teiid SVN: r4470 - trunk/engine/src/main/java/org/teiid/query/validator.

teiid-commits at lists.jboss.org teiid-commits at lists.jboss.org
Tue Sep 25 14:56:07 EDT 2012


Author: shawkins
Date: 2012-09-25 14:56:07 -0400 (Tue, 25 Sep 2012)
New Revision: 4470

Modified:
   trunk/engine/src/main/java/org/teiid/query/validator/ValidationVisitor.java
Log:
TEIID-2222 fix for npe with dyanmic sql in trigger

Modified: trunk/engine/src/main/java/org/teiid/query/validator/ValidationVisitor.java
===================================================================
--- trunk/engine/src/main/java/org/teiid/query/validator/ValidationVisitor.java	2012-09-25 18:19:53 UTC (rev 4469)
+++ trunk/engine/src/main/java/org/teiid/query/validator/ValidationVisitor.java	2012-09-25 18:56:07 UTC (rev 4470)
@@ -408,10 +408,12 @@
 
     public void visit(CreateProcedureCommand obj) {
         //check that the procedure does not contain references to itself
-        if (GroupCollectorVisitor.getGroups(obj,true).contains(obj.getVirtualGroup())) {
-        	handleValidationError(QueryPlugin.Util.getString("ValidationVisitor.Procedure_has_group_self_reference"),obj); //$NON-NLS-1$
-        }
-        this.createProc = obj;
+    	if (obj.getUpdateType() == null) {
+	        if (GroupCollectorVisitor.getGroups(obj,true).contains(obj.getVirtualGroup())) {
+	        	handleValidationError(QueryPlugin.Util.getString("ValidationVisitor.Procedure_has_group_self_reference"),obj); //$NON-NLS-1$
+	        }
+	        this.createProc = obj;
+    	}
     }
 
     public void visit(CompoundCriteria obj) {



More information about the teiid-commits mailing list