In very specific circumstances we can bulkDistribute 0 notices

Seems to be what caused an infinite loop on quitter.es, or I guess so anyway.
This commit is contained in:
Mikael Nordfeldth 2016-01-16 17:18:14 +01:00
parent 912d65c767
commit c393bc9563
2 changed files with 2 additions and 2 deletions

View File

@ -226,7 +226,7 @@ class HubSub extends Managed_DataObject
* @param string $atom well-formed Atom feed
* @param array $pushCallbacks list of callback URLs
*/
function bulkDistribute($atom, $pushCallbacks)
function bulkDistribute($atom, array $pushCallbacks)
{
if (empty($pushCallbacks)) {
common_log(LOG_ERR, 'Callback list empty for bulkDistribute.');

View File

@ -257,7 +257,7 @@ class OStatusQueueHandler extends QueueHandler
}
}
}
if (count($batch) >= 0) {
if (count($batch) > 0) {
$sub->bulkDistribute($atom, $batch);
}
}