Nixyri/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-02-16 07:40:36 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Fix duplicated entries in TranslationServer::get_loaded_locales()

Browse Source
This commit is contained in:
Haoyu Qiu 2024-10-26 14:17:25 +08:00
parent 61accf0605
commit f451997666
2 changed files with 24 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
core/string/translation_domain.cpp
View File

@ -247,7 +247,10 @@ PackedStringArray TranslationDomain::get_loaded_locales() const {
PackedStringArray locales; PackedStringArray locales;
for (const Ref<Translation> &E : translations) { for (const Ref<Translation> &E : translations) {
ERR_CONTINUE(E.is_null()); ERR_CONTINUE(E.is_null());
locales.push_back(E->get_locale()); const String &locale = E->get_locale();
if (!locales.has(locale)) {
locales.push_back(locale);
}
} }
return locales; return locales;
} }

30
tests/core/string/test_translation_server.h
View File

@ -37,27 +37,37 @@
namespace TestTranslationServer { namespace TestTranslationServer {
TEST_CASE("[TranslationServer] Translation operations") { TEST_CASE("[TranslationServer] Translation operations") {
Ref<Translation> t = memnew(Translation); Ref<Translation> t1 = memnew(Translation);
t->set_locale("uk"); t1->set_locale("uk");
t->add_message("Good Morning", String::utf8("Добрий ранок")); t1->add_message("Good Morning", String(U"Добрий ранок"));
Ref<Translation> t2 = memnew(Translation);
t2->set_locale("uk");
t2->add_message("Hello Godot", String(U"你好戈多"));
TranslationServer *ts = TranslationServer::get_singleton(); TranslationServer *ts = TranslationServer::get_singleton();
// Adds translation for UK locale for the first time.
int l_count_before = ts->get_loaded_locales().size(); int l_count_before = ts->get_loaded_locales().size();
ts->add_translation(t); ts->add_translation(t1);
int l_count_after = ts->get_loaded_locales().size(); int l_count_after = ts->get_loaded_locales().size();
// Newly created Translation object should be added to the list, so the counter should increase, too.
CHECK(l_count_after > l_count_before); CHECK(l_count_after > l_count_before);
Ref<Translation> trans = ts->get_translation_object("uk"); // Adds translation for UK locale again.
CHECK(trans.is_valid()); ts->add_translation(t2);
CHECK_EQ(ts->get_loaded_locales().size(), l_count_after);
// Removing that translation.
ts->remove_translation(t2);
CHECK_EQ(ts->get_loaded_locales().size(), l_count_after);
CHECK(ts->get_translation_object("uk").is_valid());
ts->set_locale("uk"); ts->set_locale("uk");
CHECK(ts->translate("Good Morning") == String::utf8("Добрий ранок")); CHECK(ts->translate("Good Morning") == String::utf8("Добрий ранок"));
ts->remove_translation(t); ts->remove_translation(t1);
trans = ts->get_translation_object("uk"); CHECK(ts->get_translation_object("uk").is_null());
CHECK(trans.is_null());
// If no suitable Translation object has been found - the original message should be returned. // If no suitable Translation object has been found - the original message should be returned.
CHECK(ts->translate("Good Morning") == "Good Morning"); CHECK(ts->translate("Good Morning") == "Good Morning");
} }