eliseuamaro/gnu-social
1
0
Fork 0
forked from GNUsocial/gnu-social
Code Issues Pull Requests Releases Wiki Activity

Use EmptyIdException in Fave deletion try-catch

Browse Source
This commit is contained in:
Mikael Nordfeldth 2016-01-03 22:57:28 +01:00
parent b4b57bba54
commit 34b25e6afc
1 changed files with 14 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
plugins/Favorite/classes/Fave.php
View File

@ -124,27 +124,30 @@ class Fave extends Managed_DataObject
$result = parent::delete($useWhere); $result = parent::delete($useWhere);
self::blowCacheForProfileId($this->user_id);
self::blowCacheForNoticeId($this->notice_id);
self::blow('popular');
if ($result !== false) { if ($result !== false) {
Event::handle('EndDisfavorNotice', array($profile, $notice)); Event::handle('EndDisfavorNotice', array($profile, $notice));
} }
} }
} catch (NoResultException $e) { } catch (NoResultException $e) {
// In case there's some inconsistency where the profile or notice was deleted without losing the fave db entry
common_log(LOG_INFO, '"'.get_class($e->obj).'" with id=='.var_export($e->obj->id, true).' object not found when deleting favorite, ignoring...'); common_log(LOG_INFO, '"'.get_class($e->obj).'" with id=='.var_export($e->obj->id, true).' object not found when deleting favorite, ignoring...');
} catch (EmptyIdException $e) {
// Some buggy instances of GNU social have had favroites with notice id==0 stored in the database
common_log(LOG_INFO, '"'.get_class($e->obj).'"object had empty id deleting favorite, ignoring...');
}
// If we catch an exception above, then $result===null because parent::delete only returns an int>=0 or boolean false
if (is_null($result)) {
// Delete it without the event, as something is wrong and we don't want it anyway. // Delete it without the event, as something is wrong and we don't want it anyway.
$result = parent::delete($useWhere); $result = parent::delete($useWhere);
}
// Err, apparently we can reference $this->user_id after parent::delete,
// I guess it's safe because this is the order it was before!
self::blowCacheForProfileId($this->user_id); self::blowCacheForProfileId($this->user_id);
self::blowCacheForNoticeId($this->notice_id); self::blowCacheForNoticeId($this->notice_id);
self::blow('popular'); self::blow('popular');
}
return $result; return $result;
} }