From ee498ad9effb7d3f9c8edea1702317f359bcaa44 Mon Sep 17 00:00:00 2001 From: Feruz Oripov Date: Tue, 27 Aug 2019 03:59:31 +0500 Subject: [PATCH] Refactor VideosController (#3806) [ci skip] --- app/controllers/videos_controller.rb | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/app/controllers/videos_controller.rb b/app/controllers/videos_controller.rb index d33f39c87..cfb09f0b2 100644 --- a/app/controllers/videos_controller.rb +++ b/app/controllers/videos_controller.rb @@ -1,10 +1,9 @@ class VideosController < ApplicationController after_action :verify_authorized, except: %i[index] before_action :set_cache_control_headers, only: %i[index] + before_action :authorize_video, only: %i[new create] - def new - authorize :video - end + def new; end def index @video_articles = Article.published. @@ -16,7 +15,6 @@ class VideosController < ApplicationController end def create - authorize :video @article = ArticleWithVideoCreationService.new(article_params, current_user).create! redirect_to @article.path + "/edit" @@ -24,6 +22,10 @@ class VideosController < ApplicationController private + def authorize_video + authorize :video + end + def article_params params.require(:article).permit(:video) end