oaeproject/Hilary

Function _createContent has 10 arguments (exceeds 4 allowed). Consider refactoring.

Open

  ctx,
  resourceSubType,
  displayName,
  description,
  visibility,

Avoid too many return statements within this function.

Open

  return callback({
    code: 400,
    msg: 'Unrecognized resourceSubType. Accepted values are "link", "file", "collabdoc" and "collabsheet"'
  });

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/revisions', (request, response) => {
  const limit = OaeUtil.getNumberParam(request.query.limit, 10, 1, 25);
  ContentAPI.getRevisions(
    request.ctx,
    request.params.contentId,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/library/:principalId', (request, response) => {
  const limit = OaeUtil.getNumberParam(request.query.limit, 12, 1, 25);
  ContentAPI.getContentLibraryItems(
    request.ctx,
    request.params.principalId,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/messages', (request, response) => {
  const limit = OaeUtil.getNumberParam(request.query.limit, 10, 1, 25);
  ContentAPI.getComments(
    request.ctx,
    request.params.contentId,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/members', (request, response) => {
  const limit = OaeUtil.getNumberParam(request.query.limit, 10, 1, 25);
  ContentAPI.getContentMembersLibrary(
    request.ctx,
    request.params.contentId,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/revisions/:revisionId', (request, response) => {
  ContentAPI.getRevision(request.ctx, request.params.contentId, request.params.revisionId, (error, revision) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('post', '/api/content/:contentId/revisions/:revisionId/restore', (request, response) => {
  ContentAPI.restoreRevision(request.ctx, request.params.contentId, request.params.revisionId, (error, newRevision) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/revisions/:revisionId/previews', (request, response) => {
  ContentAPI.getPreviewItems(request.ctx, request.params.contentId, request.params.revisionId, (error, previews) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('delete', '/api/content/:contentId/messages/:created', (request, response) => {
  ContentAPI.deleteComment(request.ctx, request.params.contentId, request.params.created, (error, deleted) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

OAE.tenantRouter.on('post', '/api/content/:contentId/invitations/:email/resend', (request, response) => {
  ContentAPI.resendContentInvitation(request.ctx, request.params.contentId, request.params.email, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId/invitations', (request, response) => {
  ContentAPI.getContentInvitations(request.ctx, request.params.contentId, (error, invitations) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

OAE.tenantRouter.on('post', '/api/content/:contentId/join', (request, response) => {
  ContentAPI.joinCollabDoc(request.ctx, request.params.contentId, (error, data) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/content/:contentId', (request, response) => {
  ContentAPI.getFullContentProfile(request.ctx, request.params.contentId, (error, contentProfile) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  ContentAPI.updateFileBody(
    request.ctx,
    request.params.contentId,
    request.files.file,
    (error, updatedContentObject) => {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

  ContentAPI.removeContentFromLibrary(request.ctx, request.params.principalId, request.params.contentId, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

  ContentAPI.shareContent(request.ctx, request.params.contentId, request.body.viewers, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  ContentAPI.updateContentMetadata(request.ctx, request.params.contentId, request.body, (error, newContentObject) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

  ContentAPI.verifySignedDownloadQueryString(request.ctx, request.query, (error, downloadInfo) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

    (error, downloadInfo) => {
      if (error) {
        return response.status(error.code).send(error.msg);
      }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

  ContentAPI.getRevisionDownloadInfo(request.ctx, request.params.contentId, null, (error, downloadInfo) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

    (error, downloadInfo) => {
      if (error) {
        return response.status(error.code).send(error.msg);
      }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  if (isResourceAFile(resourceSubType)) {
    return ContentAPI.createFile(
      ctx,
      { displayName, description, visibility, file: uploadedFile, additionalMembers, folders },
      callback

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  if (isResourceALink(resourceSubType)) {
    return ContentAPI.createLink(
      ctx,
      {
        displayName,

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  if (!request.files || !request.files.file) {
    return response.status(400).send('Missing file parameter');
  }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  const signature = {
    signature: request.query.signature,
    expires: request.query.expires,
    lastModified: request.query.lastmodified
  };

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

  _.each(request.body.viewers, (userId) => {
    additionalMembers[userId] = AuthzConstants.role.VIEWER;
  });

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

  _.each(request.body.editors, (userId) => {
    additionalMembers[userId] = AuthzConstants.role.EDITOR;
  });

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

  _.each(request.body.managers, (userId) => {
    additionalMembers[userId] = AuthzConstants.role.MANAGER;
  });

• Read up
  Read up