Fixed NullReferenceException when requests are unsuccessful and no request configuration is setup

gmarz · gmarz · commit 64926240671b · 2014-05-12T08:35:41.000-04:00
diff --git a/src/Nest/ElasticClient.cs b/src/Nest/ElasticClient.cs
@@ -94,9 +94,7 @@ D descriptor
 			where R : BaseResponse
 		{
 			var config = descriptor.RequestConfiguration as IRequestConfiguration;
-			var statusCodeAllowed = config == null 
-				|| (config.AllowedStatusCodes.HasAny() 
-				&& config.AllowedStatusCodes.Any(i=>i==c.HttpStatusCode));
+			var statusCodeAllowed = config != null ? config.AllowedStatusCodes.Any(i => i == c.HttpStatusCode) : false;
 
 			if (c.Success || statusCodeAllowed)
 			{

Original file line number	Diff line number	Diff line change
`@@ -94,9 +94,7 @@ D descriptor`
`94`	`94`	`where R : BaseResponse`
`95`	`95`	`{`
`96`	`96`	`var config = descriptor.RequestConfiguration as IRequestConfiguration;`
`97`		`- var statusCodeAllowed = config == null`
`98`		`- \|\| (config.AllowedStatusCodes.HasAny()`
`99`		`- && config.AllowedStatusCodes.Any(i=>i==c.HttpStatusCode));`
	`97`	`+ var statusCodeAllowed = config != null ? config.AllowedStatusCodes.Any(i => i == c.HttpStatusCode) : false;`
`100`	`98`
`101`	`99`	`if (c.Success \|\| statusCodeAllowed)`
`102`	`100`	`{`