diff options
Diffstat (limited to 'server/models/video')
4 files changed, 17 insertions, 14 deletions
diff --git a/server/models/video/sql/shared/abstract-videos-model-query-builder.ts b/server/models/video/sql/shared/abstract-videos-model-query-builder.ts index d959cb5d0..0d7e64574 100644 --- a/server/models/video/sql/shared/abstract-videos-model-query-builder.ts +++ b/server/models/video/sql/shared/abstract-videos-model-query-builder.ts | |||
@@ -102,9 +102,8 @@ export class AbstractVideosModelQueryBuilder extends AbstractVideosQueryBuilder | |||
102 | } | 102 | } |
103 | } | 103 | } |
104 | 104 | ||
105 | protected includeWebtorrentFiles (required: boolean) { | 105 | protected includeWebtorrentFiles () { |
106 | const joinType = required ? 'INNER' : 'LEFT' | 106 | this.addJoin('LEFT JOIN "videoFile" AS "VideoFiles" ON "VideoFiles"."videoId" = "video"."id"') |
107 | this.addJoin(joinType + ' JOIN "videoFile" AS "VideoFiles" ON "VideoFiles"."videoId" = "video"."id"') | ||
108 | 107 | ||
109 | this.attributes = { | 108 | this.attributes = { |
110 | ...this.attributes, | 109 | ...this.attributes, |
@@ -113,15 +112,13 @@ export class AbstractVideosModelQueryBuilder extends AbstractVideosQueryBuilder | |||
113 | } | 112 | } |
114 | } | 113 | } |
115 | 114 | ||
116 | protected includeStreamingPlaylistFiles (required: boolean) { | 115 | protected includeStreamingPlaylistFiles () { |
117 | const joinType = required ? 'INNER' : 'LEFT' | ||
118 | |||
119 | this.addJoin( | 116 | this.addJoin( |
120 | joinType + ' JOIN "videoStreamingPlaylist" AS "VideoStreamingPlaylists" ON "VideoStreamingPlaylists"."videoId" = "video"."id"' | 117 | 'LEFT JOIN "videoStreamingPlaylist" AS "VideoStreamingPlaylists" ON "VideoStreamingPlaylists"."videoId" = "video"."id"' |
121 | ) | 118 | ) |
122 | 119 | ||
123 | this.addJoin( | 120 | this.addJoin( |
124 | joinType + ' JOIN "videoFile" AS "VideoStreamingPlaylists->VideoFiles" ' + | 121 | 'LEFT JOIN "videoFile" AS "VideoStreamingPlaylists->VideoFiles" ' + |
125 | 'ON "VideoStreamingPlaylists->VideoFiles"."videoStreamingPlaylistId" = "VideoStreamingPlaylists"."id"' | 122 | 'ON "VideoStreamingPlaylists->VideoFiles"."videoStreamingPlaylistId" = "VideoStreamingPlaylists"."id"' |
126 | ) | 123 | ) |
127 | 124 | ||
diff --git a/server/models/video/sql/shared/video-file-query-builder.ts b/server/models/video/sql/shared/video-file-query-builder.ts index a62fa64f8..6b15c3b69 100644 --- a/server/models/video/sql/shared/video-file-query-builder.ts +++ b/server/models/video/sql/shared/video-file-query-builder.ts | |||
@@ -32,7 +32,7 @@ export class VideoFileQueryBuilder extends AbstractVideosModelQueryBuilder { | |||
32 | '"video"."id"': '' | 32 | '"video"."id"': '' |
33 | } | 33 | } |
34 | 34 | ||
35 | this.includeWebtorrentFiles(true) | 35 | this.includeWebtorrentFiles() |
36 | 36 | ||
37 | if (this.shouldIncludeRedundancies(options)) { | 37 | if (this.shouldIncludeRedundancies(options)) { |
38 | this.includeWebTorrentRedundancies() | 38 | this.includeWebTorrentRedundancies() |
@@ -48,7 +48,7 @@ export class VideoFileQueryBuilder extends AbstractVideosModelQueryBuilder { | |||
48 | '"video"."id"': '' | 48 | '"video"."id"': '' |
49 | } | 49 | } |
50 | 50 | ||
51 | this.includeStreamingPlaylistFiles(true) | 51 | this.includeStreamingPlaylistFiles() |
52 | 52 | ||
53 | if (this.shouldIncludeRedundancies(options)) { | 53 | if (this.shouldIncludeRedundancies(options)) { |
54 | this.includeStreamingPlaylistRedundancies() | 54 | this.includeStreamingPlaylistRedundancies() |
diff --git a/server/models/video/sql/shared/video-model-builder.ts b/server/models/video/sql/shared/video-model-builder.ts index 467a9378a..cc71192c0 100644 --- a/server/models/video/sql/shared/video-model-builder.ts +++ b/server/models/video/sql/shared/video-model-builder.ts | |||
@@ -108,9 +108,12 @@ export class VideoModelBuilder { | |||
108 | if (!rowsWebTorrentFiles) return | 108 | if (!rowsWebTorrentFiles) return |
109 | 109 | ||
110 | for (const row of rowsWebTorrentFiles) { | 110 | for (const row of rowsWebTorrentFiles) { |
111 | const id = row['VideoFiles.id'] | ||
112 | if (!id) continue | ||
113 | |||
111 | const videoModel = this.videosMemo[row.id] | 114 | const videoModel = this.videosMemo[row.id] |
112 | this.addWebTorrentFile(row, videoModel) | 115 | this.addWebTorrentFile(row, videoModel) |
113 | this.addRedundancy(row, 'VideoFiles.RedundancyVideos', this.videoFileMemo[row['VideoFiles.id']]) | 116 | this.addRedundancy(row, 'VideoFiles.RedundancyVideos', this.videoFileMemo[id]) |
114 | } | 117 | } |
115 | } | 118 | } |
116 | 119 | ||
@@ -118,6 +121,9 @@ export class VideoModelBuilder { | |||
118 | if (!rowsStreamingPlaylist) return | 121 | if (!rowsStreamingPlaylist) return |
119 | 122 | ||
120 | for (const row of rowsStreamingPlaylist || []) { | 123 | for (const row of rowsStreamingPlaylist || []) { |
124 | const id = row['VideoStreamingPlaylists.id'] | ||
125 | if (!id) continue | ||
126 | |||
121 | const videoModel = this.videosMemo[row.id] | 127 | const videoModel = this.videosMemo[row.id] |
122 | 128 | ||
123 | this.addStreamingPlaylist(row, videoModel) | 129 | this.addStreamingPlaylist(row, videoModel) |
@@ -125,7 +131,7 @@ export class VideoModelBuilder { | |||
125 | this.addRedundancy( | 131 | this.addRedundancy( |
126 | row, | 132 | row, |
127 | 'VideoStreamingPlaylists.RedundancyVideos', | 133 | 'VideoStreamingPlaylists.RedundancyVideos', |
128 | this.videoStreamingPlaylistMemo[row['VideoStreamingPlaylists.id']] | 134 | this.videoStreamingPlaylistMemo[id] |
129 | ) | 135 | ) |
130 | } | 136 | } |
131 | } | 137 | } |
diff --git a/server/models/video/sql/videos-model-list-query-builder.ts b/server/models/video/sql/videos-model-list-query-builder.ts index 43040fc5e..e61c51de8 100644 --- a/server/models/video/sql/videos-model-list-query-builder.ts +++ b/server/models/video/sql/videos-model-list-query-builder.ts | |||
@@ -52,8 +52,8 @@ export class VideosModelListQueryBuilder extends AbstractVideosModelQueryBuilder | |||
52 | this.includeThumbnails() | 52 | this.includeThumbnails() |
53 | 53 | ||
54 | if (options.withFiles) { | 54 | if (options.withFiles) { |
55 | this.includeWebtorrentFiles(false) | 55 | this.includeWebtorrentFiles() |
56 | this.includeStreamingPlaylistFiles(false) | 56 | this.includeStreamingPlaylistFiles() |
57 | } | 57 | } |
58 | 58 | ||
59 | if (options.user) { | 59 | if (options.user) { |