@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
Diffusion/BrowseFile: Set highlighted with blame as default view
Summary: As per title: when browsing files in Diffusion, set "Highlighted with blame" as the default view instead of falling back to "Highlighted".
Test Plan: Tested view with a local Diffusion setup, defaults correctly.
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin
Maniphest Tasks: T1278
Differential Revision: https://secure.phabricator.com/D2669
authored by
Two9A
and committed by
epriestley
2d52881d
a8b5ca63
+6
-2
+6
-2
src/applications/diffusion/controller/DiffusionBrowseFileController.php
+6
-2
src/applications/diffusion/controller/DiffusionBrowseFileController.php
···
32
32
33
33
$path = $drequest->getPath();
34
34
$selected = $request->getStr('view');
35
+
// If requested without a view, assume that blame is required (see T1278).
36
+
if (!$selected) {
37
+
$selected = 'blame';
38
+
}
35
39
$needs_blame = ($selected == 'blame' || $selected == 'plainblame');
36
40
37
41
$file_query = DiffusionFileContentQuery::newFromDiffusionRequest(
···
220
224
$select = AphrontFormSelectControl::renderSelectTag(
221
225
$request->getStr('view'),
222
226
array(
223
-
'highlighted' => 'View as Highlighted Text',
224
227
'blame' => 'View as Highlighted Text with Blame',
225
-
'plain' => 'View as Plain Text',
228
+
'highlighted' => 'View as Highlighted Text',
226
229
'plainblame' => 'View as Plain Text with Blame',
230
+
'plain' => 'View as Plain Text',
227
231
'raw' => 'View as raw document',
228
232
),
229
233
array(