revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

Use ?view= in Blame previous but not in line link

Browse source 
Summary: I didn't notice that D3494 is revert of D3453.

Test Plan: Checked both line and Blame previous links.

Reviewers: epriestley, fdoemges

Reviewed By: fdoemges

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D3566
This commit is contained in:
vrana 2012-09-28 16:41:39 -07:00
parent b43f84d6dc
commit e5e3617e46
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/diffusion/controller/DiffusionBrowseFileController.php
View file

@ -469,7 +469,6 @@ final class DiffusionBrowseFileController extends DiffusionController {
'action' => 'browse', 'action' => 'browse',
'line' => $line['line'], 'line' => $line['line'],
'stable' => true, 'stable' => true,
'params' => array('view' => 'blame'),
)); ));
$blame = array(); $blame = array();
@ -540,10 +539,11 @@ final class DiffusionBrowseFileController extends DiffusionController {
'D'.$revision_id); 'D'.$revision_id);
} }
$uri = $line_href->alter('before', $commit);
$before_link = javelin_render_tag( $before_link = javelin_render_tag(
'a', 'a',
array( array(
'href' => $line_href->alter('before', $commit), 'href' => $uri->setQueryParam('view', 'blame'),
'sigil' => 'has-tooltip', 'sigil' => 'has-tooltip',
'meta' => array( 'meta' => array(
'tip' => 'Skip Past This Commit', 'tip' => 'Skip Past This Commit',