diff options
author | paladox <thomasmulhall410@yahoo.com> | 2021-07-16 23:18:57 +0100 |
---|---|---|
committer | Paladox none <thomasmulhall410@yahoo.com> | 2021-07-16 23:40:00 +0000 |
commit | 007fad573e8e74719d256c3c1bebd55662e9b739 (patch) | |
tree | a665b5e37b364fd94b7177a195ff5c9f22291b9a | |
parent | ad53e8d38de334a30b5df15e429041d97c320a10 (diff) |
Revert "Redirect /edit to ,edit"
This reverts commit fd06827951d5e69c1db653e8419cfd30a87a0d26.
This broke it for when a change edit existed on a merged change.
And also broke selecting a edit patchset in the selector.
Change-Id: Ife339528e2b446ab21619670e021e58d738b5954
-rw-r--r-- | polygerrit-ui/app/elements/core/gr-router/gr-router.js | 9 | ||||
-rw-r--r-- | polygerrit-ui/app/elements/core/gr-router/gr-router_test.html | 23 |
2 files changed, 3 insertions, 29 deletions
diff --git a/polygerrit-ui/app/elements/core/gr-router/gr-router.js b/polygerrit-ui/app/elements/core/gr-router/gr-router.js index ae8162f7c0..e861f2ef0c 100644 --- a/polygerrit-ui/app/elements/core/gr-router/gr-router.js +++ b/polygerrit-ui/app/elements/core/gr-router/gr-router.js @@ -1367,15 +1367,6 @@ class GrRouter extends mixinBehaviors( [ queryMap: ctx.queryMap, }; - // We do not want to allow "edit" to be used as a - // patch number. Instead redirect to ,edit. - if (ctx.params[4] === 'edit' && !ctx.params[6]) { - params.basePatchNum = undefined; - params.edit = true; - this._redirect(this._generateUrl(params)); - return; - } - this.$.reporting.setRepoName(params.project); this._redirectOrNavigate(params); } diff --git a/polygerrit-ui/app/elements/core/gr-router/gr-router_test.html b/polygerrit-ui/app/elements/core/gr-router/gr-router_test.html index 0e82a543a8..2b2db0b3a1 100644 --- a/polygerrit-ui/app/elements/core/gr-router/gr-router_test.html +++ b/polygerrit-ui/app/elements/core/gr-router/gr-router_test.html @@ -1428,16 +1428,16 @@ suite('gr-router tests', () => { suite('_handleChangeRoute', () => { let normalizeRangeStub; - function makeParams(path, hash, baseNum, patchNum) { + function makeParams(path, hash) { return { params: [ 'foo/bar', // 0 Project 1234, // 1 Change number null, // 2 Unused null, // 3 Unused - baseNum ? baseNum : 4, // 4 Base patch number + 4, // 4 Base patch number null, // 5 Unused - patchNum ? patchNum : 7, // 6 Patch number + 7, // 6 Patch number ], queryMap: new Map(), }; @@ -1475,23 +1475,6 @@ suite('gr-router tests', () => { assert.isFalse(redirectStub.called); assert.isTrue(normalizeRangeStub.called); }); - - test('redirect due to patchNum being an edit', () => { - normalizeRangeStub.returns(true); - const ctx = makeParams(null, ''); - element._handleChangeRoute(ctx, undefined, 'edit', false); - assert.isTrue(normalizeRangeStub.called); - assert.isFalse(setParamsStub.called); - assert.isTrue(redirectStub.calledOnce); - - const params = { - view: GerritNav.View.CHANGE, - changeNum: '1234', - project: 'test', - edit: true, - }; - assert.equal(element._generateUrl(params), '/c/test/+/1234,edit'); - }); }); suite('_handleDiffRoute', () => { |