@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Only revert commits in the same repository
Summary: Fixes T8215. When adding a revert edge, only consider commits within the same repository.
Test Plan: Imported the arcanist repository three times and inspected the reverts relationships on rARC490984936bb7ddfc589ef3d376ec2ac2365f070d.
Reviewers: hach-que, epriestley, #blessed_reviewers
Reviewed By: epriestley, #blessed_reviewers
Subscribers: Korvin, epriestley
Maniphest Tasks: T8215
Differential Revision: https://secure.phabricator.com/D12875
+1
+1
src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php
+1
src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php
···
68
68
if ($reverts) {
69
69
$reverted_commits = id(new DiffusionCommitQuery())
70
70
->setViewer(PhabricatorUser::getOmnipotentUser())
71
+
->withRepository($repository)
71
72
->withIdentifiers($reverts)
72
73
->execute();
73
74
$reverted_commit_phids = mpull($reverted_commits, 'getPHID', 'getPHID');