- Fix "Error: title is required" by removing find() from all of migrations.

Thanks to xet7 !

Closes #1576
pull/1715/merge
Lauri Ojansivu 7 years ago
parent c372ea02f7
commit 97922c90cb
  1. 18
      server/migrations.js

@ -55,7 +55,7 @@ Migrations.add('lowercase-board-permission', () => {
// Security migration: see https://github.com/wekan/wekan/issues/99 // Security migration: see https://github.com/wekan/wekan/issues/99
Migrations.add('change-attachments-type-for-non-images', () => { Migrations.add('change-attachments-type-for-non-images', () => {
const newTypeForNonImage = 'application/octet-stream'; const newTypeForNonImage = 'application/octet-stream';
Attachments.find().forEach((file) => { Attachments.forEach((file) => {
if (!file.isImage()) { if (!file.isImage()) {
Attachments.update(file._id, { Attachments.update(file._id, {
$set: { $set: {
@ -68,7 +68,7 @@ Migrations.add('change-attachments-type-for-non-images', () => {
}); });
Migrations.add('card-covers', () => { Migrations.add('card-covers', () => {
Cards.find().forEach((card) => { Cards.forEach((card) => {
const cover = Attachments.findOne({ cardId: card._id, cover: true }); const cover = Attachments.findOne({ cardId: card._id, cover: true });
if (cover) { if (cover) {
Cards.update(card._id, {$set: {coverId: cover._id}}, noValidate); Cards.update(card._id, {$set: {coverId: cover._id}}, noValidate);
@ -86,7 +86,7 @@ Migrations.add('use-css-class-for-boards-colors', () => {
'#2C3E50': 'midnight', '#2C3E50': 'midnight',
'#E67E22': 'pumpkin', '#E67E22': 'pumpkin',
}; };
Boards.find().forEach((board) => { Boards.forEach((board) => {
const oldBoardColor = board.background.color; const oldBoardColor = board.background.color;
const newBoardColor = associationTable[oldBoardColor]; const newBoardColor = associationTable[oldBoardColor];
Boards.update(board._id, { Boards.update(board._id, {
@ -97,7 +97,7 @@ Migrations.add('use-css-class-for-boards-colors', () => {
}); });
Migrations.add('denormalize-star-number-per-board', () => { Migrations.add('denormalize-star-number-per-board', () => {
Boards.find().forEach((board) => { Boards.forEach((board) => {
const nStars = Users.find({'profile.starredBoards': board._id}).count(); const nStars = Users.find({'profile.starredBoards': board._id}).count();
Boards.update(board._id, {$set: {stars: nStars}}, noValidate); Boards.update(board._id, {$set: {stars: nStars}}, noValidate);
}); });
@ -132,7 +132,7 @@ Migrations.add('add-member-isactive-field', () => {
}); });
Migrations.add('add-sort-checklists', () => { Migrations.add('add-sort-checklists', () => {
Checklists.find().forEach((checklist, index) => { Checklists.forEach((checklist, index) => {
if (!checklist.hasOwnProperty('sort')) { if (!checklist.hasOwnProperty('sort')) {
Checklists.direct.update( Checklists.direct.update(
checklist._id, checklist._id,
@ -153,7 +153,7 @@ Migrations.add('add-sort-checklists', () => {
}); });
Migrations.add('add-swimlanes', () => { Migrations.add('add-swimlanes', () => {
Boards.find().forEach((board) => { Boards.forEach((board) => {
const swimlane = Swimlanes.findOne({ boardId: board._id }); const swimlane = Swimlanes.findOne({ boardId: board._id });
let swimlaneId = ''; let swimlaneId = '';
if (swimlane) if (swimlane)
@ -177,7 +177,7 @@ Migrations.add('add-swimlanes', () => {
}); });
Migrations.add('add-views', () => { Migrations.add('add-views', () => {
Boards.find().forEach((board) => { Boards.forEach((board) => {
if (!board.hasOwnProperty('view')) { if (!board.hasOwnProperty('view')) {
Boards.direct.update( Boards.direct.update(
{ _id: board._id }, { _id: board._id },
@ -189,7 +189,7 @@ Migrations.add('add-views', () => {
}); });
Migrations.add('add-checklist-items', () => { Migrations.add('add-checklist-items', () => {
Checklists.find().forEach((checklist) => { Checklists.forEach((checklist) => {
// Create new items // Create new items
_.sortBy(checklist.items, 'sort').forEach((item, index) => { _.sortBy(checklist.items, 'sort').forEach((item, index) => {
ChecklistItems.direct.insert({ ChecklistItems.direct.insert({
@ -210,7 +210,7 @@ Migrations.add('add-checklist-items', () => {
}); });
Migrations.add('add-profile-view', () => { Migrations.add('add-profile-view', () => {
Users.find().forEach((user) => { Users.forEach((user) => {
if (!user.hasOwnProperty('profile.boardView')) { if (!user.hasOwnProperty('profile.boardView')) {
// Set default view // Set default view
Users.direct.update( Users.direct.update(

Loading…
Cancel
Save