X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Flib%2Factivitypub%2Fprocess%2Fprocess-like.ts;h=5f2ffe7eaa6cc2aae265683163ce2b98e07e199d;hb=f00984c0077e9b666fe8005452768e53d1c3f421;hp=d77b30f2449aa0cee284422b0aa2cc6041d5c477;hpb=0032ebe94aa83fab761c7de3ceb6210ac4532824;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/lib/activitypub/process/process-like.ts b/server/lib/activitypub/process/process-like.ts index d77b30f24..5f2ffe7ea 100644 --- a/server/lib/activitypub/process/process-like.ts +++ b/server/lib/activitypub/process/process-like.ts @@ -1,14 +1,14 @@ import { ActivityLike } from '../../../../shared/models/activitypub/activity' +import { retryTransactionWrapper } from '../../../helpers/database-utils' import { database as db } from '../../../initializers' import { AccountInstance } from '../../../models/account/account-interface' import { getOrCreateAccountAndServer } from '../account' -import { sendLikeToVideoFollowers } from '../send/send-like' -import { retryTransactionWrapper } from '../../../helpers/database-utils' +import { forwardActivity } from '../send/misc' async function processLikeActivity (activity: ActivityLike) { const account = await getOrCreateAccountAndServer(activity.actor) - return processLikeVideo(account, activity.object) + return processLikeVideo(account, activity) } // --------------------------------------------------------------------------- @@ -19,16 +19,18 @@ export { // --------------------------------------------------------------------------- -async function processLikeVideo (byAccount: AccountInstance, videoUrl: string) { +async function processLikeVideo (byAccount: AccountInstance, activity: ActivityLike) { const options = { - arguments: [ byAccount, videoUrl ], + arguments: [ byAccount, activity ], errorMessage: 'Cannot like the video with many retries.' } return retryTransactionWrapper(createVideoLike, options) } -function createVideoLike (byAccount: AccountInstance, videoUrl: string) { +function createVideoLike (byAccount: AccountInstance, activity: ActivityLike) { + const videoUrl = activity.object + return db.sequelize.transaction(async t => { const video = await db.Video.loadByUrlAndPopulateAccount(videoUrl) @@ -41,10 +43,15 @@ function createVideoLike (byAccount: AccountInstance, videoUrl: string) { } const [ , created ] = await db.AccountVideoRate.findOrCreate({ where: rate, - defaults: rate + defaults: rate, + transaction: t }) - await video.increment('likes') + if (created === true) await video.increment('likes', { transaction: t }) - if (video.isOwned() && created === true) await sendLikeToVideoFollowers(byAccount, video, undefined) + if (video.isOwned() && created === true) { + // Don't resend the activity to the sender + const exceptions = [ byAccount ] + await forwardActivity(activity, t, exceptions) + } }) }