Changeset View
Changeset View
Standalone View
Standalone View
src/applications/feed/PhabricatorFeedStoryPublisher.php
| Show First 20 Lines • Show All 118 Lines • ▼ Show 20 Lines | if ($this->relatedPHIDs) { | ||||
| $ref->getTableName(), | $ref->getTableName(), | ||||
| implode(', ', $sql)); | implode(', ', $sql)); | ||||
| } | } | ||||
| $subscribed_phids = $this->subscribedPHIDs; | $subscribed_phids = $this->subscribedPHIDs; | ||||
| if ($subscribed_phids) { | if ($subscribed_phids) { | ||||
| $subscribed_phids = $this->filterSubscribedPHIDs($subscribed_phids); | $subscribed_phids = $this->filterSubscribedPHIDs($subscribed_phids); | ||||
| $this->insertNotifications($chrono_key, $subscribed_phids); | $this->insertNotifications($chrono_key, $subscribed_phids); | ||||
| // We also want to send this notification to anyone who has the object's | |||||
| // page open. | |||||
| $subscribed_phids[] = $this->primaryObjectPHID; | |||||
epriestley: This fixes a bug where you would not receive notifications about `T123` if you had it open… | |||||
| $this->sendNotification($chrono_key, $subscribed_phids); | $this->sendNotification($chrono_key, $subscribed_phids); | ||||
| } | } | ||||
| PhabricatorWorker::scheduleTask( | PhabricatorWorker::scheduleTask( | ||||
| 'FeedPublisherWorker', | 'FeedPublisherWorker', | ||||
| array( | array( | ||||
| 'key' => $chrono_key, | 'key' => $chrono_key, | ||||
| )); | )); | ||||
| ▲ Show 20 Lines • Show All 145 Lines • Show Last 20 Lines | |||||
This fixes a bug where you would not receive notifications about T123 if you had it open unless you were also subscribed to it.