Changeset View
Changeset View
Standalone View
Standalone View
src/applications/feed/PhabricatorFeedStoryPublisher.php
| Show First 20 Lines • Show All 169 Lines • ▼ Show 20 Lines | queryfx( | ||||
| 'INSERT INTO %T | 'INSERT INTO %T | ||||
| (primaryObjectPHID, userPHID, chronologicalKey, hasViewed) | (primaryObjectPHID, userPHID, chronologicalKey, hasViewed) | ||||
| VALUES %Q', | VALUES %Q', | ||||
| $notif->getTableName(), | $notif->getTableName(), | ||||
| implode(', ', $sql)); | implode(', ', $sql)); | ||||
| } | } | ||||
| private function sendNotification($chrono_key) { | private function sendNotification($chrono_key) { | ||||
| $data = array( | $data = array( | ||||
| 'data' => array( | |||||
| 'key' => (string)$chrono_key, | 'key' => (string)$chrono_key, | ||||
| 'type' => 'notification', | 'type' => 'notification', | ||||
| ), | |||||
| 'subscribers' => $this->subscribedPHIDs, | 'subscribers' => $this->subscribedPHIDs, | ||||
| ); | ); | ||||
epriestley: Oh, durr, it should be OK because of this. This is probably fine. Anyway, double-check it and… | |||||
| try { | try { | ||||
| PhabricatorNotificationClient::postMessage($data); | PhabricatorNotificationClient::postMessage($data); | ||||
| } catch (Exception $ex) { | } catch (Exception $ex) { | ||||
| // Ignore, these are not critical. | // Ignore, these are not critical. | ||||
| } | } | ||||
| } | } | ||||
| ▲ Show 20 Lines • Show All 43 Lines • Show Last 20 Lines | |||||
Oh, durr, it should be OK because of this. This is probably fine. Anyway, double-check it and this looks good otherwise.