Move bitwise functions to Function folder

Author: fredericDelaporte

src/NHibernate/Dialect/Dialect.cs

@@ -117,10 +117,10 @@ protected Dialect()
 			RegisterFunction("year", new SQLFunctionTemplate(NHibernateUtil.Int32, "extract(year from ?1)"));
 
 			// Bitwise operations
-			RegisterFunction("band", new BitwiseNativeOperation("&"));
-			RegisterFunction("bor", new BitwiseNativeOperation("|"));
-			RegisterFunction("bxor", new BitwiseNativeOperation("^"));
-			RegisterFunction("bnot", new BitwiseNativeOperation("~", true));
+			RegisterFunction("band", new Function.BitwiseNativeOperation("&"));
+			RegisterFunction("bor", new Function.BitwiseNativeOperation("|"));
+			RegisterFunction("bxor", new Function.BitwiseNativeOperation("^"));
+			RegisterFunction("bnot", new Function.BitwiseNativeOperation("~", true));
 
 			RegisterFunction("str", new SQLFunctionTemplate(NHibernateUtil.String, "cast(?1 as char)"));
 

src/NHibernate/Dialect/FirebirdDialect.cs

@@ -422,10 +422,10 @@ private void OverrideStandardHQLFunctions()
 			RegisterFunction("sysdate", new CastedFunction("today", NHibernateUtil.Date));
 			RegisterFunction("date", new SQLFunctionTemplate(NHibernateUtil.Date, "cast(?1 as date)"));
 			// Bitwise operations
-			RegisterFunction("band", new BitwiseFunctionOperation("bin_and"));
-			RegisterFunction("bor", new BitwiseFunctionOperation("bin_or"));
-			RegisterFunction("bxor", new BitwiseFunctionOperation("bin_xor"));
-			RegisterFunction("bnot", new BitwiseFunctionOperation("bin_not"));
+			RegisterFunction("band", new Function.BitwiseFunctionOperation("bin_and"));
+			RegisterFunction("bor", new Function.BitwiseFunctionOperation("bin_or"));
+			RegisterFunction("bxor", new Function.BitwiseFunctionOperation("bin_xor"));
+			RegisterFunction("bnot", new Function.BitwiseFunctionOperation("bin_not"));
 		}
 
 		private void RegisterFirebirdServerEmbeddedFunctions()

…nate/Dialect/BitwiseFunctionOperation.cs …ect/Function/BitwiseFunctionOperation.cssrc/NHibernate/Dialect/BitwiseFunctionOperation.cs renamed to src/NHibernate/Dialect/Function/BitwiseFunctionOperation.cs src/NHibernate/Dialect/BitwiseFunctionOperation.cs renamed to src/NHibernate/Dialect/Function/BitwiseFunctionOperation.cs

@@ -1,11 +1,27 @@
 using System;
 using System.Collections;
-using NHibernate.Dialect.Function;
 using NHibernate.Engine;
 using NHibernate.SqlCommand;
 using NHibernate.Type;
 
+// 6.0 TODO: remove NHibernate.Dialect.BitwiseFunctionOperation,
+// and remove "Function." prefix where the non obsolete one is used.
 namespace NHibernate.Dialect
+{
+	/// <inheritdoc />
+	[Serializable]
+	// Since 5.2
+	[Obsolete("Use NHibernate.Dialect.Function.BitwiseFunctionOperation instead")]
+	public class BitwiseFunctionOperation : Function.BitwiseFunctionOperation
+	{
+		/// <inheritdoc />
+		public BitwiseFunctionOperation(string functionName): base(functionName)
+		{
+		}
+	}
+}
+
+namespace NHibernate.Dialect.Function
 {
 	/// <summary>
 	/// Treats bitwise operations as SQL function calls.

…ernate/Dialect/BitwiseNativeOperation.cs …alect/Function/BitwiseNativeOperation.cssrc/NHibernate/Dialect/BitwiseNativeOperation.cs renamed to src/NHibernate/Dialect/Function/BitwiseNativeOperation.cs src/NHibernate/Dialect/BitwiseNativeOperation.cs renamed to src/NHibernate/Dialect/Function/BitwiseNativeOperation.cs

@@ -1,11 +1,32 @@
 using System;
 using System.Collections;
-using NHibernate.Dialect.Function;
 using NHibernate.Engine;
 using NHibernate.SqlCommand;
 using NHibernate.Type;
 
+// 6.0 TODO: remove NHibernate.Dialect.BitwiseNativeOperation,
+// and remove "Function." prefix where the non obsolete one is used.
 namespace NHibernate.Dialect
+{
+	/// <inheritdoc />
+	[Serializable]
+	// Since 5.2
+	[Obsolete("Use NHibernate.Dialect.Function.BitwiseNativeOperation instead")]
+	public class BitwiseNativeOperation : Function.BitwiseNativeOperation
+	{
+		/// <inheritdoc />
+		public BitwiseNativeOperation(string sqlOpToken) : base(sqlOpToken)
+		{
+		}
+
+		/// <inheritdoc />
+		public BitwiseNativeOperation(string sqlOpToken, bool isNot) : base(sqlOpToken, isNot)
+		{
+		}
+	}
+}
+
+namespace NHibernate.Dialect.Function
 {
 	/// <summary>
 	/// Treats bitwise operations as native operations.
@@ -14,7 +35,7 @@ namespace NHibernate.Dialect
 	public class BitwiseNativeOperation : ISQLFunction
 	{
 		private readonly string _sqlOpToken;
-		private readonly bool _isNot;
+		private readonly bool _isUnary;
 
 		/// <summary>
 		/// Creates an instance using the giving token.
@@ -34,11 +55,11 @@ public BitwiseNativeOperation(string sqlOpToken)
 		/// Creates an instance using the giving token and the flag indicating if it is an unary operator.
 		/// </summary>
 		/// <param name="sqlOpToken">The operation token.</param>
-		/// <param name="isNot">Whether the operation is unary or not.</param>
-		public BitwiseNativeOperation(string sqlOpToken, bool isNot)
+		/// <param name="isUnary">Whether the operation is unary or not.</param>
+		public BitwiseNativeOperation(string sqlOpToken, bool isUnary)
 		{
 			_sqlOpToken = sqlOpToken;
-			_isNot = isNot;
+			_isUnary = isUnary;
 		}
 
 		#region ISQLFunction Members
@@ -63,11 +84,11 @@ public SqlString Render(IList args, ISessionFactoryImplementor factory)
 
 			var sqlBuffer = new SqlStringBuilder();
 
-			if (!_isNot)
+			if (!_isUnary)
 				AddToBuffer(args[0], sqlBuffer);
 
 			sqlBuffer.Add(" ").Add(_sqlOpToken).Add(" ");
-			for (var i = _isNot ? 0 : 1; i < args.Count; i++)
+			for (var i = _isUnary ? 0 : 1; i < args.Count; i++)
 			{
 				AddToBuffer(args[i], sqlBuffer);
 			}

src/NHibernate/Dialect/HanaDialectBase.cs

@@ -384,10 +384,10 @@ protected virtual void RegisterFunctions()
 
 		protected virtual void RegisterNHibernateFunctions()
 		{
-			RegisterFunction("band", new BitwiseFunctionOperation("bitand"));
-			RegisterFunction("bor", new BitwiseFunctionOperation("bitor"));
-			RegisterFunction("bxor", new BitwiseFunctionOperation("bitxor"));
-			RegisterFunction("bnot", new BitwiseFunctionOperation("bitnot"));
+			RegisterFunction("band", new Function.BitwiseFunctionOperation("bitand"));
+			RegisterFunction("bor", new Function.BitwiseFunctionOperation("bitor"));
+			RegisterFunction("bxor", new Function.BitwiseFunctionOperation("bitxor"));
+			RegisterFunction("bnot", new Function.BitwiseFunctionOperation("bitnot"));
 			RegisterFunction("bit_length", new SQLFunctionTemplate(NHibernateUtil.Int32, "length(to_binary(?1))*8"));
 			RegisterFunction("ceiling", new StandardSQLFunction("ceil"));
 			RegisterFunction("chr", new StandardSQLFunction("char", NHibernateUtil.AnsiChar));

src/NHibernate/Dialect/Oracle8iDialect.cs

@@ -303,7 +303,7 @@ protected virtual void RegisterFunctions()
 
 			RegisterFunction("iif", new SQLFunctionTemplate(null, "case when ?1 then ?2 else ?3 end"));
 
-			RegisterFunction("band", new BitwiseFunctionOperation("bitand"));
+			RegisterFunction("band", new Function.BitwiseFunctionOperation("bitand"));
 			RegisterFunction("bor", new SQLFunctionTemplate(null, "?1 + ?2 - BITAND(?1, ?2)"));
 			RegisterFunction("bxor", new SQLFunctionTemplate(null, "?1 + ?2 - BITAND(?1, ?2) * 2"));
 			RegisterFunction("bnot", new SQLFunctionTemplate(null, "(-1 - ?1)"));

src/NHibernate/Dialect/PostgreSQLDialect.cs

@@ -84,7 +84,7 @@ public PostgreSQLDialect()
 			RegisterFunction("atan2", new StandardSQLFunction("atan2", NHibernateUtil.Double));
 
 			RegisterFunction("power", new StandardSQLFunction("power", NHibernateUtil.Double));
-			RegisterFunction("bxor", new BitwiseNativeOperation("#"));
+			RegisterFunction("bxor", new Function.BitwiseNativeOperation("#"));
 
 			RegisterFunction("floor", new StandardSQLFunction("floor"));
 			RegisterFunction("ceiling", new StandardSQLFunction("ceiling"));