Page MenuHomePhorge
Files F2874797

D25570.1736962391.diff
No OneTemporary
Actions

D25570.1736962391.diff
View Options

diff --git a/src/applications/maniphest/controller/ManiphestTaskDetailController.php b/src/applications/maniphest/controller/ManiphestTaskDetailController.php
--- a/src/applications/maniphest/controller/ManiphestTaskDetailController.php
+++ b/src/applications/maniphest/controller/ManiphestTaskDetailController.php
@@ -81,7 +81,7 @@
$graph_menu = null;
$graph_limit = 200;
- $overflow_message = null;
+ $graph_error_message = null;
$task_graph = id(new ManiphestTaskGraph())
->setViewer($viewer)
->setSeedPHID($task->getPHID())
@@ -102,8 +102,9 @@
// the search button to browse tasks with the search UI instead.
$direct_count = count($parent_list) + count($subtask_list);
+ $graph_table = null;
if ($direct_count > $graph_limit) {
- $overflow_message = pht(
+ $graph_error_message = pht(
'This task is directly connected to more than %s other tasks. '.
'Use %s to browse parents or subtasks, or %s to show more of the '.
'graph.',
@@ -111,14 +112,13 @@
phutil_tag('strong', array(), pht('Search...')),
phutil_tag('strong', array(), pht('View Standalone Graph')));
- $graph_table = null;
} else {
// If there aren't too many direct tasks, but there are too many total
// tasks, we'll only render directly connected tasks.
if ($task_graph->isOverLimit()) {
$task_graph->setRenderOnlyAdjacentNodes(true);
- $overflow_message = pht(
+ $graph_error_message = pht(
'This task is connected to more than %s other tasks. '.
'Only direct parents and subtasks are shown here. Use '.
'%s to show more of the graph.',
@@ -126,13 +126,22 @@
phutil_tag('strong', array(), pht('View Standalone Graph')));
}
- $graph_table = $task_graph->newGraphTable();
+ try {
+ $graph_table = $task_graph->newGraphTable();
+ } catch (Throwable $ex) {
+ phlog($ex);
+ $graph_error_message = pht(
+ 'There was an unexpected error displaying the task graph. '.
+ 'Use %s to browse parents or subtasks, or %s to show the graph.',
+ phutil_tag('strong', array(), pht('Search...')),
+ phutil_tag('strong', array(), pht('View Standalone Graph')));
+ }
}
- if ($overflow_message) {
+ if ($graph_error_message) {
$overflow_view = $this->newTaskGraphOverflowView(
$task,
- $overflow_message,
+ $graph_error_message,
true);
$graph_table = array(

File Metadata

Mime Type
text/plain
Expires
Wed, Jan 15, 17:33 (2 d, 11 h ago)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
1116121
Default Alt Text
D25570.1736962391.diff (2 KB)

Event Timeline

Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0