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

Correct whitespace option is passed to 'Show All Lines' request.

Browse source 
Summary:
Expanding lines duplicated some lines occasionally, because whitespace
option was different for the original request and the following request.

Test Plan:
Tested that the broken changeset was correct now.

Reviewed By: epriestley
Reviewers: epriestley
CC: jungejason, aran, epriestley
Differential Revision: 263
This commit is contained in:
tuomaspelkonen 2011-05-10 17:36:06 -07:00
parent e8744e7eeb
commit e4f42dcd7d
2 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/applications/differential/controller/changesetview/DifferentialChangesetViewController.php
View file

@ -189,6 +189,7 @@ class DifferentialChangesetViewController extends DifferentialController {
Javelin::initBehavior('differential-show-more', array( Javelin::initBehavior('differential-show-more', array(
'uri' => '/differential/changeset/', 'uri' => '/differential/changeset/',
'whitespace' => $request->getStr('whitespace'),
)); ));
$detail = new DifferentialChangesetDetailView(); $detail = new DifferentialChangesetDetailView();

1
webroot/rsrc/js/application/differential/behavior-show-more.js
View file

@ -26,6 +26,7 @@ JX.behavior('differential-show-more', function(config) {
JX.DOM.setContent(container, 'Loading...'); JX.DOM.setContent(container, 'Loading...');
JX.DOM.alterClass(context, 'differential-show-more-loading', true); JX.DOM.alterClass(context, 'differential-show-more-loading', true);
var data = e.getNodeData('show-more'); var data = e.getNodeData('show-more');
data['whitespace'] = config.whitespace;
new JX.Request(config.uri, JX.bind(null, onresponse, e)) new JX.Request(config.uri, JX.bind(null, onresponse, e))
.setData(data) .setData(data)
.send(); .send();