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.

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

Add null checks on all the field retrievals to avoid NullPointerException #8

Closed
aarahmanqa opened this issue Oct 29, 2023 · 1 comment
Closed

Add null checks on all the field retrievals to avoid NullPointerException #8

aarahmanqa opened this issue Oct 29, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@aarahmanqa
Copy link
Contributor

aarahmanqa commented Oct 29, 2023 •
edited
Loading

Whenever we retrieve a value from the json field, we use .toString() which could throw NullPointerException. That field may be allowed to have null but since we have .toString() we would end up with NPE. This PR aims at avoiding such NPE. Its a better alternative to null check.
PFB the PR raised for the same.
#7
@shivam1608 shivam1608 mentioned this issue Oct 29, 2023
Merged
@shivam1608 shivam1608 added the enhancement New feature or request label Oct 29, 2023
@shivam1608
Copy link
Owner

Done in #7
Thanks for the contribution @aarahmanqa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aarahmanqa @shivam1608