NH-3884 - Alternate implementation without a new method.

Author: fredericDelaporte

src/NHibernate.Test/TypesTest/ChangeDefaultTypeFixture.cs

@@ -27,15 +27,15 @@ protected override void Configure(Configuration configuration)
 			_testDefaultDateTimeType = NHibernateUtil.DateTime.Equals(_originalDefaultDateTimeType)
 				? (IType) NHibernateUtil.Timestamp
 				: NHibernateUtil.DateTime;
-			TypeFactory.SetDefaultType<DateTime>(_testDefaultDateTimeType);
+			TypeFactory.RegisterType(typeof(DateTime), _testDefaultDateTimeType, TypeFactory.EmptyAliases);
 			base.Configure(configuration);
 		}
 
 		protected override void DropSchema()
 		{
 			base.DropSchema();
 			if (_originalDefaultDateTimeType != null)
-				TypeFactory.SetDefaultType<DateTime>(_originalDefaultDateTimeType);
+				TypeFactory.RegisterType(typeof(DateTime), _originalDefaultDateTimeType, TypeFactory.EmptyAliases);
 		}
 
 		[Test]

src/NHibernate/Type/TypeFactory.cs

@@ -33,8 +33,8 @@ private enum TypeClassification
 			Length,
 			PrecisionScale
 		}
-		
-		private static readonly string[] EmptyAliases= new string[0];
+
+		public static readonly string[] EmptyAliases = new string[0];
 		private static readonly char[] PrecisionScaleSplit = { '(', ')', ',' };
 		private static readonly char[] LengthSplit = { '(', ')' };
 
@@ -92,7 +92,15 @@ private enum TypeClassification
 
 		private delegate NullableType NullableTypeCreatorDelegate(SqlType sqlType);
 
-		private static void RegisterType(System.Type systemType, IType nhibernateType, IEnumerable<string> aliases)
+		/// <summary>
+		/// <para>Defines which NHibernate type should be chosen by default for handling a given .Net type.</para>
+		/// <para>This must be done before any operation on NHibernate, including building its
+		/// <see cref="Configuration" /> and building session factory. Otherwise the behavior will be undefined.</para>
+		/// </summary>
+		/// <param name="systemType">The .Net type.</param>
+		/// <param name="nhibernateType">The NHibernate type.</param>
+		/// <param name="aliases">The additional aliases to map to the type. Use <see cref="EmptyAliases"/> if none.</param>
+		public static void RegisterType(System.Type systemType, IType nhibernateType, IEnumerable<string> aliases)
 		{
 			var typeAliases = new List<string>(aliases);
 			typeAliases.AddRange(GetClrTypeAliases(systemType));
@@ -280,25 +288,6 @@ private static void RegisterBuiltInTypes()
 								 len => new SerializableType(typeof (object), SqlTypeFactory.GetBinary(len))));
 		}
 
-		/// <summary>
-		/// <para>Defines which NHibernate type should be chosen by default for handling a given .Net type.</para>
-		/// <para>This must be done before any operation on NHibernate, including building its
-		/// <see cref="Configuration" /> and building session factory. Otherwise the behavior will be undefined.</para>
-		/// </summary>
-		/// <param name="targetType">The NHibernate type.</param>
-		/// <typeparam name="T">The .Net type.</typeparam>
-		public static void SetDefaultType<T>(IType targetType)
-		{
-			if (targetType == null)
-				throw new ArgumentNullException(nameof(targetType));
-
-			var type = typeof(T);
-			foreach (var alias in GetClrTypeAliases(type))
-			{
-				typeByTypeOfName[alias] = targetType;
-			}
-		}
-
 		private static ICollectionTypeFactory CollectionTypeFactory =>
 			Environment.BytecodeProvider.CollectionTypeFactory;