wamo/Qvitter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'ondeleterelated_crash' into 'master'

Browse Source 
Don't crash the event if Profile not found



See merge request !56
This commit is contained in:
hannes 2016-05-10 05:59:07 -04:00
commit eccef8b103
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
QvitterPlugin.php
View File

@ -1061,14 +1061,17 @@ class QvitterPlugin extends Plugin {
$notif->delete();
// outputs an activity notice that this notice was deleted
$profile = $notice->getProfile();
// don't delete if this is a user is being deleted
// because that creates an infinite loop of deleting and creating notices...
$user_is_deleted = false;
$user = User::getKV('id',$profile->id);
if($user instanceof User && $user->hasRole(Profile_role::DELETED)) {
try {
// outputs an activity notice that this notice was deleted
$profile = $notice->getProfile();
$user = User::getKV('id',$profile->id);
if($user instanceof User && $user->hasRole(Profile_role::DELETED)) {
$user_is_deleted = true;
}
} catch (NoProfileException $e) {
$user_is_deleted = true;
}