cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add destroy note subscription api endpoint

Browse source
This commit is contained in:
Anton Khorev 2024-10-24 04:46:57 +03:00
parent edd12c6995
commit 53a33110f0
4 changed files with 76 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
app/abilities/api_capability.rb
View file

@ -9,7 +9,7 @@ class ApiCapability
if user&.active?
can [:create, :comment, :close, :reopen], Note if scope?(token, :write_notes)
can :create, NoteSubscription if scope?(token, :write_notes)
can [:create, :destroy], NoteSubscription if scope?(token, :write_notes)
can [:show, :data], Trace if scope?(token, :read_gpx)
can [:create, :update, :destroy], Trace if scope?(token, :write_gpx)
can [:details], User if scope?(token, :read_prefs)

9
app/controllers/api/note_subscriptions_controller.rb
View file

@ -14,5 +14,14 @@ module Api
rescue ActiveRecord::RecordNotUnique
report_error "You are already subscribed to note #{note_id}.", :conflict
end
def destroy
note_id = params[:note_id].to_i
note = Note.find(note_id)
count = note.subscriptions.where(:user => current_user).delete_all
report_error "You are not subscribed to note #{note_id}.", :not_found if count.zero?
rescue ActiveRecord::RecordNotFound
report_error "Note #{note_id} not found.", :not_found
end
end
end