Fix notifications.

satanasov · Aug 5, 2017 · 5633490 · 5633490
1 parent e9681bd
commit 5633490
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/controller/notifyhelper.php b/controller/notifyhelper.php
@@ -68,7 +68,8 @@ public function notify($type, $topic_id, $post_id, $post_subject, $poster_user,
 					$phpbb_notifications->add_notifications('notification.type.postlove', $notification_data);
 				break;
 				case 'remove':
-					$phpbb_notifications->delete_notifications('notification.type.postlove', $notification_data);
+					$notifications = $phpbb_notifications->get_item_type_class('notification.type.postlove');
+					$phpbb_notifications->delete_notifications('notification.type.postlove', $notifications->get_item_id($notification_data));
 				break;
 			}
 		}

diff --git a/notification/postlove.php b/notification/postlove.php
@@ -95,10 +95,12 @@ public function find_users_for_notification($data, $options = array())
 		$options = array_merge(array(
 			'ignore_users'			=> array(),
 		), $options);
+		$users = array(
+			$data['user_id']	=> 0,
+		);
+		$this->user_loader->load_users(array_keys($users));
 
-		$this->user_loader->load_users(array_keys($data['user_id']));
-
-		return $this->check_user_notification_options(array_keys($data['user_id']), $options);
+		return $this->check_user_notification_options(array_keys($users), $options);
 	}
 
 	/**