mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Echo
synced 2024-11-12 09:26:05 +00:00
Merge "Avoid use of IDatabase::delete() return values"
This commit is contained in:
commit
155d582026
|
@ -399,20 +399,19 @@ class EchoNotificationMapper extends EchoAbstractMapper {
|
|||
if ( !$idsToDelete ) {
|
||||
return true;
|
||||
}
|
||||
$success = true;
|
||||
foreach ( array_chunk( $idsToDelete, $wgUpdateRowsPerQuery ) as $batch ) {
|
||||
$success = $dbw->delete(
|
||||
$dbw->delete(
|
||||
'echo_notification',
|
||||
[
|
||||
'notification_user' => $userId,
|
||||
'notification_event' => $batch,
|
||||
],
|
||||
__METHOD__
|
||||
) && $success;
|
||||
);
|
||||
$lbFactory->commitAndWaitForReplication(
|
||||
__METHOD__, $ticket, [ 'domain' => $domainId ] );
|
||||
}
|
||||
return $success;
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
Loading…
Reference in a new issue