ReactionsController: Don't check for status reaction existence in destroy

UnreactService checks for its existence in the background anyway, so remove redundant checks.
This commit is contained in:
Jeremy Kescher 2023-05-11 13:40:24 +02:00
parent 4252c6fe1e
commit f158fb7151
No known key found for this signature in database
GPG key ID: 80A419A7A613DFA4
2 changed files with 3 additions and 11 deletions

View file

@ -5,7 +5,7 @@ class Api::V1::Statuses::ReactionsController < Api::BaseController
before_action -> { doorkeeper_authorize! :write, :'write:favourites' }
before_action :require_user!
before_action :set_status, only: [:create]
before_action :set_status
def create
ReactService.new.call(current_account, @status, params[:id])
@ -13,15 +13,7 @@ class Api::V1::Statuses::ReactionsController < Api::BaseController
end
def destroy
react = current_account.status_reactions.find_by(status_id: params[:status_id], name: params[:id])
if react
@status = react.status
UnreactWorker.perform_async(current_account.id, @status.id, params[:id])
else
@status = Status.find(params[:status_id])
authorize @status, :show?
end
UnreactWorker.perform_async(current_account.id, @status.id, params[:id])
render json: @status, serializer: REST::StatusSerializer, relationships: StatusRelationshipsPresenter.new([@status], current_account.id, reactions_map: { @status.id => false })
rescue Mastodon::NotPermittedError

View file

@ -25,7 +25,7 @@ module Mastodon
end
def suffix_version
'+1.1.3'
'+1.1.4'
end
def to_a