Differential D9609 Diff 23074 src/applications/differential/customfield/DifferentialRepositoryField.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/differential/customfield/DifferentialRepositoryField.php
| Show First 20 Lines • Show All 137 Lines • ▼ Show 20 Lines | public function getRequiredHandlePHIDsForPropertyView() { | ||||
| } | } | ||||
| return array(); | return array(); | ||||
| } | } | ||||
| public function renderPropertyViewValue(array $handles) { | public function renderPropertyViewValue(array $handles) { | ||||
| return $this->renderHandleList($handles); | return $this->renderHandleList($handles); | ||||
| } | } | ||||
| public function shouldAppearInTransactionMail() { | |||||
| return true; | |||||
| } | |||||
| public function updateTransactionMailBody( | |||||
| PhabricatorMetaMTAMailBody $body, | |||||
| PhabricatorApplicationTransactionEditor $editor, | |||||
| array $xactions) { | |||||
| if (!$editor->getDiffUpdateTransaction($xactions)) { | |||||
| return; | |||||
| } | |||||
epriestley: I think this should use the revision's `getRepositoryPHID()`, not the latest diff's. | |||||
| $repository = $this->getObject()->getRepository(); | |||||
| if ($repository === null) { | |||||
| return; | |||||
| } | |||||
| $body->addTextSection(pht('REPOSITORY'), | |||||
| $repository->getMonogram().' '.$repository->getName()); | |||||
| } | |||||
| } | } | ||||
I think this should use the revision's getRepositoryPHID(), not the latest diff's.