forks/peertube
1
0
Fork 0
Code Releases Activity

Include transaction in video hooks

Browse source
This commit is contained in:
Chocobozzz 2021-06-11 16:22:54 +02:00
parent adddb12b04
commit 9f7657b6b6
No known key found for this signature in database
GPG key ID: 583A612D890159BE
1 changed files with 6 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
server/models/video/video.ts
View file

@ -756,14 +756,14 @@ export class VideoModel extends Model<Partial<AttributesOnly<VideoModel>>> {
} }
@BeforeDestroy @BeforeDestroy
static async removeFiles (instance: VideoModel) { static async removeFiles (instance: VideoModel, options) {
const tasks: Promise<any>[] = [] const tasks: Promise<any>[] = []
logger.info('Removing files of video %s.', instance.url) logger.info('Removing files of video %s.', instance.url)
if (instance.isOwned()) { if (instance.isOwned()) {
if (!Array.isArray(instance.VideoFiles)) { if (!Array.isArray(instance.VideoFiles)) {
instance.VideoFiles = await instance.$get('VideoFiles') instance.VideoFiles = await instance.$get('VideoFiles', { transaction: options.transaction })
} }
// Remove physical files and torrents // Remove physical files and torrents
@ -774,7 +774,7 @@ export class VideoModel extends Model<Partial<AttributesOnly<VideoModel>>> {
// Remove playlists file // Remove playlists file
if (!Array.isArray(instance.VideoStreamingPlaylists)) { if (!Array.isArray(instance.VideoStreamingPlaylists)) {
instance.VideoStreamingPlaylists = await instance.$get('VideoStreamingPlaylists') instance.VideoStreamingPlaylists = await instance.$get('VideoStreamingPlaylists', { transaction: options.transaction })
} }
for (const p of instance.VideoStreamingPlaylists) { for (const p of instance.VideoStreamingPlaylists) {
@ -797,7 +797,7 @@ export class VideoModel extends Model<Partial<AttributesOnly<VideoModel>>> {
logger.info('Stopping live of video %s after video deletion.', instance.uuid) logger.info('Stopping live of video %s after video deletion.', instance.uuid)
return LiveManager.Instance.stopSessionOf(instance.id) LiveManager.Instance.stopSessionOf(instance.id)
} }
@BeforeDestroy @BeforeDestroy
@ -810,7 +810,7 @@ export class VideoModel extends Model<Partial<AttributesOnly<VideoModel>>> {
const tasks: Promise<any>[] = [] const tasks: Promise<any>[] = []
if (!Array.isArray(instance.VideoAbuses)) { if (!Array.isArray(instance.VideoAbuses)) {
instance.VideoAbuses = await instance.$get('VideoAbuses') instance.VideoAbuses = await instance.$get('VideoAbuses', { transaction: options.transaction })
if (instance.VideoAbuses.length === 0) return undefined if (instance.VideoAbuses.length === 0) return undefined
} }
@ -825,12 +825,7 @@ export class VideoModel extends Model<Partial<AttributesOnly<VideoModel>>> {
tasks.push(abuse.save({ transaction: options.transaction })) tasks.push(abuse.save({ transaction: options.transaction }))
} }
Promise.all(tasks) await Promise.all(tasks)
.catch(err => {
logger.error('Some errors when saving details of video %s in its abuses before destroy hook.', instance.uuid, { err })
})
return undefined
} }
static listLocal (): Promise<MVideo[]> { static listLocal (): Promise<MVideo[]> {