EmptyIdException doesn't carry ->obj

This commit is contained in:
Mikael Nordfeldth 2016-01-16 21:12:53 +01:00
parent 3019f8f23f
commit 0797ee0871

View File

@ -133,8 +133,8 @@ class Fave extends Managed_DataObject
// In case there's some inconsistency where the profile or notice was deleted without losing the fave db entry // 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) { } catch (EmptyIdException $e) {
// Some buggy instances of GNU social have had favroites with notice id==0 stored in the database // Some buggy instances of GNU social have had favorites with notice id==0 stored in the database
common_log(LOG_INFO, '"'.get_class($e->obj).'"object had empty id deleting favorite, ignoring...'); common_log(LOG_INFO, _ve($e->getMessage()));
} }
// If we catch an exception above, then $result===null because parent::delete only returns an int>=0 or boolean false // If we catch an exception above, then $result===null because parent::delete only returns an int>=0 or boolean false