search

caffeinalab / ti.goosh

GCM/FCM module for Titanium.
MIT License
52 stars 34 forks source link

NPE on Notification #72

Closed brandonfuller closed 6 years ago

brandonfuller commented 7 years ago

TiGooshModule.java Line 81:

if (!notification.isEmpty()) {

notification is initialized with an empty string but then gets a null data value on line 74.

Could you simply add a null check?

davidbnk commented 6 years ago

This bug is still happening on release 3.0.0

progress44 commented 6 years ago

Should be fixed with this commit https://github.com/caffeinalab/ti.goosh/commit/74b28f911e18d239fad6eee790bd373cb1941db6