From 4cb6d4578893db310297d7e118ce2fb7ecb952a3 Mon Sep 17 00:00:00 2001 From: Chocobozzz Date: Thu, 4 Jan 2018 11:19:16 +0100 Subject: Add ability to delete comments --- server/controllers/api/videos/comment.ts | 31 +++++++++++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-) (limited to 'server/controllers/api/videos') diff --git a/server/controllers/api/videos/comment.ts b/server/controllers/api/videos/comment.ts index e09b242ed..65fcf6b35 100644 --- a/server/controllers/api/videos/comment.ts +++ b/server/controllers/api/videos/comment.ts @@ -2,14 +2,15 @@ import * as express from 'express' import { ResultList } from '../../../../shared/models' import { VideoCommentCreate } from '../../../../shared/models/videos/video-comment.model' import { retryTransactionWrapper } from '../../../helpers/database-utils' +import { logger } from '../../../helpers/logger' import { getFormattedObjects } from '../../../helpers/utils' import { sequelizeTypescript } from '../../../initializers' import { buildFormattedCommentTree, createVideoComment } from '../../../lib/video-comment' import { asyncMiddleware, authenticate, paginationValidator, setPagination, setVideoCommentThreadsSort } from '../../../middlewares' import { videoCommentThreadsSortValidator } from '../../../middlewares/validators' import { - addVideoCommentReplyValidator, addVideoCommentThreadValidator, listVideoCommentThreadsValidator, - listVideoThreadCommentsValidator + addVideoCommentReplyValidator, addVideoCommentThreadValidator, listVideoCommentThreadsValidator, listVideoThreadCommentsValidator, + removeVideoCommentValidator } from '../../../middlewares/validators/video-comments' import { VideoModel } from '../../../models/video/video' import { VideoCommentModel } from '../../../models/video/video-comment' @@ -39,6 +40,11 @@ videoCommentRouter.post('/:videoId/comments/:commentId', asyncMiddleware(addVideoCommentReplyValidator), asyncMiddleware(addVideoCommentReplyRetryWrapper) ) +videoCommentRouter.delete('/:videoId/comments/:commentId', + authenticate, + asyncMiddleware(removeVideoCommentValidator), + asyncMiddleware(removeVideoCommentRetryWrapper) +) // --------------------------------------------------------------------------- @@ -131,3 +137,24 @@ function addVideoCommentReply (req: express.Request, res: express.Response, next }, t) }) } + +async function removeVideoCommentRetryWrapper (req: express.Request, res: express.Response, next: express.NextFunction) { + const options = { + arguments: [ req, res ], + errorMessage: 'Cannot remove the video comment with many retries.' + } + + await retryTransactionWrapper(removeVideoComment, options) + + return res.type('json').status(204).end() +} + +async function removeVideoComment (req: express.Request, res: express.Response) { + const videoCommentInstance: VideoCommentModel = res.locals.videoComment + + await sequelizeTypescript.transaction(async t => { + await videoCommentInstance.destroy({ transaction: t }) + }) + + logger.info('Video comment %d deleted.', videoCommentInstance.id) +} -- cgit v1.2.3