skobkin/point-tools
1
1
Fork 0
Code Issues 4 Pull requests 1 Releases Activity

Fix #13.

Browse source
This commit is contained in:
Alexey Skobkin 2015-06-02 05:27:19 +03:00
parent 3742f4b06c
commit b7e4374dac
1 changed files with 19 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
src/Skobkin/Bundle/PointToolsBundle/Service/SubscriptionsManager.php
View file

@ -39,6 +39,14 @@ class SubscriptionsManager
$oldSubscribersList[] = $subscription->getSubscriber(); $oldSubscribersList[] = $subscription->getSubscriber();
} }
$isFirstTime = false;
// Preventing to add garbage subscriptions for first processing
// @todo improve algorithm
if ((count($oldSubscribersList) === 0) && (count($newSubscribersList) > 1)) {
$isFirstTime = true;
}
unset($tmpOldSubscribers); unset($tmpOldSubscribers);
$subscribedList = $this->getUsersListsDiff($newSubscribersList, $oldSubscribersList); $subscribedList = $this->getUsersListsDiff($newSubscribersList, $oldSubscribersList);
@ -54,17 +62,20 @@ class SubscriptionsManager
$user->addSubscriber($subscription); $user->addSubscriber($subscription);
$logEvent = new SubscriptionEvent(); // If it's not first processing
$logEvent if (!$isFirstTime) {
->setSubscriber($subscribedUser) $logEvent = new SubscriptionEvent();
->setAuthor($user) $logEvent
->setAction(SubscriptionEvent::ACTION_SUBSCRIBE) ->setSubscriber($subscribedUser)
; ->setAuthor($user)
->setAction(SubscriptionEvent::ACTION_SUBSCRIBE);
$user->addNewSubscriberEvent($logEvent); $user->addNewSubscriberEvent($logEvent);
$this->em->persist($logEvent);
}
$this->em->persist($subscription); $this->em->persist($subscription);
$this->em->persist($logEvent);
} }
unset($subscribedList); unset($subscribedList);