X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Flib%2Factivitypub%2Fvideo-rates.ts;h=2e7920f4e2d0d8dc83ea1fdb3800a45c22d0cf17;hb=866b5d3f5230204d611a556260102996c1aefe10;hp=1619251c3b4f424f107f4203e1ccb79e00cb6535;hpb=1297eb5db651a230474670c5da1517862fb9cc3e;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/lib/activitypub/video-rates.ts b/server/lib/activitypub/video-rates.ts index 1619251c3..2e7920f4e 100644 --- a/server/lib/activitypub/video-rates.ts +++ b/server/lib/activitypub/video-rates.ts @@ -1,52 +1,48 @@ import { Transaction } from 'sequelize' -import { AccountModel } from '../../models/account/account' -import { VideoModel } from '../../models/video/video' -import { sendCreateDislike, sendLike, sendUndoDislike, sendUndoLike } from './send' import { VideoRateType } from '../../../shared/models/videos' -import * as Bluebird from 'bluebird' -import { getOrCreateActorAndServerAndModel } from './actor' -import { AccountVideoRateModel } from '../../models/account/account-video-rate' -import { logger } from '../../helpers/logger' -import { CRAWL_REQUEST_CONCURRENCY } from '../../initializers' +import { MAccountActor, MActorUrl, MVideoAccountLight, MVideoFullLight, MVideoId } from '../../types/models' +import { sendLike, sendUndoDislike, sendUndoLike } from './send' +import { sendDislike } from './send/send-dislike' +import { getVideoDislikeActivityPubUrlByLocalActor, getVideoLikeActivityPubUrlByLocalActor } from './url' +import { federateVideoIfNeeded } from './videos' + +async function sendVideoRateChange ( + account: MAccountActor, + video: MVideoFullLight, + likes: number, + dislikes: number, + t: Transaction +) { + if (video.isOwned()) return federateVideoIfNeeded(video, false, t) + + return sendVideoRateChangeToOrigin(account, video, likes, dislikes, t) +} -async function createRates (actorUrls: string[], video: VideoModel, rate: VideoRateType) { - let rateCounts = 0 +function getLocalRateUrl (rateType: VideoRateType, actor: MActorUrl, video: MVideoId) { + return rateType === 'like' + ? getVideoLikeActivityPubUrlByLocalActor(actor, video) + : getVideoDislikeActivityPubUrlByLocalActor(actor, video) +} - await Bluebird.map(actorUrls, async actorUrl => { - try { - const actor = await getOrCreateActorAndServerAndModel(actorUrl) - const [ , created ] = await AccountVideoRateModel - .findOrCreate({ - where: { - videoId: video.id, - accountId: actor.Account.id - }, - defaults: { - videoId: video.id, - accountId: actor.Account.id, - type: rate - } - }) +// --------------------------------------------------------------------------- - if (created) rateCounts += 1 - } catch (err) { - logger.warn('Cannot add rate %s for actor %s.', rate, actorUrl, { err }) - } - }, { concurrency: CRAWL_REQUEST_CONCURRENCY }) +export { + getLocalRateUrl, + sendVideoRateChange +} - logger.info('Adding %d %s to video %s.', rateCounts, rate, video.uuid) +// --------------------------------------------------------------------------- - // This is "likes" and "dislikes" - if (rateCounts !== 0) await video.increment(rate + 's', { by: rateCounts }) +async function sendVideoRateChangeToOrigin ( + account: MAccountActor, + video: MVideoAccountLight, + likes: number, + dislikes: number, + t: Transaction +) { + // Local video, we don't need to send like + if (video.isOwned()) return - return -} - -async function sendVideoRateChange (account: AccountModel, - video: VideoModel, - likes: number, - dislikes: number, - t: Transaction) { const actor = account.Actor // Keep the order: first we undo and then we create @@ -59,10 +55,5 @@ async function sendVideoRateChange (account: AccountModel, // Like if (likes > 0) await sendLike(actor, video, t) // Dislike - if (dislikes > 0) await sendCreateDislike(actor, video, t) -} - -export { - createRates, - sendVideoRateChange + if (dislikes > 0) await sendDislike(actor, video, t) }