From 88b2c34534e0c4d9d99ce4ea7a1cbdd4dc785d35 Mon Sep 17 00:00:00 2001 From: michael Date: Tue, 11 Feb 2003 11:01:58 +0000 Subject: [PATCH] added error check --- lib/subscriptions/SubscriptionEngine.inc | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/lib/subscriptions/SubscriptionEngine.inc b/lib/subscriptions/SubscriptionEngine.inc index bb7d201..266a781 100644 --- a/lib/subscriptions/SubscriptionEngine.inc +++ b/lib/subscriptions/SubscriptionEngine.inc @@ -41,13 +41,19 @@ class SubscriptionEngine { // if the subscription type is document, fire the folder subscriptions also if ($iSubscriptionType == SubscriptionConstants::subscriptionType("DocumentSubscription")) { + $default->log->info("SubscriptionEngine::fireSubscription firing folder subscriptions for the document subscription, id=$iExternalID"); $oDocument = & Document::get($iExternalID); - $default->log->debug("document=$oDocument"); - $iSubscriptionsSent = SubscriptionEngine::fireSubscription($oDocument->getFolderID(), - $iSubscriptionAlertType, - SubscriptionConstants::subscriptionType("FolderSubscription"), - $aValues); - $default->log->info("fired folder subscribers, count=$iSubscriptionsSent"); + if ($oDocument) { + $default->log->debug("document=$oDocument"); + $iSubscriptionsSent = SubscriptionEngine::fireSubscription($oDocument->getFolderID(), + $iSubscriptionAlertType, + SubscriptionConstants::subscriptionType("FolderSubscription"), + $aValues); + $default->log->info("fired folder subscribers, count=$iSubscriptionsSent"); + } else { + $default->log->error("SubscriptionEngine::fireSubscription could not retrieve document from db, id=$iExternalID"); + $default->log->error("SubscriptionEngine::fireSubscription not firing folder subscriptions for the parent folder of this document"); + } } $default->log->info("subscribers=" . arrayToString($aSubscribers)); -- libgit2 0.21.4