Merge pull request #75814 from timothyqiu/connect-unicode

Fix connect signal dialog not allowing Unicode method name
This commit is contained in:
Yuri Sizov 2023-04-10 13:03:00 +02:00 committed by GitHub
commit 220953b625
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -119,7 +119,7 @@ void ConnectDialog::ok_pressed() {
return;
}
if (!method_name.strip_edges().is_valid_identifier()) {
if (!TS->is_valid_identifier(method_name.strip_edges())) {
error->set_text(TTR("Method name must be a valid identifier."));
error->popup_centered();
return;
@ -228,7 +228,7 @@ StringName ConnectDialog::generate_method_callback_name(Node *p_source, String p
String node_name = p_source->get_name();
for (int i = 0; i < node_name.length(); i++) { // TODO: Regex filter may be cleaner.
char32_t c = node_name[i];
if (!is_ascii_identifier_char(c)) {
if ((i == 0 && !is_unicode_identifier_start(c)) || (i > 0 && !is_unicode_identifier_continue(c))) {
if (c == ' ') {
// Replace spaces with underlines.
c = '_';