From 120c7b973059cefef32b4fa0fea4206e5f220f1f Mon Sep 17 00:00:00 2001 From: Umar Bolatov Date: Mon, 29 Mar 2021 22:26:33 -0700 Subject: [PATCH] Ensure submitting empty currency does not break update PoS page (#2376) --- BTCPayServer/Controllers/AppsController.PointOfSale.cs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/BTCPayServer/Controllers/AppsController.PointOfSale.cs b/BTCPayServer/Controllers/AppsController.PointOfSale.cs index 6830c2fc8..fb46bf12d 100644 --- a/BTCPayServer/Controllers/AppsController.PointOfSale.cs +++ b/BTCPayServer/Controllers/AppsController.PointOfSale.cs @@ -161,6 +161,11 @@ namespace BTCPayServer.Controllers [Route("{appId}/settings/pos")] public async Task UpdatePointOfSale(string appId, UpdatePointOfSaleViewModel vm) { + if (!ModelState.IsValid) + { + return View(vm); + } + if (_currencies.GetCurrencyData(vm.Currency, false) == null) ModelState.AddModelError(nameof(vm.Currency), "Invalid currency"); try