From: Alexander Moisseev <moiseev@mezonplus.ru>
Date: Mon, 6 Oct 2025 06:45:23 +0000 (+0300)
Subject: [Fix] WebUI: repopulate classifier dropdown
X-Git-Tag: 3.14.0~92^2~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=621a9f6d5d7b4d869836f2f16af22a9701d0f72f;p=thirdparty%2Frspamd.git

[Fix] WebUI: repopulate classifier dropdown

after reconnect from read-only

Ensure classifiers are fetched when the dropdown is empty even if cache suggests skipping,
preventing an empty selector on Scan tab after RO → Disconnect → Enable.
---

diff --git a/interface/js/app/upload.js b/interface/js/app/upload.js
index f95d5cc905..9f6c6e7a50 100644
--- a/interface/js/app/upload.js
+++ b/interface/js/app/upload.js
@@ -330,10 +330,15 @@ define(["jquery", "app/common", "app/libft"],
 
         ui.getClassifiers = function () {
             const server = common.getServer();
-            if (shouldSkipRequest(server, "classifiers")) return;
 
             if (!common.read_only) {
+                const hadOptions = $("#classifier").children().length > 0; // remember pre-state
+
                 const sel = $("#classifier").empty().append($("<option>", {value: "", text: "All classifiers"}));
+
+                // Skip request only if we already had options populated for this config/server
+                if (shouldSkipRequest(server, "classifiers") && hadOptions) return;
+
                 common.query("bayes/classifiers", {
                     success: function (data) {
                         data[0].data.forEach((c) => sel.append($("<option>", {value: c, text: c})));