diff options
author | RĂ©mi Verschelde <remi@verschelde.fr> | 2021-08-06 10:13:09 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-06 10:13:09 +0200 |
commit | 6ded4f52e8c2d48b98b631a1ec8ab5be8aa4768a (patch) | |
tree | dffe487f8a691433a502ff96b52a477f34867915 | |
parent | 2f1bc509dce5f953dc1b92e89943006d199f64ac (diff) | |
parent | 03f8fa9f62c97648491a0d8f0dd7fac6808eb9b4 (diff) |
Merge pull request #51283 from Razoric480/lsp-parse-from
Fix LSP parsing get_node only from the scene root
-rw-r--r-- | modules/gdscript/language_server/gdscript_workspace.cpp | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/modules/gdscript/language_server/gdscript_workspace.cpp b/modules/gdscript/language_server/gdscript_workspace.cpp index 148aeec47a..4fc229a0f8 100644 --- a/modules/gdscript/language_server/gdscript_workspace.cpp +++ b/modules/gdscript/language_server/gdscript_workspace.cpp @@ -520,8 +520,29 @@ void GDScriptWorkspace::completion(const lsp::CompletionParams &p_params, List<S if (const ExtendGDScriptParser *parser = get_parse_result(path)) { Node *owner_scene_node = _get_owner_scene_node(path); + + Array stack; + Node *current = nullptr; + stack.push_back(owner_scene_node); + + while (!stack.is_empty()) { + current = stack.pop_back(); + Ref<GDScript> script = current->get_script(); + if (script.is_valid() && script->get_path() == path) { + break; + } + for (int i = 0; i < current->get_child_count(); ++i) { + stack.push_back(current->get_child(i)); + } + } + + Ref<GDScript> script = current->get_script(); + if (!script.is_valid() || script->get_path() != path) { + current = owner_scene_node; + } + String code = parser->get_text_for_completion(p_params.position); - GDScriptLanguage::get_singleton()->complete_code(code, path, owner_scene_node, r_options, forced, call_hint); + GDScriptLanguage::get_singleton()->complete_code(code, path, current, r_options, forced, call_hint); if (owner_scene_node) { memdelete(owner_scene_node); } |