aljaz/btcpayserver
1
0
Fork 0
mirror of https://github.com/aljazceru/btcpayserver.git synced 2025-12-19 06:54:19 +01:00
Code Issues Packages Projects Releases Wiki Activity

Update validation of crowdfund app settings (#3708)

Browse Source
This commit is contained in:
Umar Bolatov
2022-05-04 01:34:40 -07:00
committed by GitHub
parent e5feda69c8
commit 642aad28ac
3 changed files with 51 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
BTCPayServer/Controllers/UIAppsController.Crowdfund.cs
View File

@@ -99,27 +99,29 @@ namespace BTCPayServer.Controllers
ModelState.AddModelError(nameof(vm.ResetEveryAmount), "You must reset the goal at a minimum of 1 ");
}
if (vm.DisplayPerksRanking && !vm.SortPerksByPopularity)
if (vm.DisplayPerksRanking)
{
ModelState.AddModelError(nameof(vm.DisplayPerksRanking), "You must sort by popularity in order to display ranking.");
vm.SortPerksByPopularity = true;
}
var parsedSounds = vm.Sounds.Split(
var parsedSounds = vm.Sounds?.Split(
new[] { "\r\n", "\r", "\n" },
StringSplitOptions.None
).Select(s => s.Trim()).ToArray();
if (vm.SoundsEnabled && !parsedSounds.Any())
if (vm.SoundsEnabled && (parsedSounds == null || !parsedSounds.Any()))
{
ModelState.AddModelError(nameof(vm.Sounds), "You must have at least one sound if you enable sounds");
vm.SoundsEnabled = false;
parsedSounds = new CrowdfundSettings().Sounds;
}
var parsedAnimationColors = vm.AnimationColors.Split(
var parsedAnimationColors = vm.AnimationColors?.Split(
new[] { "\r\n", "\r", "\n" },
StringSplitOptions.None
).Select(s => s.Trim()).ToArray();
if (vm.AnimationsEnabled && !parsedAnimationColors.Any())
if (vm.AnimationsEnabled && (parsedAnimationColors == null || !parsedAnimationColors.Any()))
{
ModelState.AddModelError(nameof(vm.AnimationColors), "You must have at least one animation color if you enable animations");
vm.AnimationsEnabled = false;
parsedAnimationColors = new CrowdfundSettings().AnimationColors;
}
if (!ModelState.IsValid)