cyberpresidentvanellope/tgstation-server
1
0
Fork 0
mirror of https://github.com/tgstation/tgstation-server.git synced 2026-08-29 08:00:19 +01:00
Code Issues Packages Projects Releases Wiki Activity

Changing chat providers implicitly deletes all associated channels

Browse Source
This commit is contained in:
Cyberboss
2018-09-20 14:58:08 -04:00
parent d04e0dd8cd
commit ae022a6348
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Tgstation.Server.Host/Controllers/ChatController.cs
+12 -4
View File
@@ -210,6 +210,8 @@ namespace Tgstation.Server.Host.Controllers
return false;
};
var oldProvider = current.Provider;
if (CheckModified(x => x.ConnectionString, ChatBotRights.WriteConnectionString)
|| CheckModified(x => x.Enabled, ChatBotRights.WriteEnabled)
|| CheckModified(x => x.Name, ChatBotRights.WriteName)
@@ -217,12 +219,18 @@ namespace Tgstation.Server.Host.Controllers
|| (model.Channels != null && !userRights.HasFlag(ChatBotRights.WriteChannels)))
return Forbid();
if (model.Channels != null)
var hasChannels = model.Channels != null;
if (hasChannels || (model.Provider.HasValue && model.Provider != oldProvider))
{
DatabaseContext.ChatChannels.RemoveRange(current.Channels);
var dbChannels = model.Channels.Select(x => ConvertApiChatChannel(x)).ToList();
DatabaseContext.ChatChannels.AddRange(dbChannels);
current.Channels = dbChannels;
if (hasChannels)
{
var dbChannels = model.Channels.Select(x => ConvertApiChatChannel(x)).ToList();
DatabaseContext.ChatChannels.AddRange(dbChannels);
current.Channels = dbChannels;
}
else
current.Channels.Clear();
}
await DatabaseContext.Save(cancellationToken).ConfigureAwait(false);