Java imports resolution failure #249
Conversation
(cherry picked from commit be8ae2636d58cccc947fef2ef356ac44357f33ad)
| const relativeFilePath = match[1]; | ||
| return this.getBBjClassesFromFile(context.container, relativeFilePath, false); | ||
| } else { | ||
| const program = AstUtils.getContainerOfType(context.container, isProgram)!; | ||
| const document = AstUtils.getDocument(program); | ||
| const locals = this.indexManager.allElements(BbjClass, new Set([document.uri.toString()])); | ||
| const locals = document.precomputedScopes?.get(program) |
There was a problem hiding this comment.
This seems to be the actual fix, I excluded all classes other than the BBj classes by accident.
Question: Is there a way to include these classes also in the completion of the USE statements?
There was a problem hiding this comment.
They are currently available in use completion, or what do you mean?
There was a problem hiding this comment.
Not for me. HashSet and LinkedList are two examples that are not in the Completion list:
There was a problem hiding this comment.
I see HashMap, but LinkedList is missing. Strange...
We could investigate next week if there is still time. It is low Prio cause only completion related, linking works for both types.
There was a problem hiding this comment.
Agreed, the completion aspect can be resolved later in that case.
| const relativeFilePath = match[1]; | ||
| return this.getBBjClassesFromFile(context.container, relativeFilePath, false); | ||
| } else { | ||
| const program = AstUtils.getContainerOfType(context.container, isProgram)!; | ||
| const document = AstUtils.getDocument(program); | ||
| const locals = this.indexManager.allElements(BbjClass, new Set([document.uri.toString()])); | ||
| const locals = document.precomputedScopes?.get(program) |
There was a problem hiding this comment.
Agreed, the completion aspect can be resolved later in that case.
Fixed java import resolution. Some additional fixes and improvements.
Following now work again: