@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
Use stable commit identifier to load repository commit
Summary: Fixes T5113. This was caught in the crossfire of cleaning up the DiffusionRequest "commit" properties.
Test Plan: Loaded `/rXnnnn` with some of the `nnn` missing.
Reviewers: btrahan
Reviewed By: btrahan
Subscribers: epriestley
Maniphest Tasks: T5113
Differential Revision: https://secure.phabricator.com/D9253
+1
-1
+1
-1
src/applications/diffusion/request/DiffusionRequest.php
+1
-1
src/applications/diffusion/request/DiffusionRequest.php
···
385
385
$commit = id(new PhabricatorRepositoryCommit())->loadOneWhere(
386
386
'repositoryID = %d AND commitIdentifier = %s',
387
387
$repository->getID(),
388
-
$this->getCommit());
388
+
$this->getStableCommit());
389
389
if ($commit) {
390
390
$commit->attachRepository($repository);
391
391
}