Merge "Hide reply button when not logged in"
diff --git a/polygerrit-ui/app/elements/change/gr-message/gr-message.js b/polygerrit-ui/app/elements/change/gr-message/gr-message.js
index c92ad07..acb95c1 100644
--- a/polygerrit-ui/app/elements/change/gr-message/gr-message.js
+++ b/polygerrit-ui/app/elements/change/gr-message/gr-message.js
@@ -56,15 +56,22 @@
},
showReplyButton: {
type: Boolean,
- computed: '_computeShowReplyButton(message)',
+ computed: '_computeShowReplyButton(message, _loggedIn)',
},
projectConfig: Object,
+ _loggedIn: {
+ type: Boolean,
+ value: false,
+ },
},
ready: function() {
this.$.restAPI.getConfig().then(function(config) {
this.config = config;
}.bind(this));
+ this.$.restAPI.getLoggedIn().then(function(loggedIn) {
+ this._loggedIn = loggedIn;
+ }.bind(this));
},
_computeAuthor: function(message) {
@@ -75,8 +82,8 @@
return !!(author && config && config.plugin && config.plugin.has_avatars);
},
- _computeShowReplyButton: function(message) {
- return !!message.message;
+ _computeShowReplyButton: function(message, loggedIn) {
+ return !!message.message && loggedIn;
},
_commentsChanged: function(value) {
diff --git a/polygerrit-ui/app/elements/change/gr-message/gr-message_test.html b/polygerrit-ui/app/elements/change/gr-message/gr-message_test.html
index c90f58a..ff8610b 100644
--- a/polygerrit-ui/app/elements/change/gr-message/gr-message_test.html
+++ b/polygerrit-ui/app/elements/change/gr-message/gr-message_test.html
@@ -85,5 +85,12 @@
assert.equal(0, content.textContent.trim().indexOf(updatedBy.name));
});
+ test('reply button hidden unless logged in', function() {
+ var message = {
+ 'message': 'Uploaded patch set 1.',
+ };
+ assert.isFalse(element._computeShowReplyButton(message, false));
+ assert.isTrue(element._computeShowReplyButton(message, true));
+ });
});
</script>
