Files F14057171

D11654.diff
No OneTemporary
Actions

Size

955 B

Referenced Files

None

Subscribers

None

D11654.diff
View Options

	diff --git a/src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php b/src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php
	--- a/src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php
	+++ b/src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php
	@@ -106,9 +106,10 @@
	$commit_reviewedby_phid = null;

	if ($revision_id) {
	- // TODO: (T603) This is probably safe to use an omnipotent user on,
	- // but check things more closely.
	- $revision = id(new DifferentialRevision())->load($revision_id);
	+ $revision = id(new DifferentialRevisionQuery())
	+ ->setViewer(PhabricatorUser::getOmnipotentUser())
	+ ->withIDs(array($revision_id))
	+ ->executeOne();
	if ($revision) {
	$revision_author_phid = $revision->getAuthorPHID();
	$commit_reviewedby_phid = $data->getCommitDetail('reviewerPHID');

File Metadata

Mime Type: text/plain
Expires: Mon, Nov 18, 1:02 AM (2 d, 12 h ago)
Storage Engine: blob
Storage Format: Encrypted (AES-256-CBC)
Storage Handle: 6716496
Default Alt Text: D11654.diff (955 B)

Event Timeline