From: Eugen Rochko <eugen@zeonfederated.com>
Date: Fri, 12 Feb 2021 06:19:15 +0000 (+0100)
Subject: Refactor Api::Web::SettingsController (#15717)
X-Git-Url: https://git.xn--scling-oua.cat.family/?a=commitdiff_plain;h=15ced8728ff89932e3f8febf119f63c78ac9a960;p=mastodon.git

Refactor Api::Web::SettingsController (#15717)
---

diff --git a/app/controllers/api/web/settings_controller.rb b/app/controllers/api/web/settings_controller.rb
index 3d65e46ed..601e25e6e 100644
--- a/app/controllers/api/web/settings_controller.rb
+++ b/app/controllers/api/web/settings_controller.rb
@@ -2,17 +2,16 @@
 
 class Api::Web::SettingsController < Api::Web::BaseController
   before_action :require_user!
+  before_action :set_setting
 
   def update
-    setting.data = params[:data]
-    setting.save!
-
+    @setting.update!(data: params[:data])
     render_empty
   end
 
   private
 
-  def setting
-    @_setting ||= ::Web::Setting.where(user: current_user).first_or_initialize(user: current_user)
+  def set_setting
+    @setting = ::Web::Setting.where(user: current_user).first_or_initialize(user: current_user)
   end
 end