diff options
author | ThibG <thib@sitedethib.com> | 2020-11-21 06:18:09 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-21 06:18:09 +0100 |
commit | c43f4cd3bbeaf4a6db5aa39477bb0c708422b49b (patch) | |
tree | ae356a4750071dba622a74cddb71cf7dc3f49268 /app | |
parent | 8b8004a9626442ae31e4dffd79e874e9cde050c6 (diff) |
Fix not being able to unfavorite toots one has lost access to (#15192)
Fixes #15191
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/api/v1/statuses/favourites_controller.rb | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/app/controllers/api/v1/statuses/favourites_controller.rb b/app/controllers/api/v1/statuses/favourites_controller.rb index 7afa822ed..2e21ce6a0 100644 --- a/app/controllers/api/v1/statuses/favourites_controller.rb +++ b/app/controllers/api/v1/statuses/favourites_controller.rb @@ -5,7 +5,7 @@ class Api::V1::Statuses::FavouritesController < Api::BaseController before_action -> { doorkeeper_authorize! :write, :'write:favourites' } before_action :require_user! - before_action :set_status + before_action :set_status, only: [:create] def create FavouriteService.new.call(current_account, @status) @@ -13,8 +13,19 @@ class Api::V1::Statuses::FavouritesController < Api::BaseController end def destroy - UnfavouriteWorker.perform_async(current_account.id, @status.id) + fav = current_account.favourites.find_by(status_id: params[:status_id]) + + if fav + @status = fav.status + UnfavouriteWorker.perform_async(current_account.id, @status.id) + else + @status = Status.find(params[:status_id]) + authorize @status, :show? + end + render json: @status, serializer: REST::StatusSerializer, relationships: StatusRelationshipsPresenter.new([@status], current_account.id, favourites_map: { @status.id => false }) + rescue Mastodon::NotPermittedError + not_found end private |