diff options
author | Chocobozzz <me@florianbigard.com> | 2018-09-18 12:00:49 +0200 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2018-09-19 09:54:37 +0200 |
commit | 627621c1e8d37c33f7b3dd59f4c8907b12c630bc (patch) | |
tree | 007e7118f482c543d3898b222b62c185fda4fd2b /server/helpers | |
parent | e972e046dbe9b499944c4fab9220eee13e31ac1b (diff) | |
download | PeerTube-627621c1e8d37c33f7b3dd59f4c8907b12c630bc.tar.gz PeerTube-627621c1e8d37c33f7b3dd59f4c8907b12c630bc.tar.zst PeerTube-627621c1e8d37c33f7b3dd59f4c8907b12c630bc.zip |
Optimize SQL requests of watch page API endpoints
Diffstat (limited to 'server/helpers')
-rw-r--r-- | server/helpers/custom-validators/videos.ts | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/server/helpers/custom-validators/videos.ts b/server/helpers/custom-validators/videos.ts index edafba6e2..dd207c787 100644 --- a/server/helpers/custom-validators/videos.ts +++ b/server/helpers/custom-validators/videos.ts | |||
@@ -152,13 +152,15 @@ function checkUserCanManageVideo (user: UserModel, video: VideoModel, right: Use | |||
152 | return true | 152 | return true |
153 | } | 153 | } |
154 | 154 | ||
155 | async function isVideoExist (id: string, res: Response) { | 155 | async function isVideoExist (id: string, res: Response, fetchType: 'all' | 'only-video' | 'id' | 'none' = 'all') { |
156 | let video: VideoModel | null | 156 | let video: VideoModel | null |
157 | 157 | ||
158 | if (validator.isInt(id)) { | 158 | if (fetchType === 'all') { |
159 | video = await VideoModel.loadAndPopulateAccountAndServerAndTags(+id) | 159 | video = await VideoModel.loadAndPopulateAccountAndServerAndTags(id) |
160 | } else { // UUID | 160 | } else if (fetchType === 'only-video') { |
161 | video = await VideoModel.loadByUUIDAndPopulateAccountAndServerAndTags(id) | 161 | video = await VideoModel.load(id) |
162 | } else if (fetchType === 'id' || fetchType === 'none') { | ||
163 | video = await VideoModel.loadOnlyId(id) | ||
162 | } | 164 | } |
163 | 165 | ||
164 | if (video === null) { | 166 | if (video === null) { |
@@ -169,7 +171,7 @@ async function isVideoExist (id: string, res: Response) { | |||
169 | return false | 171 | return false |
170 | } | 172 | } |
171 | 173 | ||
172 | res.locals.video = video | 174 | if (fetchType !== 'none') res.locals.video = video |
173 | return true | 175 | return true |
174 | } | 176 | } |
175 | 177 | ||