Skip to content

Commit

Permalink
Revert variant selector normalization
Browse files Browse the repository at this point in the history
Probably worth tackling later, but for now it's not worth worrying about; some other implementations (e.g. Misskey's) look to have the same behavior anyways.
  • Loading branch information
TheEssem committed Dec 2, 2024
1 parent 1d8225c commit b4f9230
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions app/controllers/api/v1/statuses/reactions_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@ class Api::V1::Statuses::ReactionsController < Api::BaseController
before_action :set_status

def create
ReactService.new.call(current_account, @status, normalize(params[:id]))
ReactService.new.call(current_account, @status, params[:id])
render json: @status, serializer: REST::StatusSerializer
end

def destroy
UnreactWorker.perform_async(current_account.id, @status.id, normalize(params[:id]))
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
Expand All @@ -22,13 +22,6 @@ def destroy

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?
Expand Down

0 comments on commit b4f9230

Please sign in to comment.