Add null checks for Order.Price

Author: vslee

src/ExchangeSharp/API/Exchanges/BL3P/ExchangeBL3PAPI.cs

@@ -252,9 +252,11 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 			switch (order.OrderType)
 			{
 				case OrderType.Limit:
+					if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 					data["price_int"] = converterToFive.FromDecimal(order.Price.Value);
 					break;
 				case OrderType.Market:
+					if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 					data["amount_funds_int"] = converterToFive.FromDecimal(roundedAmount * order.Price.Value);
 					break;
 				default:

src/ExchangeSharp/API/Exchanges/BinanceGroup/BinanceGroupCommon.cs

@@ -548,6 +548,7 @@ protected override async Task<Dictionary<string, decimal>> OnGetAmountsAvailable
 
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
+			if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 			Dictionary<string, object> payload = await GetNoncePayloadAsync();
 			payload["symbol"] = order.MarketSymbol;
 			payload["newClientOrderId"] = order.ClientOrderId;

src/ExchangeSharp/API/Exchanges/Bitfinex/ExchangeBitfinexAPI.cs

@@ -481,6 +481,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
 			if (order.OrderType != OrderType.Market)
 			{
+				if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 				payload["price"] = (await ClampOrderPrice(marketSymbol, order.Price.Value)).ToStringInvariant();
 			}
 			else

src/ExchangeSharp/API/Exchanges/Bitstamp/ExchangeBitstampAPI.cs

@@ -202,7 +202,8 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
             if (order.OrderType != OrderType.Market)
             {
-                payload["price"] = order.Price.ToStringInvariant();
+				if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
+				payload["price"] = order.Price.ToStringInvariant();
             }
 
             payload["amount"] = order.RoundAmount().ToStringInvariant();

src/ExchangeSharp/API/Exchanges/Bittrex/ExchangeBittrexAPI.cs

@@ -378,6 +378,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 		{
 
 			decimal orderAmount = await ClampOrderQuantity(order.MarketSymbol, order.Amount);
+			if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 			decimal orderPrice = await ClampOrderPrice(order.MarketSymbol, order.Price.Value);
 			string url = "/orders";
 			Dictionary<string, object> orderParams = await GetNoncePayloadAsync();

src/ExchangeSharp/API/Exchanges/Coinbase/ExchangeCoinbaseAPI.cs

@@ -573,12 +573,14 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 					// set payload["post_only"] to true for default scenario when order.ExtraParameters["post_only"] is not specified
 					// to place non-post-only limit order one can set and pass order.ExtraParameters["post_only"]="false"
 					payload["post_only"] = order.ExtraParameters.TryGetValueOrDefault("post_only", "true");
+					if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 					payload["price"] = order.Price.ToStringInvariant();
 					break;
 
 				case OrderType.Stop:
 					payload["stop"] = (order.IsBuy ? "entry" : "loss");
 					payload["stop_price"] = order.StopPrice.ToStringInvariant();
+					if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 					payload["type"] = order.Price > 0m ? "limit" : "market";
 					break;
 

src/ExchangeSharp/API/Exchanges/Huobi/ExchangeHuobiAPI.cs

@@ -693,7 +693,8 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
             else
             {
                 payload["type"] += "-limit";
-                payload["price"] = outputPrice.ToStringInvariant();
+				if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
+				payload["price"] = outputPrice.ToStringInvariant();
             }
 
             order.ExtraParameters.CopyTo(payload);

src/ExchangeSharp/API/Exchanges/Kraken/ExchangeKrakenAPI.cs

@@ -761,6 +761,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 			if (order.OrderType != OrderType.Market)
 			{
 				int precision = BitConverter.GetBytes(Decimal.GetBits((decimal)market.PriceStepSize)[3])[2];
+				if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
 				payload.Add("price", Math.Round(order.Price.Value, precision).ToStringInvariant());
 			}
 			order.ExtraParameters.CopyTo(payload);

src/ExchangeSharp/API/Exchanges/OKGroup/OKGroupCommon.cs

@@ -413,7 +413,8 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
             // Okex has strict rules on which prices and quantities are allowed. They have to match the rules defined in the market definition.
             decimal outputQuantity = await ClampOrderQuantity(order.MarketSymbol, order.Amount);
-            decimal outputPrice = await ClampOrderPrice(order.MarketSymbol, order.Price.Value);
+			if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
+			decimal outputPrice = await ClampOrderPrice(order.MarketSymbol, order.Price.Value);
 
             if (order.OrderType == OrderType.Market)
             {

src/ExchangeSharp/API/Exchanges/Poloniex/ExchangePoloniexAPI.cs

@@ -769,7 +769,8 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
             }
 
             decimal orderAmount = await ClampOrderQuantity(order.MarketSymbol, order.Amount);
-            decimal orderPrice = await ClampOrderPrice(order.MarketSymbol, order.Price.Value);
+			if (order.Price == null) throw new ArgumentNullException(nameof(order.Price));
+			decimal orderPrice = await ClampOrderPrice(order.MarketSymbol, order.Price.Value);
 
             List<object> orderParams = new List<object>
             {