diff --git a/src/app/add_invoice.rs b/src/app/add_invoice.rs index 9dc3c031..2b517eb6 100644 --- a/src/app/add_invoice.rs +++ b/src/app/add_invoice.rs @@ -73,7 +73,7 @@ pub async fn add_invoice_action( { Ok(_) => payment_request, Err(_) => { - send_new_order_msg(None, Action::IncorrectInvoiceAmount, None, &event.sender) + send_new_order_msg(Some(order.id), Action::IncorrectInvoiceAmount, None, &event.sender) .await; return Ok(()); } diff --git a/src/app/take_buy.rs b/src/app/take_buy.rs index 34fb5127..cc4cbecf 100644 --- a/src/app/take_buy.rs +++ b/src/app/take_buy.rs @@ -45,20 +45,6 @@ pub async fn take_buy_action( return Ok(()); } - // Get amount request if user requested one for range order - fiat amount will be used below - if let Some(am) = get_fiat_amount_requested(&order, &msg) { - order.fiat_amount = am; - } else { - send_new_order_msg( - Some(order.id), - Action::OutOfRangeFiatAmount, - None, - &event.sender, - ) - .await; - return Ok(()); - } - let order_status = match Status::from_str(&order.status) { Ok(s) => s, Err(e) => { @@ -90,6 +76,21 @@ pub async fn take_buy_action( return Ok(()); } } + + // Get amount request if user requested one for range order - fiat amount will be used below + if let Some(am) = get_fiat_amount_requested(&order, &msg) { + order.fiat_amount = am; + } else { + send_new_order_msg( + Some(order.id), + Action::OutOfRangeFiatAmount, + None, + &event.sender, + ) + .await; + return Ok(()); + } + // Check market price value in sats - if order was with market price then calculate if order.amount == 0 { let (new_sats_amount, fee) = diff --git a/src/app/take_sell.rs b/src/app/take_sell.rs index 9e4c4eb7..078b67ae 100644 --- a/src/app/take_sell.rs +++ b/src/app/take_sell.rs @@ -34,20 +34,6 @@ pub async fn take_sell_action( } }; - // Get amount request if user requested one for range order - fiat amount will be used below - if let Some(am) = get_fiat_amount_requested(&order, &msg) { - order.fiat_amount = am; - } else { - send_new_order_msg( - Some(order.id), - Action::OutOfRangeFiatAmount, - None, - &event.sender, - ) - .await; - return Ok(()); - } - // Maker can't take own order if order.creator_pubkey == event.sender.to_hex() { send_cant_do_msg(Some(order.id), None, &event.sender).await; @@ -110,6 +96,20 @@ pub async fn take_sell_action( return Ok(()); } } + + // Get amount request if user requested one for range order - fiat amount will be used below + if let Some(am) = get_fiat_amount_requested(&order, &msg) { + order.fiat_amount = am; + } else { + send_new_order_msg( + Some(order.id), + Action::OutOfRangeFiatAmount, + None, + &event.sender, + ) + .await; + return Ok(()); + } // Add buyer pubkey to order order.buyer_pubkey = Some(buyer_pubkey.to_string());