opendev/gerrit
Code Issues Proposed changes

Only show related changes from latest patch set

Browse Source 
This is to keep with the UI model that only the file list area
should change according to which patch set is selected.

Additionally fixed an issue where a thenable function was being
passed to then(), which accepts a success and failure function
and not a Promise.

Bug: Issue 4477
Change-Id: Id83db0ad77c4ec6ba2840fa27d13291f2c927797
This commit is contained in:
Andrew Bonventre
2016-09-14 14:42:07 -04:00
parent 2962ec29c7
commit b461ea9d58
3 changed files with 59 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.html
View File

@@ -256,7 +256,7 @@ limitations under the License.
<div class="relatedChanges">
<gr-related-changes-list id="relatedChanges"
change="[[_change]]"
patch-num="[[_patchRange.patchNum]]"></gr-related-changes-list>
patch-num="[[_computeLatestPatchNum(_allPatchSets)]]"></gr-related-changes-list>
</div>
</div>
</div>

2
polygerrit-ui/app/elements/change/gr-related-changes-list/gr-related-changes-list.html
View File

@@ -74,6 +74,7 @@ limitations under the License.
}
</style>
<div hidden$="[[!_loading]]">Loading...</div>
<div hidden$="[[_loading]]">
<section class="relatedChanges" hidden$="[[!_relatedResponse.changes.length]]" hidden>
<h4>Relation chain</h4>
<template
@@ -127,6 +128,7 @@ limitations under the License.
</a>
</template>
</section>
</div>
<gr-rest-api-interface id="restAPI"></gr-rest-api-interface>
</template>
<script src="gr-related-changes-list.js"></script>

6
polygerrit-ui/app/elements/change/gr-related-changes-list/gr-related-changes-list.js
View File

@@ -68,7 +68,7 @@
}.bind(this)),
];
return this._getServerConfig().then(function(config) {
promises.push(this._getServerConfig().then(function(config) {
if (this.change.topic && !config.change.submit_whole_topic) {
return this._getChangesWithSameTopic().then(function(response) {
this._sameTopic = response;
@@ -77,7 +77,9 @@
this._sameTopic = [];
}
return this._sameTopic;
}.bind(this)).then(Promise.all(promises)).then(function() {
}.bind(this)));
return Promise.all(promises).then(function() {
this._loading = false;
}.bind(this));
},