Show less misleading summary, test plan authors
Summary: Instead of assuming the test plan and summary are written by the author of the differential revision, let's assume they are written by the author of the latest differential diff. Test Plan: viewed a drev that had been commandeered but not updated to check authors Reviewers: epriestley, jungejason, vrana Reviewed By: epriestley CC: aran, Koolvin Maniphest Tasks: T1235 Differential Revision: https://secure.phabricator.com/D2550
This commit is contained in:
@@ -364,8 +364,10 @@ final class DifferentialRevisionViewController extends DifferentialController {
|
||||
|
||||
private function getImplicitComments(DifferentialRevision $revision) {
|
||||
|
||||
$diff = $revision->loadActiveDiff();
|
||||
|
||||
$template = new DifferentialComment();
|
||||
$template->setAuthorPHID($revision->getAuthorPHID());
|
||||
$template->setAuthorPHID($diff->getAuthorPHID());
|
||||
$template->setRevisionID($revision->getID());
|
||||
$template->setDateCreated($revision->getDateCreated());
|
||||
|
||||
|
||||
Reference in New Issue
Block a user