summaryrefslogtreecommitdiffstats
path: root/client/components/activities/activities.js
diff options
context:
space:
mode:
authorMaxime Quandalle <maxime@quandalle.com>2015-09-03 23:12:46 +0200
committerMaxime Quandalle <maxime@quandalle.com>2015-09-03 23:12:46 +0200
commitb3851817ecd59b039f2c2228d08a1c6fd8e60d60 (patch)
tree82a50f69788d5c20632f3ec9c7d3e136502b93b4 /client/components/activities/activities.js
parent039cfe7edf8faf901069a94b3ca9b66f7973b26a (diff)
downloadwekan-b3851817ecd59b039f2c2228d08a1c6fd8e60d60.tar.gz
wekan-b3851817ecd59b039f2c2228d08a1c6fd8e60d60.tar.bz2
wekan-b3851817ecd59b039f2c2228d08a1c6fd8e60d60.zip
Enforce a consistent ES6 coding style
Replace the old (and broken) jshint + jscsrc by eslint and configure it to support some of the ES6 features. The command `eslint` currently has one error which is a bug that was discovered by its static analysis and should be fixed (usage of a dead object).
Diffstat (limited to 'client/components/activities/activities.js')
-rw-r--r--client/components/activities/activities.js75
1 files changed, 37 insertions, 38 deletions
diff --git a/client/components/activities/activities.js b/client/components/activities/activities.js
index 400e3848..d6a0cf7f 100644
--- a/client/components/activities/activities.js
+++ b/client/components/activities/activities.js
@@ -1,99 +1,98 @@
-var activitiesPerPage = 20;
+const activitiesPerPage = 20;
BlazeComponent.extendComponent({
- template: function() {
+ template() {
return 'activities';
},
- onCreated: function() {
- var self = this;
+ onCreated() {
// XXX Should we use ReactiveNumber?
- self.page = new ReactiveVar(1);
- self.loadNextPageLocked = false;
- var sidebar = self.componentParent(); // XXX for some reason not working
+ this.page = new ReactiveVar(1);
+ this.loadNextPageLocked = false;
+ const sidebar = this.componentParent(); // XXX for some reason not working
sidebar.callFirstWith(null, 'resetNextPeak');
- self.autorun(function() {
- var mode = self.data().mode;
- var capitalizedMode = Utils.capitalize(mode);
- var id = Session.get('current' + capitalizedMode);
- var limit = self.page.get() * activitiesPerPage;
+ this.autorun(() => {
+ const mode = this.data().mode;
+ const capitalizedMode = Utils.capitalize(mode);
+ const id = Session.get(`current${capitalizedMode}`);
+ const limit = this.page.get() * activitiesPerPage;
if (id === null)
return;
- self.subscribe('activities', mode, id, limit, function() {
- self.loadNextPageLocked = false;
+ this.subscribe('activities', mode, id, limit, () => {
+ this.loadNextPageLocked = false;
// If the sibear peak hasn't increased, that mean that there are no more
// activities, and we can stop calling new subscriptions.
// XXX This is hacky! We need to know excatly and reactively how many
// activities there are, we probably want to denormalize this number
// dirrectly into card and board documents.
- var a = sidebar.callFirstWith(null, 'getNextPeak');
+ const nextPeakBefore = sidebar.callFirstWith(null, 'getNextPeak');
sidebar.calculateNextPeak();
- var b = sidebar.callFirstWith(null, 'getNextPeak');
- if (a === b) {
+ const nextPeakAfter = sidebar.callFirstWith(null, 'getNextPeak');
+ if (nextPeakBefore === nextPeakAfter) {
sidebar.callFirstWith(null, 'resetNextPeak');
}
});
});
},
- loadNextPage: function() {
+ loadNextPage() {
if (this.loadNextPageLocked === false) {
this.page.set(this.page.get() + 1);
this.loadNextPageLocked = true;
}
},
- boardLabel: function() {
+ boardLabel() {
return TAPi18n.__('this-board');
},
- cardLabel: function() {
+ cardLabel() {
return TAPi18n.__('this-card');
},
- cardLink: function() {
- var card = this.currentData().card();
+ cardLink() {
+ const card = this.currentData().card();
return card && Blaze.toHTML(HTML.A({
href: card.absoluteUrl(),
- 'class': 'action-card'
+ 'class': 'action-card',
}, card.title));
},
- memberLink: function() {
+ memberLink() {
return Blaze.toHTMLWithData(Template.memberName, {
- user: this.currentData().member()
+ user: this.currentData().member(),
});
},
- attachmentLink: function() {
- var attachment = this.currentData().attachment();
+ attachmentLink() {
+ const attachment = this.currentData().attachment();
return attachment && Blaze.toHTML(HTML.A({
href: attachment.url(),
- 'class': 'js-open-attachment-viewer'
+ 'class': 'js-open-attachment-viewer',
}, attachment.name()));
},
- events: function() {
+ events() {
return [{
// XXX We should use Popup.afterConfirmation here
- 'click .js-delete-comment': function() {
- var commentId = this.currentData().commentId;
+ 'click .js-delete-comment'() {
+ const commentId = this.currentData().commentId;
CardComments.remove(commentId);
},
- 'submit .js-edit-comment': function(evt) {
+ 'submit .js-edit-comment'(evt) {
evt.preventDefault();
- var commentText = this.currentComponent().getValue();
- var commentId = Template.parentData().commentId;
+ const commentText = this.currentComponent().getValue();
+ const commentId = Template.parentData().commentId;
if ($.trim(commentText)) {
CardComments.update(commentId, {
$set: {
- text: commentText
- }
+ text: commentText,
+ },
});
}
- }
+ },
}];
- }
+ },
}).register('activities');