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', | |||||
epriestley: I //think// this type should just be "notification" or similar. `storyType` has a bunch of… | |||||
Not Done Inline ActionsI agree. Is there an easy way to get this though, from within the PhabricatorFeedStoryPublisher class? Or we could pass it in as a parameter perhaps. joshuaspence: I agree. Is there an easy way to get this though, from within the… | |||||
| ), | |||||
| 'subscribers' => $this->subscribedPHIDs, | |||||
| ); | ); | ||||
| 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 44 Lines • Show Last 20 Lines | |||||
I think this type should just be "notification" or similar. storyType has a bunch of class names?