Move models to typescript-sequelize

[oweals/peertube.git] / server / lib / activitypub / videos.ts

diff --git a/server/lib/activitypub/videos.ts b/server/lib/activitypub/videos.ts
index acee4fe16e38fd1c0f236154b5529a47f42ca948..14c07fec0e196d72381bda9747021d9c24c982f1 100644 (file)
--- a/server/lib/activitypub/videos.ts
+++ b/server/lib/activitypub/videos.ts
@@ -2,21 +2,22 @@ import { join } from 'path'
 import * as request from 'request'
 import { Transaction } from 'sequelize'
 import { ActivityIconObject } from '../../../shared/index'
-import { doRequest, doRequestAndSaveToFile } from '../../helpers/requests'
-import { CONFIG, REMOTE_SCHEME, STATIC_PATHS } from '../../initializers/constants'
-import { AccountInstance } from '../../models/account/account-interface'
-import { VideoInstance } from '../../models/video/video-interface'
-import { sendLikeToOrigin } from './index'
-import { sendCreateDislikeToOrigin, sendCreateDislikeToVideoFollowers } from './send/send-create'
-import { sendLikeToVideoFollowers } from './send/send-like'
+import { doRequest, doRequestAndSaveToFile } from '../../helpers'
+import { CONFIG, REMOTE_SCHEME, STATIC_PATHS } from '../../initializers'
+import { AccountModel } from '../../models/account/account'
+import { VideoModel } from '../../models/video/video'
 import {
+  sendCreateDislikeToOrigin,
+  sendCreateDislikeToVideoFollowers,
+  sendLikeToOrigin,
+  sendLikeToVideoFollowers,
   sendUndoDislikeToOrigin,
   sendUndoDislikeToVideoFollowers,
   sendUndoLikeToOrigin,
   sendUndoLikeToVideoFollowers
-} from './send/send-undo'
+} from './send'
 
-function fetchRemoteVideoPreview (video: VideoInstance) {
+function fetchRemoteVideoPreview (video: VideoModel) {
   // FIXME: use url
   const host = video.VideoChannel.Account.Server.host
   const path = join(STATIC_PATHS.PREVIEWS, video.getPreviewName())
@@ -24,7 +25,7 @@ function fetchRemoteVideoPreview (video: VideoInstance) {
   return request.get(REMOTE_SCHEME.HTTP + '://' + host + path)
 }
 
-async function fetchRemoteVideoDescription (video: VideoInstance) {
+async function fetchRemoteVideoDescription (video: VideoModel) {
   // FIXME: use url
   const host = video.VideoChannel.Account.Server.host
   const path = video.getDescriptionPath()
@@ -37,7 +38,7 @@ async function fetchRemoteVideoDescription (video: VideoInstance) {
   return body.description ? body.description : ''
 }
 
-function generateThumbnailFromUrl (video: VideoInstance, icon: ActivityIconObject) {
+function generateThumbnailFromUrl (video: VideoModel, icon: ActivityIconObject) {
   const thumbnailName = video.getThumbnailName()
   const thumbnailPath = join(CONFIG.STORAGE.THUMBNAILS_DIR, thumbnailName)
 
@@ -48,36 +49,44 @@ function generateThumbnailFromUrl (video: VideoInstance, icon: ActivityIconObjec
   return doRequestAndSaveToFile(options, thumbnailPath)
 }
 
-function sendVideoRateChangeToFollowers (account: AccountInstance, video: VideoInstance, likes: number, dislikes: number, t: Transaction) {
-  const tasks: Promise<any>[] = []
+async function sendVideoRateChangeToFollowers (
+    account: AccountModel,
+  video: VideoModel,
+  likes: number,
+  dislikes: number,
+  t: Transaction
+) {
+  // Keep the order: first we undo and then we create
 
   // Undo Like
-  if (likes < 0) tasks.push(sendUndoLikeToVideoFollowers(account, video, t))
-  // Like
-  if (likes > 0) tasks.push(sendLikeToVideoFollowers(account, video, t))
-
+  if (likes < 0) await sendUndoLikeToVideoFollowers(account, video, t)
   // Undo Dislike
-  if (dislikes < 0) tasks.push(sendUndoDislikeToVideoFollowers(account, video, t))
-  // Dislike
-  if (dislikes > 0) tasks.push(sendCreateDislikeToVideoFollowers(account, video, t))
+  if (dislikes < 0) await sendUndoDislikeToVideoFollowers(account, video, t)
 
-  return Promise.all(tasks)
+  // Like
+  if (likes > 0) await sendLikeToVideoFollowers(account, video, t)
+  // Dislike
+  if (dislikes > 0) await sendCreateDislikeToVideoFollowers(account, video, t)
 }
 
-function sendVideoRateChangeToOrigin (account: AccountInstance, video: VideoInstance, likes: number, dislikes: number, t: Transaction) {
-  const tasks: Promise<any>[] = []
+async function sendVideoRateChangeToOrigin (
+    account: AccountModel,
+  video: VideoModel,
+  likes: number,
+  dislikes: number,
+  t: Transaction
+) {
+  // Keep the order: first we undo and then we create
 
   // Undo Like
-  if (likes < 0) tasks.push(sendUndoLikeToOrigin(account, video, t))
-  // Like
-  if (likes > 0) tasks.push(sendLikeToOrigin(account, video, t))
-
+  if (likes < 0) await sendUndoLikeToOrigin(account, video, t)
   // Undo Dislike
-  if (dislikes < 0) tasks.push(sendUndoDislikeToOrigin(account, video, t))
-  // Dislike
-  if (dislikes > 0) tasks.push(sendCreateDislikeToOrigin(account, video, t))
+  if (dislikes < 0) await sendUndoDislikeToOrigin(account, video, t)
 
-  return Promise.all(tasks)
+  // Like
+  if (likes > 0) await sendLikeToOrigin(account, video, t)
+  // Dislike
+  if (dislikes > 0) await sendCreateDislikeToOrigin(account, video, t)
 }
 
 export {