Code review changes

Author: bahusoid

src/NHibernate.Test/Criteria/AddNumberProjection.cs

@@ -35,7 +35,7 @@ public override SqlString ToSqlString(ICriteria criteria, int position, ICriteri
 				.Add(" + ")
 				.Add(criteriaQuery.NewQueryParameter(typedValue).Single())
 				.Add(") as ")
-				.Add(GetColumnAlias(0, 0 ))
+				.Add(GetColumnAlias(0))
 				.ToSqlString();
 		}
 

src/NHibernate/Criterion/AvgProjection.cs

@@ -30,7 +30,7 @@ public override SqlString ToSqlString(ICriteria criteria, int loc, ICriteriaQuer
 				sql.Add(criteriaQuery.GetColumn(criteria, propertyName));
 			}
 			sql.Add(" as ").Add(sqlType).Add(")");
-			sql.Add(") as ").Add(GetColumnAlias(loc, 0));
+			sql.Add(") as ").Add(GetColumnAlias(loc));
 			return sql.ToSqlString();
 		}
 

src/NHibernate/Criterion/CastProjection.cs

@@ -39,7 +39,7 @@ public override SqlString ToSqlString(ICriteria criteria, int position, ICriteri
 			string sqlType = factory.Dialect.GetCastTypeName(sqlTypeCodes[0]);
 			var val = CriterionUtil.GetColumnNameAsSqlStringPart(projection, criteriaQuery, criteria);
 
-			return new SqlString("cast( ", val, " as ", sqlType, ") as ", GetColumnAlias(position, 0));
+			return new SqlString("cast( ", val, " as ", sqlType, ") as ", GetColumnAlias(position));
 		}
 
 		public override IType[] GetTypes(ICriteria criteria, ICriteriaQuery criteriaQuery)

src/NHibernate/Criterion/ConditionalProjection.cs

@@ -48,7 +48,7 @@ public override SqlString ToSqlString(ICriteria criteria, int position, ICriteri
 			var ifTrue = CriterionUtil.GetColumnNameAsSqlStringPart(whenTrue, criteriaQuery, criteria);
 			var ifFalse = CriterionUtil.GetColumnNameAsSqlStringPart(whenFalse, criteriaQuery, criteria);
 			return new SqlString("(case when ", condition, " then ", ifTrue, " else ", ifFalse, " end) as ",
-			                     GetColumnAlias(position, 0));
+			                     GetColumnAlias(position));
 		}
 
 		public override IType[] GetTypes(ICriteria criteria, ICriteriaQuery criteriaQuery)

src/NHibernate/Criterion/ConstantProjection.cs

@@ -45,7 +45,7 @@ public override SqlString ToSqlString(ICriteria criteria, int position, ICriteri
 			return new SqlString(
 				criteriaQuery.NewQueryParameter(TypedValue).Single(),
 				" as ",
-				GetColumnAlias(position, 0));
+				GetColumnAlias(position));
 		}
 
 		public override IType[] GetTypes(ICriteria criteria, ICriteriaQuery criteriaQuery)

src/NHibernate/Criterion/SimpleProjection.cs

@@ -26,16 +26,16 @@ public virtual IType[] GetTypes(string alias, ICriteria criteria, ICriteriaQuery
 			return null;
 		}
 
-		// Since v5.3
+		// Since v5.4
 		[Obsolete("This method has no more usage in NHibernate and will be removed in a future version.")]
 		public virtual string[] GetColumnAliases(int loc)
 		{
-			return new[] {GetColumnAlias(loc, 0)};
+			return new[] {GetColumnAlias(loc)};
 		}
 
-		public virtual string GetColumnAlias(int position, int columnIndex)
+		protected string GetColumnAlias(int position)
 		{
-			return "y" + (position + columnIndex) + "_";
+			return "y" + position + "_";
 		}
 
 		public string[] GetColumnAliases(string alias, int position, ICriteria criteria, ICriteriaQuery criteriaQuery)
@@ -47,9 +47,9 @@ public String[] GetColumnAliases(int position, ICriteria criteria, ICriteriaQuer
 		{
 			int numColumns = this.GetColumnCount(criteria, criteriaQuery);
 			string[] aliases = new string[numColumns];
-			for (int i = 0; i < numColumns; i++) 
+			for (int i = 0; i < numColumns; i++)
 			{
-				aliases[i] = GetColumnAlias(position, i);
+				aliases[i] = GetColumnAlias(position + i);
 			}
 			return aliases;
 		}

src/NHibernate/Criterion/SqlFunctionProjection.cs

@@ -89,7 +89,7 @@ public override SqlString ToSqlString(ICriteria criteria, int position, ICriteri
 			return new SqlString(
 				sqlFunction.Render(arguments, criteriaQuery.Factory),
 				" as ",
-				GetColumnAlias(position, 0));
+				GetColumnAlias(position));
 		}
 
 		private ISQLFunction GetFunction(ICriteriaQuery criteriaQuery)