From 483611702ebd8fb8eb17cb5fa7c6d1c0667c9748 Mon Sep 17 00:00:00 2001 From: Ske Date: Thu, 18 Jul 2019 12:09:19 +0200 Subject: [PATCH] Fix error when switching with no previous switches --- PluralKit.API/Controllers/SystemController.cs | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/PluralKit.API/Controllers/SystemController.cs b/PluralKit.API/Controllers/SystemController.cs index 436d1f5d..372100f7 100644 --- a/PluralKit.API/Controllers/SystemController.cs +++ b/PluralKit.API/Controllers/SystemController.cs @@ -143,12 +143,15 @@ namespace PluralKit.API.Controllers // We get the current switch, if it exists var latestSwitch = await _switches.GetLatestSwitch(_auth.CurrentSystem); - var latestSwitchMembers = await _switches.GetSwitchMembers(latestSwitch); + if (latestSwitch != null) + { + var latestSwitchMembers = await _switches.GetSwitchMembers(latestSwitch); + + // Bail if this switch is identical to the latest one + if (latestSwitchMembers.Select(m => m.Hid).SequenceEqual(param.Members)) + return BadRequest("New members identical to existing fronters."); + } - // Bail if this switch is identical to the latest one - if (latestSwitchMembers.Select(m => m.Hid).SequenceEqual(param.Members)) - return BadRequest("New members identical to existing fronters."); - // Resolve member objects for all given IDs IEnumerable membersList; using (var conn = await _conn.Obtain())