重构获取公共参数

internal/apps/admin/system_config/routers.go

@@ -34,12 +34,14 @@ type CreateSystemConfigRequest struct {
 	Key         string `json:"key" binding:"required,max=64"`
 	Value       string `json:"value" binding:"required,max=255"`
 	Type        string `json:"type" binding:"required,oneof=system business"`
+	Visibility  int    `json:"visibility" binding:"oneof=0 1"`
 	Description string `json:"description" binding:"max=255"`
 }
 
 // UpdateSystemConfigRequest 更新系统配置请求
 type UpdateSystemConfigRequest struct {
 	Value       string `json:"value" binding:"required,max=255"`
+	Visibility  *int   `json:"visibility" binding:"omitempty,oneof=0 1"`
 	Description string `json:"description" binding:"max=255"`
 }
 
@@ -78,6 +80,7 @@ func CreateSystemConfig(c *gin.Context) {
 		Key:         req.Key,
 		Value:       req.Value,
 		Type:        req.Type,
+		Visibility:  req.Visibility,
 		Description: req.Description,
 	}
 
@@ -206,6 +209,10 @@ func UpdateSystemConfig(c *gin.Context) {
 		updates := map[string]interface{}{
 			"description": req.Description,
 		}
+		if req.Visibility != nil {
+			updates["visibility"] = *req.Visibility
+			config.Visibility = *req.Visibility
+		}
 		if key != model.ConfigKeySMTPPassword || req.Value != "******" {
 			updates["value"] = req.Value
 			config.Value = req.Value

internal/apps/admin/system_config/routers_test.go

@@ -73,6 +73,7 @@ func TestCreateSystemConfig(t *testing.T) {
 			Key:         "custom_key",
 			Value:       "custom_value",
 			Type:        "system",
+			Visibility:  model.ConfigVisibilityVisible,
 			Description: "desc",
 		}
 		body, _ := json.Marshal(payload)
@@ -99,7 +100,10 @@ func TestCreateSystemConfig(t *testing.T) {
 			t.Fatalf("failed to find system config in Redis: %v", err)
 		}
 		if redisConfig.Value != "custom_value" {
-			t.Errorf("expected value 'custom_value', got '%s'", redisConfig.Value)
+			t.Errorf("CreateSystemConfig(custom_key).Value = %q, want %q", redisConfig.Value, "custom_value")
+		}
+		if redisConfig.Visibility != model.ConfigVisibilityVisible {
+			t.Errorf("CreateSystemConfig(custom_key).Visibility = %d, want %d", redisConfig.Visibility, model.ConfigVisibilityVisible)
 		}
 	})
 
@@ -217,8 +221,10 @@ func TestUpdateSystemConfig(t *testing.T) {
 	router := setupTestRouter(adminUser)
 
 	t.Run("update successfully", func(t *testing.T) {
+		hidden := model.ConfigVisibilityHidden
 		payload := UpdateSystemConfigRequest{
 			Value:       "Super Site Name",
+			Visibility:  &hidden,
 			Description: "Updated Description",
 		}
 		body, _ := json.Marshal(payload)
@@ -234,7 +240,7 @@ func TestUpdateSystemConfig(t *testing.T) {
 		// Verify database
 		var cfg model.SystemConfig
 		dbConn.Where("key = ?", model.ConfigKeySiteName).First(&cfg)
-		if cfg.Value != "Super Site Name" || cfg.Description != "Updated Description" {
+		if cfg.Value != "Super Site Name" || cfg.Description != "Updated Description" || cfg.Visibility != model.ConfigVisibilityHidden {
 			t.Errorf("database values not updated: %+v", cfg)
 		}
 
@@ -244,6 +250,9 @@ func TestUpdateSystemConfig(t *testing.T) {
 		if redisConfig.Value != "Super Site Name" {
 			t.Errorf("redis cache value not updated, got '%s'", redisConfig.Value)
 		}
+		if redisConfig.Visibility != model.ConfigVisibilityHidden {
+			t.Errorf("redis cache visibility = %d, want %d", redisConfig.Visibility, model.ConfigVisibilityHidden)
+		}
 	})
 
 	t.Run("update non-existent config", func(t *testing.T) {