fix #651 create a better Delete extension method if you want to delete an document without specifying a CLR type for the operation

Author: Mpdreamz

src/Nest/ConvenienceExtensions/DeleteExtensions.cs

@@ -9,6 +9,24 @@ namespace Nest
 	/// </summary>
 	public static class DeleteExtensions
 	{
+
+		/// <summary>
+		///The delete API allows to delete a typed JSON document from a specific index based on its id. 
+		/// <para> </para>>http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-delete.html
+		/// </summary>
+		/// <typeparam name="T">The type used to infer the default index and typename</typeparam>
+		/// <param name="client"></param>
+		/// <param name="index">The name of the index as string</param>
+		/// <param name="type">The type name of the document you wish to delete</param>
+		/// <param name="id">The id as string of the document you want to delete</param>
+		/// <param name="selector">An optional descriptor to further describe the delete operation</param>
+		public static IDeleteResponse Delete(this IElasticClient client, string index, string type, string id, 
+			Func<DeleteDescriptor<object>, DeleteDescriptor<object>> selector = null) 
+		{
+			selector = selector ?? (s => s);
+			return client.Delete<object>(s => selector(s.Index(index).Type(type).Id(id)));
+		}
+		
 		/// <summary>
 		///The delete API allows to delete a typed JSON document from a specific index based on its id. 
 		/// <para> </para>>http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-delete.html
@@ -51,6 +69,23 @@ public static Task<IDeleteResponse> DeleteAsync<T>(this IElasticClient client, l
 			return client.DeleteAsync<T>(s => selector(s.Id(id)));
 		}
 
+		/// <summary>
+		///The delete API allows to delete a typed JSON document from a specific index based on its id. 
+		/// <para> </para>>http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-delete.html
+		/// </summary>
+		/// <typeparam name="T">The type used to infer the default index and typename</typeparam>
+		/// <param name="client"></param>
+		/// <param name="index">The name of the index as string</param>
+		/// <param name="type">The type name of the document you wish to delete</param>
+		/// <param name="id">The id as string of the document you want to delete</param>
+		/// <param name="selector">An optional descriptor to further describe the delete operation</param>
+		public static Task<IDeleteResponse> DeleteAsync(this IElasticClient client, 
+			string index, string type, string id, 
+			Func<DeleteDescriptor<object>, DeleteDescriptor<object>> selector = null) 
+		{
+			selector = selector ?? (s => s);
+			return client.DeleteAsync<object>(s => selector(s.Index(index).Type(type).Id(id)));
+		}
 		/// <summary>
 		///The delete API allows to delete a typed JSON document from a specific index based on its id. 
 		/// <para> </para>>http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-delete.html

src/Tests/Nest.Tests.Integration/Core/DeleteTests.cs

@@ -17,7 +17,15 @@ protected override void ResetIndexes()
 			IntegrationSetup.TearDown();
 			IntegrationSetup.Setup();
 		}
-
+		
+		[Test]
+		public void ShouldNotThrowOnIdOverload()
+		{
+			Assert.Throws<DispatchException>(() =>
+			{
+				this._client.Delete("id", d=>d.Index("someindex").Type("sometype"));
+			});
+		}
 
 		[Test]
 		public void ShouldThowOnNullId()