Skip to content

Fixing NullPointerException #1511

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.

Sign up for GitHub

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

Jump to bottom
Open
Gowtham-arja wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixing NullPointerException #1511

Gowtham-arja wants to merge 2 commits into from
+200 −2

Conversation

@Gowtham-arja
Copy link

@Gowtham-arja Gowtham-arja commented Dec 28, 2025 •
edited by netomi
Loading

This fixes #1502

@netomi netomi changed the title #1502 Fixing NullPointerException Fixing NullPointerException Dec 28, 2025
@netomi
Copy link
Contributor

netomi commented Dec 28, 2025

thanks for providing a fix, could you sign an ECA as described here: https://www.eclipse.org/legal/eca/ ?

Also could you work on a unit test for that issue?

@Gowtham-arja
Copy link
Author

Gowtham-arja commented Dec 29, 2025

Hi @netomi
Signed the ECA.
Added the unit test case as well. Please review.

@netomi
Copy link
Contributor

netomi commented Jan 1, 2026

Thanks for the PR, it would fix the problem afaict, however I still do not fully understand how this exception happens to occur in the first place.

The preview field does not correspond to the actual extension version that is returned, but rather to the latest active version of the same extension. I could not find a test case where this would return nothing for a given extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

NullPointerException when doing a query

2 participants

@Gowtham-arja @netomi