From c6976110643fdcfd846c8ce245ce6fd840554fe9 Mon Sep 17 00:00:00 2001
From: Veloman Yunkan <veloman.yunkan@gmail.com>
Date: Sun, 28 Jan 2024 17:58:16 +0400
Subject: [PATCH] Dropped defaultUserLanguage from viewer_settings.js

---
 src/server/internalServer.cpp       |  3 +-
 static/templates/viewer_settings.js |  3 +-
 test/server.cpp                     | 51 +++--------------------------
 3 files changed, 6 insertions(+), 51 deletions(-)

diff --git a/src/server/internalServer.cpp b/src/server/internalServer.cpp
index 2c1ea518..5c9d733d 100644
--- a/src/server/internalServer.cpp
+++ b/src/server/internalServer.cpp
@@ -767,8 +767,7 @@ std::unique_ptr<Response> InternalServer::handle_viewer_settings(const RequestCo
   const kainjow::mustache::object data{
     {"enable_toolbar", m_withTaskbar ? "true" : "false" },
     {"enable_link_blocking", m_blockExternalLinks ? "true" : "false" },
-    {"enable_library_button", m_withLibraryButton ? "true" : "false" },
-    {"default_user_language", request.get_user_language() }
+    {"enable_library_button", m_withLibraryButton ? "true" : "false" }
   };
   return ContentResponse::build(RESOURCE::templates::viewer_settings_js, data, "application/javascript; charset=utf-8");
 }
diff --git a/static/templates/viewer_settings.js b/static/templates/viewer_settings.js
index 37669bb1..0ccf6579 100644
--- a/static/templates/viewer_settings.js
+++ b/static/templates/viewer_settings.js
@@ -1,6 +1,5 @@
 const viewerSettings = {
   toolbarEnabled:       {{enable_toolbar}},
   linkBlockingEnabled:  {{enable_link_blocking}},
-  libraryButtonEnabled: {{enable_library_button}},
-  defaultUserLanguage:  "{{default_user_language}}"
+  libraryButtonEnabled: {{enable_library_button}}
 }
diff --git a/test/server.cpp b/test/server.cpp
index 74e082c1..5a0bd072 100644
--- a/test/server.cpp
+++ b/test/server.cpp
@@ -2077,8 +2077,7 @@ TEST_F(ServerTest, viewerSettings)
 R"(const viewerSettings = {
   toolbarEnabled:       false,
   linkBlockingEnabled:  false,
-  libraryButtonEnabled: false,
-  defaultUserLanguage:  "en"
+  libraryButtonEnabled: false
 }
 )");
   }
@@ -2089,8 +2088,7 @@ R"(const viewerSettings = {
 R"(const viewerSettings = {
   toolbarEnabled:       false,
   linkBlockingEnabled:  true,
-  libraryButtonEnabled: false,
-  defaultUserLanguage:  "en"
+  libraryButtonEnabled: false
 }
 )");
   }
@@ -2101,8 +2099,7 @@ R"(const viewerSettings = {
 R"(const viewerSettings = {
   toolbarEnabled:       true,
   linkBlockingEnabled:  false,
-  libraryButtonEnabled: false,
-  defaultUserLanguage:  "en"
+  libraryButtonEnabled: false
 }
 )");
   }
@@ -2113,47 +2110,7 @@ R"(const viewerSettings = {
 R"(const viewerSettings = {
   toolbarEnabled:       true,
   linkBlockingEnabled:  false,
-  libraryButtonEnabled: true,
-  defaultUserLanguage:  "en"
-}
-)");
-  }
-
-  {
-    resetServer(ZimFileServer::WITH_TASKBAR_AND_LIBRARY_BUTTON);
-    const Headers headers{ {"Accept-Language", "fr"} };
-    ASSERT_EQ(zfs1_->GET("/ROOT%23%3F/viewer_settings.js", headers)->body,
-R"(const viewerSettings = {
-  toolbarEnabled:       true,
-  linkBlockingEnabled:  false,
-  libraryButtonEnabled: true,
-  defaultUserLanguage:  "fr"
-}
-)");
-  }
-
-  {
-    resetServer(ZimFileServer::WITH_TASKBAR_AND_LIBRARY_BUTTON);
-    const Headers headers{ {"Accept-Language", "test;q=0.2, en;q=0.9"} };
-    ASSERT_EQ(zfs1_->GET("/ROOT%23%3F/viewer_settings.js", headers)->body,
-R"(const viewerSettings = {
-  toolbarEnabled:       true,
-  linkBlockingEnabled:  false,
-  libraryButtonEnabled: true,
-  defaultUserLanguage:  "en"
-}
-)");
-  }
-
-  {
-    resetServer(ZimFileServer::WITH_TASKBAR_AND_LIBRARY_BUTTON);
-    const Headers headers{ {"Accept-Language", "test;q=0.9, en;q=0.2"} };
-    ASSERT_EQ(zfs1_->GET("/ROOT%23%3F/viewer_settings.js", headers)->body,
-R"(const viewerSettings = {
-  toolbarEnabled:       true,
-  linkBlockingEnabled:  false,
-  libraryButtonEnabled: true,
-  defaultUserLanguage:  "test"
+  libraryButtonEnabled: true
 }
 )");
   }