Skip to content

Null Observer Exception#161

Merged
coenm merged 3 commits intocoenm:mainfrom
gabriel-vanca:repoObserversNull
Sep 7, 2024
Merged

Null Observer Exception#161
coenm merged 3 commits intocoenm:mainfrom
gabriel-vanca:repoObserversNull

Conversation

@gabriel-vanca
Copy link
Copy Markdown
Contributor

@gabriel-vanca gabriel-vanca commented Sep 6, 2024

NullRef exception is thrown in Reset() upon pressing the Clear button.

This PR adds a NULL check verification and cleans up the file a bit.

⚠️ However, while this PR fixes the crash, further investigation is required to determine what causes the Null element to be added. Maybe a JSON parsing error when reading the file from the cache?

See #160

@coenm coenm added the bug 🐛 Bug Fixes label Sep 7, 2024
@coenm
Copy link
Copy Markdown
Owner

coenm commented Sep 7, 2024

Thanks.

@coenm coenm changed the title Null Observer Exception. Closes #160 Null Observer Exception Sep 7, 2024
@coenm coenm merged commit 374d611 into coenm:main Sep 7, 2024
@gabriel-vanca gabriel-vanca deleted the repoObserversNull branch September 7, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🐛 Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants