Critical error in push_hub_notify_subscriber()
Closed this issue · 3 comments
push_hub_notify_subscriber() currently reads as follows:
function push_hub_notify_subscriber($info) {
if (!push_hub()->notify($info['topic'], $info['subscriber'], isset($info['content']) ? $info['content'] : NULL)) {
// On failure, queue for later notification.
push_hub_queue()->createItem($info, time() + 5 * 60);
}
}
The only problem is there is no such thing as $info['content'], so fatPings are never sent. It should be $info['changed'].
Right, here is what's going on: $info['content'] should be $info['changed'] - look at what's being added to the queue or the batch API operations in push_hub_notify().
I've committed this to my fork: http://github.com/Crell/push_hub/commit/6801d9eecedcf3705fc119b70a48ddf17bd17d94. Merge when ready.
This is merged.