Move reaction normalization to API controller
This commit is contained in:
parent
3361f8851e
commit
00234f401c
2 changed files with 9 additions and 5 deletions
|
@ -8,12 +8,12 @@ class Api::V1::Statuses::ReactionsController < Api::BaseController
|
|||
before_action :set_status
|
||||
|
||||
def create
|
||||
ReactService.new.call(current_account, @status, params[:id])
|
||||
ReactService.new.call(current_account, @status, normalize(params[:id]))
|
||||
render json: @status, serializer: REST::StatusSerializer
|
||||
end
|
||||
|
||||
def destroy
|
||||
UnreactWorker.perform_async(current_account.id, @status.id, params[:id])
|
||||
UnreactWorker.perform_async(current_account.id, @status.id, normalize(params[:id]))
|
||||
|
||||
render json: @status, serializer: REST::StatusSerializer, relationships: StatusRelationshipsPresenter.new([@status], current_account.id, reactions_map: { @status.id => false })
|
||||
rescue Mastodon::NotPermittedError
|
||||
|
@ -22,6 +22,13 @@ class Api::V1::Statuses::ReactionsController < Api::BaseController
|
|||
|
||||
private
|
||||
|
||||
def normalize(name)
|
||||
normalized = "#{name}\uFE0F"
|
||||
return normalized if StatusReactionValidator::SUPPORTED_EMOJIS.include?(normalized)
|
||||
|
||||
name
|
||||
end
|
||||
|
||||
def set_status
|
||||
@status = Status.find(params[:status_id])
|
||||
authorize @status, :show?
|
||||
|
|
|
@ -10,9 +10,6 @@ class ReactService < BaseService
|
|||
name, domain = emoji.split('@')
|
||||
return unless domain.nil? || status.local?
|
||||
|
||||
normalized = "#{name}\uFE0F"
|
||||
name = normalized if StatusReactionValidator::SUPPORTED_EMOJIS.include?(normalized)
|
||||
|
||||
custom_emoji = CustomEmoji.find_by(shortcode: name, domain: domain)
|
||||
reaction = StatusReaction.find_by(account: account, status: status, name: name, custom_emoji: custom_emoji)
|
||||
return reaction unless reaction.nil?
|
||||
|
|
Loading…
Reference in a new issue