Page MenuHomePhabricator
Files F15400230

D14964.id36151.diff
No OneTemporary
Actions

D14964.id36151.diff
View Options

diff --git a/src/applications/diffusion/controller/DiffusionBrowseController.php b/src/applications/diffusion/controller/DiffusionBrowseController.php
--- a/src/applications/diffusion/controller/DiffusionBrowseController.php
+++ b/src/applications/diffusion/controller/DiffusionBrowseController.php
@@ -591,10 +591,16 @@
$data) {
$viewer = $this->getViewer();
+ $blame_timeout = 15;
+ $blame_failed = false;
if ($needs_blame) {
- $blame = $this->loadBlame($path, $drequest->getCommit());
+ $blame = $this->loadBlame($path, $drequest->getCommit(), $blame_timeout);
list($blame_list, $blame_commits) = $blame;
+ if ($blame_list === null) {
+ $blame_failed = true;
+ $blame_list = array();
+ }
} else {
$blame_list = array();
$blame_commits = array();
@@ -697,16 +703,26 @@
->appendChild($corpus)
->setCollapsed(true);
+ $messages = array();
+
if (!$can_highlight) {
- $message = pht(
+ $messages[] = pht(
'This file is larger than %s, so syntax highlighting is disabled '.
'by default.',
phutil_format_bytes($highlight_limit));
+ }
+
+ if ($blame_failed) {
+ $messages[] = pht(
+ 'Failed to load blame information for this file in %s second(s).',
+ new PhutilNumber($blame_timeout));
+ }
+ if ($messages) {
$corpus->setInfoView(
id(new PHUIInfoView())
->setSeverity(PHUIInfoView::SEVERITY_WARNING)
- ->setErrors(array($message)));
+ ->setErrors($messages));
}
return $corpus;
@@ -1709,15 +1725,16 @@
return $view;
}
- private function loadBlame($path, $commit) {
+ private function loadBlame($path, $commit, $timeout) {
$blame = $this->callConduitWithDiffusionRequest(
'diffusion.blame',
array(
'commit' => $commit,
'paths' => array($path),
+ 'timeout' => $timeout,
));
- $identifiers = idx($blame, $path, array());
+ $identifiers = idx($blame, $path, null);
if ($identifiers) {
$viewer = $this->getViewer();

File Metadata

Mime Type
text/plain
Expires
Mar 18 2025, 10:26 AM (5 w, 23 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7709663
Default Alt Text
D14964.id36151.diff (2 KB)

Event Timeline

Phabricator · Built by Phacility