Moar

server/api/documents.js

@@ -5,7 +5,7 @@ import httpErrors from 'http-errors';
 import auth from './middlewares/authentication';
 import pagination from './middlewares/pagination';
 import { presentDocument } from '../presenters';
-import { User, Document, Collection, Star, View } from '../models';
+import { Document, Collection, Star, View } from '../models';
 
 const router = new Router();
 router.post('documents.list', auth(), pagination(), async ctx => {
@@ -118,7 +118,6 @@ router.post('documents.search', auth(), async ctx => {
     documents.map(async document => {
       data.push(
         await presentDocument(ctx, document, {
-          includeCollection: true,
           includeCollaborators: true,
         })
       );
@@ -206,9 +205,7 @@ router.post('documents.create', auth(), async ctx => {
 
   ctx.body = {
     data: await presentDocument(ctx, newDocument, {
-      includeCollection: true,
       includeCollaborators: true,
-      collection: ownerCollection,
     }),
   };
 });
@@ -236,9 +233,7 @@ router.post('documents.update', auth(), async ctx => {
 
   ctx.body = {
     data: await presentDocument(ctx, document, {
-      includeCollection: true,
       includeCollaborators: true,
-      collection: collection,
     }),
   };
 });
@@ -279,7 +274,6 @@ router.post('documents.move', auth(), async ctx => {
 
   ctx.body = {
     data: await presentDocument(ctx, document, {
-      includeCollection: true,
       includeCollaborators: true,
       collection: collection,
     }),

server/presenters/document.js

@@ -1,5 +1,5 @@
 // @flow
-import { Star, User, Document, View } from '../models';
+import { User, Document, View } from '../models';
 import presentUser from './user';
 import presentCollection from './collection';
 
@@ -19,7 +19,6 @@ async function present(ctx: Object, document: Document, options: Object = {}) {
     text: document.text,
     html: document.html,
     preview: document.preview,
-    collection: presentCollection(ctx, document.collection),
     createdAt: document.createdAt,
     createdBy: presentUser(ctx, document.createdBy),
     updatedAt: document.updatedAt,
@@ -27,9 +26,14 @@ async function present(ctx: Object, document: Document, options: Object = {}) {
     team: document.teamId,
     collaborators: [],
     starred: !!document.starred,
+    collection: undefined,
     views: undefined,
   };
 
+  if (options.includeCollection) {
+    data.collection = presentCollection(ctx, document.collection);
+  }
+
   if (options.includeViews) {
     data.views = await View.sum('count', {
       where: { documentId: document.id },

server/presenters/team.js

@@ -1,4 +1,7 @@
-function present(ctx, team) {
+// @flow
+import { Team } from '../models';
+
+function present(ctx: Object, team: Team) {
   ctx.cache.set(team.id, team);
 
   return {