Skip to content

_showForegroundNotification - Null check on title/body crash#4547

Merged
mdmohsin7 merged 1 commit intomainfrom
fix/android-crash-foreground-notification-null
Feb 5, 2026
Merged

_showForegroundNotification - Null check on title/body crash#4547
mdmohsin7 merged 1 commit intomainfrom
fix/android-crash-foreground-notification-null

Conversation

@aaravgarg
Copy link
Collaborator

Summary

  • Add null check for noti.title and noti.body before using ! operator
  • Prevents null check crash when FCM sends a data-only notification without title or body
  • Skip showing the notification instead of crashing

Crash Stats

Test plan

  • Verify foreground notifications still show correctly
  • Test with data-only FCM messages (no title/body) - should not crash

🤖 Generated with Claude Code

…ification

Check noti.title and noti.body for null before using ! operator to
prevent null check crash when FCM sends a notification without title or body.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@aaravgarg aaravgarg requested a review from mdmohsin7 February 1, 2026 23:28
@gemini-code-assist
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@mdmohsin7 mdmohsin7 merged commit c9be2a3 into main Feb 5, 2026
1 check passed
@mdmohsin7 mdmohsin7 deleted the fix/android-crash-foreground-notification-null branch February 5, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants