diff options
author | jordi <creptthrust@gmail.com> | 2023-01-07 20:23:26 -0600 |
---|---|---|
committer | jordi <creptthrust@gmail.com> | 2023-01-07 20:23:26 -0600 |
commit | d5e1b4a85760773758507501a4aa7c3b89f9cfdc (patch) | |
tree | a21fe96181931c71930b1fdb3f2f486f58b30b9f | |
parent | fcba87e696d58912838d8a4a6987b10efa28e78f (diff) |
Force double quotes for NodePaths with apostrophes
-rw-r--r-- | modules/gdscript/gdscript_editor.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/modules/gdscript/gdscript_editor.cpp b/modules/gdscript/gdscript_editor.cpp index 6f8b90bd06..dcf17b9fe7 100644 --- a/modules/gdscript/gdscript_editor.cpp +++ b/modules/gdscript/gdscript_editor.cpp @@ -2977,7 +2977,9 @@ static void _find_call_arguments(GDScriptParser::CompletionContext &p_context, c // The path needs quotes if it's not a valid identifier (with an exception // for "/" as path separator, which also doesn't require quotes). if (!opt.replace("/", "_").is_valid_identifier()) { - opt = opt.quote(quote_style); // Handle user preference. + // Ignore quote_style and just use double quotes for paths with apostrophes. + // Double quotes don't need to be checked because they're not valid in node and property names. + opt = opt.quote(opt.contains("'") ? "\"" : quote_style); // Handle user preference. } ScriptLanguage::CodeCompletionOption option(opt, ScriptLanguage::CODE_COMPLETION_KIND_NODE_PATH); options.insert(option.display, option); |