Merge pull request #764 from elasticsearch/fix/sort-methods

Replace confusing ToggleSort(bool) with Order(SortOrder) on sort descrip...

Author: Mpdreamz

src/Nest/DSL/Search/SortFieldDescriptor.cs

@@ -125,6 +125,12 @@ public virtual SortFieldDescriptor<T> Descending()
 			return this;
 		}
 
+		public virtual SortFieldDescriptor<T> Order(SortOrder order)
+		{
+			Self.Order = order;
+			return this;
+		}
+
         public virtual SortFieldDescriptor<T> NestedMin()
         {
             Self.Mode = ScoreMode.Min;
@@ -169,14 +175,5 @@ public SortFieldDescriptor<T> NestedPath(Expression<Func<T, object>> objectPath)
 			Self.NestedPath = objectPath;
 			return this;
 		}
-
-		/// <summary>
-		/// Pass true to sort ascending false to sort descending
-		/// </summary>
-		public virtual SortFieldDescriptor<T> ToggleSort(bool ascending)
-		{
-			Self.Order = ascending ? SortOrder.Descending : SortOrder.Ascending;
-			return this;
-		}
 	}
 }

src/Nest/DSL/Search/SortGeoDistanceDescriptor.cs

@@ -111,12 +111,10 @@ public SortGeoDistanceDescriptor<T> Descending()
 			Self.Order = SortOrder.Descending;
 			return this;
 		}
-		/// <summary>
-		/// Pass true to sort ascending false to sort descending
-		/// </summary>
-		public SortGeoDistanceDescriptor<T> ToggleSort(bool ascending)
+
+		public SortGeoDistanceDescriptor<T> Order(SortOrder order)
 		{
-			Self.Order = ascending ? SortOrder.Ascending : SortOrder.Descending;
+			Self.Order = order;
 			return this;
 		}
 

src/Nest/DSL/Search/SortScriptDescriptor.cs

@@ -93,12 +93,9 @@ public virtual SortScriptDescriptor<T> Descending()
 			return this;
 		}
 
-		/// <summary>
-		/// Pass true to sort ascending false to sort descending
-		/// </summary>
-		public SortScriptDescriptor<T> ToggleSort(bool ascending)
+		public SortScriptDescriptor<T> Order(SortOrder order)
 		{
-			Self.Order = ascending ? SortOrder.Ascending : SortOrder.Descending;
+			Self.Order = order;
 			return this;
 		}
 	}