Commit 409bc054 authored by Douwe Maan's avatar Douwe Maan

Fix bug causing “Jump to discussion” button not to show

parent 8b05e8c2
...@@ -14,11 +14,7 @@ ...@@ -14,11 +14,7 @@
return this.discussions[this.discussionId]; return this.discussions[this.discussionId];
}, },
allResolved: function () { allResolved: function () {
if (this.discussion) { return this.unresolvedDiscussionCount === 0;
return this.unresolvedDiscussionCount === 0;
} else {
return true;
}
}, },
showButton: function () { showButton: function () {
if (this.discussionId) { if (this.discussionId) {
...@@ -66,7 +62,7 @@ ...@@ -66,7 +62,7 @@
discussionIdsInScope = discussionIdsForElements($(discussionsSelector)); discussionIdsInScope = discussionIdsForElements($(discussionsSelector));
let unresolvedDiscussionCount = 0; let unresolvedDiscussionCount = 0;
for (let i = 0; i < discussionIdsInScope.length; i++) { for (let i = 0; i < discussionIdsInScope.length; i++) {
const discussionId = discussionIdsInScope[i]; const discussionId = discussionIdsInScope[i];
if (discussion && !discussion.isResolved()) { if (discussion && !discussion.isResolved()) {
......
...@@ -23,18 +23,18 @@ ...@@ -23,18 +23,18 @@
if (this.discussion) { if (this.discussion) {
return this.discussion.isResolvable(); return this.discussion.isResolvable();
} else { } else {
return undefined; return false;
} }
}, },
allResolved: function () { isDiscussionResolved: function () {
if (this.discussion) { if (this.discussion) {
return this.discussion.isResolved(); return this.discussion.isResolved();
} else { } else {
return undefined; return false;
} }
}, },
buttonText: function () { buttonText: function () {
if (this.allResolved) { if (this.isDiscussionResolved) {
return "Unresolve discussion"; return "Unresolve discussion";
} else { } else {
return "Resolve discussion"; return "Resolve discussion";
...@@ -44,7 +44,7 @@ ...@@ -44,7 +44,7 @@
if (this.discussion) { if (this.discussion) {
return this.discussion.loading; return this.discussion.loading;
} else { } else {
return undefined; return false;
} }
} }
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment