Refactor setting type (#12425)

### What problem does this PR solve?

Refactor setting type

### Type of change

- [x] Refactoring

---------

Signed-off-by: Jin Hai <haijin.chn@gmail.com>
This commit is contained in:
Jin Hai
2026-01-04 20:26:12 +08:00
committed by GitHub
parent 932496a8ec
commit 5ebe334a2f
4 changed files with 26 additions and 26 deletions

View File

@ -274,7 +274,7 @@ class SettingsMgr:
for setting in settings: for setting in settings:
result.append({ result.append({
'name': setting.name, 'name': setting.name,
'setting_type': setting.setting_type, 'source': setting.source,
'data_type': setting.data_type, 'data_type': setting.data_type,
'value': setting.value, 'value': setting.value,
}) })
@ -289,7 +289,7 @@ class SettingsMgr:
for setting in settings: for setting in settings:
result.append({ result.append({
'name': setting.name, 'name': setting.name,
'setting_type': setting.setting_type, 'source': setting.source,
'data_type': setting.data_type, 'data_type': setting.data_type,
'value': setting.value, 'value': setting.value,
}) })

View File

@ -1199,7 +1199,7 @@ class Memory(DataBaseModel):
class SystemSettings(DataBaseModel): class SystemSettings(DataBaseModel):
name = CharField(max_length=128, primary_key=True) name = CharField(max_length=128, primary_key=True)
setting_type = CharField(max_length=32, null=False, index=False) source = CharField(max_length=32, null=False, index=False)
data_type = CharField(max_length=32, null=False, index=False) data_type = CharField(max_length=32, null=False, index=False)
value = CharField(max_length=1024, null=False, index=False) value = CharField(max_length=1024, null=False, index=False)
class Meta: class Meta:

View File

@ -2,61 +2,61 @@
"system_settings": [ "system_settings": [
{ {
"name": "enable_whitelist", "name": "enable_whitelist",
"setting_type": "config", "source": "variable",
"data_type": "bool", "data_type": "bool",
"value": "true" "value": "true"
}, },
{ {
"name": "default_role", "name": "default_role",
"setting_type": "config", "source": "variable",
"data_type": "string", "data_type": "string",
"value": "" "value": ""
}, },
{ {
"name": "mail.server", "name": "mail.server",
"setting_type": "config", "source": "variable",
"data_type": "string", "data_type": "string",
"value": "" "value": ""
}, },
{ {
"name": "mail.port", "name": "mail.port",
"setting_type": "config", "source": "variable",
"data_type": "integer", "data_type": "integer",
"value": "" "value": ""
}, },
{ {
"name": "mail.use_ssl", "name": "mail.use_ssl",
"setting_type": "config", "source": "variable",
"data_type": "bool", "data_type": "bool",
"value": "false" "value": "false"
}, },
{ {
"name": "mail.use_tls", "name": "mail.use_tls",
"setting_type": "config", "source": "variable",
"data_type": "bool", "data_type": "bool",
"value": "false" "value": "false"
}, },
{ {
"name": "mail.username", "name": "mail.username",
"setting_type": "config", "source": "variable",
"data_type": "string", "data_type": "string",
"value": "" "value": ""
}, },
{ {
"name": "mail.password", "name": "mail.password",
"setting_type": "config", "source": "variable",
"data_type": "string", "data_type": "string",
"value": "" "value": ""
}, },
{ {
"name": "mail.timeout", "name": "mail.timeout",
"setting_type": "config", "source": "variable",
"data_type": "integer", "data_type": "integer",
"value": "10" "value": "10"
}, },
{ {
"name": "mail.default_sender", "name": "mail.default_sender",
"setting_type": "config", "source": "variable",
"data_type": "string", "data_type": "string",
"value": "" "value": ""
} }

View File

@ -405,18 +405,18 @@ Revoke successfully!
``` ```
admin> list vars; admin> list vars;
+-----------+---------------------+--------------+-----------+ +-----------+---------------------+--------------+-----------+
| data_type | name | setting_type | value | | data_type | name | source | value |
+-----------+---------------------+--------------+-----------+ +-----------+---------------------+--------------+-----------+
| string | default_role | config | user | | string | default_role | variable | user |
| bool | enable_whitelist | config | true | | bool | enable_whitelist | variable | true |
| string | mail.default_sender | config | | | string | mail.default_sender | variable | |
| string | mail.password | config | | | string | mail.password | variable | |
| integer | mail.port | config | 15 | | integer | mail.port | variable | 15 |
| string | mail.server | config | localhost | | string | mail.server | variable | localhost |
| integer | mail.timeout | config | 10 | | integer | mail.timeout | variable | 10 |
| bool | mail.use_ssl | config | true | | bool | mail.use_ssl | variable | true |
| bool | mail.use_tls | config | false | | bool | mail.use_tls | variable | false |
| string | mail.username | config | | | string | mail.username | variable | |
+-----------+---------------------+--------------+-----------+ +-----------+---------------------+--------------+-----------+
``` ```
@ -427,9 +427,9 @@ admin> list vars;
``` ```
admin> show var mail.server; admin> show var mail.server;
+-----------+-------------+--------------+-----------+ +-----------+-------------+--------------+-----------+
| data_type | name | setting_type | value | | data_type | name | source | value |
+-----------+-------------+--------------+-----------+ +-----------+-------------+--------------+-----------+
| string | mail.server | config | localhost | | string | mail.server | variable | localhost |
+-----------+-------------+--------------+-----------+ +-----------+-------------+--------------+-----------+
``` ```