Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Content Assist sometimes not Picking up local workfields #294

Open
Wikus86 opened this issue Jan 24, 2024 · 4 comments
Open

Content Assist sometimes not Picking up local workfields #294

Wikus86 opened this issue Jan 24, 2024 · 4 comments
Assignees
Labels
info-needed More information is requied from the issuer

Comments

@Wikus86
Copy link

Wikus86 commented Jan 24, 2024

Here is an exmaple of it not picking up the local workfield.

image

Another thing i noticed, linter picks up that it has not been references

image

@worksofliam worksofliam self-assigned this Jan 24, 2024
@worksofliam
Copy link
Contributor

worksofliam commented Jan 25, 2024

@Wikus86 I have tried recreating this with no luck.

Are you able to paste your code (not take a screenshot of it) so I can copy and paste it directly?

image image

Thanks,
Liam

@worksofliam worksofliam added the info-needed More information is requied from the issuer label Jan 25, 2024
@Wikus86
Copy link
Author

Wikus86 commented Jan 30, 2024

Yeah, I can also not reproduce it, only happens in some programs. Let me try doing a debug on the program giving the issue. Will advise if I find something

@Wikus86
Copy link
Author

Wikus86 commented Jan 30, 2024

I found something, looks like it gets confused with a procedure in my copybook.

This is an example of the code

image

in completionItem.ts, in the completionItemProvider function, it picks up i am on line 318 which is almost correct, actually on line 319

Screenshot 2024-01-30 112042

Then it tries to find the procedure based on the line number

image

The problem is, in my copy book I have a procedure in that line range

Screenshot 2024-01-30 111944

and the index is before the actual procedure I am in

Screenshot 2024-01-30 112015

I hope this makes a bit of sense :)

@EVLSDE
Copy link

EVLSDE commented Sep 12, 2024

The problem is, in my copy book I have a procedure in that line range

I just stumbled over this sentence. Could it be I accidentally posted a duplicate issue in #323? If yes then I'm sorry!
The code in pull request #324 worked it out for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed More information is requied from the issuer
Projects
None yet
Development

No branches or pull requests

3 participants