@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
Fix an issue with URI index updates from the daemons
Summary:
Ref T10923. This extension needs to load a little more data (with `needURIs`) to function correctly now.
(There's a recent migration does this, so indexes got updated correctly when it ran, so it hasn't been obvious that they weren't getting updated properly after that.)
Test Plan: Made an arbitrary edit to a repository, observed no more error in daemon logs.
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T10923
Differential Revision: https://secure.phabricator.com/D15908
+12
+12
src/applications/diffusion/engineextension/DiffusionRepositoryURIsIndexEngineExtension.php
+12
src/applications/diffusion/engineextension/DiffusionRepositoryURIsIndexEngineExtension.php
···
16
16
public function indexObject(
17
17
PhabricatorIndexEngine $engine,
18
18
$object) {
19
+
20
+
// Reload the repository to pick up URIs, which we need in order to update
21
+
// the URI index.
22
+
$object = id(new PhabricatorRepositoryQuery())
23
+
->setViewer(PhabricatorUser::getOmnipotentUser())
24
+
->withPHIDs(array($object->getPHID()))
25
+
->needURIs(true)
26
+
->executeOne();
27
+
if (!$object) {
28
+
return;
29
+
}
30
+
19
31
$object->updateURIIndex();
20
32
}
21
33