[Favorite] Fix "Properly trigger DisfavorNotice on profile deletion"

Calling find() inside delete() is probably not a good idea.
This commit is contained in:
Alexei Sorokin 2020-08-04 13:23:31 +03:00 committed by Diogo Peralta Cordeiro
parent 10f0a15614
commit 3429a3644d
2 changed files with 14 additions and 11 deletions

View File

@ -342,7 +342,14 @@ class FavoriteModule extends ActivityVerbHandlerModule
{ {
$fave = new Fave(); $fave = new Fave();
$fave->user_id = $profile->id; $fave->user_id = $profile->id;
$fave->delete(); // Will perform a DELETE matching "user_id = {$user->id}"
// Will perform a DELETE matching "user_id = {$user->id}"
if ($fave->find()) {
while ($fave->fetch()) {
$fave->delete();
}
}
$fave->free(); $fave->free();
Fave::blowCacheForProfileId($profile->id); Fave::blowCacheForProfileId($profile->id);

View File

@ -137,18 +137,14 @@ class Fave extends Managed_DataObject
$result = null; $result = null;
try { try {
if ($this->find()) { $profile = $this->getActor();
while ($this->fetch()) { $notice = $this->getTarget();
$profile = $this->getActor();
$notice = $this->getTarget();
if (Event::handle('StartDisfavorNotice', [$profile, $notice, &$result])) { if (Event::handle('StartDisfavorNotice', [$profile, $notice, &$result])) {
$result = parent::delete($useWhere); $result = parent::delete($useWhere);
if ($result !== false) { if ($result !== false) {
Event::handle('EndDisfavorNotice', [$profile, $notice]); Event::handle('EndDisfavorNotice', [$profile, $notice]);
}
}
} }
} }
} catch (NoResultException $e) { } catch (NoResultException $e) {