Andreas/wekan
1
0
Fork 0
mirror of https://github.com/wekan/wekan.git synced 2026-03-09 07:02:34 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Use an arrow function inside forEach() instead of an anonymous function

Browse source 
Suggested by deepcode.ai.
This commit is contained in:
Marc Hartmayer 2020-05-24 12:37:54 +02:00
parent 06515559a6
commit b2fee6a6c1
5 changed files with 9 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

4
client/components/boards/boardBody.js
View file

@ -365,12 +365,12 @@ BlazeComponent.extendComponent({
}; };
currentBoard currentBoard
.cardsInInterval(start.toDate(), end.toDate()) .cardsInInterval(start.toDate(), end.toDate())
.forEach(function(card) { .forEach(card => {
pushEvent(card); pushEvent(card);
}); });
currentBoard currentBoard
.cardsDueInBetween(start.toDate(), end.toDate()) .cardsDueInBetween(start.toDate(), end.toDate())
.forEach(function(card) { .forEach(card => {
pushEvent( pushEvent(
card, card,
`${card.title} ${TAPi18n.__('card-due')}`, `${card.title} ${TAPi18n.__('card-due')}`,

4
client/components/cards/cardDetails.js
View file

@ -805,9 +805,9 @@ Template.copyChecklistToManyCardsPopup.events({
// copy subtasks // copy subtasks
cursor = Cards.find({ parentId: oldId }); cursor = Cards.find({ parentId: oldId });
cursor.forEach(function() { cursor.forEach(cur => {
'use strict'; 'use strict';
const subtask = arguments[0]; const subtask = cur;
subtask.parentId = _id; subtask.parentId = _id;
subtask._id = null; subtask._id = null;
/* const newSubtaskId = */ Cards.insert(subtask); /* const newSubtaskId = */ Cards.insert(subtask);

2
client/components/sidebar/sidebar.js
View file

@ -653,7 +653,7 @@ BlazeComponent.extendComponent({
'subtext-with-parent', 'subtext-with-parent',
'no-parent', 'no-parent',
]; ];
options.forEach(function(element) { options.forEach(element => {
if (element !== value) { if (element !== value) {
$(`#${element} ${MCB}`).toggleClass(CKCLS, false); $(`#${element} ${MCB}`).toggleClass(CKCLS, false);
$(`#${element}`).toggleClass(CKCLS, false); $(`#${element}`).toggleClass(CKCLS, false);

6
models/checklists.js
View file

@ -100,13 +100,13 @@ Checklists.helpers({
}, },
checkAllItems() { checkAllItems() {
const checkItems = ChecklistItems.find({ checklistId: this._id }); const checkItems = ChecklistItems.find({ checklistId: this._id });
checkItems.forEach(function(item) { checkItems.forEach(item => {
item.check(); item.check();
}); });
}, },
uncheckAllItems() { uncheckAllItems() {
const checkItems = ChecklistItems.find({ checklistId: this._id }); const checkItems = ChecklistItems.find({ checklistId: this._id });
checkItems.forEach(function(item) { checkItems.forEach(item => {
item.uncheck(); item.uncheck();
}); });
}, },
@ -307,7 +307,7 @@ if (Meteor.isServer) {
items = [items]; items = [items];
} }
} }
items.forEach(function(item, idx) { items.forEach((item, idx) => {
ChecklistItems.insert({ ChecklistItems.insert({
cardId: paramCardId, cardId: paramCardId,
checklistId: id, checklistId: id,

2
server/rulesHelper.js
View file

@ -17,7 +17,7 @@ RulesHelper = {
const matchingMap = this.buildMatchingFieldsMap(activity, matchingFields); const matchingMap = this.buildMatchingFieldsMap(activity, matchingFields);
const matchingTriggers = Triggers.find(matchingMap); const matchingTriggers = Triggers.find(matchingMap);
const matchingRules = []; const matchingRules = [];
matchingTriggers.forEach(function(trigger) { matchingTriggers.forEach(trigger => {
const rule = trigger.getRule(); const rule = trigger.getRule();
// Check that for some unknown reason there are some leftover triggers // Check that for some unknown reason there are some leftover triggers
// not connected to any rules // not connected to any rules