Finalize API for posts and topics

jacnils · jacnils · commit 1b1cac621271 · 2025-06-08T19:48:04.000+02:00
diff --git a/.clang-format b/.clang-format
@@ -0,0 +1,22 @@
+---
+Language:        Cpp
+BasedOnStyle:    LLVM
+IndentWidth:     4
+TabWidth:        4
+UseTab:          Always
+BreakBeforeBraces: Attach
+PointerAlignment: Left
+AllowShortIfStatementsOnASingleLine: false
+AllowShortFunctionsOnASingleLine: Inline
+AlwaysBreakTemplateDeclarations: Yes
+SpacesInParentheses: false
+SpaceBeforeParens: ControlStatements
+SpaceAfterCStyleCast: true
+KeepEmptyLinesAtTheStartOfBlocks: false
+ColumnLimit:     100
+AlignConsecutiveAssignments: true
+AlignConsecutiveDeclarations: true
+AlignTrailingComments: true
+IncludeBlocks:   Regroup
+ReflowComments:  true
+...
diff --git a/css/ff.css b/css/ff.css
@@ -36,6 +36,10 @@
     background-color: #000000;
     color: #fff;
 }
+#forum-button, #forum-window {
+    background-color: #f0f0f0;
+    color: #000;
+}
 #sandbox-button, #sandbox-window, #file-window {
     background-color: #c4c4c4;
     color: #000;
@@ -203,7 +207,7 @@ body {
     min-height: 60%;
 }
 
-#view-window, #file-window {
+#view-window, #file-window, #forum-window {
     min-width: 70%;
     min-height: 70%;
 }
diff --git a/http/create_topic.http b/http/create_topic.http
@@ -0,0 +1,11 @@
+POST /api/create_topic HTTP/1.1
+Host: localhost:8080
+Content-Type: application/json
+
+{
+    "title": "New Topic",
+    "description": "This is a new topic created for testing purposes.",
+
+    "username": "jacob",
+    "key": "baR5ZE7v07z5MygYKlHuWUPJbB0pDDkfmLFNbMuna6lrrwxOH96zQXJojUrUtHq2"
+}
diff --git a/http/rate_forwarder.http b/http/rate_forwarder.http
@@ -5,5 +5,5 @@ Content-Type: application/json
     "forwarder_id": "jRF5k1YB",
     "rating": 5,
     "username": "jacob",
-    "key": "DQFLbDpU8tmK0uWrVi3vxXeL0iVCOf5PhXH9YgAz9wpOBaYOmovXBBvE7aeLLHVO"
+    "key": "baR5ZE7v07z5MygYKlHuWUPJbB0pDDkfmLFNbMuna6lrrwxOH96zQXJojUrUtHq2"
 }
diff --git a/include/ff.hpp b/include/ff.hpp
@@ -116,6 +116,7 @@ namespace ff {
         bool cache_exists{false};
         bool convert_images_to_webp{true};
         bool convert_videos_to_webm{false};
+        bool topics_require_admin{false};
     };
 
     enum class AccountCreationStatus {
@@ -418,4 +419,18 @@ namespace ff {
     limhamn::http::server::response handle_api_delete_comment_file_endpoint(const limhamn::http::server::request& request, database& db);
     limhamn::http::server::response handle_api_stay_logged_in(const limhamn::http::server::request& request, database& db);
     limhamn::http::server::response handle_api_try_logout_endpoint(const limhamn::http::server::request& request, database& db);
+
+    limhamn::http::server::response handle_api_create_post(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_delete_post(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_edit_post(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_close_post(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_get_posts(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_comment_post(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_delete_comment_post(const limhamn::http::server::request& request, database& db);
+
+    limhamn::http::server::response handle_api_create_topic(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_delete_topic(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_get_topics(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_edit_topic(const limhamn::http::server::request& request, database& db);
+    limhamn::http::server::response handle_api_close_topic(const limhamn::http::server::request& request, database& db);
 } // namespace ff
diff --git a/js/ff.js b/js/ff.js
@@ -5210,6 +5210,13 @@ const generate_stars = (n, w) => {
     }
 }
 
+function show_forum_index() {
+    set_path('/topic');
+    hide_initial();
+
+    const forum = create_window('forum-window');
+}
+
 function show_credits() {
     set_path('/');
     hide_initial();
@@ -5501,6 +5508,13 @@ function init_page() {
         id: "sandbox-button",
         onclick: "play_click(); show_sandbox()"
     }))
+    list.push(get_link_box({
+       title: "Forum",
+       description: "Check out the Forwarder Factory forum.",
+        background_color: "",
+        id: "forum-button",
+        onclick: "play_click(); show_forum_index()",
+    }))
 
     if (get_cookie("username") === null) {
         list.push(get_link_box({
@@ -5629,6 +5643,7 @@ document.addEventListener('DOMContentLoaded', async () => {
     if (get_path() === "/admin" && is_logged_in()) show_admin();
     if (get_path() === "/admin" && !is_logged_in()) show_login();
     if (get_path() === "/logout" && is_logged_in()) show_logout();
+    if (get_path() === "/topic") show_forum_index();
 
     if (get_path().startsWith("/view/")) {
         const id = get_path().substring(6);
diff --git a/src/config.cpp b/src/config.cpp
@@ -79,6 +79,7 @@ ff::Settings ff::load_settings(const std::string& _config_file) {
         if (config["upload"]["convert_images_to_webp"]) settings.convert_images_to_webp = config["upload"]["convert_images_to_webp"].as<bool>();
         if (config["upload"]["convert_videos_to_webm"]) settings.convert_videos_to_webm = config["upload"]["convert_videos_to_webm"].as<bool>();
         if (config["download"]["preview_files"]) settings.preview_files = config["download"]["preview_files"].as<bool>();
+    	if (config["topic"]["topics_require_admin"]) settings.topics_require_admin = config["topic"]["topics_require_admin"].as<bool>();
         if (config["smtp"]["server"]) settings.smtp_server = config["smtp"]["server"].as<std::string>();
         if (config["smtp"]["port"]) settings.smtp_port = config["smtp"]["port"].as<int>();
         if (config["smtp"]["username"]) settings.email_username = config["smtp"]["username"].as<std::string>();
@@ -278,6 +279,10 @@ std::string ff::generate_default_config() {
     ss << "  convert_images_to_webp: " << (ff::settings.convert_images_to_webp ? "true" : "false") << "\n";
     ss << "  convert_videos_to_webm: " << (ff::settings.convert_videos_to_webm ? "true" : "false") << "\n";
     ss << "\n";
+	ss << "# Topic options:\n";
+	ss << "#   topics_require_admin: Whether to require admin when creating topics.\n";
+	ss << "topic:\n";
+	ss << "  topics_require_admin: " << (ff::settings.topics_require_admin ? "true" : "false") << "\n";
     ss << "# Download options:\n";
     ss << "#   preview_files: Whether to preview files in the browser when downloading them.\n";
     ss << "download:\n";
diff --git a/src/database.cpp b/src/database.cpp
@@ -63,6 +63,20 @@ void ff::setup_database(database& database) {
         throw std::runtime_error{"Error creating the general table."};
     }
 
+    // id: the topic id
+    // identifier: the topic identifier
+    // json: the json of the topic (including title, description, etc.)
+    if (!database.exec("CREATE TABLE IF NOT EXISTS topics (" + primary + ", identifier TEXT NOT NULL, json TEXT NOT NULL);")) {
+        throw std::runtime_error{"Error creating the topics table."};
+    }
+
+    // id: the post id
+    // identifier: the post identifier
+    // json: the json of the post (including content, author, etc.)
+    if (!database.exec("CREATE TABLE IF NOT EXISTS posts (" + primary + ", identifier INTEGER NOT NULL, json TEXT NOT NULL);")) {
+        throw std::runtime_error{"Error creating the posts table."};
+    }
+
     const auto query = database.query("SELECT * FROM general;");
     if (query.empty()) {
         nlohmann::json json;
diff --git a/src/ff.cpp b/src/ff.cpp
@@ -290,6 +290,20 @@ void ff::start_server() {
                 {"/api/try_logout", ff::handle_api_try_logout_endpoint},
                 {"/api/delete_forwarder", ff::handle_api_delete_forwarder_endpoint},
                 {"/api/delete_file", ff::handle_api_delete_file_endpoint},
+
+                {"/api/create_post", ff::handle_api_create_post},
+                {"/api/delete_post", ff::handle_api_delete_post},
+                {"/api/edit_post", ff::handle_api_edit_post},
+                {"/api/close_post", ff::handle_api_close_post},
+                {"/api/get_posts", ff::handle_api_get_posts},
+                {"/api/comment_post", ff::handle_api_comment_post},
+                {"/api/delete_comment_post", ff::handle_api_delete_comment_post},
+                {"/api/create_topic", ff::handle_api_create_topic},
+                {"/api/delete_topic", ff::handle_api_delete_topic},
+                {"/api/get_topics", ff::handle_api_get_topics},
+                {"/api/edit_topic", ff::handle_api_edit_topic},
+                {"/api/close_topic", ff::handle_api_close_topic},
+                //{"/api/pin_post_to_topic", ff::handle_api_pin_post_to_topic},
             };
             const std::unordered_map<std::string, std::function<limhamn::http::server::response(const limhamn::http::server::request&, ff::database&)>> setup_handlers{
                 {virtual_favicon_path, ff::handle_virtual_favicon_endpoint},
@@ -377,6 +391,10 @@ void ff::start_server() {
                 return handlers.at("/")(request, *database);
             } else if (file.find("/profile/") != std::string::npos) {
                 return handlers.at("/")(request, *database);
+            } else if (file.find("/topic") != std::string::npos) {
+                return handlers.at("/")(request, *database);
+            } else if (file.find("/post/") != std::string::npos) {
+                return handlers.at("/")(request, *database);
             }
 
             // handle activation URLs
diff --git a/src/path_handlers.cpp b/src/path_handlers.cpp

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,10 @@`
`36`	`36`	`background-color: #000000;`
`37`	`37`	`color: #fff;`
`38`	`38`	`}`
	`39`	`+#forum-button, #forum-window {`
	`40`	`+ background-color: #f0f0f0;`
	`41`	`+ color: #000;`
	`42`	`+}`
`39`	`43`	`#sandbox-button, #sandbox-window, #file-window {`
`40`	`44`	`background-color: #c4c4c4;`
`41`	`45`	`color: #000;`
`@@ -203,7 +207,7 @@ body {`
`203`	`207`	`min-height: 60%;`
`204`	`208`	`}`
`205`	`209`
`206`		`-#view-window, #file-window {`
	`210`	`+#view-window, #file-window, #forum-window {`
`207`	`211`	`min-width: 70%;`
`208`	`212`	`min-height: 70%;`
`209`	`213`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,5 +5,5 @@ Content-Type: application/json`
`5`	`5`	`"forwarder_id": "jRF5k1YB",`
`6`	`6`	`"rating": 5,`
`7`	`7`	`"username": "jacob",`
`8`		`- "key": "DQFLbDpU8tmK0uWrVi3vxXeL0iVCOf5PhXH9YgAz9wpOBaYOmovXBBvE7aeLLHVO"`
	`8`	`+ "key": "baR5ZE7v07z5MygYKlHuWUPJbB0pDDkfmLFNbMuna6lrrwxOH96zQXJojUrUtHq2"`
`9`	`9`	`}`