From: Chocobozzz <me@florianbigard.com>
Date: Wed, 25 Apr 2018 15:30:46 +0000 (+0200)
Subject: Implement video channel feeds
X-Git-Tag: v1.0.0-beta.4~18
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=e0ea4b1d550d20271a74f1b84e7b3babeec9e0b3;p=oweals%2Fpeertube.git

Implement video channel feeds
---

diff --git a/server/controllers/feeds.ts b/server/controllers/feeds.ts
index 7dcaf7004..08f179509 100644
--- a/server/controllers/feeds.ts
+++ b/server/controllers/feeds.ts
@@ -3,10 +3,9 @@ import { CONFIG, FEEDS } from '../initializers/constants'
 import { asyncMiddleware, feedsValidator, setDefaultSort, videosSortValidator } from '../middlewares'
 import { VideoModel } from '../models/video/video'
 import * as Feed from 'pfeed'
-import { ResultList } from '../../shared/models'
 import { AccountModel } from '../models/account/account'
 import { cacheRoute } from '../middlewares/cache'
-import { VideoSortField } from '../../client/src/app/shared/video/sort-field.type'
+import { VideoChannelModel } from '../models/video/video-channel'
 
 const feedsRouter = express.Router()
 
@@ -31,6 +30,7 @@ async function generateFeed (req: express.Request, res: express.Response, next:
   const start = 0
 
   const account: AccountModel = res.locals.account
+  const videoChannel: VideoChannelModel = res.locals.videoChannel
   const hideNSFW = CONFIG.INSTANCE.DEFAULT_NSFW_POLICY === 'do_not_list'
 
   const resultList = await VideoModel.listForApi({
@@ -40,7 +40,8 @@ async function generateFeed (req: express.Request, res: express.Response, next:
     hideNSFW,
     filter: req.query.filter,
     withFiles: true,
-    accountId: account ? account.id : null
+    accountId: account ? account.id : null,
+    videoChannelId: videoChannel ? videoChannel.id : null
   })
 
   // Adding video items to the feed, one at a time
diff --git a/server/middlewares/validators/feeds.ts b/server/middlewares/validators/feeds.ts
index 6a8cfce86..b55190559 100644
--- a/server/middlewares/validators/feeds.ts
+++ b/server/middlewares/validators/feeds.ts
@@ -1,11 +1,12 @@
 import * as express from 'express'
 import { param, query } from 'express-validator/check'
-import { isAccountIdExist, isAccountNameValid, isLocalAccountNameExist } from '../../helpers/custom-validators/accounts'
+import { isAccountIdExist, isAccountNameValid } from '../../helpers/custom-validators/accounts'
 import { join } from 'path'
 import { isIdOrUUIDValid } from '../../helpers/custom-validators/misc'
 import { logger } from '../../helpers/logger'
 import { areValidationErrors } from './utils'
 import { isValidRSSFeed } from '../../helpers/custom-validators/feeds'
+import { isVideoChannelExist } from '../../helpers/custom-validators/video-channels'
 
 const feedsValidator = [
   param('format').optional().custom(isValidRSSFeed).withMessage('Should have a valid format (rss, atom, json)'),
@@ -18,11 +19,8 @@ const feedsValidator = [
 
     if (areValidationErrors(req, res)) return
 
-    if (req.query.accountId) {
-      if (!await isAccountIdExist(req.query.accountId, res)) return
-    } else if (req.query.accountName) {
-      if (!await isLocalAccountNameExist(req.query.accountName, res)) return
-    }
+    if (req.query.accountId && !await isAccountIdExist(req.query.accountId, res)) return
+    if (req.query.videoChannelId && !await isVideoChannelExist(req.query.videoChannelId, res)) return
 
     return next()
   }