summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorDanil Alexeev <danil@alexeev.xyz>2022-05-17 21:46:40 +0300
committerDanil Alexeev <danil@alexeev.xyz>2022-06-07 18:05:49 +0300
commit1ca227af1d76dd2b4ce756d822dc44bd4ce91f4a (patch)
treee46ed3929eb74d7bda54fdf3f5dd8bc58da74fdd /core
parente24e4935ffc13715cf7a3dff189ac69d0c7a44e7 (diff)
Fix saving section-less keys in `ConfigFile`
Diffstat (limited to 'core')
-rw-r--r--core/io/config_file.cpp13
1 files changed, 8 insertions, 5 deletions
diff --git a/core/io/config_file.cpp b/core/io/config_file.cpp
index dd0191f43f..ae421654ca 100644
--- a/core/io/config_file.cpp
+++ b/core/io/config_file.cpp
@@ -61,19 +61,19 @@ PackedStringArray ConfigFile::_get_section_keys(const String &p_section) const {
}
void ConfigFile::set_value(const String &p_section, const String &p_key, const Variant &p_value) {
- if (p_value.get_type() == Variant::NIL) {
- //erase
+ if (p_value.get_type() == Variant::NIL) { // Erase key.
if (!values.has(p_section)) {
- return; // ?
+ return;
}
+
values[p_section].erase(p_key);
if (values[p_section].is_empty()) {
values.erase(p_section);
}
-
} else {
if (!values.has(p_section)) {
- values[p_section] = HashMap<String, Variant>();
+ // Insert section-less keys at the beginning.
+ values.insert(p_section, HashMap<String, Variant>(), p_section.is_empty());
}
values[p_section][p_key] = p_value;
@@ -125,6 +125,9 @@ void ConfigFile::erase_section_key(const String &p_section, const String &p_key)
ERR_FAIL_COND_MSG(!values[p_section].has(p_key), vformat("Cannot erase nonexistent key \"%s\" from section \"%s\".", p_key, p_section));
values[p_section].erase(p_key);
+ if (values[p_section].is_empty()) {
+ values.erase(p_section);
+ }
}
Error ConfigFile::save(const String &p_path) {