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

Make api show/update/destroy node actions resourceful

Browse source
This commit is contained in:
Anton Khorev 2025-02-01 14:01:01 +03:00
parent 7cf9bf0e55
commit 3b0be171b9
4 changed files with 11 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

12
app/controllers/api/nodes_controller.rb
View file

@ -2,14 +2,14 @@
module Api
class NodesController < ApiController
before_action :check_api_writable, :only => [:create, :update, :delete]
before_action :authorize, :only => [:create, :update, :delete]
before_action :check_api_writable, :only => [:create, :update, :destroy]
before_action :authorize, :only => [:create, :update, :destroy]
authorize_resource
before_action :require_public_data, :only => [:create, :update, :delete]
before_action :set_request_formats, :except => [:create, :update, :delete]
before_action :check_rate_limit, :only => [:create, :update, :delete]
before_action :require_public_data, :only => [:create, :update, :destroy]
before_action :set_request_formats, :except => [:create, :update, :destroy]
before_action :check_rate_limit, :only => [:create, :update, :destroy]
# Dump the details on many nodes whose ids are given in the "nodes" parameter.
def index
@ -68,7 +68,7 @@ module Api
# Delete a node. Doesn't actually delete it, but retains its history
# in a wiki-like way. We therefore treat it like an update, so the delete
# method returns the new version number.
def delete
def destroy
node = Node.find(params[:id])
new_node = Node.from_xml(request.raw_post)