From 9d47f41109e8c5c499e48a9f96b6569c148977ce Mon Sep 17 00:00:00 2001 From: Jicheng Lu <103353@smsassist.com> Date: Tue, 2 Jan 2024 13:02:07 -0600 Subject: [PATCH] clean code --- .../Controllers/AgentController.cs | 7 ------- .../Repository/MongoRepository.cs | 19 ++----------------- 2 files changed, 2 insertions(+), 24 deletions(-) diff --git a/src/Infrastructure/BotSharp.OpenAPI/Controllers/AgentController.cs b/src/Infrastructure/BotSharp.OpenAPI/Controllers/AgentController.cs index de1060dd..36c48476 100644 --- a/src/Infrastructure/BotSharp.OpenAPI/Controllers/AgentController.cs +++ b/src/Infrastructure/BotSharp.OpenAPI/Controllers/AgentController.cs @@ -15,13 +15,6 @@ public class AgentController : ControllerBase _services = services; } - [HttpGet("/test")] - public async Task Test() - { - - } - - [HttpGet("/agent/settings")] public AgentSettings GetSettings() { diff --git a/src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.cs b/src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.cs index 5baf65c2..3efc6cb3 100644 --- a/src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.cs +++ b/src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.cs @@ -716,13 +716,7 @@ public class MongoRepository : IBotSharpRepository if (string.IsNullOrEmpty(conversationId)) return; var filterConv = Builders.Filter.Eq(x => x.Id, conversationId); - var foundConv = _dc.Conversations.Find(filterConv).FirstOrDefault(); - if (foundConv == null) return; - var filterDialog = Builders.Filter.Eq(x => x.ConversationId, conversationId); - var foundDialog = _dc.ConversationDialogs.Find(filterDialog).FirstOrDefault(); - if (foundDialog == null) return; - var dialogElements = dialogs.Select(x => DialogMongoElement.ToMongoElement(x)).ToList(); var updateDialog = Builders.Update.PushEach(x => x.Dialogs, dialogElements); var updateConv = Builders.Update.Set(x => x.UpdatedTime, DateTime.UtcNow); @@ -736,9 +730,6 @@ public class MongoRepository : IBotSharpRepository if (string.IsNullOrEmpty(conversationId)) return; var filterConv = Builders.Filter.Eq(x => x.Id, conversationId); - var foundConv = _dc.Conversations.Find(filterConv).FirstOrDefault(); - if (foundConv == null) return; - var updateConv = Builders.Update .Set(x => x.UpdatedTime, DateTime.UtcNow) .Set(x => x.Title, title); @@ -764,9 +755,6 @@ public class MongoRepository : IBotSharpRepository if (string.IsNullOrEmpty(conversationId) || states.IsNullOrEmpty()) return; var filterStates = Builders.Filter.Eq(x => x.ConversationId, conversationId); - var foundStates = _dc.ConversationStates.Find(filterStates).FirstOrDefault(); - if (foundStates == null) return; - var saveStates = states.Select(x => StateMongoElement.ToMongoElement(x)).ToList(); var updateStates = Builders.Update.Set(x => x.States, saveStates); @@ -778,9 +766,6 @@ public class MongoRepository : IBotSharpRepository if (string.IsNullOrEmpty(conversationId) || string.IsNullOrEmpty(status)) return; var filter = Builders.Filter.Eq(x => x.Id, conversationId); - var foundConv = _dc.Conversations.Find(filter).FirstOrDefault(); - if (foundConv == null) return; - var update = Builders.Update .Set(x => x.Status, status) .Set(x => x.UpdatedTime, DateTime.UtcNow); @@ -938,8 +923,8 @@ public class MongoRepository : IBotSharpRepository var filter = Builders.Filter.Eq(x => x.ConversationId, conversationId); var update = Builders.Update - .SetOnInsert(x => x.Id, Guid.NewGuid().ToString()) - .PushEach(x => x.Logs, logs); + .SetOnInsert(x => x.Id, Guid.NewGuid().ToString()) + .PushEach(x => x.Logs, logs); _dc.ExectionLogs.UpdateOne(filter, update, _options); }