GNUsocial/gnu-social
4
0
Fork 1
Code Issues Releases Wiki Activity

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

Browse Source 
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
parent a9c365a5eb
commit ab4120721f
2 changed files with 14 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
modules/Favorite/FavoriteModule.php
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);

16
modules/Favorite/classes/Fave.php
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) {