diff options
author | Brion Vibber <brion@pobox.com> | 2010-09-03 12:34:05 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-09-03 12:34:05 -0700 |
commit | 6786bbcfbfb445b190fd03e602ee0b1a85abb2af (patch) | |
tree | 24a92212831fb89d27c17e3844d577b7a7dabd44 /plugins/RSSCloud | |
parent | 0ef422593ba351d363c584ad781ff92548bc6e9b (diff) |
Drop RSSCloud queue items if the notice has a bogus profile, rather than attempting to rerun it due to the initial erroring-out. That's not a recoverable error
Diffstat (limited to 'plugins/RSSCloud')
-rw-r--r-- | plugins/RSSCloud/RSSCloudQueueHandler.php | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/plugins/RSSCloud/RSSCloudQueueHandler.php b/plugins/RSSCloud/RSSCloudQueueHandler.php index 295c26189..ef11eda2e 100644 --- a/plugins/RSSCloud/RSSCloudQueueHandler.php +++ b/plugins/RSSCloud/RSSCloudQueueHandler.php @@ -28,7 +28,12 @@ class RSSCloudQueueHandler extends QueueHandler function handle($notice) { - $profile = $notice->getProfile(); + try { + $profile = $notice->getProfile(); + } catch (Exception $e) { + common_log(LOG_ERR, "Dropping RSSCloud item for notice with bogus profile: " . $e->getMessage()); + return true; + } $notifier = new RSSCloudNotifier(); return $notifier->notify($profile); } |