Skip to content

Commit

Permalink
The plugin is now compatible with Elgg 2.3
Browse files Browse the repository at this point in the history
Fixes fatal error caused by the fact that in Elgg 2.3 instant notifications
are using the new Elgg\Notifications\InstantNotificationEvent class, and
therefore they behave differently.
  • Loading branch information
juho-jaakkola committed Dec 12, 2017
1 parent d942c56 commit c779be7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion start.php
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ function notifier_notification_send($hook, $type, $result, $params) {
$event = $params['event'];
/* @var Elgg_Notifications_Event $event */

if (!$event) {
if (!$event || !$event->getObject()) {
// Plugin is calling notify_user() so stop here and let
// the NotificationService handle the notification later.
return false;
Expand Down

0 comments on commit c779be7

Please sign in to comment.