diff --git a/src/addon/notifications/providers/push-click-handler.ts b/src/addon/notifications/providers/push-click-handler.ts index b6504debd..e95c9440d 100644 --- a/src/addon/notifications/providers/push-click-handler.ts +++ b/src/addon/notifications/providers/push-click-handler.ts @@ -39,6 +39,11 @@ export class AddonNotificationsPushClickHandler implements CorePushNotifications * @return Whether the notification click is handled by this handler */ handles(notification: any): boolean | Promise { + if (!notification.moodlecomponent) { + // The notification doesn't come from Moodle. Handle it. + return true; + } + if (this.utils.isTrueOrOne(notification.notif)) { // Notification clicked, mark as read. Don't block for this. const notifId = notification.savedmessageid || notification.id; @@ -74,7 +79,7 @@ export class AddonNotificationsPushClickHandler implements CorePushNotifications // Try to handle the appurl. if (notification.customdata && notification.customdata.appurl) { switch (notification.customdata.appurlopenin) { - case 'inappbrowser': + case 'inapp': this.utils.openInApp(notification.customdata.appurl); return; diff --git a/src/core/courses/providers/enrol-push-click-handler.ts b/src/core/courses/providers/enrol-push-click-handler.ts index 635a776e5..17539a81a 100644 --- a/src/core/courses/providers/enrol-push-click-handler.ts +++ b/src/core/courses/providers/enrol-push-click-handler.ts @@ -37,8 +37,8 @@ export class CoreCoursesEnrolPushClickHandler implements CorePushNotificationsCl * @return Whether the notification click is handled by this handler */ handles(notification: any): boolean | Promise { - return this.utils.isTrueOrOne(notification.notif) && notification.moodlecomponent.indexOf('enrol_') === 0 && - notification.name == 'expiry_notification'; + return this.utils.isTrueOrOne(notification.notif) && notification.moodlecomponent && + notification.moodlecomponent.indexOf('enrol_') === 0 && notification.name == 'expiry_notification'; } /**