opendev/gerrit
Code Issues Proposed changes

Two small PolyGerrit fixes

Browse Source 
+ Specify whitespace parameter when fetching a diff.
+ Pass the correct var to Promise.all within gr-date-formatter_test.html

Change-Id: Ife49a67639891adae7d99e735516b86e54132724
This commit is contained in:
Andrew Bonventre
2016-03-17 18:20:07 -04:00
parent 5992220391
commit 8d30975266
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/elements/shared/gr-date-formatter/gr-date-formatter_test.html
View File

@@ -125,7 +125,7 @@ limitations under the License.
getLoggedIn: sinon.stub().returns(loggedInPromise), getLoggedIn: sinon.stub().returns(loggedInPromise),
getPreferences: sinon.stub().returns(preferencesPromise), getPreferences: sinon.stub().returns(preferencesPromise),
}); });
return Promise.all([loggedInPromise, preferences]); return Promise.all([loggedInPromise, preferencesPromise]);
} }
suite('logged in', function() { suite('logged in', function() {

3
polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.js
View File

@@ -176,7 +176,8 @@
var url = this._getDiffFetchURL(changeNum, patchNum, path); var url = this._getDiffFetchURL(changeNum, patchNum, path);
var params = { var params = {
context: 'ALL', context: 'ALL',
intraline: null intraline: null,
whitespace: 'IGNORE_NONE',
}; };
if (basePatchNum != PARENT_PATCH_NUM) { if (basePatchNum != PARENT_PATCH_NUM) {
params.base = basePatchNum; params.base = basePatchNum;