Added better error messages for situation where required assemblies to

create the connection and command are not in the bin or gac.


SVN: trunk@1149

Author: Mike Doerfler

src/NHibernate/Driver/FirebirdDriver.cs

@@ -1,6 +1,4 @@
 using System;
-using System.Data;
-using System.Reflection;
 
 namespace NHibernate.Driver
 {
@@ -16,6 +14,16 @@ public FirebirdDriver()
 		{
 			connectionType = System.Type.GetType("FirebirdSql.Data.Firebird.FbConnection, FirebirdSql.Data.Firebird");
 			commandType = System.Type.GetType("FirebirdSql.Data.Firebird.FbCommand, FirebirdSql.Data.Firebird");
+			
+			if( connectionType==null || commandType==null )
+			{
+				throw new HibernateException( 
+					"The IDbCommand and IDbConnection implementation in the Assembly FirebirdSql.Data could not be found.  " +					
+					"Please ensure that the Assemblies needed to communicate with Firebird " +
+					"are in the Global Assembly Cache or in a location that NHibernate " +
+					"can use System.Type.GetType(string) to load the types from."
+					);
+			}
 		}
 
 		public override System.Type CommandType

src/NHibernate/Driver/MySqlDataDriver.cs

@@ -1,9 +1,4 @@
 using System;
-using System.Data;
-using System.Reflection;
-
-using NHibernate.Dialect;
-using NHibernate.SqlCommand;
 
 namespace NHibernate.Driver 
 {
@@ -18,7 +13,8 @@ namespace NHibernate.Driver
 	/// the Assembly <c>MySql.Data.dll</c>.
 	/// </p>
 	/// <p>
-	/// Please check the products website <a href="http://www.mysql.com/products/connector/net/">http://www.mysql.com/products/connector/net/</a>
+	/// Please check the products website 
+	/// <a href="http://www.mysql.com/products/connector/net/">http://www.mysql.com/products/connector/net/</a>
 	/// for any updates and or documentation.
 	/// </p>
 	/// </remarks>
@@ -31,6 +27,16 @@ public MySqlDataDriver()
 		{
 			connectionType = System.Type.GetType("MySql.Data.MySqlClient.MySqlConnection, MySql.Data");
 			commandType = System.Type.GetType("MySql.Data.MySqlClient.MySqlCommand, MySql.Data");
+
+			if( connectionType==null || commandType==null )
+			{
+				throw new HibernateException( 
+					"The IDbCommand and IDbConnection implementation in the Assembly MySql.Data could not be found.  " +					
+					"Please ensure that the Assemblies MySql.Data.dll and ICSharpCode.SharpZipLib.dll " +
+					"are in the Global Assembly Cache or in a location that NHibernate " +
+					"can use System.Type.GetType(string) to load the types from."
+					);
+			}
 		}
 
 		public override System.Type CommandType
@@ -44,26 +50,33 @@ public override System.Type ConnectionType
 		}
 
 		/// <summary>
-		/// MySqlData does not use named parameters in the sql.
+		/// MySql.Data uses named parameters in the sql.
 		/// </summary>
-		/// <value><c>false</c> becuase MySql uses <c>?</c> in the sql.</value>
+		/// <value><c>true</c> - MySql uses <c>?</c> in the sql.</value>
 		public override bool UseNamedPrefixInSql 
 		{
 			get {return true;}
 		}
 
+		/// <summary>
+		/// 
+		/// </summary>
 		public override bool UseNamedPrefixInParameter 
 		{
 			get {return true;}
 		}
 
+		/// <summary>
+		/// MySql.Data use the <c>?</c> to locate parameters in sql.
+		/// </summary>
+		/// <value><c>?</c> is used to locate parameters in sql.</value>
 		public override string NamedPrefix 	
 		{
 			get {return "?";}
 		}
 
 		/// <summary>
-		/// The ByteFX driver does NOT support more than 1 open IDataReader
+		/// The MySql.Data driver does NOT support more than 1 open IDataReader
 		/// with only 1 IDbConnection.
 		/// </summary>
 		/// <value><c>false</c> - it is not supported.</value>
@@ -76,6 +89,7 @@ public override bool SupportsMultipleOpenReaders
 		/// With the Gamma MySql.Data provider it is throwing an exception with the 
 		/// message "Expected End of data packet" when a select command is prepared.
 		/// </summary>
+		/// <value><c>false</c> - it is not supported.</value>
 		public override bool SupportsPreparingCommands
 		{
 			get { return false; }

src/NHibernate/Driver/NpgsqlDriver.cs

@@ -1,6 +1,4 @@
 using System;
-using System.Data;
-using System.Reflection;
 
 namespace NHibernate.Driver
 {
@@ -16,7 +14,8 @@ namespace NHibernate.Driver
 	/// NHibernate to load it.
 	/// </p>
 	/// <p>
-	/// Please check the products website <a href="http://www.postgresql.org/">http://www.postgresql.org/</a>
+	/// Please check the products website 
+	/// <a href="http://www.postgresql.org/">http://www.postgresql.org/</a>
 	/// for any updates and or documentation.
 	/// </p>
 	/// <p>
@@ -33,6 +32,16 @@ public NpgsqlDriver()
 		{
 			connectionType = System.Type.GetType("Npgsql.NpgsqlConnection, Npgsql");
 			commandType = System.Type.GetType("Npgsql.NpgsqlCommand, Npgsql");
+
+			if( connectionType==null || commandType==null )
+			{
+				throw new HibernateException( 
+					"The IDbCommand and IDbConnection implementation in the Assembly Npgsql.dll could not be found.  " +					
+					"Please ensure that the Assemblies needed to communicate with PostgreSQL " +
+					"are in the Global Assembly Cache or in a location that NHibernate " +
+					"can use System.Type.GetType(string) to load the types from."
+					);
+			}
 		}
 
 		public override System.Type CommandType

src/NHibernate/Driver/OracleClientDriver.cs

@@ -5,8 +5,7 @@
 namespace NHibernate.Driver
 {
 	/// <summary>
-	/// A NHibernate Driver for using the Oracle DataProvider	
-	/// <see cref="System.Data.OracleClient.OracleConnection"/>.
+	/// A NHibernate Driver for using the Oracle DataProvider.
 	/// </summary>
 	public class OracleClientDriver: DriverBase
 	{