mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-27 01:02:42 +01:00
Catch exception in differential
Summary: This masks possible configuration issues and slightly degrades functionality with the tradeoff of having differential work when phabricator isn't quite configured correctly. Test Plan: remove directory for a repository, load differential revision from that repo, and see differential load. Reviewers: epriestley, vrana Reviewed By: epriestley CC: aran, Korvin Maniphest Tasks: T2512 Differential Revision: https://secure.phabricator.com/D4859
This commit is contained in:
parent
275f708f14
commit
714a3cc536
1 changed files with 9 additions and 4 deletions
|
@ -251,10 +251,15 @@ final class DifferentialChangesetListView extends AphrontView {
|
||||||
|
|
||||||
$repository = $this->repository;
|
$repository = $this->repository;
|
||||||
if ($repository) {
|
if ($repository) {
|
||||||
$meta['diffusionURI'] = (string)$repository->getDiffusionBrowseURIForPath(
|
try {
|
||||||
$changeset->getAbsoluteRepositoryPath($repository, $this->diff),
|
$meta['diffusionURI'] =
|
||||||
idx($changeset->getMetadata(), 'line:first'),
|
(string)$repository->getDiffusionBrowseURIForPath(
|
||||||
$this->getBranch());
|
$changeset->getAbsoluteRepositoryPath($repository, $this->diff),
|
||||||
|
idx($changeset->getMetadata(), 'line:first'),
|
||||||
|
$this->getBranch());
|
||||||
|
} catch (DiffusionSetupException $e) {
|
||||||
|
// Ignore
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
$change = $changeset->getChangeType();
|
$change = $changeset->getChangeType();
|
||||||
|
|
Loading…
Reference in a new issue