From 799788d366ec3a680d987e5d138f89bdd6bb0f61 Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Fri, 5 Jul 2013 18:03:16 -0500
Subject: [PATCH] IDEMPIERE-1119 Improve error message when Model Validator is
 not found

---
 .../src/org/compiere/model/ModelValidationEngine.java      | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/org.adempiere.base/src/org/compiere/model/ModelValidationEngine.java b/org.adempiere.base/src/org/compiere/model/ModelValidationEngine.java
index 68fc8cdf45..e5986554e7 100644
--- a/org.adempiere.base/src/org/compiere/model/ModelValidationEngine.java
+++ b/org.adempiere.base/src/org/compiere/model/ModelValidationEngine.java
@@ -41,6 +41,7 @@ import org.compiere.acct.Fact;
 import org.compiere.util.CLogger;
 import org.compiere.util.Env;
 import org.compiere.util.KeyNamePair;
+import org.compiere.util.Util;
 import org.osgi.service.event.Event;
 
 /**
@@ -79,7 +80,7 @@ public class ModelValidationEngine
 	/** Engine Singleton				*/
 	private static ModelValidationEngine s_engine = null;
 	/* flag to indicate a missing model validation class */
-	private static String missingModelValidationMessage = null;
+	private static String missingModelValidationMessage = "";
 
 
 	/**************************************************************************
@@ -159,7 +160,7 @@ public class ModelValidationEngine
 
 			if (validator == null)
 			{
-				missingModelValidationMessage = missingModelValidationMessage +
+				missingModelValidationMessage = missingModelValidationMessage + " Missing class " + className +
 						 (client != null ? (" on client " + client.getName()) : " global") + '\n';
 			}
 			else
@@ -273,7 +274,7 @@ public class ModelValidationEngine
 		if (AD_User_ID == 0 && AD_Role_ID == 0)
 			; // don't validate for user system on role system
 		else
-			if (missingModelValidationMessage != null) {
+			if (! Util.isEmpty(missingModelValidationMessage)) {
 				MSystem system = MSystem.get(Env.getCtx());
 				if (system.isFailOnMissingModelValidator())
 					return missingModelValidationMessage;