From 122009c4c5cf8d718852f661210dec7d4386fa97 Mon Sep 17 00:00:00 2001
From: C-xC-c <boku@plum.moe>
Date: Thu, 19 Dec 2019 13:01:42 +0000
Subject: [PATCH] Move valid boards to appsettings.

---
 BantFlags/Controllers/FlagsController.cs   | 4 ++--
 BantFlags/Data/Database/DatabaseService.cs | 6 ++++++
 BantFlags/Data/PostModel.cs                | 3 +--
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/BantFlags/Controllers/FlagsController.cs b/BantFlags/Controllers/FlagsController.cs
index 00af538..cd078f8 100644
--- a/BantFlags/Controllers/FlagsController.cs
+++ b/BantFlags/Controllers/FlagsController.cs
@@ -57,7 +57,7 @@ namespace BantFlags.Controllers
         {
             string splitFlag = (version ?? 0) > 1 ? "," : "||"; // comma for v2+, else || for backwards compatibility.
 
-            Result<PostModel> post = PostModel.Create(post_nr, board, regions, splitFlag, Database.KnownFlags);
+            Result<PostModel> post = PostModel.Create(post_nr, board, regions, splitFlag, Database.KnownFlags, Database.Boards);
 
             if (post.Failed)
             {
@@ -66,7 +66,7 @@ namespace BantFlags.Controllers
 
             await Database.InsertPost(post.Value);
 
-            return Ok(post);
+            return Ok(post.Value);
         }
 
         /// <summary>
diff --git a/BantFlags/Data/Database/DatabaseService.cs b/BantFlags/Data/Database/DatabaseService.cs
index be886a5..89afbb3 100644
--- a/BantFlags/Data/Database/DatabaseService.cs
+++ b/BantFlags/Data/Database/DatabaseService.cs
@@ -16,10 +16,14 @@ namespace BantFlags.Data.Database
 
         public HashSet<string> KnownFlags { get; private set; }
 
+        public HashSet<string> Boards { get; private set; }
+
         public DatabaseService(DatabaseServiceConfig dbConfig)
         {
             ConnectionPool = new MySqlConnectionPool(dbConfig.ConnectionString, dbConfig.PoolSize);
 
+            Boards = dbConfig.Boards;
+
             UpdateKnownFlags().Wait(); // It's okay to deadlock here since it's only initialised at startup.
         }
 
@@ -101,5 +105,7 @@ namespace BantFlags.Data.Database
         public string ConnectionString { get; set; }
 
         public int PoolSize { get; set; }
+
+        public HashSet<string> Boards { get; set; }
     }
 }
\ No newline at end of file
diff --git a/BantFlags/Data/PostModel.cs b/BantFlags/Data/PostModel.cs
index 45764ed..7f719e7 100644
--- a/BantFlags/Data/PostModel.cs
+++ b/BantFlags/Data/PostModel.cs
@@ -19,10 +19,9 @@ namespace BantFlags.Data
             Flags = flags;
         }
 
-        public static Result<PostModel> Create(string post_nr, string board, string regions, string splitFlag, HashSet<string> knownFlags)
+        public static Result<PostModel> Create(string post_nr, string board, string regions, string splitFlag, HashSet<string> knownFlags, HashSet<string> boards)
         {
             string[] empty = { "empty, or there were errors. Re-set your flags." };
-            string[] boards = { "bant", "nap", "srsbsn" }; // TODO: Move this to appsettings and make a singleton for it.
 
             if (!int.TryParse(post_nr, out int postNumber))
                 return Result<PostModel>.Fail("Invalid post number.");