X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=classes%2FSubscription_queue.php;h=a60293e2158e9bf8b42ff6f01ff079b1a9056c1b;hb=aa41afc28b9933d2c1da34504ad1c1ef4f020639;hp=ea0e906d43fca01c1208eeafc1508fc556d135ab;hpb=861e838adda95bb3b6144ce52468459ca9cb191c;p=quix0rs-gnu-social.git diff --git a/classes/Subscription_queue.php b/classes/Subscription_queue.php index ea0e906d43..a60293e215 100644 --- a/classes/Subscription_queue.php +++ b/classes/Subscription_queue.php @@ -1,22 +1,18 @@ subscriber = $subscriber->id; $rq->subscribed = $subscribed->id; @@ -48,11 +47,23 @@ class Subscription_queue extends Managed_DataObject return $rq; } - function exists($subscriber, $other) + static function exists(Profile $subscriber, Profile $other) { - $sub = Subscription_queue::pkeyGet(array('subscriber' => $subscriber->id, - 'subscribed' => $other->id)); - return (empty($sub)) ? false : true; + $sub = Subscription_queue::pkeyGet(array('subscriber' => $subscriber->getID(), + 'subscribed' => $other->getID())); + return ($sub instanceof Subscription_queue); + } + + static function getSubQueue(Profile $subscriber, Profile $other) + { + // This is essentially a pkeyGet but we have an object to return in NoResultException + $sub = new Subscription_queue(); + $sub->subscriber = $subscriber->id; + $sub->subscribed = $other->id; + if (!$sub->find(true)) { + throw new NoResultException($sub); + } + return $sub; } /** @@ -64,9 +75,11 @@ class Subscription_queue extends Managed_DataObject { $subscriber = Profile::getKV('id', $this->subscriber); $subscribed = Profile::getKV('id', $this->subscribed); - $sub = Subscription::start($subscriber, $subscribed, Subscription::FORCE); - if ($sub) { + try { + $sub = Subscription::start($subscriber, $subscribed, Subscription::FORCE); $this->delete(); + } catch (AlreadyFulfilledException $e) { + common_debug('Tried to start a subscription which already existed.'); } return $sub; }